Navigation

    Logo
    • Register
    • Login
    • Search
    • Recent
    • Tags
    • Unread
    • Categories
    • Unreplied
    • Popular
    • GitHub
    • Docu
    • Hilfe
    1. Home
    2. Deutsch
    3. Skripten / Logik
    4. JavaScript
    5. setState findet manuell erzeugtes Objekt nicht

    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

    setState findet manuell erzeugtes Objekt nicht

    This topic has been deleted. Only users with topic management privileges can see it.
    • Codierknecht
      Codierknecht Developer Most Active @Shai0Hulud last edited by

      @shai0hulud
      Die Frage ist: Mit welchem Wert für "idSetPrefix" wird die Funktion aufgerufen?

      function calculateAverageConsumption(startDateString, consTarget, idSetPrefix)
      
      S 1 Reply Last reply Reply Quote 0
      • paul53
        paul53 @Shai0Hulud last edited by paul53

        @shai0hulud sagte: Verwirrt mich aber als JS-Noob etwas.

        Du hast uns den Aufruf der Funktion calculateAverageConsumption(startDateString, consTarget, idSetPrefix) nicht gezeigt. Man kann beim Aufruf auch eine Variable übergeben, die allerdings den richtigen String enthalten muss.

        @shai0hulud sagte in setState findet manuell erzeugtes Objekt nicht:

        mit ChatGPT

        ist kein guter JS-Programmierer.

        S 2 Replies Last reply Reply Quote 0
        • S
          Shai0Hulud @Codierknecht last edited by

          @codierknecht

          Bisher mit 'idSetConsPerDay'

          nun mit '0_userdata.0.Zuhause.ConsPerDay'

          Alle Warnungen weg, Werte im Dashboard werden wieder aktualisiert.

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

            @paul53

            Doch, ich hatte es erwähnt:

            "Aufgerufen wiurd die Funktion wie folgt:

                    calculateAverageConsumption("2023/01/01", 6321.6, 'idSetConsPerDay');
            
            

            "

            Nun nutze ich

                    calculateAverageConsumption("2023/01/01", 6321.6, '0_userdata.0.Zuhause.ConsPerDay');
            
            
            1 Reply Last reply Reply Quote 0
            • Codierknecht
              Codierknecht Developer Most Active @Shai0Hulud last edited by Codierknecht

              @shai0hulud sagte in setState findet manuell erzeugtes Objekt nicht:

              Bisher mit 'idSetConsPerDay'

              In Hochkommata? Dann wird nur dieser String übergeben.
              Du willst ja nicht den Namen dieser Variablen übergeben, sondern den Inhalt derselben.

              Also so:

                      calculateAverageConsumption("2023/01/01", 6321.6, idSetConsPerDay);
              
              S 1 Reply Last reply Reply Quote 0
              • S
                Shai0Hulud @paul53 last edited by

                @paul53 said in setState findet manuell erzeugtes Objekt nicht:

                @shai0hulud sagte in setState findet manuell erzeugtes Objekt nicht:

                mit ChatGPT

                ist kein guter JS-Programmierer.

                Besser als ich 😏

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

                  @codierknecht

                  idSetConsPerDay ist keine Variable, sondern nur das Prefix, welches über die Funktion mit dem jeweiligen Datum ergänzt wird und zur korrekten Variable zusammengebastelt wird, also z.B. idSetConsPerDay230101.

                  calculateAverageConsumption("2023/01/01", 6321.6, idSetConsPerDay);
                  

                  gibt direkt einen Fehler "Cannot find name". Kann er ja auch nicht finden.

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

                    @shai0hulud sagte in setState findet manuell erzeugtes Objekt nicht:

                    idSetConsPerDay ist keine Variable

                    Sehe ich in Deinem Eingangspost aber anders:

                    var idSetConsPerDay230101 = '0_userdata.0.Zuhause.ConsPerDay230101'; 
                    

                    Natürlich ist das etwas anderes.
                    Wobei da auch besser const stehen sollte.

                    Wozu wird das zu Beginn mit "230101" definiert?
                    Alles etwas schwierig ohne den Gesamtüberblick

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

                      @codierknecht Sorry, nicht eindeutig genug ausgeführt. Der 01.01.23 war nur beispielhaft gezeigt, es gibt mehrere Daten, für die der tägliche Durchschnittsverbrauch angezeigt wird:

                      bd215b55-2cad-4c37-be95-0352377630df-image.png

                      Gesamtes Funktionsteil sieht so aus (wird alle 60minuten aktuialisiert):

                      function calculateAverageConsumption(startDateString, consPast, idSetPrefix) {
                          const startDate = new Date(startDateString);
                          const year = startDate.getFullYear().toString().slice(-2); // Die letzten beiden Stellen des Jahres
                          const month = ('0' + (startDate.getMonth() + 1)).slice(-2); // Monat mit führender Null, um zweistellig zu sein
                          const day = ('0' + startDate.getDate()).slice(-2); // Tag mit führender Null, um zweistellig zu sein
                      
                          const fullID = `${idSetPrefix}${year}${month}${day}`;
                      //    console.log(fullID); // Debugging-Anweisung hinzufügen
                      
                          const currentDate = new Date().valueOf();
                          const daysDifference = Math.round((currentDate - startDate.valueOf()) / (1000 * 60 * 60 * 24));
                      
                          const consumptionDifference = getState(idGetStrombezug).val - consPast;
                          const averageConsumption = (Math.round((consumptionDifference / daysDifference) * 1000) / 1000).toLocaleString('de-DE', { maximumFractionDigits: 3 });
                      //    console.log(averageConsumption);
                      
                          setState(fullID, `${averageConsumption.replace(".", ",")} kWh`, true);
                      }
                      
                          // Initialisierung der Variable, um mehrfache Aufrufe zu verhindern
                          if (!NRDB.averageConsumptionCalculated) {
                              calculateAverageConsumption("2023/01/01", 6321.6, '0_userdata.0.Zuhause.ConsPerDay');
                              calculateAverageConsumption("2023/07/01", 8165.5, '0_userdata.0.Zuhause.ConsPerDay');
                              calculateAverageConsumption("2023/09/01", 8340.0, '0_userdata.0.Zuhause.ConsPerDay');
                              calculateAverageConsumption("2023/11/01", 8667.0, '0_userdata.0.Zuhause.ConsPerDay');
                              calculateAverageConsumption("2024/01/01", 9220.3, '0_userdata.0.Zuhause.ConsPerDay');
                              NRDB.averageConsumptionCalculated = true;
                      
                              // Setzen von NRDB.averageConsumptionCalculated auf false und Starten des Timers
                              setTimeout(function () {
                                  NRDB.averageConsumptionCalculated = false;
                              }, 60 * 60 * 1000); // 60 Minuten in Millisekunden
                          }
                      
                      
                      Codierknecht 1 Reply Last reply Reply Quote 0
                      • Codierknecht
                        Codierknecht Developer Most Active @Shai0Hulud last edited by

                        @shai0hulud

                        Ich habe noch immer nicht verstanden, warum Du das zu Beginn gleich für "230101" definierst.

                        var idSetConsPerDay230101 = '0_userdata.0.Zuhause.ConsPerDay230101'; 
                        

                        Ich würde das (prinzipiell) in etwa so machen:

                        var moment = require('moment');
                        
                        const statePrefix = '0_userdata.0.Zuhause.ConsPerDay'; 
                        
                        function calculateAverageConsumption(startDateString, consTarget, idSetPrefix) {
                            let date = moment(startDateString).format('YYMMDD');
                            const fullID = `${statePrefix}${date}`;
                            ...
                        }
                        

                        Eine stündliche Ausführung würde ich über einen Schedule lösen und nicht per Timeout.

                        Dass Du den Code ab Zeile 20 eingerückt hast, ist der Lesbarkeit nicht wirklich zuträglich.
                        Ich weiß: Klingt nach Erbesenzählerei, aber die Lesbarkeit ist das A und O für die spätere Wartung und Erweiterung von Code.

                        S 1 Reply Last reply Reply Quote 1
                        • S
                          Shai0Hulud @Codierknecht last edited by

                          @codierknecht Ich kann Dir das nicht erklären, warum ich da so in dieser Form zu Beginn definiere. JS-Noob und so. Ich habe mir das alles mit Google zusammengeschustert und war happy, dass der NodeRed-Adapter am Ende meine Werte so anzeigen konnte wie von mir gewünscht.

                          ChatGPT kam erst am Ende ins Spiel, habe nur um Vereinfachung gebeten. Woraufhin das Script mit 170 Zeilen auf immerhin 130 Zeilen geschrumpft ist. Und bis auf diesen Teil mit den täglichen Durchschnittsverbrauchswerten ab bestimmten Tagen hat auch alles sofort funktioniert.

                          Das Einrücken ist nicht bewusst passiert, ist glaube ich durch das Copy und Paste von ChatGPT gekommen.

                          Danke für Deinen Vorschlag, das probier ich mal aus!

                          VG!

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

                            @shai0hulud

                            Vorschlag funktioniert, konnte "idSetPrefix" aus der Funktion so natürlich entfernen, brauche jetzt nur noch 2 Optionen für die Funktion und hab mir von ChatGPT mit den Datumsformaten helfen lassen, da bei der restlichen Funktion ein anderes Datumsformat genutzt wurde und damit die Differenz in Tagen extrem groß wurde (knapp 20000 Tage statt nur 400 z.B.).

                            Danke nochmal!

                            Dass mit dem Schedule statt Timeout schaue ich mir auch noch an.

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

                            Support us

                            ioBroker
                            Community Adapters
                            Donate

                            722
                            Online

                            31.8k
                            Users

                            80.0k
                            Topics

                            1.3m
                            Posts

                            javascript
                            3
                            19
                            480
                            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