[gelöst] DOIF-Ärger

[Tuxi70]

Ich habe mir da was gebastelt, was die Heizung Wetterabhängig steuern soll. Aber dieser DOIF-Abschnitt bereitet mir bisher gut 48h Kopfschmerzen.

Code Auswählen Erweitern


define AutoRadiator DOIF (([WetterProplanta_temperature] >=15) and
                                           ([WetterProplanta_weather] eq 'sonnig') (set Heizung closed) 
                               DOELSE (set Heizung desired-temp 22))
attr AutoRadiator devStateIcon .*:sani_heating_level_90
attr AutoRadiator room automatik
attr AutoRadiator state enabled


Aber ich erhalte die Fehlermeldung auf dem Webinterface (siehe Screenshot), die ich nicht via Google und diese Seite lösen konnte.

[Frank_Huber]

Code Auswählen Erweitern

define AutoRadiator DOIF (([WetterProplanta:temperature] >=15) and
                                           ([WetterProplanta:weather] eq 'sonnig') (set Heizung closed) 
                               DOELSE (set Heizung desired-temp 22))

DEVICE:READING

nicht DEVICE_READING

[marwal]

Du hast deine Anweisung in extra Klammern gesetzt, versuche es mal ohne.

Code Auswählen Erweitern

define AutoRadiator DOIF ([WetterProplanta_temperature] >=15) and
                                           ([WetterProplanta_weather] eq 'sonnig') (set Heizung closed)
                               DOELSE (set Heizung desired-temp 22)

LG

[Otto123]

Zähl mal die Klammern Deiner Bedingung 

Code Auswählen Erweitern

(([WetterProplanta_temperature] >=15) and ([WetterProplanta_weather] eq 'sonnig'))
Da fehlt eine bzw sind 3 zuviel

Code Auswählen Erweitern

([WetterProplanta_temperature] >=15 and [WetterProplanta_weather] eq 'sonnig')
http://perldoc.perl.org/perlop.html#Operator-Precedence-and-Associativity

Bei ELSE genauso - eine zuviel

Code Auswählen Erweitern

(set Heizung desired-temp 22)

Gruß Otto

[Ellert]

... und wenn Du die Klammern nicht selbst zählen willst: https://wiki.fhem.de/wiki/DOIF/Tipps_zur_leichteren_Bedienung

oder 48h Kopfschmerzen umgehen möchtest: https://wiki.fhem.de/wiki/DOIF/Tools_und_Fehlersuche

[Otto123]

Naja das macht der integrierte Editor auch...
Wie ich jetzt erst  verstanden habe, hat er aus unerfindlichen Gründen das komplette DOIF geklammert 

Gruß Otto

[Tuxi70]

Für eure Hilfe besten Dank. Aber jetzt es überhaupt nicht mehr.

Jetzt ist bei mir die Verwirrung komplet. 

Code Auswählen Erweitern

configfile: AutoRadiator DOIF: expected DOELSEIF or DOELSE:  and ([WetterProplanta_weather] eq 'sonnig') (set Heizung closed) DOELSE (set Heizung desired-temp 22)

Eure Korrekturen habe via Weboberfläche nach der vorherigen Bereinigung eingepflegt.

[nils_]

bereitet mir bisher gut 48h Kopfschmerzen.

wow, sehr schmerzhaft, und dann auch noch so lange. üble sache!!!


nur nochmal als info:
screenshots posten ist nicht immer hilfreich.
hier wäre ein auszug aus dem logfile oder aber ein list <device> zielführender gewesen.

[Otto123]

So sieht es jetzt bei Dir aus?

Code Auswählen Erweitern

define AutoRadiator DOIF ([WetterProplanta_temperature] >=15 and [WetterProplanta_weather] eq 'sonnig') (set Heizung closed)  DOELSE (set Heizung desired-temp 22)

[Ellert]

Für eure Hilfe besten Dank. Aber jetzt es überhaupt nicht mehr.

Jetzt ist bei mir die Verwirrung komplet. 

Code Auswählen Erweitern

configfile: AutoRadiator DOIF: expected DOELSEIF or DOELSE:  and ([WetterProplanta_weather] eq 'sonnig') (set Heizung closed) DOELSE (set Heizung desired-temp 22)

Eure Korrekturen habe via Weboberfläche nach der vorherigen Bereinigung eingepflegt.

Das "configfile" zu bearbeiten ist keine gute Idee, wenn man nicht genau weiss wie.

[Tuxi70]

So sieht es jetzt bei Dir aus?

Code Auswählen Erweitern

define AutoRadiator DOIF ([WetterProplanta_temperature] >=15 and [WetterProplanta_weather] eq 'sonnig') (set Heizung closed)  DOELSE (set Heizung desired-temp 22)

Jetzt ja. Jetzt kommt nur eine Warnung.

Code Auswählen Erweitern

setstate AutoRadiator 2018-03-13 12:45:27 warning condition c01: Argument "" isn't numeric in numeric ge (>=)\

[Frank_Huber]

poste mal ein list von Proplante bitte.
ich vermute immernoch dass du die bedingungen falsch hast. (_ anstatt

[Per]

Zuviele Klammern, zuwenig Klammern, Unterstriche statt Doppelpunkte...
Und eine extrem aussagekräftige Überschrift .

Wahrscheinlich soll es das werden:

Code Auswählen Erweitern

define AutoRadiator DOIF ([WetterProplanta:temperature] >=15 and [WetterProplanta:weather] eq 'sonnig') (set Heizung closed) DOELSE (set Heizung desired-temp 22)
PS: Ansichtlich als Einzeiler geschrieben!

[Otto123]

In seinem Ausgangsthread hatte er ja im DOIF ordentliche Werte drin stehen. Deswegen war ich davon ausgegangen, die Namen (mit Unterstrich) sind richtig. Aber wahrscheinlich hat er es dort anders definiert und dann im DOIF die Readings abgeschrieben?

Habe kein Proplanta Wetter

Gruß Otto

[CoolTux]

WetterProplanta_temperature

Wird wohl ein Dummy sein. Aber vielleicht erzählt der Threadersteller mal genaueres und liefert vor allem mal ein list aller Devices.