Licht vergessen funktioniert nicht mehr korrekt

[TT-Tom]

@fuchs1978
Bin zwar nicht der Profi, aber ich würde die Variable „timer“ nicht zweimal verwenden. Den Timer würde ich z.B. „Wartezeit“ nennen. Zeile 22 und 33

[paul53]

@fuchs1978 sagte: Erkennt ihr einen Fehler im Skript

Es muss auf Änderung getriggert werden, da sonst mehrere Timer gestartet werden können, die sich nicht mehr löschen lassen. Außerdem verwendet man nur einen Trigger pro Datenpunkt (Ressourcen).

on (melder, function (dp) { // Trigger auf Wertänderung
    if (dp.state.val) {
        // Aktionen bei Bewegung
    } else {
        // Aktionen ohne Bewegung
    }
});

@tt-tom sagte in Licht vergessen funktioniert nicht mehr korrekt:

Variable „timer“ nicht zweimal verwenden.

Sie wird nur einmal verwendet: In einem Teil gesetzt und in einem anderen Teil gelöscht. Da Timer-Variablen spezielle Objekte sind, setzt man Timer-Variablen auf null anstatt auf false.

[TT-Tom]

@paul53

@tt-tom sagte in Licht vergessen funktioniert nicht mehr korrekt:

Variable „timer“ nicht zweimal verwenden.

Sie wird nur einmal verwendet: In einem Teil gesetzt und in einem anderen Teil gelöscht. Da Timer-Variablen spezielle Objekte sind, setzt man Timer-Variablen auf null anstatt auf false.

Dann habe ich da noch eine Verständnisfrage: Zeile 21-23
Das If bezieht sich doch auf die Variable timer, die in 23 auf false gesetzt wird. Um das Objekt timer in 22 zu löschen.

Sollte nicht Objekt und Variable unterschiedliche Namen haben?

[paul53]

@tt-tom sagte: Variable timer, die in 23 auf false gesetzt wird. Um das Objekt timer in 22 zu löschen.

clearTimeout(timer) stoppt den Timer mit der Referenz timer, setzt aber nicht die Variable zurück. Damit man mit if(timer) die Variable abfragen kann, wird sie nach dem Stoppen auf false (besser null) gesetzt.

[TT-Tom]

@paul53

Okay so weit verstanden. Müssen also beide den gleichen Namen timer haben, oder?

[paul53]

@tt-tom sagte: Müssen also beide den gleichen Namen timer haben, oder?

Ja, es gibt nur ein setTimeout(), das die Variable timer setzt.

[TT-Tom]

@paul53

Danke, wieder etwas dazu gelernt. Schönes Wochenende

[fuchs1978]

@paul53 said in Licht vergessen funktioniert nicht mehr korrekt:

@fuchs1978 sagte: Erkennt ihr einen Fehler im Skript

Es muss auf Änderung getriggert werden, da sonst mehrere Timer gestartet werden können, die sich nicht mehr löschen lassen. Außerdem verwendet man nur einen Trigger pro Datenpunkt (Ressourcen).

on (melder, function (dp) { // Trigger auf Wertänderung
    if (dp.state.val) {
        // Aktionen bei Bewegung
    } else {
        // Aktionen ohne Bewegung
    }
});

Mache ich das nicht? Ich habe ja die Trigger mit Val unterschieden. Wir würdest du den Rest aufbauen. Komisch nur, das es erst seit kurzem nicht mehr korrekt funktioniert. Ich meine Seite node 16

[paul53]

@fuchs1978 sagte: Mache ich das nicht? Ich habe ja die Trigger mit Val unterschieden.

Nein. Wenn ein zweiter Trigger mit val: false kommt ohne vorherige Wertänderung?

[fuchs1978]

@paul53 said in Licht vergessen funktioniert nicht mehr korrekt:

@fuchs1978 sagte: Mache ich das nicht? Ich habe ja die Trigger mit Val unterschieden.

Nein. Wenn ein zweiter Trigger mit val: false kommt ohne vorherige Wertänderung?

Also dort noch eine Abfrage rein If Timer und damit nicht auf zwei reagieren