if else ...

[tm]

Liebes Forum,

Wahrscheinlich nur eine Kleinigkeit - aber ich komm nicht weiter.

Relais sollen von einer Fernbedienung über eine I²C Schnittstelle geschaltet werden. Ein/Aus soll über die selbe Taste geschaltet werden. Einzeln geht das:

Code Auswählen Erweitern


RC_KLEIN:cSceneSet:.5 set relais PortA1 on


ausschalten:

Code Auswählen Erweitern


RC_KLEIN:cSceneSet:.5 set relais PortA1 off


In Abhängigkeit vom aktuellen Zustand jeweils:

Code Auswählen Erweitern


RC_KLEIN:cSceneSet:.5 {if ("relais PortA1" == "off") {fhem("set relais PortA1 on")} }


Code Auswählen Erweitern


RC_KLEIN:cSceneSet:.5 {if ("relais PortA1" == "on") {fhem("set relais PortA1 off")} }



Jetzt zum Problem - Soll beides miteinander verknüpft werden, funktioniert nur die Anweisung _vor_ dem else:

Code Auswählen Erweitern


RC_KLEIN:cSceneSet:.5 {if ("relais PortA1" == "off") {fhem("set relais PortA1 on")} else {fhem("set relais PortA1 off")}}


Hab ich da einen Denk- oder Schreibfehler?

Vielen Dank vorab,
Thomas

[Hans Franz]

Besser:

Code Auswählen Erweitern

{if ("relais PortA1" eq "off")...
Hoffentlich reicht das.

Gruß
Hans

[tm]

Servus Hans,

Danke für Deine Hilfe.
Damit hatte ich es schon versucht - und es läuft nicht ...

also

Code Auswählen Erweitern


RC_KLEIN:cSceneSet:.5 {if ("relais PortA1" =="off") {fhem("set relais PortA1 on")}}

funktioniert, wohingegen

Code Auswählen Erweitern


RC_KLEIN:cSceneSet:.5 {if ("relais PortA1" eq "off") {fhem("set relais PortA1 on")}}

ohne Wirkung bleibt.

[Prof. Dr. Peter Henning]

Natürlich kann das nicht funktionieren. Mit diesem Code

Code Auswählen Erweitern

"relais PortA1" eq "off"
werden zwei Strings verglichen - und der erste String ist nun einmal nicht gleich dem Zweiten.

Entweder Perl - dann schlage ich vor, die Konzepte Strings, Variable, Kontrollstrukturen etc. in Perl anzuschauen. Siehe Ausgabe von "man perl".

Oder FHEM - dann schlage ich vor, sich die Comandref zu DOIF anzusehen.

LG

pah

[Hans Franz]

Zapperlot, zu blind.
pah hat natürlich recht.
Wahrscheinlich geht's so:

Code Auswählen Erweitern

{if (ReadingsVal("relais","PortA1","") eq "off")...

Gruß
Hans

[tm]

OK

Code Auswählen Erweitern


RC_KLEIN:cSceneSet:.5 {if (ReadingsVal("relais","PortA1","") eq "off") set relais PortA1 on}


geht schon mal nicht ...

[CoolTux]

OK

Code Auswählen Erweitern


RC_KLEIN:cSceneSet:.5 {if (ReadingsVal("relais","PortA1","") eq "off") set relais PortA1 on}


geht schon mal nicht ...

Kann ja auch nicht. Schaue dir bitte noch mal Hans sein Beispiel an. Der Bedingungsteil war fehlerhafte aber nicht der Ausführungsteil.

[Prof. Dr. Peter Henning]

@tm: Das erscheint mir etwas experimentell - so lange tippen, bis etwas funktioniert. Bitte erst einmal die Anfängerdoku lesen.

Eine gute Einführung in Perl findet man hier: https://wiki.selfhtml.org/wiki/Perl

LG

pah

[tm]

Lieber Herr Kollege,

Vielen Dank für diesen wertvollen Hinweis.

LG,
TM