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] Elegantere Programmierung?

    NEWS

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

    • ioBroker goes Matter ... Matter Adapter in Stable

    • Monatsrückblick - April 2025

    [gelöst] Elegantere Programmierung?

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

      Habe mir zwei Scripte geschrieben. Ziel der Scripte ist es:

      Nur, wenn ich NICHT im Urlaub bin, soll bei ABwesenheit der Boiler UND die Boiler Zeitsteuerung ausgeschaltet werden. Bin ich ANwesend soll der Boiler wieder einschalten und auch die Zeitsteuerung wieder aktiv werden.

      Script einaus bei AN/ ABwesenheit:

      // ###########################################
      //            Deklarationen
      // ###########################################
      
      const IDAnwesend = 'hm-rega.0.39533'/*Anwesend*/;
      
      // Datenpunkt erstellen (für andere Scripte);
      createState('BenutzerVariablen.BoilerProgrammSchalter', '', {
          name: 'Boiler AnAus',
          type: 'boolean'
      });
      
      // ProgrSchalter standardmaessig auf true (damit Zeitsteuerung aktiv)
          const ProgrSchalter = 'javascript.0.BenutzerVariablen.BoilerProgrammSchalter'/*Boiler AnAus*/;
          setState(ProgrSchalter, true);
      
      // ##################################
      //              Programm
      // ##################################
      
      on({id: IDAnwesend, change: 'ne'}, function () {
          const IDBoiler = 'hm-rpc.0.LEQ0197823.1.STATE'/*Zw -Stecker Küche:1 STATE*/;
          const IDUrlaub = 'hm-rega.0.8359'/*URLAUB Hzg  Wohnb  ALLES (Var )*/;
          
          // wenn ABwesend und Urlaub AUS (nur prüfen) , dann Boiler AUS und Zeitsteuerung Boiler AUS;
          if(getState(IDUrlaub).val == 0 && getState(IDAnwesend).val == false) {    
          setState(IDBoiler,false);
          setState(ProgrSchalter, false);
          };
      
          // wenn ANwesend und Urlaub AUS (nur prüfen) , dann Boiler EIN Zeitsteuerung Boiler EIN;
          if(getState(IDUrlaub).val == 0  && getState(IDAnwesend).val == true) {    
          setState(IDBoiler,true);
          setState(ProgrSchalter, true);
          };
      });
      

      Programm Zeitsteuerung:

      // ###########################################
      //            Deklarationen
      // ###########################################
      
      const IDZwBoilerKueche = 'hm-rpc.0.LEQ0197823.1.STATE'/*Zw -Stecker Küche:1 STATE*/;
      const BoilerSchalter = 'javascript.0.BenutzerVariablen.BoilerSchalter'/*Boiler AnAus*/;
      
      // ##################################
      //              Programm
      // ##################################
      
          schedule({hour: 6, minute: 30}, function () {
          //Wird um 06:30 eingeschaltet, wenn der Boilerschalter true;
          if(getState(BoilerSchalter).val == true) { 
          setState(IDZwBoilerKueche, true);
          };
          });
      
          schedule({hour: 22, minute: 30}, function () {
          //Wird um 22:30 AUSgeschaltet, wenn der Boilerschalter true;
          if(getState(BoilerSchalter).val == true) { 
          setState(IDZwBoilerKueche, false);
          };
          });
      

      Mich interessiert ob das so funktioniert. Mir ist natürlich bewußt, dass das viel "eleganter" geschrieben werden kann (oder sogar muß). Im ersten Schritt geht es mir jedoch nur um die richtige Funktionalität. Könnt ihr mal drüber schauen? Ich teste es prarallel im Realbetrieb.

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

        @skorpil
        Es fehlt ein Trigger auf IDUrlaub. Was soll im Urlaub anders laufen als sonst?
        IDs sollte man außerhalb von Funktionen (am Anfang) deklarieren und initialisieren.
        Beide Skripte lassen sich zu einem Skript zusammenfassen, was doppelte Deklarationen erspart.
        Eigene Datenpunkte erstellt man unter "0_userdata.0".

        Im ersten Skript wird der DP "javascript.0.BenutzerVariablen.BoilerProgrammSchalter" im zweiten der DP "javascript.0.BenutzerVariablen.BoilerSchalter" verwendet.

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

          @paul53 sagte in [gelöst] Elegantere Programmierung?:

          Im ersten Skript wird der DP "javascript.0.BenutzerVariablen.BoilerProgrammSchalter" im zweiten der DP "javascript.0.BenutzerVariablen.BoilerSchalter" verwendet.

          Dankeschön. Du bist ein Luchs Ich hatte diesen nachher noch einmal umbenannt.

          Im Urlaub schalte ich den Boiler (und weitere Geräte) in weiteren Scripts komplett aus. Und die Variable "Urlaub" wird auf wahr gesetzt.

          ID's am Anfang definieren, also besser so?

          const IDBoiler = 'hm-rpc.0.LEQ0197823.1.STATE'/*Zw -Stecker Küche:1 STATE*/;
          const IDUrlaub = 'hm-rega.0.8359'/*URLAUB Hzg  Wohnb  ALLES (Var )*/;
          
          on({id: IDAnwesend, change: 'ne'}, function () {
          

          Ich bin ja schon mal riesig froh, dass ich wenigstens die Funktionalität richtig hinbekommen habe.

          Ja, das könnte man auch in einem Script zusammenfassen. Für mich ist das aver so übersichtlicher.

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

            @skorpil sagte: also besser so?

            Ja, alle IDs. Konstanten muss man ohnehin vor ihrer Verwendung deklarieren.

            const IDBoiler = 'hm-rpc.0.LEQ0197823.1.STATE'/*Zw -Stecker Küche:1 STATE*/;
            const IDUrlaub = 'hm-rega.0.8359'/*URLAUB Hzg  Wohnb  ALLES (Var )*/;
            const IDAnwesend = 'hm-rega.0.39533'/*Anwesend*/;
             
            on({id: IDAnwesend, change: 'ne'}, function () {
            
            S 1 Reply Last reply Reply Quote 1
            • S
              skorpil @paul53 last edited by

              @paul53 Danke Dir sehr

              Eigene Datenpunkte erstellt man unter "0_userdata.0".. Dazu der Hinweis: irgendwann hatte mir pix mal beim Scripten unter die Arme gegriffen. Und da ist dann dieser Folder entstanden. Was meinst Du, sollte ich das in allen Scripten ändern? Vor- und Nachteile? Bisher stehen diese "userdata" alle in Verzeichnissen unter "javascrip".

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

                @skorpil sagte: Bisher stehen diese "userdata" alle in Verzeichnissen unter "javascrip".

                Wenn es schon viele eigene Datenpunkte sind, dann lass es besser so und bleibe dabei.

                S 1 Reply Last reply Reply Quote 1
                • S
                  skorpil @paul53 last edited by

                  @paul53 ich hänge gerade wieder mal und bruache Hilfe:

                  ich möchte eine Aktion auslösen, aber nur nachts. Konkret, wenn der Bewegungsmelder eine Bewegung - aber eben nur nachts - detektiert, dann soll das Licht angehen.

                  Geht das mit schedule (Astrofunktion nur nachts) und einer if abfrage? Aber wie müßte das geschachtelt werden? Ich komme nicht dahinter! Danke für Hilfe.

                  Gruß Bernd52 😁

                  haus-automatisierung paul53 2 Replies Last reply Reply Quote 0
                  • haus-automatisierung
                    haus-automatisierung Developer Most Active @skorpil last edited by

                    @skorpil Du kannst den Trigger nicht auf nachts einschränken. Du musst im Trigger bei der Bewegung prüfen, ob gerade Nacht ist.

                    S 1 Reply Last reply Reply Quote 1
                    • paul53
                      paul53 @skorpil last edited by paul53

                      @skorpil sagte: Bewegung - aber eben nur nachts

                      Innerhalb des Bewegungstriggers muss "nacht" abgefragt werden.
                      Dafür liefert der Javascript-Adapter den Standard-Datenpunkt "javascript.0.variables.isDayTime", dessen Verhalten in der JS-Instanz-Konfiguration vorgegeben wird.

                      JS_Tageszeit.JPG

                      const IDAnwesend = 'hm-rega.0.39533'/*Anwesend*/;
                      const idTag = 'javascript.0.variables.isDayTime';
                      
                      on({id: IDAnwesend, change: 'ne'}, function () {
                          if(!getState(idTag).val) {
                              // Aktionen bei Bewegung nachts
                          }
                      });
                      
                      S 1 Reply Last reply Reply Quote 1
                      • S
                        skorpil @paul53 last edited by

                        @paul53 Dankeschön. Das hilft.

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

                          @haus-automatisierung merci

                          1 Reply Last reply Reply Quote 0
                          • S
                            skorpil last edited by skorpil

                            ich brauche bitte nochmal eine Erklärung, weil ich offenbar die ON Funktion noch nicht richtig verstanden habe resp. werde ich aus der Beschreibung "ioBroker/ioBroker.javascript" nicht wirklich schlau. Genauer gefragt, welche Rolle spielt in der ON Funktion der Teil "function" ?

                            on({id: IDAnwesend, val: false} , function (data) {
                            

                            Hier kommt hinter function "data". Und hier:

                            on({id: IDAnwesend, change: 'ne'}, function () {
                            

                            ist der Klammerausdruck leer. Was ist der Unterschied? Wann muß da data rein und wann kann die Klammer leer bleiben.

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

                              @skorpil sagte in [gelöst] Elegantere Programmierung?:

                              ist der Klammerausdruck leer. Was ist der Unterschied? Wann muß da data rein und wann kann die Klammer leer bleiben.

                              Das ist eine Callback-Funktion. Diese nimmt einen Parameter entgegen und enthält ein Objekt, mit dem neuen State. Oben haben wir dafür schon einige Beispiele geliefert. Darin ist z.B. der neue und der vorige Wert des States enthalten, welches den Trigger ausgelöst hat.

                              Wenn Du diese Daten nutzen möchtest, dann musst Du den Parameter natürlich definieren. Wenn Du den eh nicht brauchst, kannst Du den Parameter weg lassen.

                              on({id: IDAnwesend, val: false} , function (data) {
                              

                              ist z.B. das gleiche wie

                              on({id: IDAnwesend} , function (data) {
                                  if (!data.state.val) {
                              

                              Wie Du den nennst, ist total egal. Der hat als Scope dann ja eh nur deine Funktion:

                              on({id: IDAnwesend} , function (blablabla) {
                                  if (!blablabla.state.val) {
                              
                              1 Reply Last reply Reply Quote 1
                              • S
                                skorpil last edited by

                                Ich möchte in der ON Anweisung prüfen, ob state1 ODER state2 true ist. Also, wenn ich das Licht "AussenSchlaf" oder das Lich "HinterTV" einschalte, soll etwas passieren. Das könnte ich ja mit zwei Scripten lösen. Aber ich würde es gerne in einem Script erledigen. Mein Versuch:

                                on({IDLichtschalterAussenSchlaf || IDLichtschalterHinterTV} , true, function () {
                                

                                ist offenbar falsch. Wie wäre die richtige Notation?

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

                                  @skorpil sagte: Licht "AussenSchlaf" oder das Lich "HinterTV" einschalte, soll etwas passieren.

                                  on({id: [IDLichtschalterAussenSchlaf, IDLichtschalterHinterTV], change: 'gt'}, function () { // triggert bei Wertänderung false --> true
                                  
                                  S 1 Reply Last reply Reply Quote 1
                                  • S
                                    skorpil @paul53 last edited by

                                    @paul53 was war an meiner Variante mit den "||" falsch?

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

                                      @skorpil
                                      Laut Doku kein zulässiges Muster.

                                      S 1 Reply Last reply Reply Quote 1
                                      • S
                                        skorpil @paul53 last edited by

                                        @paul53 ...und wie wäre das so gegangen?

                                        u can use following parameters to specify the trigger:
                                        
                                        parameter	type/value	description
                                        logic	string	"and" or "or" logic to combine the conditions (default: "and")
                                        
                                        paul53 1 Reply Last reply Reply Quote 0
                                        • paul53
                                          paul53 @skorpil last edited by

                                          @skorpil sagte in [gelöst] Elegantere Programmierung?:

                                          wäre das so gegangen?

                                          Nein, das bezieht sich auf die verschiedenen Bedingungen.

                                          on({id: ID1, val: true, ack: true}, function() {});
                                          

                                          bedeutet: Wert == true UND Bestätigung == true.
                                          Das kann man ändern in Wert == true ODER Bestätigung == true:

                                          on({id: ID1, logic: 'or', val: true, ack: true}, function() {});
                                          

                                          Habe ich selbst noch nie gemacht.

                                          1 Reply Last reply Reply Quote 1
                                          • S
                                            skorpil last edited by skorpil

                                            in der WebUI der homematic gibt es die Formulierung "wenn" und "sonst".

                                            Ich habe jetzt so einen Fall, wo ich dies einsetzen möchte. Momentan habe ich das in JavaScript mit zwei On Anweisungen in einem Script gelöst:

                                            on({id: IDAussenbeleuchtung , val: true}, function () {
                                            
                                            on({id: IDAussenbeleuchtung , val: false}, function () {
                                            

                                            also den false Fall in einer weiteren ON Anweisungen. Geht das? Ist das so richtig?

                                            Oder besser mit

                                            on({id: IDAussenbeleuchtung , change: 'ne'}, function () {
                                            

                                            und einer folgenden if Abfrage?

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

                                            Support us

                                            ioBroker
                                            Community Adapters
                                            Donate

                                            972
                                            Online

                                            31.7k
                                            Users

                                            79.7k
                                            Topics

                                            1.3m
                                            Posts

                                            10
                                            325
                                            37438
                                            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