Rolladen holiday wird nicht beachtet

[pointde]

Ich wollte, dass meine Rolladen um 07:15 Uhr hochgehen, wenn nicht Wochenende ist, nicht wenn ein Ferientag ist, und nicht wenn Urlaub ist.

Code Auswählen Erweitern

define Rollo_auf at *7:15:00 { if ( !($we) && !(Value("Bayern_Ferientag") && !(Value("Feiertag_Urlaub")) ) { fhem ("set RolloWohnenEssen 100");; fhem("set LichtEssen off");; fhem("set StehlampeWohnen off");; } }

Wenn Wochenende und Feiertag und Urlaub ist, hätte ich dann eine zweite Zeile geschrieben, bei der dann eine andere Zeit steht. Oder könnte man das evtl. in eine schreiben?

Testweise habe ich mal einen freien Tag in die Holiday geschreiben.
Ergebnis:

Code Auswählen Erweitern

Internals:
   NAME       Feiertag_Urlaub
   NR         25
   STATE      none
   TRIGGERTIME 1430172002.99922
   TYPE       holiday
   Readings:
     2015-04-27 16:12:13   state           none
     2015-04-27 16:12:13   tomorrow        Frei
     2015-04-27 16:12:13   yesterday       none
Attributes:
   room       Kalender

Wenn ich mir aber jetzt das Rolloprogramm anschaue, steht da, dass es morgen doch ausgeführt wird.
Wo liegt mein Fehler?

Code Auswählen Erweitern

Internals:
   COMMAND    { if ( !($we) && !(Value("Bayern_Ferientag") && !(Value("Feiertag_Urlaub")) ) { fhem ("set RolloWohnenEssen 100"); fhem("set LichtEssen off"); fhem("set StehlampeWohnen off"); } }
   DEF        *7:15:00 { if ( !($we) && !(Value("Bayern_Ferientag") && !(Value("Feiertag_Urlaub")) ) { fhem ("set RolloWohnenEssen 100"); fhem("set LichtEssen off"); fhem("set StehlampeWohnen off"); } }
   NAME       Rollo_auf
   NR         192
   PERIODIC   yes
   RELATIVE   no
   REP        -1
   STATE      Next: 07:15:00
   TIMESPEC   7:15:00
   TRIGGERTIME 1430198100
   TRIGGERTIME_FMT 2015-04-28 07:15:00
   TYPE       at
   Readings:
     2015-04-27 16:13:07   state           Next: 07:15:00
Attributes:

[Paul]

Das at wird doch immer um 7:15 Uhr ausgeführt, ob dann das Rollo öffnet entscheidet dein if.

[pointde]

Hast recht, war ein Denkfehler, wird ja immer um 07:15 auseführt.
So klappt es jetzt:

Code Auswählen Erweitern

define Rollo_auf at *21:19 { if ( !($we) && !(Value("Bayern_Ferientag")) ) { fhem("set RolloKuecheFenster 100");; } }
Wenn ich aber noch die Variable Ferientag_Urlaub einbinde, geht es nicht. Hier steht auch nicht 0 drin, sondern none. Also habe ich folgendes getestet, was nicht geht:

Code Auswählen Erweitern

define Rollo_auf at *21:25 { if ( !($we) && !(Value("Bayern_Ferientag")) && {if (Value("Feiertag_Urlaub") eq "none")) ) { fhem("set RolloKuecheFenster 100");; } }
Ich habe mir gedacht, wenn in der Variable als Wert none drin steht,... geht aber nicht:

Code Auswählen Erweitern

Internals:
   NAME       Feiertag_Urlaub
   NR         25
   STATE      none
   TRIGGERTIME 1430344802.03538
   TYPE       holiday
   Readings:
     2015-04-29 00:00:02   state           none
     2015-04-29 00:00:02   tomorrow        none
     2015-04-29 00:00:02   yesterday       Frei
Attributes:
   room       Kalende

[Paul]

Weshalb ein zweites if?

Du willst doch vergleichen

Wenn nicht WE und nicht Ferien und nicht sonstwas

[pointde]

Nicht Wochenende, nicht Ferien, nicht Feiertag und Urlaub

[Mr. P]

Ich glaub, was du möchtest, müsste so aussehen:

Code Auswählen Erweitern

define Rollo_auf at *21:25 { if ( !$we && Value("Bayern_Ferientag") ne "none" && Value("Feiertag_Urlaub") ne "none" ) { fhem("set RolloKuecheFenster 100");; }


Ergibt logisch:
Wenn nicht Wochenende und Bayern_Ferientag irgendwas nur nicht "none" und Feiertag_Urlaub irgendwas nur nicht "none", dann mach die Küchenrollo ganz auf.

Im übrigen ist vielleicht das Attribut was für dich:

Code Auswählen Erweitern

attr RolloKuecheFenster param levelInverse
Dadurch werden die Values umgedreht und dann ist 0 auch 'offen' und 100 'geschlossen.

[pointde]

Danke für die Antwort!
Hinten fehlt die geschweifte Klammer, oder?...
Bayern_Ferientag kennt nur 0 oder 1

Code Auswählen Erweitern

Internals:
   NAME       Bayern_Ferientag
   NR         23
   STATE      0
   TYPE       dummy
   Readings:
     2015-04-30 16:11:58   state           0
Attributes:
   room       Kalender

Feiertag_Urlaub:

Code Auswählen Erweitern

Internals:
   NAME       Feiertag_Urlaub
   NR         25
   STATE      none
   TRIGGERTIME 1430431202.05461
   TYPE       holiday
   Readings:
     2015-04-30 00:00:02   state           none
     2015-04-30 00:00:02   tomorrow        Tag der Arbeit
     2015-04-30 00:00:02   yesterday       none
Attributes:
   room       Kalender

Also müsste es doch eigentlich so sein:
Wenn nicht Wochenende und kein Ferientag (also 0) und kein Feiertag_Urlaub (none)..
Ich habe keine Fehler im Log, er führt es einfach nicht aus....

edit:
oder so:
define Rollo_auf at *17:52 { if ( !$we && Value("Bayern_Ferientag") eq "0" && Value("Feiertag_Urlaub") eq "none" ) { fhem("set RolloKuecheFenster 100");; } }

[Mr. P]

Hinten fehlt die geschweifte Klammer, oder?...

Hast natürlich recht. Sollte keinen Code mit TapaTalk schreiben.

Bayern_Ferientag kennt nur 0 oder 1
.
.
.
oder so:
define Rollo_auf at *17:52 { if ( !$we && Value("Bayern_Ferientag") eq "0" && Value("Feiertag_Urlaub") eq "none" ) { fhem("set RolloKuecheFenster 100");; } }

Code Auswählen Erweitern

define Rollo_auf at *17:52 { \
   if ( !$we && Value("Bayern_Ferientag") == 0  && Value("Feiertag_Urlaub") eq "none" ) { \
      fhem("set RolloKuecheFenster 100");; \
   } \
}


Habs zwecks Lesbarkeit jetzt einmal etwas aufgesplittet und korrigiert, müsste so funktionieren.
Was mir noch aufgefallen ist: Warum nutzt du eigentlich nicht die holiday2we-Funktion?
Und was das Mitloggen angeht:

Code Auswählen Erweitern

attr global verbose 4
vorm Testen in FHEM ausführen. Dann bekommst im Logfile von FHEM schon etwas mehr Output.

[pointde]

Ich habe es jetzt mal so gelöst, obwohl der 2. Befehl nicht funktioniert:

Code Auswählen Erweitern

define Rollo_auf_Werktag at *07:15 { if ( !$we && Value("Bayern_Ferientag") == 0 && Value("Feiertag_Urlaub") eq "none" ) { fhem("set RolloWohnenEssen 100");; fhem("set RolloKuecheFenster 100");; fhem("set LichtEssen off");; fhem("set StehlampeWohnen off");; } }

Code Auswählen Erweitern

define Rollo_auf_Wochenende at *08:45 { if ( $we && Value("Bayern_Ferientag") == 1 && Value("Feiertag_Urlaub") ne "none" ) { fhem("set RolloWohnenEssen 100");; fhem("set RolloKuecheFenster 100");; fhem("set LichtEssen off");; fhem("set StehlampeWohnen off");; } }

[pointde]

Es hat gestern, am Vatertag mal funktoniert. Mein Code ist zwar bestimmt nicht sauber, hat aber wie gesagt funktoniert. Kann man das verkürzen oder besser machen.

Code Auswählen Erweitern


define Rollo_auf_Wochenende at *08:45 {if ($we) { fhem("set RolloWohnenEssen 100");; fhem("set RolloKuecheFenster 100");; fhem("set LichtEssen off");; fhem("set StehlampeWohnen off")} elsif (Value("Bayern_Ferientag") == 1) { fhem("set RolloWohnenEssen 100");; fhem("set RolloKuecheFenster 100");; fhem("set LichtEssen off");; fhem("set StehlampeWohnen off")} elsif (Value("Feiertag_Urlaub") ne "none" ) { fhem("set RolloWohnenEssen 100");; fhem("set RolloKuecheFenster 100");; fhem("set LichtEssen off");; fhem("set StehlampeWohnen off")}}



[Mr. P]

Es hat gestern, am Vatertag mal funktoniert. Mein Code ist zwar bestimmt nicht sauber, hat aber wie gesagt funktoniert. Kann man das verkürzen oder besser machen.

Code Auswählen Erweitern


define Rollo_auf_Wochenende at *08:45 {if ($we) { fhem("set RolloWohnenEssen 100");; fhem("set RolloKuecheFenster 100");; fhem("set LichtEssen off");; fhem("set StehlampeWohnen off")} elsif (Value("Bayern_Ferientag") == 1) { fhem("set RolloWohnenEssen 100");; fhem("set RolloKuecheFenster 100");; fhem("set LichtEssen off");; fhem("set StehlampeWohnen off")} elsif (Value("Feiertag_Urlaub") ne "none" ) { fhem("set RolloWohnenEssen 100");; fhem("set RolloKuecheFenster 100");; fhem("set LichtEssen off");; fhem("set StehlampeWohnen off")}}



Zwei Dinge:

1) Schreib nicht alles in eine Zeile. Das ist zwar nett platzsparend... Aber das Wesentliche, nämlich das du auf einen Blick gut lesen kannst, was in deinem Code passieren soll, bleibt dabei auf der Strecke.

2) Wie es aussieht, bist du auf deinen Fehler drauf gekommen und hast ihn (wenn auch etwas umständlich, wie du selbst schon angemerkt hast) gelöst.
Optimiert sieht dein Code folgendermaßen aus:

Code Auswählen Erweitern

define Rollo_auf_Wochenende at *08:45 { \
if ($we || Value("Bayern_Ferientag") == 1 || Value("Feiertag_Urlaub") ne "none" ) { \
fhem("set RolloWohnenEssen,RolloKuecheFenster 100";; set LichtEssen,StehlampeWohnen off") \
} \
}

Du hattest alles ursprünglich mit logischen && verknüpft gehabt. Es mussten also alle drei Werte wahr ergeben, damit deine Befehle ausgeführt werden.
Mit deinen elsif's hast du im Prinzip logische oder gebaut, die ich in einem if mit || darstellst. Der einzige Unterschied (und der ist hier offensichtlich unwichtig) zwischen dem elsif und der Verknüpfung mit logischem || in einem if besteht darin, dass beim if einfach einer der Werte zutreffen muss, ganz egal an welcher Position er steht. Beim elsif hingegen klappert das if jede einzelne Möglichkeit in der gewählten Reihenfolge ab und führt im Falle eines Treffers die darunter liegenden Befehle aus und beendet anschließend die if-Anweisung.

Have fun!

[pointde]

Dein Code funktioniert leider nicht!
Ich sehe aber keinen Fehler. Kannst du mir bitte nochmal helfen?

[Treibhaus]

Dein Code funktioniert leider nicht!
Ich sehe aber keinen Fehler. Kannst du mir bitte nochmal helfen?

Meines Erachtens fehlt es an einer weiteren ) hinter dem none.