IF Abfrage

[jutil]

Hallo zusammen,

ich kriege hier ums erbrechen leider die If Abfrage nicht hin.
Heißt, der Befehl wird ausgeführt obwohl die Bedingung nicht erfüllt wurde.

Denke das ich wieder mit Klammer und / oder Kommata durchn tüttel komme.
Kann mir bitte jemand kurz dabei helfen? Danke!

Code Auswählen Erweitern


Wohnung_komplett:off { fhem "set Aussen_1,Aussen_2,Aussen_3, Licht, $EVENT; set WZLicht_1 $EVENT; sleep 3; set WZLicht_2 $EVENT; sleep 3; set WZLicht_3 $EVENT; sleep 10; set Wohnung_komplett on; set Gute_Nacht $EVENT" }
{ if ( "TV" eq "on" ) {
fhem("set harmony_36285656 command PowerToggle; set harmony_51989440 command PowerToggle; set harmony_37540046 command PowerToggle; sleep 5; set TV off; set Canton off") }}

[MadMax-FHEM]

{ fhem "set Aussen_1,Aussen_2,Aussen_3, Licht, $EVENT; set WZLicht_1 $EVENT; sleep 3; set WZLicht_2 $EVENT; sleep 3; set WZLicht_3 $EVENT; sleep 10; set Wohnung_komplett on; set Gute_Nacht $EVENT" }

Du gehst nach Perl -> geschweifte Klammer auf
Machst was: fhem-Befehl (Folge)
Verlässt Perl -> geschweifte Klammer zu

-> fertig

Was danach kommt müsste einen Fehler werfen?

EDIT: wenn du nur das gewollt hättest, hättest du es auch einfach ohne geschweifte Klammern und ohne fhem-Befehl eingeben können

if ( "TV" eq "on" )

Ist NIE wahr, weil der String "TV" nun mal nicht (und nie wird   ) "on" ist.
Da ist ja mal 0% Übereinstimmung

Also du müsstest für 1. Problem das if noch in die erste öffnende geschweifte Klammer bringen.
Befehle in Perl trennt man mittels Strichpunkt.
Also nach dem fhem-Befehl einen Strichpunkt (wenn im DEF-Code-Fenster / 2 Strichpunkte wenn per define in FhemWeb) und dann die if-Abfrage...

Vermutlich soll "TV" eq "on" den Status vom Fernseher abfragen/vergleichen?

if(ReadingsVal("TV", "status", "n.a.") eq "on")  ?

EDIT:
Also in Summe in etwa so (wobei das mit ReadingsVal usw. musst du wissen was da gemeint ist UND der erste fhem-Befehl [Folge] wird [weiterhin] IMMER ausgeführt):

Code Auswählen Erweitern


Wohnung_komplett:off { fhem ("set Aussen_1,Aussen_2,Aussen_3, Licht, $EVENT; set WZLicht_1 $EVENT; sleep 3; set WZLicht_2 $EVENT; sleep 3; set WZLicht_3 $EVENT; sleep 10; set Wohnung_komplett on; set Gute_Nacht $EVENT");
if ( ReadingsVal("TV","state","n.a.") eq "on" ) {
fhem("set harmony_36285656 command PowerToggle; set harmony_51989440 command PowerToggle; set harmony_37540046 command PowerToggle; sleep 5; set TV off; set Canton off") }}


set Aussen_1,Aussen_2,Aussen_3, Licht, $EVENT; set WZLicht_1 $EVENT;

Da kann man doch WZLicht_1 einfach auch vorne mit dazu nehmen?

Gruß, Joachim

[jutil]

Mega Antwort, danke dafür!
Ich komme mit dieser Sprache einfach nicht klar, mache dafür einfach zu wenig ...

Ich lese mich da ersteinmal in deine Antwort ein

Beste Grüße
Timo

[jutil]

Joachim ich muss noch einmal nerven, sorry.

Das funktioniert bei mir nicht.
Also, ja stimmt

Code Auswählen Erweitern

Wohnung_komplett:off { fhem ("set Aussen_1,Aussen_2,Aussen_3, Licht, $EVENT; set WZLicht_1 $EVENT; sleep 3; set WZLicht_2 $EVENT; sleep 3; set WZLicht_3 $EVENT; sleep 10; set Wohnung_komplett on; set Gute_Nacht $EVENT");
Wird immer ausgeführt und das soll er auch.

Das,

Code Auswählen Erweitern

if ( ReadingsVal("TV","state","n.a.") eq "on" ) {
fhem("set harmony_36285656 command PowerToggle; set harmony_51989440 command PowerToggle; set harmony_37540046 command PowerToggle; sleep 5; set TV off; set Canton off") }}
funktioniert so leider nicht.
Habe es auch mit If (Value(.... etc. sowie diversen anderen Variablen hinter "state", "?" ausprobiert, leider erfolglos.

TV ist im Grunde nur eine IT Steckdose deren Status abgefragt und in Abhängigkeit eine Funktion ausgeführt werden soll, aber ich denke das war soweit klar.

Zu deinem Hinweis mit dem Kompakten Code. Ich hatte den Code schon einmal etwas komprimierter aber da wollten komischerweise die milight brigde (WZLicht...) nicht so richtig mitspielen daher hab ich es wieder getrennt.

Hoffe Du kannst hier noch einmal einsteigen.

Beste Grüße
Timo

[MadMax-FHEM]

Naja, ohne ein list des/der Devices konnte ich nur raten.

Ich weiß nicht, ob "state" das richtige Reading ist...

EDIT: und das Device heißt aber auch "TV"? EDIT: ok, heißt es wohl, nachdem du es ja mittels set TV off ausschaltest

Und lass Value!

Value() "frägt" den STATE (ja das INTERNAL!) ab und STATE wird durch z.B. stateFormat beeinflusst und dann hat mal zwar passende "Optik" aber die Logik (mittels Value-Abfrage) funktioniert nicht (mehr).

Das notify triggert aber generell?
Aus dem Eventmonitor erzeugt?

-> davon gehe (ging) ich aus.

Ansonsten wären halt lists der beteiligten Devices, mindestens derer von denen etwas abgefragt werden soll(en) hilfreich.

Hier zu lesen: https://forum.fhem.de/index.php/topic,71806.0.html

EDIT: wie/wo gibst du ein? Frage wegen der Strichpunkte. Und: geht nicht? Gar nicht, nur das if nicht? Fehler im Log? Also generell: ohne ausreichende Infos kann ich (und andere Helfer) nur raten und das ist anstrengend...

Gruß, Joachim

[Wernieman]

Wenn Du in FHEM eingiebst (Beim Device), kannst Du die Lesbarkeit/Logik auch durch Formatieren verbessern:

Code Auswählen Erweitern

Wohnung_komplett:off
{
  fhem ("set Aussen_1,Aussen_2,Aussen_3, Licht, $EVENT; set WZLicht_1 $EVENT; sleep 3; set WZLicht_2 $EVENT; sleep 3; set WZLicht_3 $EVENT; sleep 10; set Wohnung_komplett on; set Gute_Nacht $EVENT");
  if ( ReadingsVal("TV","state","n.a.") eq "on" )
   {
     fhem("set harmony_36285656 command PowerToggle; set harmony_51989440 command PowerToggle; set harmony_37540046 command PowerToggle; sleep 5; set TV off; set Canton off")
   }
}


Hast Du Dir mal den state (nicht STATE) vom Device angesehen?