wieder das leidige "if"

[fh555]

Da finde ich für mich aber die Schreibweise

Code Auswählen Erweitern


define schalter1machwas notify schalter1 { if ("ReadingsVal("schalter1","state", "not found") eq "0") { fhem "set B_Dose_Fenster_LED on" } }


wesentlich logischer. Erst "die Bedingung" und dann "mach was".

Danke noch einmal für deine Hilfe 

Gruß Jens

[Dietmar63]

Hat sich der Erfinder von Perl auch gedacht.

In der Umgangsprache heißt es doch auch:

geh nach Hause, wenn es 8 Uhr ist.

[fh555]

oder

wenn es 8:00 Uhr ist, geh nach Hause, denn es ist Feierabend, ansonsten musst du noch arbeiten

if .. then .. else 

[marvin78]

Und wenn es kein "ansonsten" gibt, ist die Kurzschreibweise mit dem if hinten dran, einfacher und kürzer. Das ganze funktioniert ja auch mit "unless" etc.

[ntruchsess]

@fh555: besser so:

Code Auswählen Erweitern


define schalter1machwas notify schalter1.(0|1) { fhem "set B_Dose_Fenster_LED ".($EVENT==0 ? "off" : "on") }


das erste Argument des notifys ist ein regulärer Ausdruck. Mit dem kannst Du den gewünschten Event genau passend rausfiltern, dann reagiert das notify nicht sinnlos auf die Transmission-state Änderungen und braucht den neuen Wert von 'state' auch nicht per ReadingsVal auslesen (weil er ja eh schon mit dem Event mitkommt).

Gruß,

Norbert