DOIF neue Features: Generalisierung, $DEVICE, $EVENT, Attribut notexist

[JoeALLb]

Noch nicht alle

Wenn wir schon am träumen sind...
Wir wäre es mit einer (einfachen) Verschachtelung von IFs ,ähnlich zu

Code Auswählen Erweitern


(set lg_light on [DOIFsub $EVENT eq "TEMPL"])



oder gar

Code Auswählen Erweitern


  DOIF ([winter])
    (set desired-temp 18 )
    CASE ([light] eq "on")
       (set xyy)
    CASE ([garage] eq "offen")
       (set xyyZZ)
    CASEEND
  DOELSEIF ([sommer])
    (set desired-temp off )


Zum Hintergrund:
Ich habe in ein großes DOIF mit sehr vielen (set ...) Einträgen, manche haben aber bestimmte Bedingungen wie Lichteinfall, etc.
Mit reinen DOELSEIF ist das schwierig, mit notify auch . Das alles in eigene DOIFS zu ziehen wird unübersichtlich...

Aber wie gesagt, nur ein Traum... vielen Dank für die tolle Arbeit!

[Per]

Ich glaube nun haben wir alle Funktionen die benötigt werden!

Ich ich hätte da auch noch Ideen.

[l2r]

@JoeALLb: ich könnte mir vorstellen dass sich mit "?:"  da was basteln lässt...

http://forum.fhem.de/index.php/topic,48364.msg402589.html#msg402589

[Ellert]

Ich versuche gerade ein DOIF zu definieren mit

Code Auswählen Erweitern

define doif DOIF ([Test:"test: "])

da kommt eine Fehlermeldung: DOIF: unknown expression format: ". Es liegt am Doppelpunkt in  "test: ".

Ist der Doppelpunkt als Zeichen nicht erlaubt? Ich dachte in der neuen DOIF Version ist er erlaubt.

[Damian]

Ideen sind natürlich immer willkommen. Zu bedenken ist allerdings immer, das Modul nicht unnötig komplizierter zu machen als es schon ist. Manche Sachen lassen sich leicht einbauen, Manche Sachen würde man wieder gerne herausnehmen, weil sie im Nachhinein Verwirrung stiften. Dann gibt es das Problem der verfügbaren Programmierzeit. Und es gibt  Dinge, die der Anwender zunächst nicht sieht, aber auf meiner Liste ganz oben stehen:

1. Nur ein Trigger bei gleichen Zeiten -> damit wird die Reihenfolge der Abarbeitung gewährleistet (ist zur Zeit nicht der Fall)

2. Setzen der Anfangszeit eines Zeitintervalls erst beim End-Trigger des Intervalls -> damit würde man bei Zeitintervallen nicht mehrfach triggern, wenn die neue Zeit sich noch innerhalb des alten Intervalls befindet z. B. [([10:00]+rand(120)) - 12:00]

Es gibt also genug zu tun. Es lebe der Karneval und die Zeit zum Programmieren

Gruß

Damian

[Damian]

Ich versuche gerade ein DOIF zu definieren mit

Code Auswählen Erweitern

define doif DOIF ([Test:"test: "])

da kommt eine Fehlermeldung: DOIF: unknown expression format: ". Es liegt am Doppelpunkt in  "test: ".

Ist der Doppelpunkt als Zeichen nicht erlaubt? Ich dachte in der neuen DOIF Version ist er erlaubt.

ja, bei

Code Auswählen Erweitern


(["Test:test: "])

habe ich es schon berücksichtigt, in dem anderen Fall wohl noch nicht.

[Ellert]

Wenn wir schon am träumen sind...
Wir wäre es mit einer (einfachen) Verschachtelung von IFs ,ähnlich zu

Code Auswählen Erweitern


(set lg_light on [DOIFsub $EVENT eq "TEMPL"])



oder gar

Code Auswählen Erweitern


  DOIF ([winter])
    (set desired-temp 18 )
    CASE ([light] eq "on")
       (set xyy)
    CASE ([garage] eq "offen")
       (set xyyZZ)
    CASEEND
  DOELSEIF ([sommer])
    (set desired-temp off )


Zum Hintergrund:
Ich habe in ein großes DOIF mit sehr vielen (set ...) Einträgen, manche haben aber bestimmte Bedingungen wie Lichteinfall, etc.
Mit reinen DOELSEIF ist das schwierig, mit notify auch . Das alles in eigene DOIFS zu ziehen wird unübersichtlich...

Aber wie gesagt, nur ein Traum... vielen Dank für die tolle Arbeit!

Das geht schon:

Code Auswählen Erweitern

  DOIF ([winter])
    (set desired-temp 18,
    IF ([light] eq "on")
       (set xyy),
    IF ([garage] eq "offen")
       (set xyyZZ))
DOELSEIF ([sommer])
    (set desired-temp off )

[JoeALLb]

Das geht schon:

VIELEN Dank!!! Das war mich echt nicht bewußt, und ich habe schon einiges darüber gesucht...
Auch aus der commandref kann ich das nicht herauslesen?!?
Beiträgen wie diese führten mich auch nicht weiter http://forum.fhem.de/index.php?topic=34935.0!!

Wie auch immer, VIELEN DANK!!! Mein Tag ist gerettet.

[Ellert]

VIELEN Dank!!! Das war mich echt nicht bewußt, und ich habe schon einiges darüber gesucht...
Auch aus der commandref kann ich das nicht herauslesen?!?
Beiträgen wie diese führten mich auch nicht weiter http://forum.fhem.de/index.php?topic=34935.0!!

Wie auch immer, VIELEN DANK!!! Mein Tag ist gerettet.

Im Nachhinein denke ich, es hat nicht die gewünschte Funktionalität, denn IF ([ABC] eq "xyz") triggert nicht, es ist ein FHEM-Befehl, s. http://fhem.de/commandref_DE.html#IF

[Damian]

Wenn wir schon am träumen sind...
Wir wäre es mit einer (einfachen) Verschachtelung von IFs ,ähnlich zu

Code Auswählen Erweitern


(set lg_light on [DOIFsub $EVENT eq "TEMPL"])



oder gar

Code Auswählen Erweitern


  DOIF ([winter])
    (set desired-temp 18 )
    CASE ([light] eq "on")
       (set xyy)
    CASE ([garage] eq "offen")
       (set xyyZZ)
    CASEEND
  DOELSEIF ([sommer])
    (set desired-temp off )


Zum Hintergrund:
Ich habe in ein großes DOIF mit sehr vielen (set ...) Einträgen, manche haben aber bestimmte Bedingungen wie Lichteinfall, etc.
Mit reinen DOELSEIF ist das schwierig, mit notify auch . Das alles in eigene DOIFS zu ziehen wird unübersichtlich...

Aber wie gesagt, nur ein Traum... vielen Dank für die tolle Arbeit!

Es ist auch nicht klar, ob der CASE-Fall ohne Winter-Trigger ausgewertet werden soll oder nicht, auf der anderen Seite kann Winter und light nicht gleichzeitig triggern. Mit anderen Worten, wenn man sich eine Syntax überlegt, dann muss man auch alle Eventualitäten berücksichtigen.

Man könnte auch definieren:

Code Auswählen Erweitern

DOIF ([winter] or [light] or [garage])
    (set desired-temp 18,
    IF ([light] eq "on")
       (set xyy),
    IF ([garage] eq "offen")
       (set xyyZZ))
DOELSEIF ([sommer])
    (set desired-temp off )

Es gibt also auch jetzt schon vielfältige Möglichkeiten seine Wünsche umzusetzen.

Gruß

Damian

[Per]

Code Auswählen Erweitern

DOIF ([winter] or [light] or [garage])

Sollte es nicht ([winter] and ([light] or [garage])) sein? Zumindest passend zur angegebenen Aufgabenstellung.

[Damian]

Sollte es nicht ([winter] and ([light] or [garage])) sein? Zumindest passend zur angegebenen Aufgabenstellung.

Nein, es geht hier nur um den Trigger, der Inhalt der Variablen ist egal und normalerweise immer wahr, weil ungleich "".

[Per]

Nein, es geht hier nur um den Trigger

Danke!

[Damian]

ja, bei

Code Auswählen Erweitern


(["Test:test: "])

habe ich es schon berücksichtigt, in dem anderen Fall wohl noch nicht.

Code Auswählen Erweitern

([Test:"test: "])

funktioniert ab morgen auch.

Gruß

Damian

[Ellert]

Danke, ich konnte es im Wirkbetrieb gleich anwenden.