js callback asynchron problem

[hanss]

Hallo,
ich komme seit Tagen mit diesem Problem nicht weiter.
Asynchrone Programmierung ist wohl nicht meine Welt.

var Raumctr = 0
// Schleife für jeden Raum ######################################
 S_Raum.forEach(function(element, index) { 
        if( SollTemp_akt != Temp_FOA && SollTemp_akt != Temp_Abw ){    
            setState(Out_Root+D_Raum[index]+".Man",SollTemp_akt,true,function(){
                       setRT(index,element,SollTemp_akt)         // function setRT wg. Callback
                       log("RaumzählerCB: "+Raumctr+" Index: "+RaumIndex) 
            })
        } else {
            setRT(index,element,SollTemp_akt)
            log("Raumzähler: "+Raumctr+" Index: "+RaumIndex) 
        }
   }); 
 // Schleife für jeden Raum ######################################
 
// Script hier, wenn alle Räume durchlaufen sind
// ##########   Script soll hier fortfahren, wenn  Schleife für jeden Raum durchlaufen ist.  ##########
log("Raumzähler nach allen Räumen: "+Raumctr+" Index: "+RaumIndex)

Monitor:
// wegen dem verzögerten Callback wird folgende Reihenfolge ausgegeben:
Raumzähler: 1 Index: 4
Raumzähler: 2 Index: 5
Raumzähler: 3 Index: 6
Raumzähler: 4 Index: 7
Raumzähler nach allen Räumen: 4 Index: 7  - zu früh, im Callback warten noch 4 Räume
RaumzählerCB: 5 Index: 0
RaumzählerCB: 6 Index: 1
RaumzählerCB: 7 Index: 2
RaumzählerCB: 8 Index: 3

  
 function setRT(index,element,SollTemp_akt){
                Raumctr++
                RaumIndex = index
 }

Das Problem dabei:
je nach "if( SollTemp_akt != Temp_FOA && SollTemp_akt != Temp_Abw )" wird die funktion setRT()
einmal direkt oder als callback aufgerufen.

"// Script hier, wenn alle Räume durchlaufen sind"
soll aber in jedem Fall erst ausgeführt werden, wenn alle Räume durchlaufen sind.
In dem Beispiel hier wird zuerst die setRT() als Callback aufgerufen, wegen der Verzögerung bei
setState wird der Raumctr aber erst incrementiert, nachdem der anschließende direkte Aufruf von setRT()
schon beendet ist und die Schleife für jeden Raum schon beendet wurde.

[OliverIO]

@hanss
du könntest den Aufruf des 2.setRT ebenfalls in einen Callback mit settimeout setzen, dann werden die Aufrufe zumindes in gleicher Reihenfolge auf den Stack geschrieben.
Allerdings kann die Abarbeitung innerhalb des Callbacks unterschiedlich lange ausfallen. Von daher hast du keine Garantie, das hier genau die gleiche Reihenfolge eingehalten wird

setTimeout(function(){
            setRT(index,element,SollTemp_akt)
            log("Raumzähler: "+Raumctr+" Index: "+RaumIndex) 
},10);

kann man in den Skripten setState nicht auch synchron aufrufen?

[hanss]

@OliverIO
Vielen Dank, aber Deine Lösung ist nicht sicher, weil zeitabhängig.
Ein synchroner setState wäre die Lösung, das hätte mir zwei Tage rumprobieren erspart.

[OliverIO]

@hanss wie gesagt bei asynchron gibt es keine garantierte abfolge.

evtl beschreibst du mal was du am ende erreichen möchtest, evtl ist dein bisheriger lösungsweg nicht optimal

[AlCalzone]

Eventuell sind die neuen async-Funktionen in 4.8.x was für dich @hanss
https://forum.iobroker.net/topic/36999/neu-diverse-async-funktionen-im-javascript-adapter

Damit lässt sich der Programmablauf trotz asynchronität schön steuern. Ich hab den obigen Schnipsel mal entsprechend umgebaut. Beachte das async vor der Funktion und await setStateAsync:

async function raumzaehler() {
	var Raumctr = 0;

	// Schleife für jeden Raum ######################################
	for (let i = 0; i < S_Raum.length; i++) {
		const element = S_Raum[i];
		if (SollTemp_akt != Temp_FOA && SollTemp_akt != Temp_Abw) {
			await setStateAsync(Out_Root + D_Raum[index] + ".Man", SollTemp_akt, true);
			setRT(index, element, SollTemp_akt); // function setRT wg. Callback
			log("RaumzählerCB: " + Raumctr + " Index: " + RaumIndex);
		} else {
			setRT(index, element, SollTemp_akt);
			log("Raumzähler: " + Raumctr + " Index: " + RaumIndex);
		}
	}
	// Schleife für jeden Raum ######################################

	// ##########   Script soll hier fortfahren, wenn  Schleife für jeden Raum durchlaufen ist.  ##########
	log("Raumzähler nach allen Räumen: " + Raumctr + " Index: " + RaumIndex);

}

// Funktion aufrufen nicht vergessen!
raumzaehler();

[liv-in-sky]

@AlCalzone

ich habe immer wieder probleme mit dem async und await - toll wäre es, wenn man so was wie mit dem type-script kurs für adapterentwickler machen würde - nur halt für dieses thema

[AlCalzone]

@liv-in-sky https://gist.github.com/AlCalzone/d14b854b69ce5e8a03718336cc650a95

[liv-in-sky]

@AlCalzone danke werd ich mal durcharbeiten

[fastfoot]

@liv-in-sky sagte in js callback asynchron problem:

....wenn man so was wie mit dem type-script kurs für adapterentwickler machen würde....

gibt es dazu einen Link?

[liv-in-sky]

@fastfoot

https://forum.iobroker.net/topic/36493/09-09-20-30-einsteiger-cursus-demonstration-von-typescript

[fastfoot]

@liv-in-sky cool, Danke!

[OliverIO]

@AlCalzone

Async und await machen aus asynchronen Funktionen wieder synchrone Funktionen.
Das will er ja nicht

[OliverIO]

Hier mal ein kurzer Artikel, der auf einen Punkt eingeht, der die Funktionsweise der asynchronen Funktionen (callback oder auch promise basiert) erklärt

https://dev.to/steelvoltage/if-javascript-is-single-threaded-how-is-it-asynchronous-56gd

[hanss]

@OliverIO
Vielen Dank, ich habe das jetzt so gelöst:

In die Funktion setRT() einen Raumzähler und das Script eingefügt, wenn alle Räume durchlaufen sind:
Das ist sehr übersichtlich und nicht so kompliziert wie async und await.
Ich glaube, das kapier ich nicht mehr.

setRT() {
  Raumctr++
  //mein Script

   if (S_Raum.length == Raumctr){              // letzter Raum
       // Script hier, wenn alle Räume durchlaufen sind 
       Raumctr=0        
   }
}

[AlCalzone]

@OliverIO wieso nicht? Wenn man etwas ausführen will, nachdem asynchrone Vorgänge durch sind, ist async/await genau das richtige Mittel.
Und IMO intuitiver als der callback Krempel. Nach dem Aufruf mit await ist es einfach fertig.

[OliverIO]

@AlCalzone
Ah hatte einer seiner Antworten falsch interpretiert. Hab nochmal gelesen.
Wenn er mit seiner Lösung nun zufrieden ist, ist ja alles gut