Navigation

    Logo
    • Register
    • Login
    • Search
    • Recent
    • Tags
    • Unread
    • Categories
    • Unreplied
    • Popular
    • GitHub
    • Docu
    • Hilfe
    1. Home
    2. Deutsch
    3. Skripten / Logik
    4. JavaScript
    5. scriptedEnabled - Problem bei Abfrage

    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

    scriptedEnabled - Problem bei Abfrage

    This topic has been deleted. Only users with topic management privileges can see it.
    • liv-in-sky
      liv-in-sky @paul53 last edited by

      @paul53

      das ist schade, das konzept dieses scriptes sollte sein - eine kleine , 2 mal am tag upgedatete , liste in der vis zu haben, die mir nur scripte anzeigt, die laufen und einen falschen trigger haben

      danke für deine hilfe

      paul53 1 Reply Last reply Reply Quote 0
      • liv-in-sky
        liv-in-sky last edited by

        @apollon77

        hi ingo - du stehst in github als entwickler drin - weißt du, wieso das so ist - siehe bitte im ersten post ?

        zusammnegefaßt: selektor bekommt falsche states im script_enabled ordner beim prüfen, ob scripte enable/disabled sind

        AlCalzone 1 Reply Last reply Reply Quote 0
        • paul53
          paul53 @liv-in-sky last edited by paul53

          @liv-in-sky sagte:

          das ist schade

          Es funktioniert mit der asynchronen Version von getState(id, callback), ist also ein Puffer-Problem:

          const idsEnabled = $('javascript.*.scriptEnabled.aktiv.*');
          idsEnabled.each(function(id, i) {
              if(existsState(id)) getState(id, function(err, state) {log(id + ': ' + state.val)});
          });
          
          1 Reply Last reply Reply Quote 0
          • AlCalzone
            AlCalzone Developer @liv-in-sky last edited by

            @liv-in-sky Welche Adapter-Version setzt du denn ein? Wir haben vor kurzem das Verhalten von Wildcards im Selektor gefixt - ggf. hat das damit zu tun.

            liv-in-sky paul53 2 Replies Last reply Reply Quote 0
            • liv-in-sky
              liv-in-sky @AlCalzone last edited by

              @AlCalzone

              momentan 4.8.4

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

                @AlCalzone sagte:

                Wir haben vor kurzem das Verhalten von Wildcards im Selektor gefixt - ggf. hat das damit zu tun.

                Nein, damit hat es nichts zu tun, denn die IDs werden richtig selektiert. Die synchrone Version von getState(id) funktioniert bei diesen Datenpunkten nicht.

                liv-in-sky 1 Reply Last reply Reply Quote 0
                • liv-in-sky
                  liv-in-sky @paul53 last edited by liv-in-sky

                  @paul53

                  was ist hier falsch

                  function lookForEnabled(pfad) {
                    let myEnabled=false;
                    let myselctor="javascript.*.scriptEnabled."+pfad
                    $(myselctor).each(function(id, i) {   
                  
                      //if(existsState(id)) {if (getState(id).val) {myEnabled=true;}}
                      if(existsState(id)) {  getState(id, function(err, state){ if (state.val) { myEnabled=true; 
                                                                                                 return myEnabled;}})
                      }
                  });
                  }
                  

                  ich nehme an, der return wert darf da nicht drin sein - wenn ich den aber rausnehme funktioniert es nicht ?

                  paul53 1 Reply Last reply Reply Quote 0
                  • paul53
                    paul53 @liv-in-sky last edited by

                    @liv-in-sky sagte:

                    was ist hier falsch

                    Man kann keine Rückgabe aus einer Callback-Funktion an die aufrufende Funktion machen, denn die Callback-Funktion läuft asynchron.

                    liv-in-sky 1 Reply Last reply Reply Quote 0
                    • liv-in-sky
                      liv-in-sky @paul53 last edited by

                      @paul53
                      leidr weiß ich nicht, wie ich mit dieser function umgehen muss - letztlich gibt es doch ein promise - wenn das erfüllt/ausgeführt ist, soll der return gegeben werden

                      habe mit then() etwas versucht - klappt aber garnicht

                      AlCalzone 1 Reply Last reply Reply Quote 0
                      • AlCalzone
                        AlCalzone Developer @liv-in-sky last edited by AlCalzone

                        @liv-in-sky Probiers mal so:

                        async function lookForEnabled(pfad) {
                          let myselctor = "javascript.*.scriptEnabled." + pfad;
                          for (const id of $(myselctor)) {
                            if (existsState(id) && (await getStateAsync(id)).val) return true;
                          }
                          return false;
                        }
                        

                        Dazu musst du aber auf 4.9.4 updaten, da die das Iterieren des Selector-Ergebnisses ermöglicht.
                        Und du musst das aus der aufrufenden Funktion ebenfalls mit await aufrufen, oder das Ergebnis mit .then(...) verarbeiten.

                        liv-in-sky AlCalzone 4 Replies Last reply Reply Quote 0
                        • liv-in-sky
                          liv-in-sky @AlCalzone last edited by

                          @AlCalzone

                          jetzt sind alle true 😞

                          Image 4.png

                          1 Reply Last reply Reply Quote 0
                          • liv-in-sky
                            liv-in-sky @AlCalzone last edited by

                            @AlCalzone

                            sorry - das ist ein fehler im verarbeiten nach dem aufruf der function !!!!

                            muss noch testen

                            1 Reply Last reply Reply Quote 0
                            • liv-in-sky
                              liv-in-sky @AlCalzone last edited by

                              @AlCalzone

                              so wird die function jetzt aufgerufen:

                              lookForEnabled(myGoal[b][1]) ? myEnabled="⚠️(aktiv)" : myEnabled="⚫" 
                              
                              
                              async function lookForEnabled(pfad) {
                                pfad=pfad.replace(".js","")   
                                pfad=pfad.replace(/\//g,".")  
                               // let scriptPfad=""
                                //let myEnabled=false;
                              
                                
                                let myselctor="javascript.*.scriptEnabled."+pfad
                                for (const id of $(myselctor)) {
                                    log((await getStateAsync(id)).val+"  "+id)
                                  if (existsState(id) && (await getStateAsync(id)).val) {return true;}
                                }
                                return false;
                              }
                              

                              das log in deiner function wird richtig angezeigt

                              muss ich lookForEnabled anders aufrufen

                              bei

                                log(lookForEnabled(myGoal[b][1]).toString())
                              

                              kommt das:

                              Image 6.png

                              1 Reply Last reply Reply Quote 0
                              • AlCalzone
                                AlCalzone Developer @AlCalzone last edited by AlCalzone

                                @liv-in-sky sagte in scriptedEnabled - Problem bei Abfrage:

                                muss ich lookForEnabled anders aufrufen

                                Ja, hab ich doch geschrieben:

                                @AlCalzone sagte in scriptedEnabled - Problem bei Abfrage:

                                Und du musst das aus der aufrufenden Funktion ebenfalls mit await aufrufen, oder das Ergebnis mit .then(...) verarbeiten.

                                z.B.

                                async function whatever() {
                                const result = await lookForEnabled("meine-id");
                                log(result);
                                }
                                

                                Ggf. empfiehlt sich die Lektüre von https://gist.github.com/AlCalzone/d14b854b69ce5e8a03718336cc650a95#

                                1 Reply Last reply Reply Quote 0
                                • paul53
                                  paul53 @liv-in-sky last edited by paul53

                                  @liv-in-sky sagte:

                                  muss ich die abfrage irgendwie anders programmieren ?

                                  Weshalb verwendest Du nicht die synchrone Funktion isScriptActive(name) ?

                                  function lookForEnabled(pfad) {
                                    pfad=pfad.replace(".js","")   
                                    pfad=pfad.replace(/\//g,".")  
                                    return isScriptActive(pfad);
                                  }
                                  
                                  liv-in-sky 1 Reply Last reply Reply Quote 0
                                  • liv-in-sky
                                    liv-in-sky @paul53 last edited by

                                    @paul53

                                    bin vor lauter frust in den regen raus und habe die blätter vom garten entsorgt - dieses await ... bekomme ich einfach nicht hin

                                    habe den link von @AlCalzone 20mal gelesen und das ganze script mit funktionen gemacht und wollte am ende eine .then kette machen

                                    die function kannte ich nicht - werd die sofort mal testen

                                    paul53 AlCalzone 2 Replies Last reply Reply Quote 0
                                    • paul53
                                      paul53 @liv-in-sky last edited by paul53

                                      @liv-in-sky sagte:

                                      die function kannte ich nicht

                                      Ich auch nicht: Ist erst seit 9. Oktober dokumentiert.

                                      1 Reply Last reply Reply Quote 0
                                      • AlCalzone
                                        AlCalzone Developer @liv-in-sky last edited by

                                        @liv-in-sky sagte in scriptedEnabled - Problem bei Abfrage:

                                        bin vor lauter frust in den regen raus und habe die blätter vom garten entsorgt - dieses await ... bekomme ich einfach nicht hin

                                        Auch wenn es eine einfachere Lösung gibt, vielleicht zeigst du mal was du hattest. So schwer ist es eigentlich nicht.
                                        Wenn ne Funktion einen Promise zurückgibt, rufst du sie mit await davor auf. Dazu muss die Funktion, in der await verwendet wird, async sein. Ansonsten wird der Code geschrieben wie synchroner Code.

                                        liv-in-sky 1 Reply Last reply Reply Quote 1
                                        • liv-in-sky
                                          liv-in-sky @AlCalzone last edited by liv-in-sky

                                          @AlCalzone

                                          erst mal danke für deine hilfe für dein angebot - darauf komme ich noch zurück - brauch jetzt erstmal eine pause
                                          bei der anderen version, hatte ich das problem, dass ich eine function hatte, in der ich mit await getStateAsync(id)).val das enabled suche - dann wollte ich aber, dass diese function ein ein promise bringt, wenn sie fertig ist und dann erst der datenpunkt geschrieben wird - ich konnte kein new promise einfügen, weil dann await getStateAsync(id)).val einen fehler anzeigte

                                          ung. so:

                                           asyncfunction writeStuff() {
                                          return new Promise((resolve) => {......
                                                                         ....... await    lookForEnabled(pfad)                                        
                                           resolve()
                                          

                                          hier kam ein fehler und await war nicht mehr erlaubt

                                          meine jetzige lösung sieht so aus (hoffe, die ist richtig):

                                          sammleDaten().then(writeStuff).then( function() { setState("javascript."+instance+".CheckScript.jsonSubScript",JSON.stringify(mxJson))
                                                                                         log(JSON.stringify(mxJson))})
                                          

                                          sammleDaten (liest scripte aus filesystem und versucht die trigger daraus zu finden - schreibt ein arr mit gefundenen non-existent datenpunkten) und writeStuff (checkt script_enabled und macht json für vis-datenpunkt )- beide functionen haben ein promise bekommen und ein resolve

                                          damit der datenpunkt zum richtigen zeitpunkt geschrieben wird

                                          @paul53 - die function hat das ganze extrem vereinfacht - nochmals danke dafür

                                          ergebnis schaut richtig aus:

                                          Image 8.png

                                          AlCalzone 1 Reply Last reply Reply Quote 0
                                          • AlCalzone
                                            AlCalzone Developer @liv-in-sky last edited by AlCalzone

                                            @liv-in-sky sagte in scriptedEnabled - Problem bei Abfrage:

                                            async function writeStuff() {
                                            return new Promise((resolve) => {......

                                                                        ....... await    lookForEnabled(pfad)                                        
                                            

                                            resolve()

                                            Zwei Fehler hast du hier:

                                            1. Eine async-Funktion braucht nicht explizit einen Promise zurückgeben. In 99% der Fälle ist es sogar einfach nur falsch. Das was ich oben sehe, kannst du wie folgt eindampfen:
                                            async function writeStuff() {
                                              // ...
                                              await lookForEnabled(pfad);
                                            }
                                            
                                            1. Wenn es unbedingt sein muss, await doch in einem Promise zu kapseln (z.B. weil du zusätzlich eine Callback-API nutzt), muss die Promise-"Callback"-Funktion als async definiert werden, nicht die außenrum:
                                            function foobar() {
                                              return new Promise(async (resolve) => {
                                                await irgendwas;
                                                irgendwasMitCallback(() => resolve());
                                              });
                                            });
                                            

                                            Meistens lässt sich das aber eleganter trennen.


                                            meine jetzige lösung sieht so aus

                                            Würde ich in etwa so machen (besser lesbar und mischt nicht async/await mit .then()-Ketten):

                                            async function doStuff() {
                                            	await sammleDaten();
                                            	await writeStuff();
                                            	// ^ Hier sollten vermutlich Daten zwischen den Methoden übergeben werden,
                                            	// damit keine globalen Variablen missbraucht werden
                                            	// const daten = await sammleDaten();
                                            	// await writeStuff(daten)
                                            	const stringified = JSON.stringify(mxJson);
                                            	await setStateAsync("javascript."+instance+".CheckScript.jsonSubScript", stringified);
                                            	log(stringified);
                                            }
                                            
                                            liv-in-sky 3 Replies Last reply Reply Quote 0
                                            • First post
                                              Last post

                                            Support us

                                            ioBroker
                                            Community Adapters
                                            Donate

                                            416
                                            Online

                                            31.8k
                                            Users

                                            80.0k
                                            Topics

                                            1.3m
                                            Posts

                                            javascript
                                            4
                                            51
                                            2444
                                            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