neuer FHEM-Befehl IF

[Puschel74]

Moin,

@tommy
Poste doch mal Alarmon und Alarmoff hier schön untereinander in einen Beitrag so wie du sie jetzt bei dir hast.

Grüße

[Tommy82]

Moin, bin leider jetzt auf der Arbeit und komme nur per Handy an fhem, das list von Alarm in sieht so aus

Code Auswählen Erweitern

Internals:
   DEF        *19:45:00 IF ([Alarmanlage_Wohnzimmer] eq "On" or [Alarmanlage_Wohnzimmer] eq "Auto")  (  IF ([TuerFensterWohnzimmer] eq "AllesZu")       (set Alarmanlage_Wohnzimmer on)     ELSE       ({FB_mail('Xyz@googlemail.com','FHEM Fenster Offen','Alarm kann nicht aktiviert werden, Fenster oder Tuer noch offen' )}) )
   NAME       alarmOn
   NR         167
   REP        -1
   STATE      Next: 19:45:00
   TRIGGERTIME 1414172700
   TRIGGERTIME_FMT 2014-10-24 19:45:00
   TYPE       at
Attributes:
Das list von AlarmOff so

Code Auswählen Erweitern

Internals:
   DEF        *19:55:00 IF ([Alarmanlage_Wohnzimmer] eq "Off" or [Alarmanlage_Wohnzimmer] eq "Auto")       (set Alarmanlage_Wohnzimmer off)     ELSE       (set Alarmanlage_Wohnzimmer on)
   NAME       alarmOff
   NR         172
   REP        -1
   STATE      Next: 19:55:00
   TRIGGERTIME 1414173300
   TRIGGERTIME_FMT 2014-10-24 19:55:00
   TYPE       at
Attributes:

Und die beiden Codes mit denen ich die Befehle gesetzt habe sehen so aus:

alarmOn

Code Auswählen Erweitern

define alarmOn at *19:45:00 IF ([Alarmanlage_Wohnzimmer] eq "On" or [Alarmanlage_Wohnzimmer] eq "Auto")
(  IF ([TuerFensterWohnzimmer] eq "AllesZu")
     (set Alarmanlage_Wohnzimmer on)
   ELSE
     ({FB_mail('Xyz@googlemail.com','FHEM Fenster Offen','Alarm kann nicht aktiviert werden, Fenster oder Tuer noch offen' )})
)

alarmOff

Code Auswählen Erweitern

define alarmOff at *19:55:00 IF ([Alarmanlage_Wohnzimmer] eq "Off" or [Alarmanlage_Wohnzimmer] eq "Auto")
     (set Alarmanlage_Wohnzimmer off)
   ELSE
     (set Alarmanlage_Wohnzimmer on)

[Puschel74]

Hallo,

das hatte ich befürchtet 
Ich hab doch vorhin geschrieben das es für FHEM durchaus einen Unterschied macht ob ich etwas on schalte und dann aber auf On prüfe.
Das nennt sich case-sensitiv 

Grüße

edith: Aber ich seh schon deine nächste Frage - was ist jetzt noch falsch 

[Tommy82]

Hi, ok das case-sensitiv was du oben schon meintest hatte ich falsch verstanden, hatte verstanden das auf off schalten keinen Sinn macht es dann auf off zu prüfen...

Werd das Heut Abend mal anpassen.

Und dann mal überlegen wie die Abfrage mehr Sinn macht bzw. funktioniert, damit ich die Baustelle mal schließen kann

[Puschel74]

Hallo,

hatte verstanden das auf off schalten keinen Sinn macht es dann auf off zu prüfen

Das habe ich auch nicht geschrieben.

Hallo,
auch macht es für FHEM einen Unterschied ob ich etwas off schalte und dann auf Off prüfe.
Grüße

Ich hatte es extra für dich noch dick markiert in der Hoffnung es springt dir ins Auge 
Aber da habe ich wohl zu viel erhofft 
Nächstesmal bekommst du Schriftgröße 36

Grüße

Edith: @Damian - ich hoffe mal das IF auch case-sensitiv ist 

[Damian]

Edith: @Damian - ich hoffe mal das IF auch case-sensitiv ist 

IF ist nichts anderes als ein Aufruf von perl-if, also kontextsensitiv, d. h.  "off" ist nicht das Gleiche wie "Off"

Gruß

Damian

[Tommy82]

So,
hab mal meine Pause genutzt, hoffe das ich jetzt das case-sensitiv komplett berücksichtigt habe:-) , konnte es leider noch nicht testen.

Code Auswählen Erweitern

define alarmOn at *19:45:00 IF ([Alarmanlage_Wohnzimmer] eq "on" or [Alarmanlage_Wohnzimmer] eq "Auto")
(  IF ([TuerFensterWohnzimmer] eq "AllesZu")
     (set Alarmanlage_Wohnzimmer on)
   ELSE
     ({FB_mail('Th.Halberstadt@googlemail.com','FHEM Fenster Offen','Alarm kann nicht aktiviert werden, Fenster oder Tuer noch offen' )})
)


define alarmOff at *19:55:00 IF ([Alarmanlage_Wohnzimmer] eq "off" or [Alarmanlage_Wohnzimmer] eq "Auto")
     (set Alarmanlage_Wohnzimmer off)
   ELSE
     (set Alarmanlage_Wohnzimmer on)

Allerdings bin ich mir bei der logik immer noch sehr unsicher............. Die Alarmanlage soll ja im "normal" zustand immer auf Auto stehn und zu den definierten Zeiten on bzw. off gehen. Wenn ich dann den Button on bzw. off betätige soll dieser Status solange gesetzt bleiben bis ich ihn wieder auf Auto stelle.

Brauch ich im alarmOff das hier berhaupt?

(set Alarmanlage_Wohnzimmer off)
   ELSE
     (set Alarmanlage_Wohnzimmer on)[/code]

Wie gesagt konnte es noch nicht testen, werd es heute abend nachholen.

[Puschel74]

Hallo,

Die Alarmanlage soll ja im "normal" zustand immer auf Auto stehn

Und in welchem Codeteil wird sie auf Auto geschalten 

Ich seh überall nur ein

Code Auswählen Erweitern

set Alarmanlage_Wohnzimmer on und

Code Auswählen Erweitern

set  Alarmanlage_Wohnzimmer off
aber nirgends ein

Code Auswählen Erweitern

set Alarmanlage_Wohnzimmer Auto

Denkst du nicht du solltest dir heute abend besser nochmal in Ruhe das Einsteiger.pdf durchlesen?
Und das war jetzt weder böse noch sarkastisch gemeint sondern nur ein gut gemeinter Tipp.

Grüße

[Tommy82]

Hallo Puschel,
hab mir deinen Vorschlag zu herzen genommen, und mir grade nochmal das Einsteiger Handbuch vorgenommen und dortz vorallem den Bereich

Einfache Programmierung: if-Bedingung

, das Problem jetzt bin ich richtig verwirrt 
Bzw. ich hab iregendwo einen Logik fehler denn ich nicht finde, denke ja was ich erreichen will ist klar.

Neuer Ansatz:

Hab alarmOn und alarmOff mal so geändert:

Code Auswählen Erweitern

define alarmOn at *19:45:00 IF ([Alarmanlage_Wohnzimmer] eq "on")
(  IF ([TuerFensterWohnzimmer] eq "AllesZu")
     (set Alarmanlage_Wohnzimmer on)
   ELSE
     ({FB_mail('xyz@googlemail.com','FHEM Fenster Offen','Alarm kann nicht aktiviert werden, Fenster oder Tuer noch offen' )})
)


define alarmOff at *19:55:00 IF ([Alarmanlage_Wohnzimmer] eq "off" )
     (set Alarmanlage_Wohnzimmer off)



Das wäre dann die normale Aktion.
Das Manuelle On /Off würde ich hier rüber regeln

Code Auswählen Erweitern

define alarmManuell { IF (Value("Alarmanlage_Wohnzimmer") eq "on" ) { fhem ( "set TuerFensterWohnzimmer eq "AllesZu") } ELSE (Value("Alarmanlage_Wohnzimmer") eq "off" { fhem ("set Alarmanlage_Wohnzimmer off") } }

Werd das jetzt mal testen, aber kommt mir noch irgendwie seltsam vor........

EDIT
Hab jetzt ein wenig rumprobiert, und wie ich erwartet hatte es funktioniert nicht

[rudolfkoenig]

Bitte dran denken: if ist nicht gleich IF, das PDF beschreibt if, und hier geht es um IF.
Das ist glaube ich so ein Fall, wovor ich in Beitrag #7 gewarnt habe.
Unter dem Strich sind aber scheinbar mehr Anweder gluecklich mit IF als ungluecklich.

[Puschel74]

Hallo,

das ist korrekt Rudi aber der Anwender sollte schon ein klein wenig Vorstellung davon haben was er erreichen möchte.
Wenn ich mir den Code so ansehe - ist es egal ob if oder IF 

Code Auswählen Erweitern

define alarmOn at *19:45:00 IF ([Alarmanlage_Wohnzimmer] eq "on")
(  IF ([TuerFensterWohnzimmer] eq "AllesZu")
     (set Alarmanlage_Wohnzimmer on)

Um 19:45 wird geprüft on die Alarmanlage on ist, dann muss noch TuerFenster gleich AllesZu sein damit - die Alarmanlage on geschaltet wird.
Wozu? Sie ist ja bereits on sonst wären wir nicht hier.

Nun der ELSE-Teil:
Um 19:45 wird geprüft ob die Alarmanlage on ist, wenn dann TuerFenster ungleich AllesZu ist wird die Mail gesendet.
Ok, das kann Sinn machen.

Code Auswählen Erweitern

define alarmOff at *19:55:00 IF ([Alarmanlage_Wohnzimmer] eq "off" )
     (set Alarmanlage_Wohnzimmer off)
Hier wird es olympisch 
Um 19:55 wird geprüft ob die Alarmanlage off ist um sie dann off zu schalten 

Tommy - egal ob if oder IF.
Du brauchst dir doch nur durchlesen was du da programmierst.
Der Code wird nicht angemeckert - syntaktisch ist er ja richtig.
Aber ergibt er auch Sinn für dich?

Grüße

Edith: Vorschlag zur Güte - ich geh mit meienr Frau in Ruhe auf die Terrasse eine rauchen.
Du schickst mir per PN WAS du genau haben möchtest - keinen Code sondern in deinen eigenen Worten (aber kein Roman).
Ich schau was ich dir daraus zaubern kann.
Das würde Damians Beitrag hier erstmal um einige Seiten entlasten 

[Tommy82]

Hi Puschel, nein er ergibt keinen Sinn da hast du leider recht... Wie gesagt, hab da grad einen absoluten Logik (denk) fehler, obwohl die Aufgabenstellung eigentlich recht abstrakt ist....., steh mir echt auf dem schlauch.. Naja ich spiel noch was 

So alarmOn mal so geändert, mal sehen was um 18:00 passiert

Code Auswählen Erweitern

define alarmOn at *18:05:00
  IF ([TuerFensterWohnzimmer] eq "AllesZu")
     (set Alarmanlage_Wohnzimmer on)
   ELSE
     ({FB_mail('xyz@googlemail.com','FHEM Fenster Offen','Alarm kann nicht aktiviert werden, Fenster oder Tuer noch offen' )})

So hier alarmOff

Code Auswählen Erweitern

define alarmOff at *18:25:00 {fhem ("set Alarmanlage_Wohnzimmer off")}

[Damian]

Bitte dran denken: if ist nicht gleich IF, das PDF beschreibt if, und hier geht es um IF.
Das ist glaube ich so ein Fall, wovor ich in Beitrag #7 gewarnt habe.
Unter dem Strich sind aber scheinbar mehr Anweder gluecklich mit IF als ungluecklich.

Ich glaube, hier geht es vorallem um das grundsätzliche Verständnis der logischen Konstruktionen für Nicht-Programmierer:

Beispiel von @Tommy82:

IF ([Alarmanlage_Wohnzimmer] eq "on")
(  IF ([TuerFensterWohnzimmer] eq "AllesZu")
     (set Alarmanlage_Wohnzimmer on)
...

Warum soll ich Alarmanlage_Wohnzimmer auf "on" schalten, wenn sie schon vorher "on" war. So etwas wird auch mit if nicht gut funktionieren.

Wenn Anwender, insb. Einsteiger, IF nutzen, dann sollte man zumindest in der Einsteiger-Doku auf die Commandref zu IF hinweisen.
Dort habe ich diverse Beispiele zu IF beschrieben.

Viele Fälle kann man aber inzwischen genauso gut mit DOIF erledigen und dazu gibt es ebenso unzählige Beispiele in der Commandref.

Gruß

Damian

[Tommy82]

Hi Puschel, nein er ergibt keinen Sinn da hast du leider recht... Wie gesagt, hab da grad einen absoluten Logik (denk) fehler, obwohl die Aufgabenstellung eigentlich recht abstrakt ist....., steh mir echt auf dem schlauch.. Naja ich spiel noch was 

So alarmOn mal so geändert, mal sehen was um 18:00 passiert

Code Auswählen Erweitern

define alarmOn at *18:05:00
  IF ([TuerFensterWohnzimmer] eq "AllesZu")
     (set Alarmanlage_Wohnzimmer on)
   ELSE
     ({FB_mail('xyz@googlemail.com','FHEM Fenster Offen','Alarm kann nicht aktiviert werden, Fenster oder Tuer noch offen' )})

So hier alarmOff

Code Auswählen Erweitern

define alarmOff at *18:25:00 {fhem ("set Alarmanlage_Wohnzimmer off")}

So funktioniert jetzt das On und Off, ist das denn so auch richtig?

Jetzt fehlt nur noch der Schalter ums Manuell zu schalten
Idee dafür wäre

Code Auswählen Erweitern

define alarmManuel notify IF ("Alarmanlage_Wohnzimmer") eq "on" ) {fhem ("set Alarmanlage_Wohnzimmer off") } ELSE ("Alarmanlage_Wohnzimmer") eq "off" ) {fhem ("set Alarmanlage_Wohnzimmer on") aber das gefällt mir auch noch nicht so richtig....., sieht noch komisch aus.....

[Puschel74]

Hallo,

Mensch tommy

Das würde Damians Beitrag hier erstmal um einige Seiten entlasten

Ich grübel da per PN und du müllst den Beitrag dennoch munter voll - gehts noch 
*grumpf*
Lesen, denken, verstehen

Grüße