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

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

      wäre das in dem Fall nicht einfacher ?

      Also ich persönlich weigere mich, solche Vergleiche auf boolsche Datenpunkte anzuwenden 🙂 Würde ich in der Programmierung ja auch nirgendwo so machen.

      true > false. Ist nicht schön (meine Meinung). Aber funktioniert auch.
      Ob das "einfacher" ist, sei mal dahingestellt 🙂 Ich finde es schwerer zu verstehen.

      Ich programmiere nun über 20 Jahre, aber habe noch nie gesehen, dass jemand mit > false auf true prüft 🙂 Zum Glück...

      Wenn man sich nicht an Typen hält und die bunt mischt, kommen lustige Dinge raus. '3' + 2 ergibt z.B. 32 (als String) und '3' - 2 ergibt 1 (als Number) - siehe "Implicit Conversions". Damit muss man sich dann erstmal auseinandersetzen.

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

        @haus-automatisierung sagte in [gelöst] Elegantere Programmierung?:

        Also ich persönlich weigere mich, solche Vergleiche auf boolsche Datenpunkte anzuwenden Würde ich in der Programmierung ja auch nirgendwo so machen.
        true > false. Ist nicht schön (meine Meinung). Aber funktioniert auch.

        Darum habe ich so meine Probleme mit un- oder nur schwach typisierten Sprachen.
        Strenge Typisierung und ein vernünftiger Compiler/Interpreter machen die Dinge deutlich einfacher und vor allem stabiler.
        Da passieren sonst gerne mal die aberwitzigsten Dinge ... die man vermutlich erst nach stundenlangen Debug-Sessions findet.

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

          @codierknecht dankeschön. Ich meine ja, man müßte die Doku für JavaScript im IObroker mal überarbeiten. Viele dieser Infos sind da in der Form nicht vorhanden (oder habe ich da was übersehen?). Solche Effekte müssten m.E. dort intensiver, eben auch für Dummies wie mich, besser dokumentiert werden.

          Ich bin leider nicht Fachmann genug, um mich an sowas heranzuwagen, wäre aber gerne bereit, zu helfen.

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

            @skorpil sagte: Viele dieser Infos sind da in der Form nicht vorhanden

            Die Doku beschreibt die Funktionen des Javascript-Adapters - nicht die Sprache Javascript.
            Dass true > false ist und implizite Typwandlungen erfolgen, sind Eigenschaften der Programmiersprache Javascript.

            1 Reply Last reply Reply Quote 1
            • Andreios
              Andreios Forum Testing @paul53 last edited by Andreios

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

              @skorpil sagte: war mein Weg total falsch?

              Ja, Zeile 6

              if(!anwesend) break;
              

              bricht die Schleife ab, wenn irgendwer abwesend ist. Wenn niemand anwesend ist, muss die Schleife ohne Abbruch voll durchlaufen werden.

              Also ihr könnt mich ja Korinthenkacker nennen und ich bin auch nur zufällig beim "Überfliegen" des Thread darauf gestossen und habe auch einen ganzen Teil weiterer Beiträge durchsucht, ob ich dazu noch etwas finden, aber nichts gefunden!

              Für mich sieht das genau umgekehrt aus, nämlich dass abgebrochen wird, wenn keiner Anwesend ist!

              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?:

                Ich meine ja, man müßte die Doku für JavaScript im IObroker mal überarbeiten.

                Scroll mal ein stück runter bei deinem Link. Da ist doch eine total ausführliche Tabelle aller Optionen für die on-Funktion. Und da steht auch, dass gt ein Vergleich ist (greater than). Dass sich dass auf den Wert bezieht, sollte klar sein 🙂

                Und dass das z.B. keinen Sinn ergibt, wenn man change: 'gt' auf einen Datenpunkt vom Typ "String" registriert (obwohl es funktioniert), sind Grundlagen von JavaScript (wie @paul53 schon schrieb). Am Ende finden da ja die Vergleiche im JavaScript-Adapter statt, welche dann entscheiden, ob die Callback-Funktion aufgerufen wird, oder nicht.

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

                  @haus-automatisierung das habe ich natürlich gelesen. Aber ich zitiere @Codierknecht „Genau das ist der beschriebene Effekt. Der Trigger feuert bei jeder Aktualisierung. Ob der Wert auch vorher schon true war ist ihm völlig Latte.“ Diese Art von Effekte sind nach meiner Einschätzung in der Doku so nicht klar. Sie ist halt schon seeeehr knapp gehalten.

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

                    @skorpil sagte: Diese Art von Effekte sind nach meiner Einschätzung in der Doku so nicht klar.

                    Man muss schon alles lesen:

                    Notice: Please note, that by default "change" is equal to "any", except when only id as string is set (like on("id", function (){});). In last case change will be set to "ne".

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

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

                      Art von Effekte sind nach meiner Einschätzung in der Doku so nicht klar. Sie ist halt schon seeeehr knapp gehalten.

                      Und die Dokumentation ist ebenfalls Open Source. Pull Requests sind immer willkommen… Pack doch gerne dazu was Dir fehlt oder was nicht ausführlich genug ist!

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

                        Logikfrage: ich hatte mir folgendes Script angelegt

                        on({id: [IDLichtschalterAussenSchlaf, IDLichtschalterHinterTV], change: 'gt'}, function () { 
                        
                            var AussenSchlaf = getState(IDLichtschalterAussenSchlaf).val;
                            var HinterTV = getState(IDLichtschalterHinterTV).val;
                        
                            if(AussenSchlaf == true || HinterTV == true) {
                                setState(IDAussenbeleuchtung, true);
                            };
                            
                            if(AussenSchlaf == false || HinterTV == false) {
                                setState(IDAussenbeleuchtung, false);
                            };
                        
                        });
                        

                        dieses Script hat denfinitiv zu komischen Ergebnissen geführt. Ich habe es dann abgeändert

                        on({id: [IDLichtschalterAussenSchlaf, IDLichtschalterHinterTV], change: 'gt'}, function () { 
                        
                            var AussenSchlaf = getState(IDLichtschalterAussenSchlaf).val;
                            var HinterTV = getState(IDLichtschalterHinterTV).val;
                        
                            if(AussenSchlaf == true || HinterTV == true) {
                                setState(IDAussenbeleuchtung, true);
                            
                            } else {
                                setState(IDAussenbeleuchtung, false);
                            
                            };
                        
                        });
                        

                        also statt der beiden if Abfragen mit If - else gearbeitet. Das funktioniert jetzt.

                        Aber, ich verstehe nicht so recht den Unterschied zwischen beiden Vorgehensweisen. Meine Hypothese: im 1. Script erzeuge ich offenbar eine Schleife. Im 2. nicht. Richtig gedacht?

                        OliverIO paul53 2 Replies Last reply Reply Quote 0
                        • OliverIO
                          OliverIO @skorpil last edited by OliverIO

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

                          die Notation im If wäre besser wie folgt zu schreiben kannst

                          if (AussenSchlaf || HinterTV ) {
                          

                          Ich interpretiere, wenn mindestens einer der Parameter wahr ist, dann soll die Aussenbeleuchtung auf wahr gestellt werden, ansonsten false?

                          Das mit einer Schleife bzw. du meinst evtl eine Rückkoplung der Skripte kann ich nicht erkennen. ausser du hast da noch ein anderes Skript welches mit den gleichen Datenpunkten arbeitet.
                          Wenn du während des entwickelns und testens immer wieder mal ein

                          console.log("mein skript ist hier");
                          

                          oder so ähnlich einfügst, dann kannst du besser die Vorgänge verfolgen, wann was passiert

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

                            @skorpil sagte: Richtig gedacht?

                            Das Gegenteil (else) von

                            a !! b // einer true
                            

                            ist

                            !a && !b // beide false
                            

                            Du hattest aber die untere Bedingung per ODER verknüpft.

                            Auf das Thema bezogen: Eleganter ist anstelle von if ... else

                                setState(IDAussenbeleuchtung, AussenSchlaf || HinterTV);
                            

                            wobei das Licht nie ausgeschaltet wird, da die Triggerbedingung change: "gt" lautet, also nur bei Änderung von false in true getriggert wird. Zum "nur einschalten" würde es auch so ausreichen:

                            on({id: [IDLichtschalterAussenSchlaf, IDLichtschalterHinterTV], change: 'gt'}, function () { 
                                setState(IDAussenbeleuchtung, true);
                            });
                            
                            S 1 Reply Last reply Reply Quote 1
                            • S
                              skorpil @paul53 last edited by

                              @paul53 Unbenannt-1.jpg

                              Um noch einmal klarzustellen, was ich erreichen möchte, habe ich hier einen Screenshot meiner WebUI Programmierung beigefügt. Von diesem Programm hatte ich 2 angelegt, eines für den Schalter HinterTV und eines für den Schalter im Schlafzimmer. Diese schalten die SV "Außenbeleuchtung" ein- und aus.

                              Im JavaScript hier habe ich versucht, die Aktionen beider Schalter in einem Script abzubilden. Aber ich komme vermutlich mit der Logik durcheinander.

                              Wie muß es sein. Ich sehe den Wald vor lauter Bäumen nicht mehr!

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

                                @skorpil sagte: Diese schalten die SV "Außenbeleuchtung" ein- und aus.

                                Wenn ausgeschaltet werden soll, wenn beide Trigger-DP auf false sind:

                                on([IDLichtschalterAussenSchlaf, IDLichtschalterHinterTV], function () {  // triggert bei Wertänderung eines DP
                                    setState(IDAussenbeleuchtung, getState(IDLichtschalterAussenSchlaf).val || getState(IDLichtschalterHinterTV).val);
                                });
                                

                                Wenn aber ausgeschaltet werden soll, wenn eines der Trigger-DP auf false wechselt (entspricht HomeMatic-Version):

                                on([IDLichtschalterAussenSchlaf, IDLichtschalterHinterTV], function (dp) {  // triggert bei Wertänderung eines DP
                                    setState(IDAussenbeleuchtung, dp.state.val);
                                });
                                
                                S 1 Reply Last reply Reply Quote 1
                                • S
                                  skorpil @paul53 last edited by

                                  @paul53 Dankeschön. Du kannst definitiv klarer denken als ich!

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

                                    Ich habe ein Scipt geschrieben, das auch funktioniert (Heureka!). Es schaltet - bei Anwesenheit - um 1 Uhr nachts den Drucker aus und um 8 Uhr morgens schaltet es den Drucker wieder ein.

                                    // ##################################
                                    //              Programm
                                    // ##################################
                                    
                                        schedule({hour: 1, minute: 0}, function () {
                                        //Wird um 01:00 ausgeschaltet, bei Anwesendheit;
                                        if(getState(IDAnwesend).val == true) { 
                                        setState(IDDrucker, false);
                                        };
                                        });
                                    
                                        schedule({hour: 8, minute: 0}, function () {
                                        //Wird um 08:00 EINgeschaltet, bei Anwesendheit;
                                        if(getState(IDAnwesend).val == true) { 
                                        setState(IDDrucker, true);
                                        };
                                        });
                                    

                                    Mir gefällt das mit den Uhrzeiten in der schedule Anweisung aber nicht. Man kann doch sicher sagen: im Intervall 01:00 bis 08:00 aus, sonst ein. Aber wie?

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

                                      Ich benötige bitte noch einmal Programmierhilfe bei einem kniffligen Problem.

                                      Ziel: ich möchte detektieren, ob der TürFensterKontakt nur eine Erschütterung meldet oder ob das Fenster offen ist. Das habe ich bislang mit einem WebUI Programm gemacht.
                                      RaspberryMatic WebUI.jpg
                                      Wenn der TFK NUR 3 Sekunden "geöffnet" meldet, ist es ein Einbruch (Erschütterung), wenn er länger als 3 Sekunden "offen" meldet, ist einfach nur das Fenster offen.

                                      Meine Versuche, das in JavaScript abzubilden, sind bisher gescheitert. Ich habe es mit "setStateDelayed" versucht und jetzt sleep.

                                      // ###########################################
                                      //            Deklarationen
                                      // ###########################################
                                      
                                      const ErschSensorGaesteLinks = 'hm-rpc.0.MEQ1135406.1.STATE'/*ErschSensorGaestezi :1 STATE*/
                                      const FnstrGaesteUntLinksLangOFFEN = 'hm-rega.0.43334'/*FnstrGaesteUntLinks lang OFFEN*/
                                      const FnstrGaesteUntLinksEINBRUCH = 'hm-rega.0.42835'/*FnstrGaesteUntLinks EINBRUCH*/
                                      
                                      async function pause() {
                                       // log("Vor der sleep-Funktion");
                                       await sleep(3000); // Pausiert die Funktion für 3 Sekunden
                                       // log("Nach der Sleep Funktion");
                                      };
                                      
                                      
                                      // ##################################
                                      //              Programm
                                      // ##################################
                                      
                                      on({id: ErschSensorGaesteLinks, change: 'gt'} , function () {
                                      
                                             if (getState(ErschSensorGaesteLinks).val == true) {
                                              //  block of code to be executed if condition1 is true
                                              log("ErschSensorGaesteLinks ist wahr");
                                              pause();
                                              setState(FnstrGaesteUntLinksLangOFFEN, true);
                                      
                                              } else if (getState(FnstrGaesteUntLinksLangOFFEN).val == true) {
                                              //  block of code to be executed if the condition1 is false and condition2 is true
                                              log("FnstrGaesteUntLinksLangOFFEN ist wahr");
                                              pause();
                                              setState(FnstrGaesteUntLinksLangOFFEN, false);
                                              } else {
                                                  
                                              //  block of code to be executed if the condition1 is false and condition2 is false
                                                setState(FnstrGaesteUntLinksEINBRUCH, true)
                                                log("FnstrGaesteUntLinksEINBRUCH ist wahr");
                                              };
                                      });
                                      

                                      Könnt ihr mir helfen?

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

                                        @skorpil sagte: Wenn der TFK NUR 3 Sekunden "geöffnet" meldet, ist es ein Einbruch (Erschütterung), wenn er länger als 3 Sekunden "offen" meldet, ist einfach nur das Fenster offen.

                                        Vorschlag:

                                        var timer = null;
                                        
                                        on(ErschSensorGaesteLinks, function (dp) {
                                            if (dp.state.val) {
                                                log("ErschSensorGaesteLinks ist wahr");
                                                timer = setTimeout(function() {
                                                    setState(FnstrGaesteUntLinksLangOFFEN, true);
                                                    timer = null;
                                                }, 3000);
                                            } else if (timer) {
                                                clearTimeout(timer);
                                                setState(FnstrGaesteUntLinksEINBRUCH, true);
                                                log("FnstrGaesteUntLinksEINBRUCH ist wahr");
                                            }
                                        });
                                        
                                        S 3 Replies Last reply Reply Quote 1
                                        • S
                                          skorpil @paul53 last edited by

                                          @paul53 Dankeschön. Werde Deinen Vorschlag gleich testen und mich melden. An timeout hatte ich auch gedacht und dann irgendwie nicht weiter verfolgt.

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

                                            @paul53 feedback is breakfast for champions! Also:

                                            1. Dein Vorschlag funktioniert. Vielen Dank!
                                            2. was nicht richtig funktioniert, das sind meine drei pushover Scripte bei
                                            • offen
                                            • Einbruch
                                            • geschlossen

                                            Im Falle "Einbruch" kommt nachher noch die Meldung "geschlossen". Das ist aber nicht gewollt. Hier die drei zugehörigen Scripte:

                                            offen

                                            // ##################################
                                            // Deklarationen
                                            // ##################################
                                            
                                            const IDAusloeser ='hm-rega.0.43334'/*FnstrGaesteUntLinks lang OFFEN*/;
                                            const IDAnwesend = 'hm-rega.0.39533'/*Anwesend*/;
                                            const titel = "Fenster GÄSTEZIMMER unten links:";
                                            const beschreibung = "OFFEN!";
                                            
                                            
                                            // ##################################
                                            // Programm
                                            // ##################################
                                            
                                            on({id: IDAusloeser, val: true}, function () {
                                            
                                                if (IDAnwesend) {
                                            
                                                    sendTo("pushover.0", {
                                                        message:  beschreibung, // mandatory - your text message
                                                        title:    titel, // optional  - your message's title, otherwise your app's name is used
                                                        sound:    'siren',     // optional  - the name of one of the sounds supported by device clients to override the user's default sound choice
                                                                              //    pushover, bike, bugle, cashregister, classical, cosmic, falling,
                                                                              //    gamelan, incoming, intermission, magic, mechanical, pianobar, siren,
                                                                              //    spacealarm, tugboat, alien, climb, persistent, echo, updown, none
                                                        priority: 1,          // optional
                                                                              //    -1 to always send as a quiet notification,
                                                                              //    1 to display as high-priority and bypass the user's quiet hours, or
                                                                              //    2 to also require confirmation from the user
                                                    });
                                                   
                                                };                                
                                            });
                                            

                                            Einbruch

                                            // ##################################
                                            // Deklarationen
                                            // ##################################
                                            
                                            const IDAusloeser ='hm-rega.0.42835'/*FnstrGaesteUntLinks EINBRUCH*/;
                                            const IDlang = 'hm-rega.0.43334'/*FnstrGaesteUntLinks lang OFFEN*/;
                                            const titel = "Fenster Gäste unten links:";
                                            const beschreibung = "EINBRUCH!";
                                            
                                            // const IDAnwesend = 'hm-rega.0.39533'/*Anwesend*/;
                                            
                                            
                                            // ##################################
                                            // Programm
                                            // ##################################
                                            
                                            on({id: IDAusloeser, val: true} , function () {
                                            
                                                // if (IDAnwesend) {
                                                    sendTo("pushover.0", {
                                                        message:  beschreibung, // mandatory - your text message
                                                        title:    titel, // optional  - your message's title, otherwise your app's name is used
                                                        sound:    'siren',     // optional  - the name of one of the sounds supported by device clients to override the user's default sound choice
                                                                              //    pushover, bike, bugle, cashregister, classical, cosmic, falling,
                                                                              //    gamelan, incoming, intermission, magic, mechanical, pianobar, siren,
                                                                              //    spacealarm, tugboat, alien, climb, persistent, echo, updown, none
                                                        priority: 1,          // optional
                                                                              //    -1 to always send as a quiet notification,
                                                                              //    1 to display as high-priority and bypass the user's quiet hours, or
                                                                              //    2 to also require confirmation from the user                              
                                                    });
                                                    
                                                    setState(IDAusloeser, false); // Auslöser zurücksetzen
                                                    setState(IDlang, false); // lang offen auch zurücksetzen
                                                        
                                                //};        
                                            });
                                            

                                            geschlossen

                                            // ##################################
                                            // Deklarationen
                                            // ##################################
                                            
                                            const IDAusloeser ='hm-rega.0.43334'/*FnstrGaesteUntLinks lang OFFEN*/;
                                            const IDAnwesend = 'hm-rega.0.39533'/*Anwesend*/;
                                            const titel = "Fenster GÄSTEZIMMER unten links:";
                                            const beschreibung = "GESCHLOSSEN!";
                                            
                                            
                                            // ##################################
                                            // Programm
                                            // ##################################
                                            
                                            on({id: IDAusloeser, val: false}, function () {
                                            
                                                if (IDAnwesend) {
                                            
                                                    sendTo("pushover.0", {
                                                        message:  beschreibung, // mandatory - your text message
                                                        title:    titel, // optional  - your message's title, otherwise your app's name is used
                                                        sound:    'siren',     // optional  - the name of one of the sounds supported by device clients to override the user's default sound choice
                                                                              //    pushover, bike, bugle, cashregister, classical, cosmic, falling,
                                                                              //    gamelan, incoming, intermission, magic, mechanical, pianobar, siren,
                                                                              //    spacealarm, tugboat, alien, climb, persistent, echo, updown, none
                                                        priority: 1,          // optional
                                                                              //    -1 to always send as a quiet notification,
                                                                              //    1 to display as high-priority and bypass the user's quiet hours, or
                                                                              //    2 to also require confirmation from the user
                                                    });
                                                   
                                                };                                
                                            });
                                            

                                            Ich denke, die Doppel resp. falsche Pushover Nachricht hängt mit der On Bedingung "val: true" resp. false zusammen. Wie müßte das richtig sein?

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

                                            Support us

                                            ioBroker
                                            Community Adapters
                                            Donate

                                            720
                                            Online

                                            31.7k
                                            Users

                                            79.7k
                                            Topics

                                            1.3m
                                            Posts

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