IF Verzweigung - Problem

[SaschaQ]

Hallo zusammen,

ich habe folgende Rule in Openhab 3 geschrieben.

Code: Alles auswählen

if (now.getHour() <= 22 && now.getHour() >= 6) {
		if (all_kind1_schlafen.state==OFF || all_kind1_schlafen_prion_schalter.state==OFF || all_kind2_schlafen.state==OFF || all_kind2_schlafen_prion_schalter.state==OFF){
		createTimer(now.plusSeconds(3), [ |
		echo_wohnzimmer_TTS.sendCommand('Willkommen zuhause Person 1')
		])
		createTimer(now.plusSeconds(3), [ |
		echo_wohnzimmer_TTS_Volume.sendCommand(25)
		])
      } 
}

Wenn die Kinder schlafen, schalte ich eines der Items, die hinter der Uhrzeitabfrage kommen auf ON, sodass dann keine Durchsagen durch meine Echos im Haus mehr kommen sollen, wenn eines der oben genannten Items auf AN steht.

Komischerweise rutscht die Rule aber immer wieder in die Durchsage obwohl eines der Items auf AN steht.
In den Logs sehe ich auch das beispielsweie das ITem all_kind1_schlafen auf ON steht.

Was mache ich falsch?

Viele Grüße
Sascha

[BOP]

Komischerweise rutscht die Rule aber immer wieder in die Durchsage obwohl eines der Items auf AN steht.

Ja, aber die anderen stehen dann noch auf OFF? Du hast alles mit ODER verknüpft. Es reicht also, wenn einer der Ausdrucke erfüllt ist, damit er in IF-Verzweigung "rutscht".

EDIT: Wieso erzeugst du eigentlich zwei Timer mit gleicher Ablaufzeit direkt hintereinander? Du kannst die zwei Befehle auch in einem Timeraufruf zusammen packen.

[Mannklappe]

So lange eins immer auf OFF steht wird die Rule ausgeführt. Du müsstest eine Abfrage rein machen, die prüft ob einer auf ON steht und die Regel abbrechen dann.

[BOP]

Du müsstest eine Abfrage rein machen, die prüft ob einer auf ON steht und die Regel abbrechen dann.

Oder schlicht alles mit && verknüpfen. Wenn ich das richtig verstehe, geht es ja nicht darum die Kinder einzeln zu betrachten. Sondern sobald eines Schläft, sollen die Befehle nicht mehr ausgeführt werden.

[Mannklappe]

Er schreibt doch, er möchte EINES der Items auf ON stellen. Somit wäre && hier falsch, weil ja nur eines auf ON steht.

[BOP]

Deswegen schrieb ich ja: "Wenn ich das richtig verstehe, geht es ja nicht darum die Kinder einzeln zu betrachten."
Sollte das nicht so sein, hast du recht. Dann reicht es nicht, alle mit && zu verknüpfen.

Oder sagen wir es so: Sobald eines auf ON steht, würde bei && Verknüpfungen die Befehle in der IF-Verzweigung nicht mehr ausgeführt.

[Mannklappe]

Vielleicht erhellt uns SachaQ

[SaschaQ]

Hallo zusammen,

ich habe mit || gearbeitet, da tatsächlich nur eines der Items ausreichen soll, damit die Durchsage nicht mehr kommen soll.

Ich verstehe den Ansatz von && nicht. Dies würde doch bewirken, dass alle Items den Status OFF haben müssen.

[Mannklappe]

Einfach eine If-Regel am Anfang einfügen, in dem du abfragst ober einer der Items auf ON steht und dann einfach die Regel abbrechen. Somit umgehst du, dass die Schleife dauerhaft ausgeführt wird.

Code: Alles auswählen

if (all_kind1_schlafen.state == ON || all_kind1_schlafen_prion_schalter.state == ON || all_kind2_schlafen.state == ON || all_kind2_schlafen_prion_schalter.state == ON)
{
return;
}

[BOP]

Ich verstehe den Ansatz von && nicht. Dies würde doch bewirken, dass alle Items den Status OFF haben müssen.

Ja genau. Oder anders ausgedrückt. Sobald eines der Items nicht OFF ist, würde sie nicht abgearbeitet.