[gelöst] Elegantere Programmierung?

[paul53]

@skorpil sagte: wie ich das anpassen kann?

Lese die Beschreibung erst in der Callback-Funktion von Timeout ein (über Zeile 53). Nur verzögert zu senden, bringt nichts.

[skorpil]

@paul53 Dankeschön, ich teste!

[paul53]

@skorpil
Zeilen 69 bis 77 kann man vereinfachen:

    log(anwesend ? "einer da" : "Keiner zuhause");
    setState(IDAnwesend, anwesend);

[skorpil]

@paul53 merci

[skorpil]

@paul53 das war es! Im Realbetrieb getestet. Mit dieser kleinen Modifikation läuft alles perfekt. Vielen Dank.

[skorpil]

bitte Logik Hilfe im Java Script. Ich habe testweise folgendes kleine Script geschrieben:

const x = false;
const y = true;

if(x == true && y == true) { 
    log("beide wahr");
    }
    else {
    log("einer ist wahr") };

Es geht mir um die ELSE Bedingung. Wie kann ich das Ergebnis der ELSE Bedingung ohne den Umweg über else direkt in der If Anweisung abfragen?

Mit anderen Worten: wenn eine der Bedingungen in der If Anweisung false ist, dann soll der nachfolgende Code NICHT ausgeführt werden. Ich sehe gerade den Wald vor lauter Bäumen nicht.

[Codierknecht]

@skorpil
Dann lass doch einfach den Else-Zweig komplett weg

const x = false;
const y = true;

if(x == true && y == true) { 
    log("beide wahr");
}

[Codierknecht]

@skorpil
Und wenn die beiden Variablen eh Boolean sind, kannst Du das noch weiter vereinfachen

const x = false;
const y = true;

if(x && y) { 
    log("beide wahr");
}

[paul53]

@skorpil sagte: wenn eine der Bedingungen in der If Anweisung false ist

Meinst Du: Nur eine der Bedingungen?

if(x && y) log('beide wahr');
if(!x && !y) log('keiner wahr');

oder

if(x && y) log('beide wahr');
else if(x || y) log('nur einer wahr');
else log('keiner wahr');

[skorpil]

Erstmal danke Euch beiden.

Beide Datenwerte entstammen einer Systemvariablen aus der CCU mit der Werteliste „ein;aus“ und können die Werte 0 und 1 oder eben false und true annehmen. Dummerweise habe ich das vor vielen Jahren so anlegt. Aus heutiger Sicht mit JavaScript ist das unlogisch: ein = 0 = false. Ist aber nun so.

Ich habe zwei Variablen dieser Art. Es geht nun darum, die If Anweisung NICHT dann auszuführen, wenn entweder einer der Werte „aus“ ist oder beide!

Ich bin ein Logik Legastheniker.

[paul53]

@skorpil sagte: Anweisung NICHT dann auszuführen, wenn entweder einer der Werte „aus“ ist oder beide!

Anders ausgedrückt: Die Anweisung ausführen, wenn beide "ein" (false oder 0) sind.

if(!x && !y) log('beide ein');
// oder
if(!(x || y)) log('beide ein');

[Codierknecht]

Hat @paul53 ja bereits geschrieben

[skorpil]

@paul53, @Codierknecht Daaaanke

[skorpil]

dieses Script sendet mir alle paar Minuten eine push Nachricht. Warum? Was ist falsch?

Leider als Bild und nicht als Code, weil ich auf dem iPad den Code nicht kopieren kann (oder zu dumm dazu bin…)

[paul53]

@skorpil sagte: alle paar Minuten eine push Nachricht. Warum?

Vermutlich wird der Trigger-DP zyklisch aktualisiert. Ändere in

on({id: IDAusloeser, val: false, change: 'ne'}, function() {
    if(getState(IDAnwesend).val) {

[skorpil]

@paul53 danke, das war es

[skorpil]

Schönes neues Jahr Euch allen!

Frage:

wenn mein Garagentor auf geht (Neigungssensor) oder der Lichtschalter betätigt wird, soll, wenn es dunkel ist (Brightness vom Bewegungsmldr. < 80) das Licht für 1 Minute an- und dann wieder ausgehen.

Ist die Programmierung so okay? Kann ich das Ausschalten mit setStateDelayed innerhalb der if-Anweisung so erreichen?

Danke für Eure Hilfe.

// ###########################################
//            Deklarationen
//     (An- UND ABwesenheit detektieren)
// ###########################################
 
const idNeigungGarageALT = 'hm-rpc.0.JEQ0498248.1.STATE'/*Neigungssensor Gar  ALT (HM-Sec-TiS JEQ0498248:1) STATE*/;
const idLichtInGarage = 'hm-rpc.0.JEQ0097321.1.STATE'/*Lichtsch  in Garage STATE*/;
const idBewMldrGartenHell = 'hm-rpc.0.LEQ0657464.1.BRIGHTNESS'/*Bew Mldr Garten:1 BRIGHTNESS*/ ;
const idLichtSchltrAlterEingang = 'hm-rpc.0.NEQ0133869.1.STATE'/*Lichtsch  alter Eingang STATE*/

// ##################################
//              Programm
// ##################################

on([idNeigungGarageALT, idLichtInGarage], function (dp) {  // triggert bei Wertänderung eines DP
    
    if(getState(idBewMldrGartenHell).val < 80) {
        // Aktionen bei Bewegung nachts
        setState(idLichtSchltrAlterEingang, true);
        setStateDelayed(idLichtSchltrAlterEingang, false, 60000);
        };
});

[paul53]

@skorpil sagte: wenn mein Garagentor auf geht (Neigungssensor)

Welchen Wert liefert dann der Datenpunkt?
Wenn das Licht nur bei Wert true einschalten soll, dann

    if(getState(idBewMldrGartenHell).val < 80 && dp.state.val) {

oder im Trigger:

on({id: [idNeigungGarageALT, idLichtInGarage], change: 'gt'}, function () {  // triggert bei Wertänderung false --> true eines DP

[skorpil]

@paul53 Dankeschön.

Welchen Wert liefert der Datenpunkt? Jetzt, wo das Tor geschlossen ist, steht es auf false. In der WebUI kennt er Zustand offen und Zustand geschlossen und Batterie leer und Batterie okay. Ich denke, wenn offen, dann true.

Ich prüfe das aber morgen noch einmal. Und werde dann auch den Trigger mit „gt“ anpassen.

Das mit setStateDelayed ist okay?

[skorpil]

Gerade habe ich bei „Chat.openai.com“ mal folgende Frage eingegeben:

Bitte erstelle ein ioBroker JavaScript für folgende Bedingung: wenn mein Garagentor auf geht (Neigungssensor) oder der Lichtschalter betätigt wird, soll, wenn es dunkel ist (Brightness vom Bewegungsmldr. < 80) das Licht für 1 Minute an- und dann wieder ausgehen.

Antwort vom Bot:

Hier ist ein Beispiel für ein ioBroker JavaScript, das die von Ihnen beschriebene Funktionalität implementiert:

on({id: "garage.door.sensor", change: "ne"}, function (obj) {
    checkLight();
});

on({id: "light.switch", change: "ne"}, function (obj) {
    checkLight();
});

function checkLight() {
    if (getState("motion.sensor.brightness").val < 80) {
        turnLightOn();
        setTimeout(turnLightOff, 60 * 1000);
    }
}

function turnLightOn() {
    setState("light", true);
}

function turnLightOff() {
    setState("light", false);
}

Hinweis: Stellen Sie sicher, dass die IDs der Zustände garage.door.sensor, light.switch und motion.sensor.brightness in Ihrem ioBroker System korrekt sind.

Ganz schön spooky!