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

    • Neuer Blog: Fotos und Eindrücke aus Solingen

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

    • ioBroker goes Matter ... Matter Adapter in Stable

    [gelöst] Elegantere Programmierung?

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

      @skorpil sagte: Kann ich mir nicht eigentlich die "If" Abfrage sparen und das direkt in der "On" Anweisung erledigen?

      Ja, wenn nur auf val = true getriggert werden soll.

      on({id: IDAusloeser, val: true}, function (data) {
      

      Beschreibung

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

        @paul53 merci

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

          So, jetzt bin ich bei einem Fall angekommen, wo ich nicht mehr eine HM Systemvariable mit den Werten true/ false als trigger habe, sondern den Zustand eines Gerätes, hier der opt. Tür/ Fenster Kontakt. Ich möchte eine Nachricht erhalten, nur, wenn ich ANwesend bin

          const IDAusloeser ='hm-rpc.1.0000DD89B24B3E.1.STATE'/*TFK opt  Sensor Terrasse:1 STATE*/;
          const IDAnwesend = 'hm-rega.0.39533'/*Anwesend*/;
          
          // ##################################
          // Programm
          // ##################################
          
          on({id: IDAusloeser, val: true} , function (data) {
          

          Der TFK kann die zustände offen u. geschlossen annehmen; CLOSED = 0 oder OPEN = 1. Wie muß ich da den Auslöser setzen?

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

            @skorpil Na auch über val...

            // CLOSED
            on({id: IDAusloeser, val: 0} , function (data) {
            });
            
            // OPEN
            on({id: IDAusloeser, val: 1} , function (data) {
            });
            
            1 Reply Last reply Reply Quote 1
            • S
              skorpil last edited by

              Ich bekomme von meinem Automower diverse Fehlermeldungen, auf die ich aber nicht alle reagieren möchte. Vielmehr nur auf Werte < 1000 sowie auf 1038 und 1052.

              const IDAusloeser ='hm-rega.0.35569'/*Automower Status-Code*/;
              var IDstatusCode = getState(IDAusloeser).val;
              
              
              on(IDAusloeser , function (data) {
              
                  if (IDstatusCode < 1000 || IDstatusCode == 1038  || IDstatusCode == 1052) {
              

              Frage 1: Ist die "If" Anweisung so richtig?

              Frage 2: muß ich den Umweg über die getState Anweisung und die Definition einer Variablen (hier: IDstatusCode) gehen? Oder könnte ich in der "If" Anweisung direkt mit der IDAusloeser arbeiten? Ich bin immer unsicher, wie und wann das mit ".val" funktioniert.

              Frage 3: benötige ich überhaupt die If Anweisung oder ließe sich das auch direkt in der On Anweisung lösen?

              Dank Euch schön.

              PS: warum ich dauernd frage: ich stelle gerade meine HM Programme auf iobroker um.

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

                @skorpil Du solltest Dich erstmal mit den Grundlagen der Programmierung beschäftigen glaube ich. Also unabhängig vom ioBroker.

                1. Die Variable IDstatusCode wird beim Script-Start EINMALIG mit dem Wert aus dem Datenpunkt gefüllt. Danach nie wieder.
                2. Diesen Wert (was auch immer zum Script-Start da drin stand) wird dann jedes Mal verglichen, wenn sich der Wert ändert. Aber der neue Wert wird halt nirgendwo berücksichtigt.

                Du willst wahrscheinlich sowas machen:

                const IDAusloeser ='hm-rega.0.35569'/*Automower Status-Code*/;
                 
                on(IDAusloeser , function (data) {
                    const IDstatusCode = getState(IDAusloeser).val;
                 
                    if (IDstatusCode < 1000 || IDstatusCode == 1038  || IDstatusCode == 1052) {
                
                

                Schlauer wäre es dann aber, den neuen Wert aus dem Parameter zu übernehmen:

                const IDAusloeser ='hm-rega.0.35569'/*Automower Status-Code*/;
                 
                on(IDAusloeser , function (data) {
                    const IDstatusCode = data.state.val;
                 
                    if (IDstatusCode < 1000 || IDstatusCode == 1038  || IDstatusCode == 1052) {
                
                
                S 1 Reply Last reply Reply Quote 1
                • paul53
                  paul53 @skorpil last edited by paul53

                  @skorpil sagte: Ist die "If" Anweisung so richtig?

                  Ja, aber die Variable IDstatusCode wird nicht aktualisiert. Außerdem enthält die Variable keine ID, was man beim Bezeichner berücksichtigen sollte.

                  const IDAusloeser ='hm-rega.0.35569'/*Automower Status-Code*/;
                  var statusCode = getState(IDAusloeser).val;
                    
                  on(IDAusloeser, function (data) {
                      statusCode = data.state.val;
                      if (statusCode < 1000 || statusCode == 1038  || statusCode == 1052) {
                  
                  1 Reply Last reply Reply Quote 1
                  • S
                    skorpil @haus-automatisierung last edited by

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

                    Du solltest Dich erstmal mit den Grundlagen der Programmierung beschäftigen

                    Das stimmt. Gibt es irgendwo einen Videokurs zu Javascript Programmierung im Zusammenhang mit iobroker? Deinen ioBroker-Master-Kurs habe ich erworben, aber da geht es meist um Blockly. Und damit tue ich mich schwer.

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

                      @skorpil Am Ende schiebst Du mit Blockly nur fertigen JavaScript-Code zusammen. Wenn Du Blockly verstehst, und dann zu JavaScript umschaltest, dann siehst Du ja sofort was daraus generiert wird.

                      Also am Ende ist für Blockly als auch für JavaScript das gleiche grundlegende Verständnis notwendig. Nur dass Du bei Blockly keine syntaktischen Fehler machen kannst. Semantisch kann man genauso "Mist bauen".

                      Über eine JavaScript-Erweiterung habe ich schon nachgedacht - aber wo fängt man da an? Wie man an diesem Beispiel gut sieht, müsste man bei den absoluten Basics starten, welche erstmal nichts mit ioBroker an sich zu tun haben. Und ich möchte ja keine Kurse für Softwareentwickler erschaffen. Daher fehlt mir da etwas die Abgrenzung und deswegen gibt es aktuell "nur" Blockly im Kurs.

                      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 am Ende ist für Blockly als auch für JavaScript das gleiche grundlegende Verständnis notwendig. Nur dass Du bei Blockly keine syntaktischen Fehler machen kannst. Semantisch kann man genauso "Mist bauen".

                        Dem würde ich mich anschließen.
                        Versuch's zunächst mit Blockly. Viel einfacher kann man kaum einsteigen, da es Dir viele Dinge abnimmt und - wie bereits gesagt - Fehler mit der Syntax ausschließt.

                        Nach meiner Erfahrung haben die meisten Anfänger Probleme damit, das grundlegende Verständnis aufzubauen wie ein "if-then-else" funktioniert und wie man passende Bedingungen mit "and/or" verküpft. Da werden die meisten Fehler gemacht, oder - um den Begriff zu verwenden - "Mist gebaut". Eine Klammer an der falschen Stelle - zack - völlig anderes Verhalten.
                        Da gleich mit JS (und womöglich Objektorientierung) einzusteigen könnte schwierig werden. Da würde ein Basic-Dialekt womöglich einfacher sein. War ja ursprünglich genau dafür ersonnen. Macht aber heute keiner mehr.

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

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

                          Da gleich mit JS (und womöglich Objektorientierung) einzusteigen könnte schwierig werden.

                          Naja, OOP braucht man nun wirklich nicht. Es reicht schon, wenn man sich mit Variablen, Datentypen (number, boolean, string, object), (anonymen) Funktionen / Callbacks und einfachen Vergleichen beschäftigt. Dann kann man 99% erschlagen.

                          Dafür gibt es ja ohne Ende guten Content auf YouTube und co. Wie gesagt - unabhängig vom ioBroker. Eventuell auch einfach mal ein paar Beispiele auf https://playcode.io durchspielen. Variablen deklarieren, Funktionen aufrufen, usw.

                          https://www.codewars.com finde ich auch ganz cool. Dort bekommt man Probleme gestellt und muss diese dann möglichst elegant lösen.

                          Wenn man das hier durcharbeitete, weiß man schon mehr als genug und muss dann nur noch das Wissen auf den ioBroker anwenden: https://github.com/jonasschmedtmann/complete-javascript-course

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

                            @haus-automatisierung
                            Nicht zu vergessen: The Tao of programming

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

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

                              Gibt es irgendwo einen Videokurs zu Javascript Programmierung im Zusammenhang mit iobroker?

                              Ich persönlich finde Videokurs nicht so optimal, da man sowieso immer wieder mal code beispiele kopieren muss.

                              https://wiki.selfhtml.org/wiki/JavaScript
                              oder hier
                              https://www.w3schools.com/js/default.asp

                              ansonsten gibt es auf youtube ne menge videos dazu

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

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

                                Ich persönlich finde Videokurs nicht so optimal, da man sowieso immer wieder mal code beispiele kopieren muss.

                                Hauptsache die "Programmierung" endet nicht so 😁 😁 😁 😁

                                dcc5290d-50a2-4791-b561-901dafb41cad-image.png

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

                                  @codierknecht

                                  o'reily ist cool.
                                  aber zum verständis für andere.
                                  das sind keine echten covers 😀 (hatte ich schon mal erwähnt das die emojis hier hässlich sind?)

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

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

                                    aber zum verständis für andere.

                                    das sind keine echten covers

                                    Das sind Fake-Covers ... die haben diverse Spaßvögel kreiert: https://dev.to/rly
                                    Aber natürlich hat O'Reilly "echte" und wirklich gute und empfehlenswerte Bücher im Angebot.

                                    hatte ich schon mal erwähnt das die emojis hier hässlich sind?

                                    Da hast Du wohl Recht. Ich war mal so frei: https://forum.iobroker.net/topic/59147/anregung-standard-smilies

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

                                      Ich danke Euch allen für die Tips. Viel Arbeit liegt vor mir. Im übrigen: learning by doing. So lerne ich a, besten.

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

                                        zurück zum Thema "elegantere Programmierung". Ziel folgenden Scripts ist es, nur auszulösen, wenn NICHT anwesend:

                                        on({id: IDAusloeser, val: true} , function (data) {
                                                
                                            if (getState(IDAnwesend).val == false) {
                                        

                                        Könnte ich nicht den Ausdruck in der IF Anweisung einfacher und so schreiben, wie in der ON Anweisung:

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

                                        Ich habe es getestet. Es funktioniert aber nicht. Offenbar wird in der IF Anweisung das false so nicht verarbeitet.

                                        Wo ist mein Denkfehler?

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

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

                                          Könnte ich nicht den Ausdruck in der IF Anweisung einfacher und so schreiben, wie in der ON Anweisung:

                                          Nein, weil es kein IF ist.

                                          on ist eine Funktion, welche als Konfiguration im ersten Parameter einen String oder ein Objekt (wie in deinem Script) erwartet. Zweiter Parameter ist ein Callback.

                                          Bei einem if kannst Du natürlich nicht so auf ein Objekt prüfen (das ist ja dann wieder statisch). Das Ergebnis wird immer true sein. Ich wiederhole mich gerne:

                                          Du solltest Dich erstmal mit den Grundlagen der Programmierung beschäftigen

                                          Schöner wäre übrigens, mit "not" zu arbeiten, statt ein Vergleich mit false. Geht aber beides.

                                          on({id: IDAusloeser, val: true} , function (data) {
                                              if (!getState(IDAnwesend).val) {
                                          

                                          <hier stand Quatsch - nicht gesehen, dass es eine zweite ID gibt, weil keine vollständiges Script geliefert wurde...>

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

                                            @paul53 jetzt habe ich den Fall, wo ich auf "false" triggern möchte

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

                                            Das funktioniert aber nicht. Ohne erkennlichen Grund bekomme ich eine Nachricht (weil ich mir mit dem Script eine Nachricht senden lasse).

                                            Ich habe das jetzt so gelöst:

                                            on(IDAusloeser, function (data) {
                                            
                                                test = getState(IDAusloeser).val;
                                                
                                                if (test == false) {
                                            

                                            Und das funktioniert auch. Ich möchte verstehen, warum val auf false triggern nicht funktioniert.

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

                                            Support us

                                            ioBroker
                                            Community Adapters
                                            Donate

                                            658
                                            Online

                                            31.8k
                                            Users

                                            80.0k
                                            Topics

                                            1.3m
                                            Posts

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