DOIF in "Etappen"

[stgeran]

Wenn ich ein DOIF habe mit einer Bedingung, die einen zweistufigen Schalter betätigt, kann ich das in einem DOIF schreiben? Oder ist es besser ein zweites DOIF zu nehmen?
z.B. DOIF  Ventilator Feuchte über 50% set Schalter1 on; Feuchte über 70% set Schalter2 on DOELSE Feuchte unter 50% set Schalter1 und Schalter2 off.
Ich habe es nicht als code geschrieben, es soll nur ein Beispiel sein.

[CoolTux]

kann man alles in einem DOIF schreiben. Das geht ja noch von der Größe/Übersicht her.

[Otto123]

Hi,

ist wie immer sicher auch Geschmacksache. Du hast ja nicht eine Bedingung, so wie Du schreibst hast Du drei!
Du solltest die Bedingungen klar definieren und dann sind getrennte DOELSEIF Zweige kein Problem. Keinen Fall unberücksichtigt lassen!

Feuchte > 50 and Feuchte <= 70 
Feuchte > 70

Gruß Otto

[stgeran]

Hier mein Versuch mit "richtigen" Definitionen

Code Auswählen Erweitern

([HM_Wand_Weather:humidity] > 60 and [HM_Wand_Weather:humidity] <= 70)(set Vent_Bad_Sw_01 on) DOELSE (set Vent_Bad_Sw_01 off)
([HM_Wand_Weather:humidity] >70)(set Vent_Bad_Sw_01, Vent_Bad_Sw_02 on) DOELSE (set Vent_Bad_Sw_01, Vent_Bad_Sw_02 off)
Würde es reichen, im zweiten DOIF nur Sw_02 einzuschalten? Ich brauche > 70% Sw_01 und Sw_02; bei < 70% nur Sw_01; und unter 60% sollen beide ausgehen.

[Otto123]

Hallo,
"richtige" Definitionen wären die Kopie der Raw Defintion oder ein list. Nur die DEF kopieren ist für mich immer ein bisschen halbherzig. 

Kommas haben in DOIF einen spezielle Bedeutung -> Du musst sie extra klammern 

Ich würde so denken?

Code Auswählen Erweitern

define di_Feuchte DOIF ([HM_Wand_Weather:humidity] > 60 and [HM_Wand_Weather:humidity] <= 70)(set Vent_Bad_Sw_01 on)\
DOELSEIF ([HM_Wand_Weather:humidity] >70)((set Vent_Bad_Sw_01,Vent_Bad_Sw_02 on))\
DOELSE ((set Vent_Bad_Sw_01,Vent_Bad_Sw_02 off))

Wenn es nur die beiden Schalter mit dem namen gibt kannst Du auch so schreiben: (set Vent_Bad_Sw_0. off)

Du schickst damit unter Umständen extra Schaltbefehle obwohl der Schaltzustand schon stimmt, kann man mit einem FILTER noch verhindern.

Gruß Otto

[stgeran]

Prima, Danke. Der "\" ist zum Zeilenumbruch, ist das korrekt? Wegen der Übersicht?

[stgeran]

Das ist jetzt das "list Luft_Bad"

Code Auswählen Erweitern

Internals:
   CFGFN
   DEF        ([HM_Wand_Weather > 60 and [HM_Wand_Weather <= 70)(set Vent_Bad_Sw_01 on)
DOELSEIF ([HM_Wand_Weather >70)((set Vent_Bad_Sw_01, Vent_Bad_Sw_02 on))
DOELSE ((set Vent_Bad_Sw_01, Vent_Bad_Sw_02 off))
   NAME       Luft_Bad
   NR         435
   NTFY_ORDER 50-Luft_Bad
   STATE      cmd_3
   TYPE       DOIF
   READINGS:
     2017-09-06 18   Device          HM_Wand_Weather
     2017-09-06 18   cmd             3
     2017-09-06 18   cmd_event       HM_Wand_Weather
     2017-09-06 18   cmd_nr          3
     2017-09-06 18   e_HM_Wand_Weather_humidity 53
     2017-09-06 18   error           set Vent_Bad_Sw_01, Vent_Bad_Sw_02 off: Unknown argument Vent_Bad_Sw_02, choose one of clear getConfig getRegRaw inhibit off on-for-timer on-till on peerBulk peerIODev press pressS pressL regBulk regSet sign statusRequest toggle
     2017-09-06 18   state           cmd_3
   condition:
     0          ReadingValDoIf($hash,'HM_Wand_Weather','humidity') > 60 and ReadingValDoIf($hash,'HM_Wand_Weather','humidity') <= 70
     1          ReadingValDoIf($hash,'HM_Wand_Weather','humidity') >70
   devices:
     0           HM_Wand_Weather
     1           HM_Wand_Weather
     all         HM_Wand_Weather
   do:
     0:
       0          set Vent_Bad_Sw_01 on
     1:
       0          (set Vent_Bad_Sw_01, Vent_Bad_Sw_02 on)
     2:
       0          (set Vent_Bad_Sw_01, Vent_Bad_Sw_02 off)
   helper:
     event      humidity: 53,T: 26.3 H: 53 D: 16.0,temperature: 26.3,dewpoint: 16.0
     globalinit 1
     last_timer 0
     sleeptimer -1
     timerdev   HM_Wand_Weather
     timerevent humidity: 53,T: 26.3 H: 53 D: 16.0,temperature: 26.3,dewpoint: 16.0
     triggerDev HM_Wand_Weather
     timerevents:
       humidity: 53
       T: 26.3 H: 53 D: 16.0
       temperature: 26.3
       dewpoint: 16.0
     timereventsState:
       humidity: 53
       T: 26.3 H: 53 D: 16.0
       temperature: 26.3
       dewpoint: 16.0
     triggerEvents:
       humidity: 53
       T: 26.3 H: 53 D: 16.0
       temperature: 26.3
       dewpoint: 16.0
     triggerEventsState:
       humidity: 53
       T: 26.3 H: 53 D: 16.0
       temperature: 26.3
       dewpoint: 16.0
   internals:
   itimer:
   readings:
     0           HM_Wand_Weather
     1           HM_Wand_Weather
     all         HM_Wand_Weather
   regexp:
     0:
     1:
     all:
   state:
     STATE:
   trigger:
Attributes:
   room       Bad
   wait       1200
Es zickt aber rum

Code Auswählen Erweitern

error

set Vent_Bad_Sw_01, Vent_Bad_Sw_02 off: Unknown argument Vent_Bad_Sw_02, choose one of clear:readings,trigger,register,oldRegs,rssi,msgEvents,attack,all getConfig:noArg getRegRaw inhibit:on,off off:noArg on-for-timer on-till on:noArg peerBulk peerIODev press pressS:HM_Wand_SwitchTr pressL:HM_Wand_SwitchTr regBulk regSet sign:on,off statusRequest:noArg toggle:noArg

[stgeran]

Sorry für den Wind.
Der Sw_02 war noch nicht mit dem ...Weather gepeert ((

[stgeran]

Ah, Sch.... ich krieg den verdammten error nicht weg!

[Damian]

Ah, Sch.... ich krieg den verdammten error nicht weg!

Korrekte Schreibweise:

ohne Leerzeichen:

Code Auswählen Erweitern

((set Vent_Bad_Sw_01,Vent_Bad_Sw_02 on))

oder

hier ist das Leerzeichen egal:

Code Auswählen Erweitern

(set Vent_Bad_Sw_01 on, set Vent_Bad_Sw_02 on)

[stgeran]

Das sieht schon besser aus

[Otto123]

Prima, Danke. Der "\" ist zum Zeilenumbruch, ist das korrekt? Wegen der Übersicht?

So ist es. Dann ist die Zeile nicht so lang.
In der Raw Def geht das so per cut & paste

Aber nimm doch

Code Auswählen Erweitern

set Vent_Bad_Sw_0. on - dann hast Du den Kuddel Muddel mit den Kommas nicht.

[Per]

Würde es reichen, im zweiten DOIF nur Sw_02 einzuschalten?

Jein. Wenn dein Fhem garantiert 24/7 läuft, ja. Ansonsten würde ich einfach auf Nummer sicher gehen. Kostet ja nix.

[stgeran]

Ich habs mit Ottos Vorschlag gemacht. Danke euch für die Hilfe