Wartung am 15.11. – Forum ab 22:00 Uhr nicht erreichbar

[Ben1983]

@apollon77 weist DU was das hier bedeuten soll?

ich habe diese zeile:

this.log.info("Schwimmer = " + this.config.schwimmerschalter);

und bekomme diesen Fehler:

zisternensteuerung.0
2022-02-02 22:59:34.398	error	Cannot read properties of undefined (reading 'log')
zisternensteuerung.0
2022-02-02 22:59:34.397	error	TypeError: Cannot read properties of undefined (reading 'log') at checkSchwimmerschalter (C:\vscode\iobroker adapter\ioBroker.zisternensteuerung\.dev-server\default\node_modules\iobroker.zisternensteuerung\main.js:61:8) at Timeout._onTimeout (C:\vscode\iobroker adapter\ioBroker.zisternensteuerung\.dev-server\default\node_modules\iobroker.js-controller\lib\adapter.js:1273:17) at listOnTimeout (node:internal/timers:557:17) at processTimers (node:internal/timers:500:7)
zisternensteuerung.0
2022-02-02 22:59:34.394	error	unhandled promise rejection: Cannot read properties of undefined (reading 'log')
zisternensteuerung.0
2022-02-02 22:59:34.393	error	Unhandled promise rejection. This error originated either by throwing inside of an async function without a catch block, or by rejecting a promise which was not handled with .catch().

Aufgerufen wird der log innerhalb einer function.
diese wird bei wertänderung aufgerufen. (Ohne Timeout klappt es)

this.setTimeout(this.checkSchwimmerschalter,1000);

wenn ich es so aufrufe, dann geht es:

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

warum geht es nicht wie oben, wenn ich die function direkt aufrufe?
auch wenn man es so schreibt, geht es nicht:

this.setTimeout(function(){
   this.checkSchwimmerschalter();
},1000);

[OliverIO]

@ben1983 said in Eigenen Adapter erstellen:

@oliverio ok.
Würde es halt gerne verstehen, wo das her kommt.
Aber zurück zu meiner anderen Frage,
Wie realisiert ihr denn zyklische Abläufe und Timer?

schau in meine adapter.

verwende setTimeout.
setInterval wird dann im Review angemeckert

[Ben1983]

@oliverio mache ich.
Wieso gibt es denn jetzt bei einem getforeignState einen callback den ich übergeben will? möchte doch nur den Wert haben.

[OliverIO]

@ben1983 said in Eigenen Adapter erstellen:

this.log.info

this.log.info
muss heißen
this.adapter.log.info

this.setTimeout(this.checkSchwimmerschalter,1000);

leider hast du die Fehlermeldung nicht mitgegeben.
grundsätzlich muss das schon funktionieren. Wahrscheinlich ist nicht das Problem, das die Funktion this.checkSchwimmerschalter nicht aufgerufen wird, sondern
das innerhalb der Funktion der scope nix mehr mit dem Adapter zu tun hat und du mit this nicht mehr auf die Adapterfunktionen zugreifen kannst.
Um das zu können, musst du entweder die Funktion in ein ES6-Konstrukt einpacken oder den gewünschten Scope mit bind mitgeben.
angewandt auf dein Beispiel mache ich das so:

this.setTimeout(this.checkSchwimmerschalter.bind(this),1000);

[OliverIO]

@ben1983 said in Eigenen Adapter erstellen:

@oliverio mache ich.
Wieso gibt es denn jetzt bei einem getforeignState einen callback den ich übergeben will? möchte doch nur den Wert haben.

du bist mit asynchronen aufrufen in javascript vertraut? ist eines der grundprinzipien.
das ermitteln der Daten dauert ein wenig. jetzt könntest du natürlich solange die programmverarbeitung anhalten, dann würde aber parallel nix anderes auf dem server mehr gehen.
daher gibtst du eine callback-adresse mit, welche aufgerufen wird, wenn der Rückgabewert bereit steht.

Wenn du auf callbacks verzichten willst, dann musst du diese Funktionen in promises umwandeln. kommt aber vom aufwand auf das selbe raus. sieht aber besser aus und der code wird strukturierter
https://developer.mozilla.org/de/docs/Web/JavaScript/Guide/Using_promises

verwendest du eigentlich einen debugger?
damit würden einige fragen sich erübrigen, wenn du zur Laufzeit in die variablen und Objekte reinschauen kannst.

und schaue in andere adapter. ich habe dir oben als Beispiele meine Adapter aufgelistet, die mehr oder weniger schon genau die Technologien verwenden, die du benötigst.

[OliverIO]

@ben1983 said in Eigenen Adapter erstellen:

@apollon77 ok,
wie würdest DU denn dann eine Schrittkette aufbauen?
asynch und await?
das ist dann auch wieder blöd, wenn ich die schrittkette abbrechen möchte, wenn das await noch nicht fertig ist.

Bin gespannt auf deinen Vorschlag?
immer einen schritt und dann wieder setTimeout?
wären möglich... nicht ganz symmetrisch, aber denkbar.
Aber wie würdest Du innerhalb einer schrittkette bspw. 10s warten?
Oder wie würde man einen ablauf machen, der entweder bei einem bestimmten wert eines datenpunkts oder nach 10s in den nächsten schritt springt?

erzähle mal genau was du in welcher reihenfolge du schalten möchtest und was das am anderen ende ist, was du da schalten willst und wie man mit denen kommunizieren kann.
die genaue vorgehensweise hängt von diesen möglichkeiten ab.

[apollon77]

@oliverio sagte in Eigenen Adapter erstellen:

this.log.info
muss heißen
this.adapter.log.info

ääähhm .. neee?

Je nachdem wie der gesamte Code ist (ob ES6 Class oder old style) ist es entweder this.log oder adapter.log ... aber am Ende sind wir jetzt an einer Stelle wo man mehr code sehen muss. am Ende ist die Frage wo Sdu deine methode hingepackt hast und wann das ausgefphrt wird. Wenn "thislog" nicht da ist dann läuft Dein Code ggf CVOR dem "on(ready)" und dann ist DAS der Fehler!

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