UPDATE 31.10.: Amazon Alexa - ioBroker Skill läuft aus ?

[apollon77]

@ben1983 Ja, am Ende ist eine "Funktion einer Instanz" intern keine "function" die man hier übergeben kann weil Sie an das Objekt gebunden ist und nicht an "global". Wenn setTimeout die funktion ausführt geht daher das Objekt-"binding" verloren - es sei denn du setzt es explizit

[Ben1983]

@apollon77 das bedeutet, mit einem direkten Aufruf als callback, oder mit function(){This.myfunctiln()} geht es nicht,
Aber in einer arrow function geht es?

Das liegt also an dem callback der settimeout Funktion?
Denn wenn ich direkt aufrufe gehts ja.

Oder solltest dann anders auf bspw. Log zugreifen?

Ps. Jemand ne Idee zu meinem Vorhaben?

[OliverIO]

@ben1983
ja, lese meinen post oben
den abschnitt mit bind
ist etwas anstrengend mit dir.
die liest die posts nicht richtig
du antwortest nicht auf die fragen
fragst die gleichen fragen dann wieder andere

[apollon77]

@ben1983 Ok, schick mal den GitHub lInk zu dem code bitte in Gänze. Wenn du eine Methiode aufrufst und dort this.log nicht da ist dann ist was anderes im argen. Ich vermute das du code ausführts ohne das vorher on(ready) aufgerufen wurde und damit ist es einfach zu früh

[Ben1983]

@apollon77 muss ich die Tage mal hoch laden.
Bin unterwegs. Aber:
Onready wurde ausgeführt und this.log soll ja ausgeführt werden.
Es geht ja auch bei direktaufruf der Funktion, oder in der arrowfunktion.
Nur als callback nicht.

Vielleicht kann ich es später noch hochladen

[Ben1983]

@oliverio Sorry. Ich stelle die Fragen ja auch nur, weil ich mit der Antwort nichts anfangen kann, oder eventuell was anderes gemeint war.

Sollte man eigentlich generell die interne Funktion zur Abonierung einer State Änderung nutzen, oder vielleicht auch die on Funktion?

[Ben1983]

@apollon77 Hir der link zu github:

ioBroker.zisternensteuerung

[apollon77]

@ben1983 Ok, und mit setInterval(() => this.checkSchwimmerschalter()) kommt das "this log gibbet nicht" auch?

[Ben1983]

@apollon77 sagte in Eigenen Adapter erstellen:

@ben1983 Ok, und mit setInterval(() => this.checkSchwimmerschalter()) kommt das "this log gibbet nicht" auch?

Nein damit geht es.
Wo hast Du das denn im Code gefunden? Ist doch gar nicht mehr da drin.

Der Fehler kommt hier:

this.activeInterval = this.setInterval(this.checkSchwimmerschalter,1000);

Das liegt dann ja wahrscheinlich am Binding.
Sollte es mit

this.activeInterval = this.setInterval(this.checkSchwimmerschalter.Bind(this),1000);

Gehen?

Da erscheint ein Fehler, dass kein Catch zum abfangen erzeugt wurde.
Aber warum brauche ich das?

Und warum liefert mir die getForeignState Funktion einen error?

Nächste Frage:
Am Anfang logge ich ja die config Daten mit.
Warum stehen die da noch auf undefined?

[apollon77]

@ben1983 sagte in Eigenen Adapter erstellen:

Das liegt dann ja wahrscheinlich am Binding.

Exakt, es ist dann "this" nicht mehr auf das objekt gesetzt und damit geht this.log schieff

[apollon77]

@ben1983 sagte in Eigenen Adapter erstellen:

Sollte es mit
this.activeInterval = this.setInterval(this.checkSchwimmerschalter.Bind(this),1000);

Gehen?

wenn du bind klein schreibst ja.

Da erscheint ein Fehler, dass kein Catch zum abfangen erzeugt wurde.
Aber warum brauche ich das?
Und warum liefert mir die getForeignState Funktion einen error?

welcher fehler kommt denn?
Was ist denn this.config.schwimmerschalter ?? Ist die Idee das der user eine State-ID angibt die du dann nutzt oder wie?

Nächste Frage:
Am Anfang logge ich ja die config Daten mit.
Warum stehen die da noch auf undefined?

naja du hast eine funktion definiert mit 2 Parametern und rufst iIe mit nur einem auf. Damit ist der zweite undefined

[Ben1983]

@apollon77 Das funktioniert schon mal so:

this.activeInterval = this.setInterval(this.checkSchwimmerschalter.bind(this),1000);

Ist nur die Frage, ob man das so machen sollte um eine Schrittkette zu starten.
Würde ja auch gerne eine async funktion erstellen, welche dann mit await einzelne schritte verzögert, aber ein await kann man nicht mehr abbrechen, oder?

Danke für den Tip mit den zwei Parametern.
Geht jetzt auch

[Ben1983]

@apollon77

welcher fehler kommt denn?
es erscheint der Fehler:

zisternensteuerung.0.2022-02-03 22:29:01.789	error	[object Object]

Was ist denn this.config.schwimmerschalter ?? Ist die Idee das der user eine State-ID angibt die du dann nutzt oder wie?

Ja, genau das ist die Idee, oder kann man die irgendwie via Dropdown auswählen?
Dachte so muss man die Datenpunkte nicht noch extra in irgend einem Skript zuweisen.
Oder würdest Du es lieber nicht so machen?

[Ben1983]

@ben1983 habe das GetForeignState so umgangen:

async checkSchwimmerschalter(){if(this._schwimmerschalter)
{
	this.setToZisterne();
}
}
async setToZisterne()
{		 
this.setForeignStateAsync(this.config.ansteuerungFrischwasserStellmotor,false,true);	
await new Promise(resolve =>this.setTimeout(resolve,this.config.verfahrzeitFrischwassermotor));
this.setForeignStateAsync(this.config.ansteuerungZisternenstellmotor,false,true);	}


	onStateChange(id, state) {
		if (state) {
			// The state was changed
			this.log.info(`state ${id} changed: ${state.val} (ack = ${state.ack})`);
			this.log.info(this.config.schwimmerschalter);
			if(id == this.config.schwimmerschalter)
			{
			
			//this.activeInterval = this.setInterval(this.checkSchwimmerschalter.bind(this),1000);
			this._schwimmerschalter = state.val;
			this.checkSchwimmerschalter();
			}

		} else {
			// The state was deleted
			this.log.info(`state ${id} deleted`);
		}
	}

Also den Wert bei Wertänderung in eine lokale Variable geschrieben.
Aber trotzdem müsste doch das getForeignState auch gehen, oder?

Was ist denn der Unterschied, wenn man mit der on funktion oder der subscribe und dann der internen onchange methode arbeitet?

[Ben1983]

@ben1983 Hier nochmal der Link zur aktuellen Version:
Zisternensteuerung

ich frage den State.val ab, aber leider (egal ob true oder false) wird irgendwie immer der Else zweig ausgeführt und damit immer (setToFrischwasser)
Hier die Abfrage

	async checkSchwimmerschalter()
	{
		this.log.info(this._schwimmerschalter.val);
		if(this._schwimmerschalter.val == this.config.zisternenlogik)
		{
			this.log.info("Start set to Zisterne");
			this.setToZisterne();
		}
		else
		{
			this.log.info("Start set to Frischwasser");
			this.setToFrischwasser();
		}
	}

Der log oben drüber liefert den korrekten Wert.

Fällt jemand ein Fehler auf?

[Ben1983]

@apollon77 natürlich sollte der letzte Post an Dich gehen.

[apollon77]

@ben1983 Naja wenn der Vergleich sagt es ist nicht gleich dann ists nicht gleich. Log mal beides und zeig mal log

[Ben1983]

@apollon77 laut log sollte es aber erfüllt sein.
Ich habe auch schon die Bedingung dahingehend geändert, dass ich auf == false abgefragt habe und egal welcher Zustand der Parameter angenommen hat wurde trotzdem immer der else Zweig ausgeführt.
Echt sehr sehr merkwürdig.
Vielleicht kann jemand mal das Ding über github testen, ob es Bei euch auch so ist?

[apollon77]

@ben1983 Logge doch bitte mal beide werte!!!! Am besten mit "JSON.stringify(wert)" weil wenn es ein boolean ist aber dein wert ggf ein string ist hast du "

false == "false"

und das stimmt natürlich nicht. beachte bitte datentypen!!

[Ben1983]

@apollon77 ah ok.
Ich bin unterwegs. Am besten schaue ich noch mal, dass der Type wirklich boolean ist.
In den Objekten musste ich bis jetzt true oder false eintragen.
Nicht anhaken.