SetState() nach createState() schreibt nichts?

ruhr70

Danke für die Antwort! Das gibt mir was zum grübeln.

Du übergibst deinem "dpAnlegen" ein en Parameter beim AUfruf. Aber die Methode an sich hat dann keinen Parameter. Also wird der nicht übergeben und du machst ja auch nichts damit.

Du musst:

function dpAnlegen(callback) {

definieren und dann wenn in dpAnlegen alles fertig ist dann ein "callback();" einbauen das die callback-Funtion aufgerufen wird. `

So hatte ich das vorher. Aber die ganzen createState() waren dann beim callback(); anscheinend noch nicht fertig.

~~@apollon77:~~

Ein Kommentar übrigens aber: Du rufst in deiner dpAnlegen wieder alle createStates "synchron" auf. Das bedeutet: Wenn die die Schleife in dpAnlegen abgearbeitet hast ist von den createStates pot,. noch nichts abgearbeitet worden. Erst wenn Di dem Prozess wieder "freie Zeit" gibst hat er Zeit die ganzen Dinge dann asynchron abzuarbeiten. `

Das habe ich beim Probieren auch vermutet. Danke, für die Bestätigung.

Ich war so naiv, dass das mit callbacks am Ende einer Funktion irgendwie anders funktioniert

~~@apollon77:~~

In einer asynchronen Welt muss man das etwas Trickreicher machen.

Vllt kannst Du Dir in https://stackoverflow.com/questions/841 … ys-or-objs (die zweite Antwort) anregungen holen ... leider ist Zeit jetzt gerade knapp ein Beispiel zu basteln `

Auch dafür danke. Die zweite Antwort sieht interessant aus. Allerdings verursacht das ganze asynchrone Thema und die Callbacks einen Knoten im Kopf. Ich habe keine eile und schau mal, ob ich damit zurecht komme

apollon77

Ja, das asynchrone Zeug macht einmal nen Knoten im Hirn bis man es mal verarbeitet hat Aber dann ists logisch.

ruhr70

~~@apollon77:~~

In einer asynchronen Welt muss man das etwas Trickreicher machen.

Vllt kannst Du Dir in https://stackoverflow.com/questions/841 … ys-or-objs (die zweite Antwort) anregungen holen ... leider ist Zeit jetzt gerade knapp ein Beispiel zu basteln `

Ich habe mir jetzt die Zeit genommen und das Beispiel von Deinen Link (Danke!) mir näher angesehen.

Der Knoten im Kopf ist nicht weg

Als Ergebnis gibt es nun eine Funktion createStates(), die aus einem Array die Datenpunkte (hoffentlich) nacheinander anlegt.

var forceCreation = true;

var datenpunkteArr =[
    {   "name":             'test.a',
        "initialValue":     true,
        "forceCreation":    forceCreation,
        "common":           {name:"Mein Boolean DP",type:"boolean", role:"value"}
    },
    {   "name":             'test.b',
        "initialValue":     40,
        "forceCreation":    forceCreation,
        "common":           {name:"mein Number DP", type:"number",  role:"value"}
    },
    {   "name":             'test.c',
        "initialValue":     "hallo",
        "forceCreation":    forceCreation,
        "common":           {name:"Mein String DP", type:"string",  role:"value"}
    }
];

function createStates(arr, func, cb) {
    var doneCounter = 0;
    arr.forEach(function alleDpBearbeiten(obj) {
        func(obj.name, obj.initialValue, obj.forceCreation, obj.common, function () {
            doneCounter += 1;
            log(doneCounter+": " + obj.name+" mit Wert: " + obj.initialValue + " ("+obj.common.type+") angelegt");
            if (doneCounter === arr.length) {
                cb("alle Datenpunkte angelegt"); // mit der Funktion (callback) geht es weiter, wenn alles erledigt ist
            }
        });
    });
}

createStates(datenpunkteArr, createState, log);

apollon77

Sieht sinnig aus … passts denn?

ruhr70

~~@apollon77:~~

Sieht sinnig aus … passts denn? `

ja und nein… :?

Die Datenpunkte werden anscheinend alle brav hintereinander angelegt. Beim Alexa-Steuerungs-Skript sind es bei mir 98 Stück.

Das scheint zu funktionieren. Ich bekomme 98x mal hintereinander brav eine Rückmeldung von jedem createState, wenn dieses fertig ist.

Als Callback für ich dann danach nicht das log() aus dem Beispiel auf, sondern die Funktion, die die Subscriptions anlegt.

! function setOn() { var reg = new RegExp("^javascript\\."+instance+"\\." + devicePfad.replace('.', '\\.') + ".*\\.(volume|pause|play|next|forward|rewind|shuffle|tunein|radio|previous)"); on({"id":reg , "change": "any"}, function (obj) { var objArr = obj.id.match(/(^.+)\.(.+)\.(.+)$/, ""); //Aufteilung in Pfad + Device + CMD var device = objArr[2]; var cmd = objArr[3]; ! switch (cmd) { case "radio": if(obj.state.val) { // default Radiostation einschalten setState(devicePfad + device +".tunein",defaultRadiostation); return; } else { // Musik auf Pause. setState(devicePfad + device +".pause",true); return; } break; ! // Buttons, true zum auslösen. Werden danach wieder auf flase gesetzt case "pause": case "play": case "next": case "forward": case "rewind": case "previous": if(getState(obj.id).val) setState(obj.id,false); // Button wieder auf false zurücksetzen, wenn er true war if(obj.state.val) alexaCommand(device,cmd,true); // Kommando war ein true -> Kommando ausführen break; ! // Switches oder Wert case "shuffle": case "volume": case "tunein": alexaCommand(device,cmd,obj.state.val); break; ! default: log("Kommando << "+cmd+" >> im Skript nicht behandelt","warn"); break; } ! }); log("--- Subscriptions angelegt ---"); } !

Und hier kommt der Punkt, den ich nicht verstehe.

Das Skript verhält sich so, wie vorher.

Wenn bei der ersten Ausführung des Skriptes die Datenpunkte angelegt werden oder das Skript mir forceCreation=true ausgeführt wird, greift das on(), obwohl es erst nach den ganzen createStates ausgeführt wird.

D.h. wenn ein Datenpunkt neu angelegt wird, wird die Subscriptions ausgeführt.

Ich bin der Meinung, dass es nicht mehr an den asynchronen Aufruf liegen dürfte. Aber warum wird trotzdem "nachträglich" die Subscriptions ausgeführt?

Bei der Skriptausführung sieht das Log so aus:

22:33:17.972	[info]	javascript.1 script.js.Steuerung.Alexa_cURL_Kommandos: 96: alexa_device_command.other_devices.Michael_s_echosim_BETA._info.device_type_str mit Wert: Gerät unbekannt (string) angelegt
22:33:17.972	[info]	javascript.1 script.js.Steuerung.Alexa_cURL_Kommandos: 97: alexa_device_command.other_devices.Michael_s_echosim_BETA._info.softwareVersion mit Wert: 0 (string) angelegt
22:33:17.972	[info]	javascript.1 script.js.Steuerung.Alexa_cURL_Kommandos: 98: alexa_device_command.other_devices.Michael_s_echosim_BETA._info.capabilities mit Wert: ["SUPPORTS_CONNECTED_HOME_CLOUD_ONLY","KINDLE_BOOKS","AUDIBLE","AUDIO_PLAYER","SLEEP","TUNE_IN","CHANGE_NAME","I_HEART_RADIO","PERSISTENT_CONNECTION","AMAZON_MUSIC","POPTART","TIMERS_AND_ALARMS"] (string) angelegt
22:33:17.972	[info]	javascript.1 script.js.Steuerung.Alexa_cURL_Kommandos: --- Alexa Kommandos - Datenpunkte angelegt ---
22:33:19.964	[info]	javascript.1 script.js.Steuerung.Alexa_cURL_Kommandos: --- Subscriptions angelegt ---
22:33:20.388	[info]	javascript.1 script.js.Steuerung.Alexa_cURL_Kommandos: --- Alexa Kommandos - Skript initalisiert und bereit ---

Die Einträge …,96,97,98 werden im Callback des createStates ausgeführt.

die Subscriptions erst, wenn alles angelegt wurde.

apollon77

Woran siehst DU das die Subscriptions vorher schon ausgeführt werden?

ruhr70

Wenn die Datenpunkte das erste Mal angelegt werden, werden alle Datenpunkte einmal angesteuert.

Ich sehe die Kommandos, die die Subscriptions auslösen und alle Alexa Lautsprecher gehen an und spielen Radio.

apollon77

Heisst, er reagiert auf das setzen der initialen Werte auch wenn das "on" vom Ablauf her noch gar nicht gesetzt wurde …hm ... ich schau morgen mal nach ...

ruhr70

~~@apollon77:~~

Heisst, er reagiert auf das setzen der initialen Werte auch wenn das "on" vom Ablauf her noch gar nicht gesetzt wurde …hm ... ich schau morgen mal nach ... `

Genau. Ich habe das heute Morgen noch einmal getestet. Mal klappt es, mal klappt es nicht.

Als ob die Subscriptions manchmal noch aktiv wären, obwohl das Skript neu gestartet wurde.

[EDIT] Auszug des erweiterten Logs im Fehlerfall:

06:54:32.661	[info]	javascript.1 script.js.Steuerung.Alexa_cURL_Kommandos: 96: alexa_device_command.other_devices.Michael_s_echosim_BETA._info.device_type mit Wert: A3H674413M2EKB (string) angelegt
06:54:32.661	[info]	javascript.1 script.js.Steuerung.Alexa_cURL_Kommandos: 97: alexa_device_command.other_devices.Michael_s_echosim_BETA._info.essid mit Wert: null (string) angelegt
06:54:32.661	[info]	javascript.1 script.js.Steuerung.Alexa_cURL_Kommandos: 98: alexa_device_command.other_devices.Michael_s_echosim_BETA._info.macAddress mit Wert: null (string) angelegt
06:54:32.661	[info]	javascript.1 script.js.Steuerung.Alexa_cURL_Kommandos: --- Alexa Kommandos - Datenpunkte angelegt ---
06:54:32.661	[info]	javascript.1 script.js.Steuerung.Alexa_cURL_Kommandos: --- Subscriptions angelegt ---
06:54:32.661	[info]	javascript.1 script.js.Steuerung.Alexa_cURL_Kommandos: Kommando für: Arbeitszimmer_Echo_Dot_sw
06:54:32.661	[info]	javascript.1 script.js.Steuerung.Alexa_cURL_Kommandos: Kommando: {"type":"tunein","station":"s100198"}
06:54:32.661	[info]	javascript.1 script.js.Steuerung.Alexa_cURL_Kommandos: Data to request body: {"type":"tunein","station":"s100198"}
06:54:32.661	[info]	javascript.1 script.js.Steuerung.Alexa_cURL_Kommandos: Kommando für: Kueche_Echo_Dot_ws
... usw.

Die Datenpunkte haben alle einzeln zurückgemeldet,dass sie angelegt wurden.

Die Subscriptions werden angelegt.

Die Subscriptions reagieren auf die gerade vorher angelegten Werte.

apollon77

ich hab da nur eine Idee, dass am Ende die "setStates" die beim ANlegen der States passieren ja auch asynchron sind und daher bei der Ausführung durchaus dann nach den "on"s ausgeführt werden könnten.

Um das ggf rauszufinden wäre eine Idee das anlegen der subscriptions um 1-2 Sekunden zu verzögern … also ein "setTimeout(function ..., 2000) einzubauen. Das sollte/könnte den setStates genug Zeit geben um auch abgearbeitet zu sein.

Dann sieht man ob es daran lag ...

Oder Du baust eine "initDone" Variable ein die am Anfang "false" ist und nach Anlegen aller Subscriptions (ggf zeitverzögert) auf "true" geht und in deinem Subscription code prüfst Du ob die true ist und machst nur dann die Logik

AlCalzone

setState hat auch eine callback-Methode. Man könnte das so handhaben (Pseudocode, => steht für Callback):

createAllStates():
  für jeden State:
    createState => Alle States angelegt? Dann setAllStates()

setAllStates():
  für jeden State:
    createState => Alle States gefüllt? Dann init()

init():
  subscriptions anlegen

@OP: Welche Node-Version verwendest du? Wenn es mindestens 4.X ist kannst du auch auf Promises gehen. Ich poste dir dann gerne Code. Dann würde sich die Initialisierung grob wie folgt gestalten inklusive Ablaufkontrolle:

var createStates = [...]; // Array mit Promises füllen, die States anlegen
var fillStates = [...]; // Array mit Promises füllen, die States füllen
Promise.all(createStates)
  .then(function() {return Promise.all(fillStates)})
  .then(function() { /* Subscriptions und Skript initialisieren */ })
  ;

apollon77

Ich glaube er nutzt hier die Möglichkeit automatisch einen ersten Wert bei createState zu setzen … gerade nachgeschcut. Der clalback sollte dann erst aufgerufen werden wenn auch das setState durch ist ... Also komisch ists

ruhr70

@apollon77 und @AlCalzone: erst einmal herzlichenDank für Eure Gedanken und Mühe! :!:

Vorweg der aktuelle Stand, gerade getestet:

das Problem ist nicht immer reproduzierbar
alle Versuche vorhin waren so, wie sie sein sollten

Ich vermute, dass es ggf. an der Auslastung des Rechners liegt und dass das Setzen des initValue nicht im callback des createState() berücksichtigt ist.

Also immer noch in Teilen asynchrone :?:

~~@apollon77:~~

ich hab da nur eine Idee, dass am Ende die "setStates" die beim ANlegen der States passieren ja auch asynchron sind und daher bei der Ausführung durchaus dann nach den "on"s ausgeführt werden könnten. `

Du meinst, da ich das initialValue beim ersten Anlegen oder beim Überschreiben (nicht der normale Vorgang, nur wenn ich Änderungen in den Bezeichnungen durchgeführt habe) verwende, ruft der Javascript-Adapter innerhalb der Funktion createState() ein setState() auf und dieses ist im Callback von createState() noch nicht berücksichtigt?

Ich habe mal in den Code vom Javascript-Adapter reingeschaut.

Im createState gibt es u.a. diese Zeile:

if (initValue !== undefined) {
    adapter.setState(name, initValue, callback);

Ob das dann beim callback vom createState mit berücksichtig ist… da bin ich definitiv raus

~~@apollon77:~~

Um das ggf rauszufinden wäre eine Idee das anlegen der subscriptions um 1-2 Sekunden zu verzögern … also ein "setTimeout(function ..., 2000) einzubauen. Das sollte/könnte den setStates genug Zeit geben um auch abgearbeitet zu sein.

Dann sieht man ob es daran lag ... `

Hatte ich gemacht, bevor ich oben gepostet hatte. Hilft etwas… aber nicht immer.

~~@apollon77:~~

Oder Du baust eine "initDone" Variable ein die am Anfang "false" ist und nach Anlegen aller Subscriptions (ggf zeitverzögert) auf "true" geht und in deinem Subscription code prüfst Du ob die true ist und machst nur dann die Logik `

Verstehe ich nicht ganz. Das Problem liegt doch vorher oder?

Es sollten alles States gesetzt sein, bevor die Subscritions angelegt werden.

~~@AlCalzone:~~

setState hat auch eine callback-Methode. Man könnte das so handhaben (Pseudocode, => steht für Callback): `

Das wäre die Variante, wenn das createState wirklich die dort enthaltenen setStates asynchrone aufruft?

Also auf initialValue verzichten.

Dann kommt hinzu, dass vorher geprüft werden muss, ob die Werte gesetzt werden müssen (nur beim ersten Start oder bei forcecreation)?

~~@AlCalzone:~~

@OP: Welche Node-Version verwendest du? Wenn es mindestens 4.X ist kannst du auch auf Promises gehen. Ich poste dir dann gerne Code. Dann würde sich die Initialisierung grob wie folgt gestalten inklusive Ablaufkontrolle: `

Ich selbst verwende 4.x. Ich überlege, ob es sich lohnt das Skript als Prototype für einen Adapter zu verwenden.

Denke da an die Abwärtskompatibelität…

Da ich das mit den Callbacks() wohl nicht mehr so schnell in den Kopf bekomme, bietet es sich an doch mal Promises anzusehen

Kann das direkt verwendet werden oder muss da noch was geladen werden?

Aber prinzipiell würde ich gerne verstehen, woran es jetzt gerade das Problem ist

AlCalzone

~~@ruhr70:~~

Da ich das mit den Callbacks() wohl nicht mehr so schnell in den Kopf bekomme, bietet es sich an doch mal Promises anzusehen

Kann das direkt verwendet werden oder muss da noch was geladen werden? `
Ab Node 4.x ist das eigentlich nativ unterstützt. Das Skript läuft dann aber nicht unter einer 0.10.x-Umgebung, das muss dir klar sein.

Ich sehe zu, dass ich dir heute Abend mal ein bisschen Code bastele zum Testen.

~~@ruhr70:~~

Aber prinzipiell würde ich gerne verstehen, woran es jetzt gerade das Problem ist `
Dass dein Problem nicht immer reproduzierbar ist, klingt für mich nach Timing-Problemen durch asynchronen Programmablauf. Wenn eine Aktion dann mal länger dauert, hakts woanders.

apollon77

Aber warte mal … vllt löst sich dein Problem bald von selbst:

http://forum.iobroker.net/viewtopic.php?p=63889#p63889

AlCalzone

Die Funktionen lassen sich vorübergehend sehr leicht "synchron" nachrüsten. Wie gesagt, Code liefere ich später

Den dazu benötigten Nachrüst-Code kann man dann - wenn es soweit ist - wieder entfernen, die Logik funktioniert weiterhin.

–-

Edit: Anscheinend sind die neuen Methoden dann wirklich synchron, und nicht als Promises gekapselt wie angenommen.

Musst du wissen @ruhr70 ob du jetzt was haben willst, oder noch ein bisschen warten kannst.

ruhr70

~~@AlCalzone:~~

Edit: Anscheinend sind die neuen Methoden dann wirklich synchron, und nicht als Promises gekapselt wie angenommen.

Musst du wissen @ruhr70 ob du jetzt was haben willst, oder noch ein bisschen warten kannst. `

Beim Skript bin ich erst einmal im Reinen. Bei der Funktion. createState() synchron mit Callback und dann setTimeout für die Subscriptions.

Ist der schlimmste Spagetti-Code geworden… aber nun ja.

Ich kann also warten ;-). Das Promises Thema werde ich mir aber trotzdem mal ansehen. Nur die Adapter-Funktionen werden ja dann synchrone. Für das ganze drumherum braucht man dann auch eine Lösung... also Promises!?

AlCalzone

Ich hatte Code versprochen…

Promise-Kapselung um eine Funktion mit Callback:

// Erwartet eine Funktion, die als letzten Parameter einen Node-JS-Callback annimmt.
function promisify(fn, context) {
	return function (args) {
		context = context || this;
		return new Promise(function (resolve, reject) {
			args.push(function (error, result) {
				if (error)
					return reject(error);
				else
					return resolve(result);
			});
			fn.apply(context, args);
		});
	};
}

setState und createState mit Promise-Kapselung:

	var 
		$setState = promisify(setState),
		$createState = promisify(createState)
		;

–-

Deine Programmlogik ist mir noch nicht 100% klar, aber ausgehend von dem Code http://forum.iobroker.net/viewtopic.php?p=63432#p63432, könntest du es mal so probieren:

	// Datenpunkte in ein Array aus Promises umwandeln
	var createDPs = datenpunkteArr.map(function(dp) {
		return $createState(dp.name, dp.initialValue, dp.forceCreation, dp.common);
	});
	// Auf Ausführung warten
	Promise.all(createDPs)
		.then(function() {
			// dann mit der Logik weitermachen, z.B. States füllen (geht ähnlich, siehe einer meiner früheren Posts)
			// oder Subscriptions starten

			// TODO
		});

ruhr70

Puh… Danke!!!

na, da habe ich ja was zu tun

NEWS

SetState() nach createState() schreibt nichts?

Support us

813
Online

31.7k
Users

79.8k
Topics

1.3m
Posts

NEWS

SetState() nach createState() schreibt nichts?

Support us

813Online

31.7kUsers

79.8kTopics

1.3mPosts

813
Online

31.7k
Users

79.8k
Topics

1.3m
Posts