if Abfrage durch DOIF ersetzen

[Supertux]

Hallo zusammen
Habe zu Testzwecke einen MCP23017 an meinem Raspberry am laufen.
Als Device hat dieser den Namen mcp_eingabe.
Außerdem bewirken zwei Taster am Port A2 und A3 ein Ereignis zwischen on und off.
Das Modul Rollo soll mit diesen beiden Tastern angesprochen werden. Stoppen des Rollos bei drücken des entgegengesetzten Tasters
Folgender Code für Rollo Ab:

Code Auswählen Erweitern

mcp_eingang:PortA3:.on { if ( ReadingsVal("Rollo","state","") eq "drive-up") {fhem("set Rollo stop")} else {fhem("set Rollo closed")}}

Folgender Code für Rollo Auf:

Code Auswählen Erweitern

mcp_eingang:PortA2:.on { if ( ReadingsVal("Rollo","state","") eq "drive-down") {fhem("set Rollo stop")} else {fhem("set Rollo open")}}

Wollte dies eleganter lösen mit einem DOIF. Doch trotz lesen der Commandref habe ich so meine Probleme mit der Umsetzung (Wahrscheinlich habe ich noch ein Fehler beim auslesen von states und ReadingsVal im DOIF)
Kann mir da jemand einen Denkanstoß geben.

Dankeschön

[Amenophis86]

Naja, ob es eleganter mit DOIF ist, ist eine Geschmacksfrage, aber da du es lernen willst helfen wir gerne. Jedoch solltest du mal posten was du bisher hast und was die Fehler Ausgabe (Log) ist. Sonst läuft es darauf hinaus, dass dir jemand einen fertigen Code schreibet und das bringt ja dann auch nix.

[Supertux]

Hallo
Folgenden Ansatz hatte ich mir überlegt:

Code Auswählen Erweitern

([mcp_eingang:PortA3] eq "on" and [Rollo] eq "drive-up") (set Rollo stop) DOELSEIF (set Rollo closed)

PortA3 wäre der Taster zum schließen des Rollos
Jedoch wird das DOIF bei einem "stehendem" Rollladen nicht aktiv, weil die Bedingung "drive-up" nicht erfüllt ist und so das DOIF abricht.

Meine Überlegung für die gesamte Steuerung:
- Zwei DOIFś
- Der Trigger ist der Eingang PortA2 (hoch) oder PortA3 (runter)
- das DOIF sollte jeweils überprüfen ob der Rollladen fährt. Wenn nicht soll es in die Richtung des dafür vorgesehenen Taster fahren
- Fährt der Rolladen soll es diesen stoppen

Mit dem Code aus dem Anfangspost läuft es und ich würde das auch so umsetzen. Mir gehts ums lernen und verstehen

Gruß

[Amenophis86]

Wenn du "DOELSEIF" schreibst, dann musst du auch eine Bedingung setzen, sonst heißt es einfach nur DOELSE. (Dann dürfte es auch bei stehendem Rollladen aktiv werden )

[Supertux]

Hallo
Danke für den Hinweis mit DOELSEIF
Folgender Code funktioniert:

Code Auswählen Erweitern

([mcp_eingang:PortA3] eq "on" and [Rollo] eq "drive-up") (set Rollo stop)
DOELSEIF ([mcp_eingang:PortA3] eq "on") (set Rollo closed)

ein

Code Auswählen Erweitern

attr test_mcp_auf do always

Hatte dann zum gewünschten Erfolg erzielt.
Wobei ich jetzt den Eingangsstehenden Code verständlicher/leichter finde.

Danke nochmals 

[Amenophis86]

Du wirst aber noch das Problem haben, dass er auch auf

Code Auswählen Erweitern

[Rollo] eq "drive-up" hören wird. Das heißt, wenn dein Rollo diesen Zustand einnimmt un der Schalter noch auf on steht, dann wird die Bedigung auch wahr. Daher solltest du mit 

Code Auswählen Erweitern

[?Rollo] eq "drive-up" arbeiten. Das heißt es wird nur geschaut, ob die Bedingung wahr ist, sie triggert aber selbst nicht das DOIF.

Und dann müsste es eigentlich auch gehen, wenn du

Code Auswählen Erweitern

DOELSEIF ([mcp_eingang:PortA3] eq "on") (set Rollo closed) durch

Code Auswählen Erweitern

DOELSE (set Rollo closed) ersetzt. Da bin ich mir aber gerade nicht 100% sicher, müsstest du mal testen.

Was man leichter findet muss jeder selbst entscheiden, der eine mag DOIF, der andere notify etc. Reine Geschmackssache

[Pfriemler]

Und dann müsste es eigentlich auch gehen, wenn du

Code Auswählen Erweitern

DOELSEIF ([mcp_eingang:PortA3] eq "on") (set Rollo closed) durch

Code Auswählen Erweitern

DOELSE (set Rollo closed) ersetzt. Da bin ich mir aber gerade nicht 100% sicher, müsstest du mal testen.

Nö. Das Triggerdevice ist mcp_eingang:PortA3 - DOELSE würde auch greifen, wenn es z.B. "off" wird.

[Amenophis86]

Stimmt, hatte vergessen, dass es nur bei on triggern soll