Navigation

    Logo
    • Register
    • Login
    • Search
    • Recent
    • Tags
    • Unread
    • Categories
    • Unreplied
    • Popular
    • GitHub
    • Docu
    • Hilfe
    1. Home
    2. Deutsch
    3. Skripten / Logik
    4. JavaScript
    5. [GELÖST] Skript gibt [objekt Objekt] aus

    NEWS

    • Neuer Blog: Fotos und Eindrücke aus Solingen

    • ioBroker@Smart Living Forum Solingen, 14.06. - Agenda added

    • ioBroker goes Matter ... Matter Adapter in Stable

    [GELÖST] Skript gibt [objekt Objekt] aus

    This topic has been deleted. Only users with topic management privileges can see it.
    • M
      MCU @fischi87 last edited by MCU

      @fischi87
      dd0942d1-2f69-4fb5-b0c7-6568f50681b3-image.png
      Sehr wahrscheinlich sollte es so sein?

      var name = getObject(id).common.name 
      /* besser */
      var name = obj.common.name
      /* obj wurde ja schon gesetzt */
      

      Warum devicename der nicht genutzt wird?

      fischi87 T 2 Replies Last reply Reply Quote 0
      • fischi87
        fischi87 @MCU last edited by fischi87

        @mcu

        Danke für deine Antwort, werde es probieren.

        Hast zu meinen anderen Fragen eine Idee?

        Codierknecht 1 Reply Last reply Reply Quote 0
        • Codierknecht
          Codierknecht Developer Most Active @fischi87 last edited by

          @fischi87 sagte in Skript gibt [objekt Objekt] aus:

          Hast zu meinen anderen Fragen eine Idee?

          Ich mag mich täuschen, aber ich lerne ja selbst noch gerne dazu.
          Kann es sein, dass da der Scope nicht passt und die Variable textUpdateAn innerhalb der anonymen Funktion für cacheSelectorState.each nicht bekannt ist?
          Was passiert, wenn Du die als globale Variable deklarierst?

          Vielleicht kann @OliverIO hier Licht ins Dunkel bringen?

          Was mir noch aufgefallen ist:
          Du schreibst
          ++variable
          Ich persönlich finde das etwas unüblich.
          Diese Notation kenne ich nur, wenn man eine Variable verwendet und sie vor der Verwendung inkrementiert werden soll.
          Will man einfach nur hochzählen, verwendet man üblicherweise
          variable++
          Ist zwar semantisch das gleiche, liest sich aber irgendwie besser.

          fischi87 1 Reply Last reply Reply Quote 0
          • T
            ticaki Developer @MCU last edited by ticaki

            @mcu sagte in Skript gibt [objekt Objekt] aus:

            @fischi87
            dd0942d1-2f69-4fb5-b0c7-6568f50681b3-image.png
            Sehr wahrscheinlich sollte es so sein?

            var name = getObject(id).common.name 
            /* besser */
            var name = obj.common.name
            /* obj wurde ja schon gesetzt */
            

            Warum devicename der nicht genutzt wird?

            ne so macht man das nicht. getObject wird mit callback verwendet.

            async function main() 
            {
              const obj = await getObjectAsync(id);
              let oName = ''
              if (obj && obj.common) {
                oName = obj.common.name
              }
            }
            

            Man testet auch die Keys die man benutzen will - wenn da einer von nicht existiert gibts sonst einen Javascript Adapter neustart oder wenn nicht in einer async funktion einen Fehler.

            ach und name ist als Variable ungünstig. Da steht AFAIK der scriptname drin.

            fischi87 1 Reply Last reply Reply Quote 0
            • fischi87
              fischi87 @ticaki last edited by

              @ticaki

              Mit deiner Antwort bin ich etwas überfordert muss ich sagen. Das Skript gibt mir ja alle shelly Geräte aus die ein Update verfügbar haben nur halt den Namen nicht.

              Ist dein Ansatz jetzt der neuen JavaScript Version geschuldet?

              Komm auch mit deinem Code Schnipsel nicht so klar. Füg ich die Funktion hinzu oder ändere ich diese ab oder wie meinst du das?

              Codierknecht 1 Reply Last reply Reply Quote 0
              • fischi87
                fischi87 @Codierknecht last edited by

                @codierknecht

                Bin auch noch einiges am lernen. Hatte mir nur gedacht das ich ein funktionieren Skript nur anpasse, deswegen wundert es mich dass es so nicht funktioniert.

                Kenn das mit ++variable dies anfang an deswegen liest sich das für mich besser 😉

                1 Reply Last reply Reply Quote 0
                • Codierknecht
                  Codierknecht Developer Most Active @fischi87 last edited by Codierknecht

                  @fischi87
                  Du liest z.B. in Zeile 51 den Namen über getObject(id).
                  In Zeile 53 willst Du den gleich weiter verarbeiten.

                  getObject() liest aber asynchron.
                  Das heißt, in Zeile 53 ist name noch gar nicht besetzt. Das asynchrone Lesen ist da nämlich mit ziemlicher Sicherheit noch gar nicht fertig.

                  @ticaki meint das vermutlich so (Code ab Zeile 49):

                      cacheSelectorState.each(function (id, i) { 
                          const obj = await getObjectAsync(id);
                          if (obj && obj.common) {
                              let oName = obj.common.name;
                          }
                  
                  fischi87 paul53 2 Replies Last reply Reply Quote 0
                  • fischi87
                    fischi87 @Codierknecht last edited by

                    @codierknecht

                    Okay werde ich probieren.

                    Nur als Verständnis, ist das schon immer so weil ich meine hier vor paar Jahren das lichter Skript kopiert zu haben was genauso aufgebaut ist und funktioniert, deswegen erschließt sich mir dies im Moment nicht. Kann ja sein das sich bei version Änderung etwas geändert hat?

                    Wie gesagt das lichter, Fenster und lowbat Skript laufen immer noch ohne Probleme und da werden mir auch die Namen angezeigt.

                    Würde es gerne verstehen.

                    Codierknecht 1 Reply Last reply Reply Quote 0
                    • Codierknecht
                      Codierknecht Developer Most Active @fischi87 last edited by

                      @fischi87 sagte in Skript gibt [objekt Objekt] aus:

                      vor paar Jahren

                      So lange bin ich noch nicht dabei.
                      Ich kenne es nur so - also dass asynchron gelesen wird.
                      Das kann dann ohne await funktionieren, muss aber nicht. In der Regel tut es das nicht.

                      An den ganzen asynchronen Kram muss man sich erstmal gewöhnen. Ich habe da auch so meine liebe Not.

                      fischi87 1 Reply Last reply Reply Quote 0
                      • fischi87
                        fischi87 @Codierknecht last edited by

                        @codierknecht

                        In deinen erwähnten Code schreibst du:

                        let oName = obj.common.name;
                        

                        Kann ich denn dann oName außerhalb dieser Funktion nutzen? Dachte immer let kann man nur innerhalb der deklarierten Funktion nutzen?

                        Codierknecht 1 Reply Last reply Reply Quote 0
                        • Codierknecht
                          Codierknecht Developer Most Active @fischi87 last edited by

                          @fischi87 sagte in Skript gibt [objekt Objekt] aus:

                          Dachte immer let kann man nur innerhalb der deklarierten Funktion nutzen?

                          Da dürftest Du Recht haben.
                          Scope in JS ist auch eine meiner Baustellen 😉

                          Mach halt ein var draus, wenn Du es brauchst.

                          fischi87 haus-automatisierung 3 Replies Last reply Reply Quote 0
                          • fischi87
                            fischi87 @Codierknecht last edited by

                            @codierknecht

                            Okay wollte es nur wissen nicht das ich da was falsches gelernt hab 😅

                            1 Reply Last reply Reply Quote 0
                            • haus-automatisierung
                              haus-automatisierung Developer Most Active @Codierknecht last edited by

                              @codierknecht sagte in Skript gibt [objekt Objekt] aus:

                              Mach halt ein var draus, wenn Du es brauchst.

                              var braucht man nie 🙂 immer nur let und const.

                              Ansonsten kann common.name natürlich auch ein Objekt sein, wenn es in mehrere Sprachen übersetzt ist. Also z.B. common.name.de wäre dann die Deutsche Übersetzung.

                              fischi87 1 Reply Last reply Reply Quote 0
                              • fischi87
                                fischi87 @Codierknecht last edited by

                                @codierknecht

                                also sollte ich es so versuchen?

                                    cacheSelectorState.each(function (id, i) { 
                                        const obj = await getObjectAsync(id);
                                
                                        if (obj && obj.common) {
                                            var oName = obj.common.name;
                                           var státus = getState(id).val;
                                        }
                                
                                         if (status) {
                                               ++anzahlUpdateAn;
                                               textUpdateAn.push(Oname);
                                         }
                                
                                
                                haus-automatisierung 1 Reply Last reply Reply Quote 0
                                • fischi87
                                  fischi87 @haus-automatisierung last edited by

                                  @haus-automatisierung

                                  wie jetzt man bruacht nie var?

                                  haus-automatisierung 1 Reply Last reply Reply Quote 0
                                  • haus-automatisierung
                                    haus-automatisierung Developer Most Active @fischi87 last edited by haus-automatisierung

                                    @fischi87 sagte in Skript gibt [objekt Objekt] aus:

                                    also sollte ich es so versuchen?

                                    Bitte nicht 🙂 Der Scope von status ist absolut blöde so.

                                    cacheSelectorState.each(function (id, i) {
                                        let anzahlUpdateAn = 0;
                                        const obj = await getObjectAsync(id);
                                    
                                        if (obj && obj.common) {
                                            const oName = typeof obj.common.name === 'object' ? obj.common.name.de : obj.common.name;
                                    
                                            if (getState(id).val) {
                                                anzahlUpdateAn++;
                                                textUpdateAn.push(oName);
                                            }
                                        }
                                    )};
                                    

                                    Und achte darauf, Variablen RICHTIG wieder zu verwenden. Du schreibst oName und unten Oname. Das sind zwei komplett verschiedene Dinge. Genau wie státus

                                    Codierknecht 1 Reply Last reply Reply Quote 0
                                    • haus-automatisierung
                                      haus-automatisierung Developer Most Active @fischi87 last edited by haus-automatisierung

                                      @fischi87 sagte in Skript gibt [objekt Objekt] aus:

                                      wie jetzt man bruacht nie var?

                                      Na einfach NIE var ... schreiben. Siehst Du ja an deinem Code. Du deklarierst innerhalb des if ... eine Variable (z.B. oName) und möchtest diese außerhalb wieder verwenden. Das ist kein schöner Code und sollte man nie so machen.

                                      Brauchst Du diese außerhalb, musst diese auch vorher außerhalb mit let deklarieren (falls diese geändert werden sollen). Ansonsten mit const arbeiten.

                                      Google: JavaScript variable scopes https://www.w3schools.com/js/js_scope.asp

                                      1 Reply Last reply Reply Quote 0
                                      • Codierknecht
                                        Codierknecht Developer Most Active @haus-automatisierung last edited by

                                        @haus-automatisierung

                                        Und achte darauf, Variablen RICHTIG wieder zu verwenden

                                        Ähämmm ... räusper

                                        textUpdateAn.push(oName);
                                        

                                        😉

                                        haus-automatisierung 1 Reply Last reply Reply Quote 0
                                        • haus-automatisierung
                                          haus-automatisierung Developer Most Active @Codierknecht last edited by

                                          @codierknecht sagte in Skript gibt [objekt Objekt] aus:

                                          Ähämmm ... räusper

                                          Danke, im Forum programmieren ist so eine Sache 🙂

                                          Codierknecht 1 Reply Last reply Reply Quote 0
                                          • Codierknecht
                                            Codierknecht Developer Most Active @haus-automatisierung last edited by Codierknecht

                                            @haus-automatisierung
                                            Nur interessehalber:
                                            Wie verwendest Du den Inkrement-Operator in einem solchen Fall:

                                            ++anzahlUpdateAn;
                                            

                                            Vor oder nach der Variablen?

                                            @fischi87
                                            Es ist halt ein himmelweiter Unterschied, ob man das so
                                            Schlechtes Beispiel. Siehe unten.

                                            for (var i = 0; i < 100; i++)
                                            

                                            oder so

                                            for (var i = 0; i < 100; ++i)
                                            

                                            verwendet.

                                            haus-automatisierung 1 Reply Last reply Reply Quote 0
                                            • First post
                                              Last post

                                            Support us

                                            ioBroker
                                            Community Adapters
                                            Donate

                                            795
                                            Online

                                            31.8k
                                            Users

                                            80.0k
                                            Topics

                                            1.3m
                                            Posts

                                            javascript
                                            6
                                            51
                                            2162
                                            Loading More Posts
                                            • Oldest to Newest
                                            • Newest to Oldest
                                            • Most Votes
                                            Reply
                                            • Reply as topic
                                            Log in to reply
                                            Community
                                            Impressum | Datenschutz-Bestimmungen | Nutzungsbedingungen
                                            The ioBroker Community 2014-2023
                                            logo