Jahresrückblick 2025 – unser neuer Blogbeitrag ist online! ✨

[Codierknecht]

@peoples sagte in GetState in Schleife liefert zuerst false:

createDeviceStates

JS ist ja nicht unbedingt meine Kernkompetenz, aber ich versuch's mal:

Alles was Du in createDeviceStates lostrittst, wird ja asynchron ausgeführt.
Nach meinem Verständnis kehrt die Methode also quasi unmittelbar nach dem Aufruf zurück.
Wenn Du dann scheduleDeviceStates aufrufst kannst Du nicht sicher sein, dass die DP auch tatsächlich angelegt sind.

Warum nicht einfach nur ein createState? (ohne "Async")

[Peoples]

@codierknecht
HI also das Problem ist nicht das erstellen der Datenpunkte das funktioniert sondern das auslesen in der Schleife.

Selbst wenn die erstellt sind, kann ich die States nicht abrufen es geht immer der erste nicht.
Macht auch keinen Unterschied ob Async oder nicht.

Sobald bei devices[i].deviceName das i den Wert 0 hat funktioniert es nicht.

[Codierknecht]

@peoples
Also zunächst mal finde ich persönlich es extrem schwierig zu lesen, wenn innerhalb der Objektdefinition Kommentare eingefügt sind. Ich würde das außerhalb der Deklaration dokumentieren (wenn überhaupt - was da gemacht wird ist eigentlich auch ohne Kommentare klar).

Auch warum die beiden Variablen instanz und pfad0 nicht direkt als Stringliterale besetzt sondern zusammengebaut werden, erschließt sich nicht. Macht es nur schwerer zu lesen.

Was sagt denn das hier:

async function scheduleDeviceStates() { 
  try {
    for(var i = 0; i < devices.length; i++) { 
      log(devices[i].deviceName + ' --->' + i);
    }
  } catch (error) {
      log(`Unexpected error - ${error}`, 'error');
  }
}

Wird das array zumindest sauber gelesen?
Bei mir schon - ich vermute immer noch, dass beim ersten Durchlauf noch nicht alle DP existieren bzw. mit ihren Defaultwerten besetzt sind.

Warum überhaupt createStateAsync? Nur um die parallel zu erzeugen?
Wenn die sauber nacheinander erzeugt werden, dauert das doch nur wenige Millisekunden.
Da wäre mir der ganze Verwaltungskram für die Parallelität zu aufwändig.
Ich würde das zunächst mal ganz klassisch synchron machen. Wenn's zu viel Performance frisst, kann man das später immer noch optimieren.
KISS ... keep it simple stupid

Rules of Optimization:
Rule 1: Don’t do it.
Rule 2 (for experts only): Don’t do it yet.

Oder um es mit den Worten von Donald Knuth zu sagen:

premature optimization is the root of all evil

[Codierknecht]

@codierknecht
Gerade genau Dein Script mal in 2 Stufen laufen lassen:
1.) Den zweiten Funktionsaufruf auskommentiert. Es werden wie erwartet alle DP erzeugt.
2.) Den ersten Funktionsaufruf auskommentiert. Ergebnis:

29.11.2022, 19:50:30.280	[info ]: javascript.1 (23493) Start javascript script.js.test.Test_JS
29.11.2022, 19:50:30.296	[info ]: javascript.1 (23493) script.js.test.Test_JS: 8:00 --->0
29.11.2022, 19:50:30.297	[info ]: javascript.1 (23493) script.js.test.Test_JS: 8:00 --->1
29.11.2022, 19:50:30.297	[info ]: javascript.1 (23493) script.js.test.Test_JS: 8:00 --->2
29.11.2022, 19:50:30.297	[info ]: javascript.1 (23493) script.js.test.Test_JS: 8:00 --->3

[Peoples]

@codierknecht

Danke ich habe jetzt meinen Fehler gefunden

[Codierknecht]

@peoples sagte in [Gelöst] - GetState in Schleife liefert zuerst false:

Nicht jeder ist ein "ausgebildeter / studierter" Programmierer, ergo man macht vieles wie man denkt.
Auch Zitate helfen oft wenig weiter und fördern nur einen Eindruck den man wahrscheinlich seinem Gegenüber nicht vermitteln will.

Akzeptiert ... war aber tatsächlich nur gut gemeint ;-)
Ich beschäftige mich mittlerweile seit Jahren viel mit dem Thema "CleanCode".
Meine Quintesenz: Gut lesbarer Code funktioniert meist - weil sich Bugs nicht so leicht verstecken können.
Ausgebildet/studiert übrigens nicht - ich hatte lediglich das große Glück, mein Hobby zum Beruf machen zu können.

Ich bin gerade dabei, mich in die Grundzüge von await und async einzulesen.
Ich hoffe, in ein paar Minuten eine Lösung für Dein Problem anbieten zu können.
Mein erster Ansatz funktioniert nämlich im Übrigen nicht. createState scheint grundsätzlich asynchron zu sein. Man weiß ohne besondere Vorkehrungen also nicht, wann alle DP tatsächlich existieren.

[Codierknecht]

@peoples sagte in [Gelöst] - GetState in Schleife liefert zuerst false:

@codierknecht

Danke ich habe jetzt meinen Fehler gefunden

Und der wäre? Interessiert mich auch.

[Peoples]

@codierknecht
Ich bin ein Idiot :-)

Ich hatte in dem ersten Datenpunkt keine Zeit '8:00' drin stehen sondern tatsächlich ein 'false'.
Deswegen hat er auch immer das false ausgegeben

[Codierknecht]

@peoples
:grin:

Ich habe mittlerweile folgende Lösung gefunden:

async function scheduleDeviceStates(){ 
   try {
        await createDeviceStates();
        for(var i = 0; i < Object.keys(devices).length; i++) { 

Die erste Funktion wird nicht bei Scriptstart aufgerufen, sondern bevor in der zweiten Funktion die DP verarbeitet werden.
Mit diesem Konstrukt wird auf die Beendigung gewartet und erst dann die Schleifenverarbeitung gestartet.

Wieder was dazugelernt - Danke!

Da wirste alt wie 'ne Kuh und lernst immer noch dazu

[fastfoot]

@codierknecht sagte in GetState in Schleife liefert zuerst false:

Ich bin gerade dabei, mich in die Grundzüge von await und async einzulesen.

dann interessiert dich möglicherweise das hier. Ist aber nichts was man unbedingt sofort versteht :-)

[Codierknecht]

@peoples
Nur der Vollständigkeit halber und weil wir am Ende des Tages dann beim Smalltalk angekommen sind:
Die Krux ist tatsächlich dieser Async-Kram. Den muss man an der Ecke halt irgendwie in den Griff kriegen.
Wirkliche "Fehler" waren in Deinem Code ja nicht vorhanden ;-)

Falls es interessiert: https://clean-code-developer.de/
Ist jetzt nicht die allerbeste Adresse, weil da kommerzielle Gedanken dahinterstehen. Aber das ist ja durchaus legitim und die Seite fasst Dinge gut zusammen, die man sich sonst aus allen möglichen Quellen zusammensuchen muss.

[Codierknecht]

@fastfoot
Danke! :+1: