Eigenen Adapter erstellen

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

[Ben1983]

@apollon77 sagte in Eigenen Adapter erstellen:

JSON.stringify(wert)

Danke, das war es. Funktioniert jetzt.
Nun habe ich nur noch die Funktion der Schrittkette zu lösen.
Hat jemand eine Idee, wie man Wartezeiten innerhalb einer Schrittkette elegant löst?
Also die Schrittkette muss natürlich auch abgebrochen werden können.
dann wäre ich ja mit einem await nicht so gut bedient, oder?

Und habe ich irgendwo eine Systemzeit, welche ich auf bspw. "22:30" vergleichen kann?

[Ben1983]

@apollon77 Was bedeutet diese Meldung?
Warum wird er teminiert?

host.dev-zisternensteuerung-DESKTOP-JHNIDES
2022-02-06 14:14:01.768	error	instance system.adapter.zisternensteuerung.0 terminated with code 7 (ADAPTER_ALREADY_RUNNING)

zisternensteuerung.0
2022-02-06 14:14:00.207	warn	Got terminate signal. Checking desired PID: 6452 vs own PID 7400
host.dev-zisternensteuerung-DESKTOP-JHNIDES
2022-02-06 14:14:00.183	info	instance system.adapter.zisternensteuerung.0 started with pid 6452

zisternensteuerung.0
2022-02-06 14:14:00.178	warn	Got terminate signal. Checking desired PID: 0 vs own PID 7400