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.
    • 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
              • haus-automatisierung
                haus-automatisierung Developer Most Active @Codierknecht last edited by haus-automatisierung

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

                Vor oder nach der Variablen?

                Wenn der alleine in einer Zeile steht ist das völlig egal. Dann aber meistens hinter der Variable. Ist ja nur entscheidend, wenn man das direkt als Parameter übergibt o.ä.

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

                @fischi87
                Es ist halt ein himmelweiter Unterschied, ob man das so

                In dem Fall nicht. Beide gehen von 0 bis 99.

                In einer for-Schleife mache ich es immer dahinter. Macht in deinem Beispiel aber auch keinen Unterschied bei der Schleife (weil es ja alleine steht). Hier wäre es ein Unterschied:

                let i = 0;
                do {
                    console.log(i);
                } while (++i < 100);
                // 0 bis 99
                
                let i = 0;
                do {
                    console.log(i);
                } while (i++ < 100);
                // 0 bis 100
                
                Codierknecht 1 Reply Last reply Reply Quote 0
                • Codierknecht
                  Codierknecht Developer Most Active @haus-automatisierung last edited by

                  @haus-automatisierung
                  Asche auf mein Haupt. Schlechtes Beispiel 😞

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

                    @codierknecht sagte: getObject() liest aber asynchron.

                    getObject(id) liest synchron, wenn keine Callback-Funktion übergeben wird.

                    Codierknecht fischi87 2 Replies Last reply Reply Quote 0
                    • Codierknecht
                      Codierknecht Developer Most Active @paul53 last edited by

                      @paul53
                      Ich sag' ja: An dem synchron/synchron Kram verzweifle ich noch 😞

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

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

                        An dem synchron/synchron Kram verzweifle ich noch

                        Ich verwende einfach immer die ...Async-Funktionen. Dann kann ich sicher sein, dass ich ein Promise zurückbekomme und kann damit dann halt mit await oder .then() weitermachen... Callbacks sind ja irgendwie retro und führen nur zu stark verschachteltem Code.

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

                          @haus-automatisierung sagte in Skript gibt [objekt Objekt] aus:

                          Ich verwende einfach immer die ...Async-Funktionen

                          Ich ja mittlerweile auch.

                          Was aber auch daran liegen mag, dass ich das ganze Konzept noch immer nicht verinnerlicht habe.
                          Ich komme halt aus der Windows-Welt. Und da muss man Klimmzüge machen, um etwas asynchron hinzukriegen und nicht, um etwas synchron ablaufen zu lassen.

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

                            @paul53

                            also müsste es so laufen wie im ersten post? jetzt bin ich verwirrt!

                            paul53 haus-automatisierung 2 Replies Last reply Reply Quote 0
                            • paul53
                              paul53 @fischi87 last edited by

                              @fischi87 sagte also müsste es so laufen wie im ersten post?

                              Nicht ganz (Zeilen 50, 51)

                                      var obj = getObject(id);
                                      var name = getObject(id).common;
                              

                              Richtig:

                                      var name = getObject(id).common.name;
                                      if(typeof name == 'object') name = name.de;
                              
                              fischi87 1 Reply Last reply Reply Quote 0
                              • haus-automatisierung
                                haus-automatisierung Developer Most Active @fischi87 last edited by

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

                                also müsste es so laufen wie im ersten post?

                                War wohl etwas viel Offtopic hier, aber fehlt Dir hier? https://forum.iobroker.net/topic/72999/skript-gibt-objekt-objekt-aus/17

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

                                  @paul53

                                  Sind denn da die Fehlermeldung bei .push und .sort auch behoben oder liegt die dann an einem anderen fehler?

                                  Danke dir erstmal.

                                  Ich glaube die Antwort auf die Frage nach dem „wie ist es richtig“ bzw „wie ist es am korrektesten“ wird man wohl nie beantworten können?

                                  paul53 haus-automatisierung 2 Replies Last reply Reply Quote 0
                                  • paul53
                                    paul53 @fischi87 last edited by paul53

                                    @fischi87
                                    Zur Überschrift: Zeile 68

                                        if (logging) log("Text: " + textUpdateAn);
                                    

                                    zeigt "Text: [objekt Objekt]", da textUpdateAn ein Array enthält. Entweder

                                        if (logging) log("Text: " + JSON.stringify(textUpdateAn));
                                    

                                    oder

                                        if (logging) log(textUpdateAn);
                                    

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

                                    Sind denn da die Fehlermeldung bei .push und .sort auch behoben

                                    Ich sehe im ersten Post keinen Grund, weshalb es nicht funktionieren sollte.

                                    EDIT: Durch falsch gewählte Variablenbezeichner (textUpdateAn) kann man sich auch selbst verwirren.

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

                                      @fischi87 Habe mir jetzt das ganze Script nochmal angeschaut. Dein Problem ist .each und async. Das geht so nicht.

                                      const logging = true;
                                       
                                      const idbutton = 'shelly.0.info.update';
                                       
                                      const idAnzahlAn = 'javascript.0.Status.Shelly.UpdateFirmware.An';
                                      const idAnzahl = 'javascript.0.Status.Shelly.UpdateFirmware.Anzahl';
                                      const idText = 'javascript.0.Status.Shelly.UpdateFirmware.Text';
                                      const iddatenpunkt = 'javascript.0.Status.Shelly.UpdateFirmware.State';
                                       
                                      createState(idAnzahl, { // wenn benötigt: Anzahl der vorhandenen Lichter
                                          type: 'number',
                                          name: 'Anzahl aller Geräte',
                                          min: 0,
                                          def: 0,
                                          role: 'value'
                                      });
                                      createState(idAnzahlAn, { // Anzahl der Lichter, die auf sind als Variable unter Javascript.0 anlegen
                                          type: 'number',
                                          name: 'Anzahl Update',
                                          min: 0,
                                          def: 0,
                                          role: 'value'
                                      });
                                      createState(idText, { // Anzahl der brennenden Lichter und deren Namen als Variable unter Javascript.0 anlegen
                                          type: 'string',
                                          name: 'Eingeschaltete Update',
                                          desc: 'Namen der Update Geräte',
                                          def: ' ',
                                          role: 'value'
                                      });
                                      createState(iddatenpunkt, {
                                          type: 'boolean',
                                          name: 'State',
                                          role: '',
                                          read: true,
                                          write: false,
                                          def: false
                                      });
                                      
                                      const cacheSelectorState = $('shelly.0.*[state.id=*.firmware]');
                                      
                                      async function checkDevices() {
                                          let anzahlUpdateAn = 0;
                                          let anzahlUpdate = 0;
                                          const textUpdateAn = [];
                                       
                                          for (const id of cacheSelectorState) {
                                              const obj = await getObjectAsync(id);
                                              const oName = typeof obj.common.name === 'object' ? obj.common.name.de : obj.common.name;
                                              const state = await getStateAsync(id).val;
                                       
                                              if (state.val) { 
                                                  anzahlUpdateAn++;
                                                  textUpdateAn.push(oName); 
                                              }
                                              anzahlUpdate++;
                                       
                                              setState(iddatenpunkt, anzahlUpdateAn > 0, true);
                                          }
                                       
                                          textUpdateAn.sort();
                                       
                                          if (logging) log(`Text: ${JSON.stringify(textUpdateAn)}`);
                                          if (logging) log(`Anzahl Geräte: ${anzahlUpdate} # davon: ${anzahlUpdateAn} Updaten`);
                                      
                                          setState(idText, textUpdateAn.join(',<br>')); 
                                          setState(idAnzahlAn, textUpdateAn.length); 
                                          setState(idAnzahl, anzahlUpdate); 
                                       
                                          if (anzahlUpdateAn > 0 ) {
                                              sendTo('telegram.0', {
                                                  text: `+++ Geräte Update +++\n\n + ${textUpdateAn.join(', ')}`,
                                                  reply_markup: {
                                                      keyboard: [['System', 'Heizung'], ['Licht', 'Strom'], ['Multimedia', 'Szenen'], ['Home']],
                                                      resize_keyboard: true,
                                                      one_time_keyboard: true
                                                  }
                                              });
                                          }
                                      }
                                      
                                      // Trigger
                                      cacheSelectorState.on(checkDevices);
                                      
                                      function main() {
                                          setTimeout(() => {
                                              if (logging) {
                                                  log('Auslöser Skriptstart');
                                              }
                                      
                                              checkDevices();
                                          }, 2000);
                                      }
                                      
                                      function button() { 
                                          setTimeout(() => {
                                              setState(idbutton, false);
                                          }, 200);
                                      }
                                      
                                      on(idbutton, (dp) => {
                                          if (dp.state.val) {
                                              button();
                                          }
                                      });
                                      
                                      main(); // Skriptstart-Auslöser
                                      
                                      paul53 fischi87 4 Replies Last reply Reply Quote 0
                                      • paul53
                                        paul53 @haus-automatisierung last edited by

                                        @haus-automatisierung sagte: Problem ist .each und async.

                                        Ich kann im ersten Post kein async entdecken.

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

                                          @haus-automatisierung

                                          Soviele Wege 🙂

                                          Das ist echt viel Input. Hab noch paar solche Skripte die eigentlich laufen aber wenn ich das jetzt sehe mit 100% Wahrscheinlichkeit auch nicht richtig sind.

                                          Danke euch erstmal, werd heut Abend Erfolg oder Misserfolg vermelden!

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

                                            @paul53

                                            Was meinst du selber verwirren? Ich möchte die Geräte ausgeben die ein verfügbares Update haben. Wie meinst du das? Wie im Skript lass ich mir das auch als Telegram Nachricht senden, da bekomme ich auch nur [objekt Objekt] ausgegeben. Also nicht nur im log?!

                                            paul53 1 Reply Last reply Reply Quote 0
                                            • First post
                                              Last post

                                            Support us

                                            ioBroker
                                            Community Adapters
                                            Donate

                                            833
                                            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