DOIF ohne Loops

[Damian]

Liebe DOIF-User,

da es immer wieder vorkommt, dass man unabsichtlich Loops (Schleifen) einbaut, habe ich nun eine DOIF-Version erstellt, die Rekursionen erkennt und verhindert.

Solcher Code z. B. führt mit der aktuellen Version zu einer endlosen Wiederholung, mit der neuen Version wird sie unterbunden:

Code Auswählen Erweitern

define di_loop DOIF ([test_1] eq "off") (set test_1 on) DOELSE (set test_2 off)
attr di_loop wait 1,1
attr di_loop do always


set test_1 on


Diese Version ist voll abwärtskompatibel zur bisherigen, sie kann hier geladen und getestet werden.

Wenn keine Beschwerden kommen, werde ich sie einchecken.

Edit: Die Version wurde eingecheckt und wird per Update installiert.

Gruß

Damian

[Toto1973]

Ich hab diese Version jetzt mal installiert.
Werde berichten, wenn es zu unerwarteten Fehlern kommen sollte

[Damian]

Ich hab diese Version jetzt mal installiert.
Werde berichten, wenn es zu unerwarteten Fehlern kommen sollte

ja,

insb. können jetzt getriggerte Abfragen und Ausführungen auf gleiches Device bedenkenlos benutzt werden:

z. B.

Code Auswählen Erweitern

DOIF ([Steckdose:power]<5) (set Steckdose off)


Gruß

Damian

[Toto1973]

Cool, das vereinfacht einiges!

[Sirel]

Hallo Damian,
wieder mal ganz großes Kino von Dir! Ich teste und werde berichten.

Viele Grüße und besten Dank für die Anpassung,
Max

[Toto1973]

Das DOIF Steuert meinen automatische Werkersteuerung. Der Dummy wecker_manu steht auf Auto und dennoch würde es seit dem 06.10. nicht mehr ausgeführt!

Code Auswählen Erweitern

Internals:
   DEF        ([00:30|7] and [?wecker_manu] eq "auto" and [?Technoclub] eq "nein") (set wakeUp_dummy 09:00,set tag Wochenende)
DOELSEIF ([00:33|8] and [?wecker_manu] eq "auto" and [?Schicht_dummy] eq "Frueh") (set wakeUp_dummy 04:00,set tag Werktags)
DOELSEIF ([00:34|8] and [?wecker_manu] eq "auto" and [?Schicht_dummy] eq "Mittel") (set wakeUp_dummy 08:00,set tag Werktags)
DOELSEIF ([00:31|7] and [?wecker_manu] eq "auto" and [?Technoclub] eq "ja") (set wakeUp_dummy 14:00,set tag Wochenende,set 4917685026123 send Noch viel Spaß beim feiern im Technoclub!)
DOELSEIF ([00:32] and [?Urlaub] eq "on") (set tag aus)
DOELSEIF ([08:35|8] and [?Schicht_dummy] eq "Mittel" and [?sz_Fenster:position] eq "100") (set wakeUp_dummy 9:00,set tag Werktags)
   NAME       wecker_stellen
   NR         94
   NTFY_ORDER 50-wecker_stellen
   STATE      9_00WE
   TYPE       DOIF
   Readings:
     2015-10-06 00:30:00   cmd_event       timer_1
     2015-10-06 00:30:00   cmd_nr          1
     2015-10-06 00:30:00   state           9_00WE
     2015-10-10 00:30:00   timer_1_c1      11.10.2015 00:30:00|7
     2015-10-10 00:33:00   timer_2_c2      11.10.2015 00:33:00|8
     2015-10-10 00:34:00   timer_3_c3      11.10.2015 00:34:00|8
     2015-10-10 00:31:00   timer_4_c4      11.10.2015 00:31:00|7
     2015-10-10 00:32:00   timer_5_c5      11.10.2015 00:32:00
     2015-10-10 08:35:00   timer_6_c6      11.10.2015 08:35:00|8
   Condition:
     0          DOIF_time_once($hash,$hash->{timer}{0},$wday,"7") and InternalDoIf('wecker_manu','STATE','') eq "auto" and InternalDoIf('Technoclub','STATE','') eq "nein"
     1          DOIF_time_once($hash,$hash->{timer}{1},$wday,"8") and InternalDoIf('wecker_manu','STATE','') eq "auto" and InternalDoIf('Schicht_dummy','STATE','') eq "Frueh"
     2          DOIF_time_once($hash,$hash->{timer}{2},$wday,"8") and InternalDoIf('wecker_manu','STATE','') eq "auto" and InternalDoIf('Schicht_dummy','STATE','') eq "Mittel"
     3          DOIF_time_once($hash,$hash->{timer}{3},$wday,"7") and InternalDoIf('wecker_manu','STATE','') eq "auto" and InternalDoIf('Technoclub','STATE','') eq "ja"
     4          DOIF_time_once($hash,$hash->{timer}{4},$wday,"") and InternalDoIf('Urlaub','STATE','') eq "on"
     5          DOIF_time_once($hash,$hash->{timer}{5},$wday,"8") and InternalDoIf('Schicht_dummy','STATE','') eq "Mittel" and ReadingValDoIf('sz_Fenster','position','') eq "100"
   Days:
     0          7
     1          8
     2          8
     3          7
     5          8
   Devices:
   Do:
     0:
       0          set wakeUp_dummy 09:00,set tag Wochenende
     1:
       0          set wakeUp_dummy 04:00,set tag Werktags
     2:
       0          set wakeUp_dummy 08:00,set tag Werktags
     3:
       0          set wakeUp_dummy 14:00,set tag Wochenende,set 4917685026123 send Noch viel Spaß beim feiern im Technoclub!
     4:
       0          set tag aus
     5:
       0          set wakeUp_dummy 9:00,set tag Werktags
     6:
   Helper:
     globalinit 1
     last_timer 6
     sleeptimer -1
   Internals:
   Itimer:
   Readings:
   Realtime:
     0          00:30:00
     1          00:33:00
     2          00:34:00
     3          00:31:00
     4          00:32:00
     5          08:35:00
   State:
   Time:
     0          00:30:00
     1          00:33:00
     2          00:34:00
     3          00:31:00
     4          00:32:00
     5          08:35:00
   Timecond:
     0          0
     1          1
     2          2
     3          3
     4          4
     5          5
   Timer:
     0          0
     1          0
     2          0
     3          0
     4          0
     5          0
   Timerfunc:
   Timers:
     0           0
     1           1
     2           2
     3           3
     4           4
     5           5
Attributes:
   cmdState   9_00WE|Fr_4_00WO|Mi_8_00WO|TE_14_00WE|Urlaub|Mi_9_00WO
   group      Wecker
   room       DOIFs
Weis jemand, warum das nicht mehr läuft?

[Damian]

wecker_manu steht auf Auto

Das reicht nicht, dann wird wohl eine der and-Verknüpfungen nicht wahr sein.

Vielleicht arbeitest du mit Aliasnamen oder hast den Status von Schicht_dummy irgendwie (z. B. per devStateIcon) umgebogen.

So etwas kannst du leicht testen, z. B.

Code Auswählen Erweitern

define di_test DOIF ([wecker_manu] eq "auto" and [?Schicht_dummy] eq "Frueh") (set blabla on)

trigger wecker_manu



Gruß

Damian

[Toto1973]

Ich weis jetzt, warum es nicht ging!
Ich habe eine Holiday-Datei eingerichtet. Und die letzte Woche hatte ich Urlaub, der auch in der Holiday-Datei eingetragen war.
Somit wurde jeden Tag State 1 getriggert und da dies ja schon gesetzt war, wurde nichts mehr gemacht.
Das ganze sollte ich ja mit do always beheben können.
Denn der Wecker soll jeden Tag gesetzt werden.

[Toto1973]

Ich hätte man noch ein Problem.
In diesem Beitrag hier habe ich dazu was geschrieben. http://forum.fhem.de/index.php/topic,42029.0.html

Es geht darum, das wenn man Zeitdummys benutzt, das DOIF nicht nach ändern der Zeit im Dummy die Timer neu setzt, sondern erst, wenn die Timerzeit (am nächsten Tag), neu gesetzt wird.
Könnte man denn DOIF so umprogrammieren, das ein Zeitdummy quasi DOIF triggert, wenn sich der wert ändert und so die Timer neu gesetzt werden?

Damian, ich hoffe Du verstehst, was ich meine!

[Damian]

Ich hätte man noch ein Problem.
In diesem Beitrag hier habe ich dazu was geschrieben. http://forum.fhem.de/index.php/topic,42029.0.html

Es geht darum, das wenn man Zeitdummys benutzt, das DOIF nicht nach ändern der Zeit im Dummy die Timer neu setzt, sondern erst, wenn die Timerzeit (am nächsten Tag), neu gesetzt wird.
Könnte man denn DOIF so umprogrammieren, das ein Zeitdummy quasi DOIF triggert, wenn sich der wert ändert und so die Timer neu gesetzt werden?

Damian, ich hoffe Du verstehst, was ich meine!

Wenn man eine indirekte Zeit angibt z. B. [[my_time]], dann wird sie bei Änderung sofort aktualisiert. Funktionsaufrufe dagegen werden einfach zum Ausführungszeitpunkt ausgeführt, sie werden vom Modul nicht verwaltet, daher kann das Modul nicht erkennen, ob sich am Aufruf etwas ändert oder nicht, insb. nicht bei der Angabe von Value("my_time").

Gruß

Damian

[Damian]

Nach erfolgreichen Tests habe ich nun diese Version eingecheckt, da es gehäuft vorkommt, dass die User sich unbewusst (meistens indirekt) Rekursionen einbauen. FHEM unterbindet zwar auch Rekursionen, allerdings bei DOIF-Definitionen greift es oft einen Schritt zu spät.

Dieser Absatz aus der Commandref zu DOIF wurde entfernt, da er jetzt keine Bedeutung hat:

Rekursionen vermeiden

Das Verändern des Status eines Devices z. B. durch set-Befehl, welches in der Bedingung bereits vorkommt, würde das Modul erneut triggern. Solche Rekursionen (Loops) werden zwar von FHEM unterbunden, können jedoch elegant durch Abfragen mit Fragezeichen [?...] gelöst werden:

statt:

define di_lamp ([brightness] < 50 and [lamp] eq "off")(set lamp on)

mit Fragezeichen abfragen:

define di_lamp ([brightness] < 50 and [?lamp] eq "off")(set lamp on)

Abfragen, die grundsätzlich nicht triggern sollen, sollten dagegen weiterhin mit Fragezeichen angegeben werden.

Gruß

Damian

[Toto1973]

Wenn man eine indirekte Zeit angibt z. B. [[my_time]], dann wird sie bei Änderung sofort aktualisiert. Funktionsaufrufe dagegen werden einfach zum Ausführungszeitpunkt ausgeführt, sie werden vom Modul nicht verwaltet, daher kann das Modul nicht erkennen, ob sich am Aufruf etwas ändert oder nicht, insb. nicht bei der Angabe von Value("my_time").

Dankeschön!
Ich habe meine sub nun so umgebaut, das die "errechnete" Zeit in ein dummy geschrieben wird.
Nachts um 1 Uhr lassen ich die Sub laufen und jetzt aktualisiert sich auch die Zeit im Timer der DOIF.

[Ellert]

Ich möchte meine Heizkörperventile im Wohnzimmer erst nach dem Lüften auf Tagbetrieb setzen. Damit der Türkontakt nicht jedes mal beim Schließen den Tagbetrieb triggert, frage ich "state" des DOIF ab. Das klappt auch alles, wie es soll, auch nach dem hier beschriebenen Unterbinden von Loops.

Meine Frage ist: Wird ein Rückbezug, wie [?meindi] !~ "cmd_2" zukünftig weiterhin erlaubt sein.

Code Auswählen Erweitern

define meindi DOIF (...) (...)
DOELSEIF ([?06:00-08:00] and [Türkontakt] eq "closed" and [?meindi] !~ "cmd_2") (set WZventil Tagbetrieb)
DOELSEIF (...) (...)
.
.
.
DOELSEIF (...) (...)

attr meindi do always


Falls das weiterhin möglich sein wird, werde ich auf auf diese Weise einen 3-fach Schrittschalter realisieren, der dann etwa so aussieht:

Code Auswählen Erweitern

define step_di DOIF ([step_du] and [step_di] =~ "cmd_3|initialized")  ({Log 3, "[step_di]"})
DOELSEIF ([step_du] and [?step_di] =~ "cmd_1") ({Log 3, "[step_di]"})
DOELSEIF ([step_du] and [?step_di] =~ "cmd_2") ({Log 3, "[step_di]"})

"step_du" triggert das Weiterschalten.

[Damian]

Ich möchte meine Heizkörperventile im Wohnzimmer erst nach dem Lüften auf Tagbetrieb setzen. Damit der Türkontakt nicht jedes mal beim Schließen den Tagbetrieb triggert, frage ich "state" des DOIF ab. Das klappt auch alles, wie es soll, auch nach dem hier beschriebenen Unterbinden von Loops.

Meine Frage ist: Wird ein Rückbezug, wie [?meindi] !~ "cmd_2" zukünftig weiterhin erlaubt sein.

Code Auswählen Erweitern

define meindi DOIF (...) (...)
DOELSEIF ([?06:00-08:00] and [Türkontakt] eq "closed" and [?meindi] !~ "cmd_2") (set WZventil Tagbetrieb)
DOELSEIF (...) (...)
.
.
.
DOELSEIF (...) (...)

attr meindi do always


Falls das weiterhin möglich sein wird, werde ich auf auf diese Weise einen 3-fach Schrittschalter realisieren, der dann etwa so aussieht:

Code Auswählen Erweitern

define step_di DOIF ([step_du] and [step_di] =~ "cmd_3|initialized")  ({Log 3, "[step_di]"})
DOELSEIF ([step_du] and [?step_di] =~ "cmd_1") ({Log 3, "[step_di]"})
DOELSEIF ([step_du] and [?step_di] =~ "cmd_2") ({Log 3, "[step_di]"})

"step_du" triggert das Weiterschalten.

Zur Zeit steht im Status der Zustand der letzten Ausführung. Ich denke, dass das so bleiben wird, alles andere wäre aus heutiger Sicht nicht sinnvoll.

Gruß

Damian