[gelöst] messageTo mit Rückgabewert?

[OliverIO]

@ben1983 sagte in messageTo mit Rückgabewert?:

warum nicht so?

let abc;
messageTo("Testmessage",data,(dp)=>{
    // Hier der callback mit
    abc = dp;
    log(abc);
});

messageTo ist eine asynchrone Funktion. Mehr dazu mit (allgemeine) Beispielen hier
https://javascript.info/callbacks

Warum willst du die Funktion mit messageTo aufrufen? Evtl bist du da auf dem falschen Weg unterwegs.
Globale funktionen im javascript adapter helfen eigentlich nur, das man den code nicht für jedes neue skript erneut schreiben muss. beim start wird technisch der code allerdings einfach nur vor dein skript kopiert.
https://github.com/ioBroker/ioBroker.javascript/blob/master/docs/en/javascript.md#global-functions
mehrere verschiedene skripte können nicht über die globalen funktionen miteinander kommunizieren. daher jedes skript hat seine eigenen variablen zu jedem globalen skript

daher, wenn du bspw nachschlage tabellen als globale funktion umsetzen willst ist das ok.
dafür musst du aber nicht onMessage und messageTo verwenden, da reicht auch einfach nur folgendes

globalskript

function test() {
    return "test";
}

normales skript

log(test());

wenn du die beiden skripte anlegst und startest (erst global dann das normale), dann gibt das normale skript "test" aus

[Ben1983]

@oliverio Ich weiß, dass die globalen in jedem skript davor gesetzt werden.
Deshalb wollt ich sie ja direkt vermeiden.

Dein Bsp. mit

let abc;
messageTo("Testmessage",data,(dp)=>{
    // Hier der callback mit
    abc = dp;
    log(abc);
});

kann ich leider nicht so machen, weil im haupt skript, welches das messageTo aufruft eben noch weiteres passiert.
(nach dem messagTo Aufruf, was aber erst nach dem Callback passieren sollte / darf.

Ich müsste dann das Skript dahingehen aufbauen, dass der callback diesen Teil dann aufruft,
aber dann müsste ich alles umbauen... glaube dann erzeuge ich lieber eine globale Funktion

[paul53]

@ben1983 sagte: Deshalb wollt ich sie ja direkt vermeiden.

Was stört Dich daran? Wenn die (globale) Funktion im Skript nicht aufgerufen wird, wird sie auch nicht kompiliert. Node.js arbeitet nicht mit Unterprogrammen (Bibliotheken), sondern verwendet Copy & Paste.

[Ben1983]

@paul53 ach so, ich dachte die wir immer davor kopiert.
Wenn die also nicht verwendet wird, dann wird sie auch nicht kopiert. Na dann ist es ja keine Frage. Ok. Danke

Aber es wird immer erst der „Hauptthread“ ausgeführt?

Also:

function getmystring(){
return „2“};
log(„1“);
log(getmystring());
log(„3“);

Liefert als log
1
2
3

Und

onMessage(„Test“,(data,callback){
log(„2“);};);

log(„1“);
messageTo(„Test“,“xyz“);
log(„3“);

Liefert:
1
3
2

Richtig? … zumindest bei mir

[paul53]

@ben1983 sagte: ich dachte die wir immer davor kopiert.

Wird sie auch unmittelbar vor dem Kompilieren, der Compiler verwendet sie aber nur beim Aufruf.

@ben1983 sagte in messageTo mit Rückgabewert?:

es wird immer erst der „Hauptthread“ ausgeführt?

Der Hauptthread ist meist schneller als die asynchrone Funktion. Die asynchrone Funktion wird nur "angestoßen", bevor es im Hauptthread weiter geht.

[Ben1983]

@paul53 OK, also wird nicht wirklich wie in einer funktion in das messagaTo (in dem Beispiel) rein gesrungen und der code ausgeführt, sondern "nur" so zu sagen vermerkt, dass diese Methode aufgerufen werden soll und dann geht es schon im Hauptthread weiter. Richtig?

[paul53]

@ben1983 sagte: vermerkt, dass diese Methode aufgerufen werden soll und dann geht es schon im Hauptthread weiter. Richtig?

So oder so ähnlich.

[Ben1983]

@paul53 Sie wird aber wirklich erst am Ende aufgerufen.
siehe dieses Beispiel:

function logAufruf(i){
    log(i);
}

onMessage("Aufrufschreife",(data,callback)=>{
    for(let i = data; i > 0 ; i--){
        log("Messageaufruf" + i);
    }
});

log("Start");
messageTo("Aufrufschreife",10);
for(let i = 10;i>0;i--){
    log("Außen" + i);
}

Das führt zu folgendem log: Ich habe es mal per screenshot eingefügt, damit man direkt sieht, es werden erst die Aüßeren Befehle ausgeführt und dann die inneren....

Ps. Ich kann auch 10000 logs eintragen, es werden auch dann erst die Äußeren komplett ausgeführt.

[paul53]

@ben1983 sagte: es werden auch dann erst die Äußeren komplett ausgeführt.

Schön, dass Du es mal getestet hast.

[fastfoot]

die sendTo Funktion arbeitet halt asynchron, d.h. ein Ergebnis wird irgendwann zurückgeliefert, nicht zwingend sofort. Das Hauptprogramm macht dann derweil andere Dinge bis das Ergebnis da ist. In dem Fall gibt man einen callback mit der dann das Ergebnis verarbeitet. Ist wie im richtigen Leben: Du rufst in die Küche bringste mal'n Bier mit, bitte? Jetzt gehts aber weiter mit Fussball in der Glotze, wenn das Bier dann da ist wirds getrunken, in der Glotze nix versäumt. Du könntest auch in den await Modus gehen, umgedreht Richtung Küche auf dein Bier wartend. Schauen kannste jetzt nicht mehr aber auf sonstige Ereignisse trotzdem reagieren(es fällt ein Tor, das hörst du und kannst sofort reagieren, Sch........... verpasst!!!).

So ähnlich verhält es sich mit async/await, während dem await ist zwar der laufende Code blockiert, auf Ereignisse(z.B onMessage anderswo im Skript) kann trotzdem reagiert werden. Um diese Funktionalität zu erhalten kann man Funktionen welche Callbachs verwenden in einen Wrapper mit Promise packen und dann mit await darauf warten.

const messageToAsync = async function (data) {
    return new Promise((resolve, reject) => {
        messageTo("addiere5", data, (erg) => {
            if (erg) resolve(erg);
        })
    })
}

log(await messageToAsync(3))
log(await messageToAsync(6))

Bei eigenen Funktionen kann man sich jetzt überlegen ob man sich das antun will oder lieber nach Alternativen sucht, viele ältere Tools (z.B request() )arbeiten jedoch noch mit Callbacks und da ist so ein Wrapper schon eine coole Sache. Für request() gibt es axios als Alternative, da kann man dann auch await verwenden

[Ben1983]

@fastfoot ok. danke.

[fastfoot]

@ben1983 das hast du aber schon geschnallt dass der cb nur einmal aufgerufen werden kann? Du bekommst auch keine 100 Bier wenn du nur einmal in die Küche rufst. Der cb ist da um ein Ergebnis zurück zu geben, der Aufrufer möchte auch nur ein Ergebnis haben und behandelt das auch nur einmal, ein mehrfacher Aufruf des cb ist Quatsch!!!

[Ben1983]

@fastfoot ja in dem Beispiel schon.
ein normaler einer funtion übergebener callback kann ja auch mehrfaxh aufgerufen werden.
natürlich macht es am meisten sinn das nur einmal zu machen.

[Ben1983]

@fastfoot Aber davon abgesehen sind für dinge die direkt zurück kommen "normale" funktionen doch die richtige Wahl.
wenn es etwas ist was dauern kann, dann asynchrone.... und wenn man warten will, dann eben die variante mit await.

oder?

[OliverIO]

@fastfoot

ich habe nur noch nicht verstanden, warum er überhaupt mit onmessage/messageto arbeiten möchte.
wenn er einfach nur eine funktion definiert, die ihm den wert zurück gibt ist es doch mit dem async egal

[fastfoot]

@oliverio der Ursprungsgedanke war etwas Globales zu haben, ohne den Ordner Global zu nutzen. Ich habe eig. nur zeigen wollen wie man auf das Ergebnis von sendTo warten kann. Über etwaige 'bessere' Lösungen hab ich mir gar keine Gedanken gemacht

[fastfoot]

@ben1983 ja aber es ging ja um das globale und deine Umsetzung mittels sendTo(). natürlich ist eine lokale Funktion da besser(und zuverlässiger!). Es hängt auch von der Funktion ab die du da global nutzen willst. Im Adapterbereich würde man wohl einen Import nutzen

[Ben1983]

@fastfoot ja da hast Du recht.
Ich wollte nur den oft versöhnten global Ordner umgehen. Aber es scheint ja doch nicht so wild zu sein den zu benutzen.

[ticaki]

@ben1983
Aber der globale Ordner macht doch nichts was mit messageTo() vergleichbar wäre. Ich dachte der wäre nur für häufig verwendete selbstgeschriebene Standardfunktionen oder Konstanten.

[Ben1983]

@ticaki ja es ging nur darum um das Mehrfache kopieren der Funktionen zu umgehen. Alles gut.