Navigation

    Logo
    • Register
    • Login
    • Search
    • Recent
    • Tags
    • Unread
    • Categories
    • Unreplied
    • Popular
    • GitHub
    • Docu
    • Hilfe
    1. Home
    2. Deutsch
    3. Skripten / Logik
    4. JavaScript
    5. asynch funktion

    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

    asynch funktion

    This topic has been deleted. Only users with topic management privileges can see it.
    • D
      Delphinis last edited by

      Hallo,
      ich möchte in einer Funktion in der ich Werte verrechne, warten, bis alle Werte eingetrudelt sind. Für "wait" muss ich ja offensichtlich eine asynch function verwenden. In den Java Scripts fehlt ja aber der Funktionskopf.
      Ich habs mal so versucht:

      async function Eigenverbrauch {
          // On change
          on('modbus.0.inputRegisters.30775_Messwert_Leistung', (data) => {
              
              if (!existsState('0_userdata.0.Solar.Leistung.Eigenverbrauch')) {
              createState('0_userdata.0.Solar.Leistung.Eigenverbrauch', {type: 'number', read: true, write: true});
              }
      
              await wait(500);
      
              const WR = getState('modbus.0.inputRegisters.30775_Messwert_Leistung').val;
              const Bezug = getState('modbus.0.inputRegisters.30865_Leistung_Bezug').val;
              const Einspeisung = getState('modbus.0.inputRegisters.30867_Messwert_Leistung_Einspeisung').val;
              const BatLad = getState('modbus.0.inputRegisters.31393_Batterieladung').val;
              const BatEntl = getState('modbus.0.inputRegisters.31395_Batterieentladung').val;
      
              let eigenverbrauch = (WR + Bezug - Einspeisung -BatLad + BatEntl);
              setState('0_userdata.0.Solar.Leistung.Eigenverbrauch', eigenverbrauch, true);
          });
      }
      

      dies liefert aber den Fehler:

      15:16:36.440	error	javascript.0 (8848) script.js.Eigene_Scripte.Solar_Leistung_Eigenverbrauch compile failed: at script.js.Eigene_Scripte.Solar_Leistung_Eigenverbrauch:1
      

      Wie kann man das realisieren?

      Btw: gibt es da irgendwo genauere Fehlermeldungen beim compilieren?

      ? Codierknecht haus-automatisierung paul53 Peoples 5 Replies Last reply Reply Quote 0
      • ?
        A Former User @Delphinis last edited by

        @delphinis da fehlt die klammer in der ersten Zeile hinter ‚async function Eigenverbrauch‘

        Muss so aussehen:

        async function Eigenverbrauch() {
        
        1 Reply Last reply Reply Quote 1
        • Codierknecht
          Codierknecht Developer Most Active @Delphinis last edited by Codierknecht

          @delphinis
          1.) Du kannst keinen Trigger in eine Funktion packen.

          2.)

          bis alle Werte eingetrudelt sind

          Wann soll das sein? Meinst Du die 500 ms? Das kann passen, muss aber nicht. Kommt drauf an auf was Du da warten willst.

          Wenn Dein Ergebnis aus 5 Werten besteht, solltest Du auf alle 5 DP triggern.
          Bei jeder einzelnen Änderung ergibt sich ein anderes Ergebnis.

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

            @delphinis sagte in asynch funktion:

            dies liefert aber den Fehler:

            Naja dein await wait(500); ist ja auch nicht Teil der Funktion Eigenverbrauch, sondern wird im Callback aufgerufen (die anonyme Funktion des Triggers), welche nicht als async definiert ist.

            Aber wie @Codierknecht schon sagte, würdest Du so mit jedem Funktionsaufruf eh einen neuen Trigger erstellen. Das ist keine gute Idee. Du suchst ggf. sowas:

            if (!existsState('0_userdata.0.Solar.Leistung.Eigenverbrauch')) {
                createState('0_userdata.0.Solar.Leistung.Eigenverbrauch', {type: 'number', read: true, write: true});
            }
            
            on('modbus.0.inputRegisters.30775_Messwert_Leistung', async (data) => {
                await wait(500);
            
                const WR = getState('modbus.0.inputRegisters.30775_Messwert_Leistung').val;
                const Bezug = getState('modbus.0.inputRegisters.30865_Leistung_Bezug').val;
                const Einspeisung = getState('modbus.0.inputRegisters.30867_Messwert_Leistung_Einspeisung').val;
                const BatLad = getState('modbus.0.inputRegisters.31393_Batterieladung').val;
                const BatEntl = getState('modbus.0.inputRegisters.31395_Batterieentladung').val;
            
                const eigenverbrauch = WR + Bezug - Einspeisung -BatLad + BatEntl;
                setState('0_userdata.0.Solar.Leistung.Eigenverbrauch', eigenverbrauch, true);
            });
            
            1 Reply Last reply Reply Quote 1
            • paul53
              paul53 @Delphinis last edited by paul53

              @delphinis sagte: warten, bis alle Werte eingetrudelt sind.

              Dann nimm setTimeout().

              if (!existsState('0_userdata.0.Solar.Leistung.Eigenverbrauch')) {
                  createState('0_userdata.0.Solar.Leistung.Eigenverbrauch', {type: 'number', read: true, write: false, role: 'value'});
              }
              
              on({id: 'modbus.0.inputRegisters.30775_Messwert_Leistung'}, (data) => {
                  const WR = data.state.val;
                  setTimeout(function() {
                      const Bezug = getState('modbus.0.inputRegisters.30865_Leistung_Bezug').val;
                      const Einspeisung = getState('modbus.0.inputRegisters.30867_Messwert_Leistung_Einspeisung').val;
                      const BatLad = getState('modbus.0.inputRegisters.31393_Batterieladung').val;
                      const BatEntl = getState('modbus.0.inputRegisters.31395_Batterieentladung').val;
               
                      const eigenverbrauch = WR + Bezug - Einspeisung - BatLad + BatEntl;
                      setState('0_userdata.0.Solar.Leistung.Eigenverbrauch', eigenverbrauch, true);
                  }, 500);
              });
              
              1 Reply Last reply Reply Quote 1
              • Codierknecht
                Codierknecht Developer Most Active last edited by Codierknecht

                Mal eben so hingeschrieben:

                const ids = ['modbus.0.inputRegisters.30775_Messwert_Leistung',
                             'modbus.0.inputRegisters.30865_Leistung_Bezug',
                             'modbus.0.inputRegisters.30867_Messwert_Leistung_Einspeisung',
                             'modbus.0.inputRegisters.31393_Batterieladung',
                             'modbus.0.inputRegisters.31395_Batterieentladung'];
                
                if (!existsState('0_userdata.0.Solar.Leistung.Eigenverbrauch')) {
                    createState('0_userdata.0.Solar.Leistung.Eigenverbrauch', {type: 'number', read: true, write: true});
                }
                
                on({id: ids, change: "ne"}, function() {
                    const WR = getState('modbus.0.inputRegisters.30775_Messwert_Leistung').val;
                    const Bezug = getState('modbus.0.inputRegisters.30865_Leistung_Bezug').val;
                    const Einspeisung = getState('modbus.0.inputRegisters.30867_Messwert_Leistung_Einspeisung').val;
                    const BatLad = getState('modbus.0.inputRegisters.31393_Batterieladung').val;
                    const BatEntl = getState('modbus.0.inputRegisters.31395_Batterieentladung').val;
                
                    let eigenverbrauch = (WR + Bezug - Einspeisung - BatLad + BatEntl);
                    setState('0_userdata.0.Solar.Leistung.Eigenverbrauch', eigenverbrauch, true);
                });
                

                Jede Änderung an einem der 5 Werte führt zu einem anderen Ergebnis.
                Also würde ich auf alle 5 triggern.

                @haus-automatisierung @paul53
                Wenn da etwas nicht passen sollte, bitte korrigieren.

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

                  @codierknecht sagte: Jede Änderung an einem der 5 Werte führt zu einem anderen Ergebnis.

                  Das ist das Problem: Es muss gewartet werden, bis der letzte der zusammen gehörenden Werte aktualisiert wurde.

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

                    @paul53
                    Werden die bei Modbus garantiert innerhalb einer bestimmte Zeitspanne alle aktualisiert?
                    Wenns mal 600 ms sind, passt der Kram ja auch nicht mehr.

                    "Isch aabe gar keine Modbus" 😁

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

                      @codierknecht sagte: "Isch aabe gar keine Modbus"

                      Ich auch nicht.

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

                        @codierknecht sagte in asynch funktion:

                        Werden die bei Modbus garantiert innerhalb einer bestimmte Zeitspanne alle aktualisiert?

                        Normalerweise werden die Register ja der Reihe nach abgefragt. Das heißt, dass das letzte gelesene Register die Logik ausführen sollte. Dann sollten die davor ja alle aktualisiert worden sein.

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

                          @haus-automatisierung

                          werden die Register ja der Reihe nach abgefragt

                          Das würde bedeuten, dass man das auch selbst in der Hand hat.
                          Triggert man auf den state, der als letztes gelesen wird, braucht's auch keinen timeout.

                          Werde ich mich wohl demnächst auch mal mit beschäftigen müssen.
                          Meine PV kommt Montag 😉

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

                            @codierknecht
                            @paul53
                            Ihr habt ja alle recht, dass das nicht super akkurat ist, mit den 500ms. Aber die Werte kommen bei mir innerhalb von 120ms +/-20ms rein und da ich bin ja nur im Lan und nicht im Internet. Aber klar, garantiert ist das natürlich nicht und es könnte sein, dass das mal falsch ist. Mit dieser Ungenauigkeit könnte ich aber leben.
                            Ich möchte aber eben auch nicht alle 10 Sekunden 5 verschiedene Werte, welche alle fast gleichzeitig reinkommen. Das ist auch nicht akkurat für eine Statistik und füllt mir nur den Speicher.

                            Vielleicht gibt es ja etwas was nur einmal triggert, nachdem alle fünf reingekommen sind?

                            Aber im Moment übernehme ich mal die Version von @haus-automatisierung. Die funktioniert schon mal recht gut. Die von @paul53 würde wahrscheinlich auch tun.

                            Das Problem ist halt, dass nicht alle States immer reinkommen, wenn es keine Änderung gibt, weshalb ich auf diesen (in diesem Fall der Erste) State triggern will. Dies wiederum wiederspricht dann auch meiner obigen Überlegung einen Trigger auf alle fünf zu legen.

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

                              @delphinis

                              Wenns unbedingt Async sein soll müsste es so gehen (ohne Garantie)

                              Beim ersten Durchlauf passiert jedoch nichts, weil erst der Datenpunkt erstellt wird.

                              on({id:'modbus.0.inputRegisters.30775_Messwert_Leistung', change: "any"}, function (dp) {
                                  Eigenverbrauch();
                              });
                              
                              async function Eigenverbrauch () {
                                  try {
                              
                                      if (!existsState('0_userdata.0.Solar.Leistung.Eigenverbrauch')) {
                                      createState('0_userdata.0.Solar.Leistung.Eigenverbrauch', {type: 'number', read: true, write: true});
                                      } else {
                               
                                          const WR = await getStateAsync('modbus.0.inputRegisters.30775_Messwert_Leistung');
                                          const Bezug = await getStateAsync('modbus.0.inputRegisters.30865_Leistung_Bezug');
                                          const Einspeisung = await getStateAsync('modbus.0.inputRegisters.30867_Messwert_Leistung_Einspeisung');
                                          const BatLad = await getStateAsync('modbus.0.inputRegisters.31393_Batterieladung');
                                          const BatEntl = await getStateAsync ('modbus.0.inputRegisters.31395_Batterieentladung');
                              
                                          let eigenverbrauch = (WR.val + Bezug.val - Einspeisung.val -BatLad.val + BatEntl.val);            
                                          await setStateAsync('0_userdata.0.Solar.Leistung.Eigenverbrauch', eigenverbrauch);
                              
                                      }    
                                  } catch (error) {
                                  log(`Unexpected error - ${error}`, 'error');
                                  }
                              }
                              
                              haus-automatisierung 1 Reply Last reply Reply Quote 0
                              • haus-automatisierung
                                haus-automatisierung Developer Most Active @Peoples last edited by

                                @peoples Na wozu dann noch die anonyme Funktion?

                                Statt

                                on({id:'modbus.0.inputRegisters.30775_Messwert_Leistung', change: "any"}, function (dp) {
                                    Eigenverbrauch();
                                });
                                 
                                async function Eigenverbrauch () {
                                

                                kannst dann auch gleich

                                on({id:'modbus.0.inputRegisters.30775_Messwert_Leistung', change: "any"}, Eigenverbrauch);
                                 
                                async function Eigenverbrauch () {
                                

                                schreiben. Aber wäre dann wieder fast 1:1 identisch zu meiner Version oben - nur dass die Funktion halt nicht anonym ist...

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

                                  @haus-automatisierung

                                  du liest die Werte doch nach einer gewissen zeit ein, ich beziehe sie async. D.H. eigentlich wenn die Werte bei dir nicht innerhalb von 500ms zur Verfügung stehen fehlen sie, bei mir wird gewartet bis sie vorhanden sind. Oder habe ich da einen Denkfehler drin?

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

                                    @delphinis sagte: nicht alle States immer reinkommen, wenn es keine Änderung gibt,

                                    Deshalb sollte auf "Aktualisierung" getriggert werden. Der Modbus-Adapter aktualisiert hoffentlich Werte, die 0 sind?

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

                                      @paul53 said in asynch funktion:

                                      Deshalb sollte auf "Aktualisierung" getriggert werden. Der Modbus-Adapter aktualisiert hoffentlich Werte, die 0 sind?

                                      Ja stimmt, das kann man einstellen.

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

                                        @peoples sagte: bei mir wird gewartet bis sie vorhanden sind.

                                        Werte sind immer vorhanden, was nicht bedeutet, dass sie zeitlich zusammen gehören.

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

                                          @paul53

                                          Da hast du auch wieder recht

                                          D 1 Reply Last reply Reply Quote 0
                                          • D
                                            Delphinis @Peoples last edited by Delphinis

                                            @peoples
                                            @Codierknecht
                                            @haus-automatisierung
                                            @paul53
                                            So, habe es jetzt so realisiert: Ich triggere auf die wichtigsten Ereignisse, lasse das Script aber nach dem Singleton ansatz nur einmal laufen. So hab ich aktuelle Werte, ohne dass das Script zig mal durchläuft. Die Zeit von 500ms sollte dazu dicke reichen, und wenn das einmal nicht klappen sollte, ist es auch egal. Aber ich bin ja nicht im Internet wo die Packete über zig ruoter rüber müssen. So glaube ich dass das für 99.9% aller Fälle reicht. Und wenn nicht, ist es auch kein Unglück.
                                            Jedenfalls kommen die Werte jetzt korrekt verrechnet und immer nur einer. Die Debug infos nehm ich natürlich wieder raus.

                                            if (!existsState('0_userdata.0.Solar.Leistung.Eigenverbrauch')) {
                                            createState('0_userdata.0.Solar.Leistung.Eigenverbrauch', {type: 'number', read: true, write: true});
                                            }
                                            
                                            var count = 0;
                                            var count2 = 0;
                                            var working = false;
                                            const ids = ['modbus.0.inputRegisters.30775_Messwert_Leistung',     // parameter auf die getriggert werden
                                                         'modbus.0.inputRegisters.30865_Leistung_Bezug',
                                                         'modbus.0.inputRegisters.31395_Batterieentladung',
                                                         'modbus.0.inputRegisters.30867_Messwert_Leistung_Einspeisung'];    
                                            
                                            
                                            // On change
                                            //on('modbus.0.inputRegisters.30775_Messwert_Leistung', async (data) => {
                                            on(ids, async (data) => {
                                                count+=1;
                                                if (working == false) {
                                                    working = true;
                                                    count2+=1;
                                                    const now1 = new Date();
                                                    const minutes1 = now1.getMinutes();
                                                    const seconds1 = now1.getSeconds();
                                                    const millis1 = now1.getMilliseconds();
                                            
                                                    await wait(500);
                                            
                                                    const WR = getState('modbus.0.inputRegisters.30775_Messwert_Leistung').val;
                                                    const Bezug = getState('modbus.0.inputRegisters.30865_Leistung_Bezug').val;
                                                    const Einspeisung = getState('modbus.0.inputRegisters.30867_Messwert_Leistung_Einspeisung').val;
                                                    const BatLad = getState('modbus.0.inputRegisters.31393_Batterieladung').val;
                                                    const BatEntl = getState('modbus.0.inputRegisters.31395_Batterieentladung').val;
                                            
                                                    let eigenverbrauch = (WR + Bezug - Einspeisung - BatLad/* + BatEntl*/);
                                                    setState('0_userdata.0.Solar.Leistung.Eigenverbrauch', eigenverbrauch, true);
                                            
                                                    const now2 = new Date();
                                                    const minutes2 = now2.getMinutes();
                                                    const millis2 = now2.getMilliseconds();
                                                    const seconds2 = now2.getSeconds();
                                                    console.log(count + ' ' + count2 + ` start:${minutes1}:${seconds1}:${millis1}. ` + `time:${minutes2}:${seconds2}:${millis2} - ` + eigenverbrauch);
                                                    working = false;
                                                }
                                            });
                                            
                                            

                                            Danke an alle, ich habe wieder einiges gelernt 🙂

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

                                            Support us

                                            ioBroker
                                            Community Adapters
                                            Donate

                                            977
                                            Online

                                            31.8k
                                            Users

                                            80.0k
                                            Topics

                                            1.3m
                                            Posts

                                            6
                                            29
                                            818
                                            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