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:

      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
                                          • fischi87
                                            fischi87 @haus-automatisierung last edited by

                                            @haus-automatisierung

                                            Wofür ist dieses Zeichen? Bzw was besagt oder besser wo ist der Unterschied ohne?

                                            =>
                                            
                                            1 Reply Last reply Reply Quote 0
                                            • First post
                                              Last post

                                            Support us

                                            ioBroker
                                            Community Adapters
                                            Donate

                                            766
                                            Online

                                            31.8k
                                            Users

                                            80.0k
                                            Topics

                                            1.3m
                                            Posts

                                            javascript
                                            6
                                            51
                                            2163
                                            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