Jahresrückblick 2025 – unser neuer Blogbeitrag ist online! ✨

[echoDave]

Hallo zusammen,

ich bin in JS nicht wirklich fit, komme jedoch meistens klar.

Doch nun habe ich ein Problem bei dem ich nicht weiterkomme.

Ich versuche folgendes nachzubilden, wenn ein Fenster aufgemacht wird, wird Zeit X abgewartet, ist das Fenster nach dem Ablauf dieser Zeit offen wird in dem Zimmer das entsprechende Thermostat runtergeschraubt. Danach wird immer wieder nach Zeit X geguckt ob das Fenster offen ist, wenn es geschlossen ist, hört die "Schleife" auf.

Das ganze wollte ich mit einer Rekursion bewerkstelligen.

Ich habe eine on() Funktion auf alle Fenstersensoren. Feuert diese, ermittelt diese das Entsprechende Thermostat (über functions+rooms).

Anschließend wird eine zweite Funktion namens checkOpenFenster innerhalb eines Timeouts 1min aufgerufen.

In dieser Funktion wird zuerst geprüft ob das Fenster offen ist

Falls ja -> wird das Thermostat auf 12 Grad gesetzt und die Funktion ruft sich selbst innerhalb eines Timeouts wieder auf.

Falls nein -> wird das Thermostat auf den ursprünglichen Wert zurückgesetzt und die Funktion ruft sich nicht mehr wieder auf, somit hört der Kreislauf auf.

Nun habe ich das Problem, dass meine Funktion checkOpenFenster an der Stelle wo sie sich innerhalb eines Timeouts sich selbst aufruft, abschmiert.

Fehlermeldung:

error javascript.0 at Object. <anonymous>(script.js.common.Fenster:41:46)

die Funktion checkOpenFenster:

function checkOpenFenster(objId, thermId, room, oldTemp) {
    console.log('Fensterid: '+objId);
    console.log('ThermostatId: '+thermId);

    var fensterValue = getState(objId).val;
    var thermValue = getState(thermId).val;

    if (oldTemp) {
        console.log("old temp gesetzt");
    } else {
        console.log("old temp nicht gesetzt");
        oldTemp = thermValue;
    }
    // nur wenn das Fenster immer noch offen ist
    if(fensterValue) {
        console.log("Fenster auf im " + room + ", setze Objekt " + thermId + " auf 12.0.")
        setState(thermId, 12.0);
        var partsOfStr = thermId.split('.');
        console.log('Setze Timer:'+partsOfStr[2]);
        if (this[partsOfStr[2]]) {clearTimeout(this[partsOfStr[2]]); this[partsOfStr[2]] = null;};
        this[partsOfStr[2]] = setTimeout(function () {
            checkOpenFenster(objId, thermId, tmpRoom, oldTemp);         //<----- Zeile 41
        }, 1*5*1000);
    } else {
        console.log("Fenster zu im " + room + ", setze Objekt " + thermId + " zurück auf "+oldTemp)
        setState(thermId, oldTemp);
    }
}

Ich komme leider nicht auf die Lösung, kann mir einer von euch bitte aushelfen.

Viele Grüße und vielen Dank, David.</anonymous>

[echoDave]

Hat sich erledigt…..

checkOpenFenster(objId, thermId, <u>tmpRoom</u>, oldTemp)

Sollte an der stelle schon die richtige Variable verwenden, mit <u>room</u> klappt es schon besser :D