Timeout in JS einbinden

[OliverIO]

@ubecker
das geht auch schon in der kostenlosen version mit chatgpt

[paul53]

@oliverio sagte: Durch die globale timeout-Variable wird verhindert, dass mehrere setTimeout()-Aufrufe gleichzeitig laufen.

Das ist bei analogen Vergleichswerten auch erforderlich, da andernfalls clearTimeout(timeout) nicht funktioniert.
Prinzip:

    if (Vergleich von Analogwerten) {
        setState('javascript.0.Heizung_Kessel_aktiv', true, true);
        // ⛔ Bereits laufenden Ausschalt-Timer abbrechen
        if (timeout) {
            clearTimeout(timeout);
            timeout = null;
        }
    } else if (!timeout) {
        timeout = setTimeout(() => {
            setState('javascript.0.Heizung_Kessel_aktiv', false, true);
        }, 300 * 60 * 1000); // 300 Minuten = 5 Stunden
    }

[OliverIO]

@paul53 das was copy/paste aus chatgpt

[ubecker]

@oliverio said in Timeout in JS einbinden:

nächster schritt

Spoiler

Perfekt! Hier ist dein Skript mit korrekt implementiertem Timeout-Handling, sodass clearTimeout(timeout) wie gewünscht funktioniert und nicht bei jedem Trigger-Ereignis verloren geht:

---

Finale Version mit persistentem Timeout

// 🔥 Heizung ein-/ausschalten bei Energieanforderung

const ids = 'javascript.0.Array_IDs_Heizung_level';       // Datenpunkt mit Array von Level-IDs
const level_value = 20.1;                                  // Schwellenwert zur Aktivierung der Heizung
const ID_Liste_Level = getState(ids).val;                  // Array der zu überwachenden IDs

let timeout = null; // ⏱ Timeout-Variable global definieren

// 📡 Trigger bei jeder Änderung eines Wertes in der Liste
on({ id: [].concat(ID_Liste_Level), change: 'ne' }, async (obj) => {
    let gesamt_value = 0.0;

    // 🔁 Summiere alle aktuellen Werte
    for (let i = 0; i < ID_Liste_Level.length; i++) {
        let val = getState(ID_Liste_Level[i]).val;
        gesamt_value += val;
    }

    // ✅ Wenn Schwelle erreicht oder überschritten → Heizung AN
    if (gesamt_value >= level_value) {
        setState('javascript.0.Heizung_Kessel_aktiv', true, true);

        // ⛔ Bereits laufenden Ausschalt-Timer abbrechen
        if (timeout !== null) {
            clearTimeout(timeout);
            timeout = null;
        }
    }
    // ❄️ Wenn unter Schwelle → Heizung AUS (nach 5 Stunden)
    else {
        if (timeout === null) {
            timeout = setTimeout(() => {
                setState('javascript.0.Heizung_Kessel_aktiv', false, true);
                timeout = null; // Timer zurücksetzen
            }, 300 * 60 * 1000); // 300 Minuten = 5 Stunden
        }
    }

    // 📋 Protokolliere aktuellen Summenwert
    console.log('Gesamtanforderung:', gesamt_value);
});

---

🧠 Vorteil der Änderung

Durch die globale timeout-Variable wird verhindert, dass mehrere setTimeout()-Aufrufe gleichzeitig laufen. Stattdessen gibt es genau einen Ausschalt-Timer, der bei Bedarf neu gesetzt oder gelöscht wird.

Wenn du möchtest, kann ich auch eine erweiterte Version mit Logging, Telegram-Benachrichtigung oder Visualisierung vorbereiten.

super; einzig was noch fehlt sind die Bedingungen wenn Heizug schon auf true steht braucht nicht wieder gesetzt zu werden, auch beim ausschalten. sind aber nur 2 if Bedingungen.

[ubecker]

@oliverio
kleiner Fehler in dem Script:
console.log('Gesamtanforderung:', gesamt_value);
funktioniert so nicht da erst ein String und dann eine Zahl.
Zahl nach String konvertieren oder einfach den String 'Gesamtanforderung:' weglassen

sonst alles ok, meine if Bedingung zum setzen hab ich auch eingebaut.

nochmals Danke.

[MCU]

@ubecker sagte in Timeout in JS einbinden:

console.log('Gesamtanforderung:', gesamt_value);

console.log('Gesamtanforderung: '+ gesamt_value);

[paul53]

@ubecker sagte: funktioniert so nicht da erst ein String und dann eine Zahl.

Funktioniert, wenn das Komma durch ein Plus ersetzt wird.

[OliverIO]

@ubecker sagte in Timeout in JS einbinden:

console.log('Gesamtanforderung:', gesamt_value);

wie gesagt, das skript ist nicht von mir überarbeitet worden, sondern von chatGPT
nach Definition ist das richtig
https://developer.mozilla.org/en-US/docs/Web/API/console/log_static
https://nodejs.org/api/console.html#consolelogdata-args
console.log nimmt 1 bis N parameter. jeder parameter wird gemäß typ so ausgegeben

mit + wird automatisch von javascript der typ aus gesamt_value nach string gewandelt und dann das ergebnis angehängt.

da scheint es im javascript adapter noch was geben
hier die ausgabe aus der browser console

und aus der node repl (einfach node starten)

habe mal geschaut.
da ist die typdefinition im javascript adapter nicht ganz korrekt.
die console befehle kennen alle nur einen parameter.
allerdings ist das auch nicht die priorität, ist ja eh nur für debugging
https://github.com/ioBroker/ioBroker.javascript/blob/64c375ea0e0840f21a7cc12582c970022f061fa2/src/types.d.ts#L570

[peterfido]

@oliverio Für solche Zeitspannen setze ich auf einen Datenpunkt, welcher minutiös heruntergezählt wird. Arbeitet man am ioBroker innerhalb der fünf Stunden, kann es sonst passieren, dass das Timeout verlorengeht.

[OliverIO]

@peterfido
Besser:
Den Ende timestamp in einem datenpunkt merken und dann regelmäßig prüfen, ob er erreicht wird. Bzw. der datenpunkt existiert nur falls der Adapter neu gestartet wird, damit man wieder neu aufsetzen kann.
Dann schreibt man nicht so oft in der Datenbank herum.

[peterfido]

@oliverio Der Schreibzyklen wegen vielleicht schon. Darüber habe ich mir zu Raspi-Zeiten tatsächlich Gedanken gemacht.Also, wie so oft, ein "kommt darauf an".

[ubecker]

@oliverio said in Timeout in JS einbinden:

allerdings ist das auch nicht die priorität, ist ja eh nur für debugging

genau.

@oliverio said in Timeout in JS einbinden:

Den Ende timestamp in einem datenpunkt merken und dann regelmäßig prüfen, ob er erreicht wird. Bzw. der datenpunkt existiert nur falls der Adapter neu gestartet wird, damit man wieder neu aufsetzen kann.
Dann schreibt man nicht so oft in der Datenbank herum

hier eigentlich unkritisch. Dann bleibt die Heizung weitere 5 Std. an.
ich lass es erst mal beim normalen Teimout und werde das beobachten.

danke an alle