Verschachteltes DOIF vereinfachen?

[andies]

Guten Tag in die Runde, ich habe jetzt eine halbe Stunde vergeblich probiert, für das nachfolgende DOIF eine einfachere Version zu finden:

Code Auswählen Erweitern

defmod KlingelAusloesen
DOIF
(([Klingelsensor:Button1_Action] eq "SINGLE") and ([Profil:housemode] eq "normal"))  (set Klingel on-for-timer 1,set TelegramBot message 🛎�)
DOELSEIF (([Klingelsensor:Button1_Action] eq "SINGLE") and ([Profil:housemode] eq "absence")) (set TelegramBot message 🛎�)

Falls jemand im Forum Lust und Zeit hat, hier eine bessere Verschachtelung zu formulieren, wäre ich sehr dankbar. Ich kriege das nicht hin und mir kommt die doppelte Abfrage irgendwie umständlich vor.

[rabehd]

... und mir kommt die doppelte Abfrage irgendwie umständlich vor.

Mir nicht, aber wenns Dich stört, warum baust Du nicht ein "if" in den Ausführungsteil ein?
Dann würde ich aber gleich ein notify draus machen.

[Damian]

Guten Tag in die Runde, ich habe jetzt eine halbe Stunde vergeblich probiert, für das nachfolgende DOIF eine einfachere Version zu finden:

Code Auswählen Erweitern

defmod KlingelAusloesen
DOIF
(([Klingelsensor:Button1_Action] eq "SINGLE") and ([Profil:housemode] eq "normal"))  (set Klingel on-for-timer 1,set TelegramBot message 🛎�)
DOELSEIF (([Klingelsensor:Button1_Action] eq "SINGLE") and ([Profil:housemode] eq "absence")) (set TelegramBot message 🛎�)

Falls jemand im Forum Lust und Zeit hat, hier eine bessere Verschachtelung zu formulieren, wäre ich sehr dankbar. Ich kriege das nicht hin und mir kommt die doppelte Abfrage irgendwie umständlich vor.

Wenn, dann im Perlmodus:

Code Auswählen Erweitern

DOIF
{ if ([Klingelsensor:Button1_Action] eq "SINGLE") {
    if  ([Profil:housemode] eq "normal")  {fhem_set "Klingel on-for-timer 1";fhem_set "TelegramBot message 🛎�"}
    elsif ([Profil:housemode] eq "absence") {fhem_set "set TelegramBot message 🛎�"}
  }
}


[andies]

Danke! Ich bin nicht auf die Idee mit dem Perl-Modus gekommen.

Toll, so ein Forum. Oder besser: die Leute da drin

[Per]

Noch etwas kürzer:

Code Auswählen Erweitern

DOIF
{ if ([Klingelsensor:Button1_Action] eq "SINGLE") {
    if  ([Profil:housemode] eq "normal")
      fhem_set "Klingel on-for-timer 1";
fhem_set "set TelegramBot message 🛎�";
  }
}
Geht auch im Fhem Mode, aber so ists eleganter.

[MadMax-FHEM]

Noch etwas kürzer

Aber nur, wenn es außer "normal" und "absence" nichts gibt...

Gruß, Joachim

[andies]

Danke! Es gibt noch andere Modi, aber die benutze ich nie. Genau danach habe ich gesucht und ich weiß nicht, wieso mir das nicht gelungen ist. Aber dafür gibt es ja das Forum.