Eigenen Adapter erstellen

[apollon77]

@ben1983 Also das ist jetzt JavaScript Language Specifics

sobald Du eine "function() .." machst ist "this" auf diese function bezogen und die hat nun mal nicht diese Methode. Um das zu verhindern deklariert man funktionen über den Arrow operator () => ..., dann wird "this" nicht geändert.

Ansonsten (und ich versuche es einfach auszudrücken): DersetTimeout Parameter ist eine "function". "this.xy" ist aber eine Funktion einer Instanz und damit keine "natürliche function" und deswegen geht das nicht. Das lässt sich auch ändern, aber würde jetzt hier zu weit führen. Ich denke die notation

this.setTimeout(() => this.checkSchwimmerschalter(),1000);

passt.

(ja in dem Fall das deine Function genau ein Kommando ist kannst du die geschweiften Klammern weglassen)

[apollon77]

@ben1983 ALso hier müsste man jetzt mal verstehen was du genau tun willst - und warum du 10s warten musst.

Wenn Du sicher sein willst das eins nach dem anderen erfolgt dann ja async/await. Der js-controller 4 wird auch ein this.delay(ms) haben was man nutzen kann wo wir sicherstellen das das beim "Unload" auch abgebrochen wird.

[Ben1983]

@apollon77 warum geht es aber nicht, wenn ich den Funktionsaufruf ohne arrow function übergebe? Da deklariere ich ja nichts neu.
Rufe ja genau das gleiche je auf wie in der arrow Funktion???

[Ben1983]

@apollon77 ich möchte folgendes machen.
Entweder wenn die systemzeit eine bestimmte Zeit erreicht hat, oder ein Eingang (sxhwimmerschalter) sich ändert, möchte ich eine schrittkette durchlaufen die folgendes macht:

1. einen State auf True setzen.
   (Damit wird physikalisch ein Shelly aktiviert und ein Stellmotor läuft).
2. dann eine einstellbare Zeit x warten (verfahrzeot des Stellmotors)
3. nächsten State auf True setzen.
4. wieder warten zeit y.
5. fertig loggen.
6. schrittkette beendet verlassen

[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.