[Gelöst] Fehler bei DOIF und DOELSEIF

[Amenophis86]

Code Auswählen Erweitern

define z_advent DOIF ([06:00-22:00] and [BM_Bewegungsmelder:brightness] <= 100 and [{Advent}] eq "1") (set advent on)
DOELSEIF ([06:00-22:00] and [BM_Bewegungsmelder:brightness] > 100 and [{Advent}] eq "1") (set advent off)
DOELSEIF ([22:00]) (set advent off)

Versuch das mal. Vom Prinzip ist immer in [] das was geprüft werden soll und da hinter kommt der Operator und dann der Wert auf den es geprüft wird.

[raspklaus]

Fehler hat sich wieder mal geändert:

Code Auswählen Erweitern

z_advent DOIF: the at function "Advent" must return a timespec and not Month '12' out of range 0..11 at ./FHEM/99_myUtils.pm line 231.
.: {Advent}

Hier mal die sub:

Code Auswählen Erweitern

sub Advent() {
my ($Sekunde, $Minute, $Stunde, $Monatstag, $Monat, $Jahr, $Wochentag, $Jahrestag, $Sommerzeit) = localtime(time);
    my $Dez=0;
    my $adv=0;
    if($Monat<1) {
     if ($Monatstag<7) {$adv=1;}
     $Dez=-1;
    }
    if($Monat>9) {
     my $t = timelocal(0, 0, 0, 25, 12, $Jahr-$Dez);
     my $wd = (localtime $t)[6];
     $t = timelocal(0, 0, 0, 25-$wd-1, 12, $Jahr-$Dez);
my $dt2 = DateTime->new(
                       year   => $Jahr-$Dez,
                       month  => 12,
                       day    => 25-$wd-1,
                       hour   => 0,
                       minute => 0,
                       second => 0
                     );
      $dt2->subtract(days => 28);
      if($dt2->months()==11) {
        if($dt2->days >= $Monatstag ) {$adv=1;}
      } else {
        if($Monat==11) {$adv=1;}
        else {if($Monatstag>=$dt2->month()) {$adv=1;}}
      }
    }
    readingsSingleUpdate($defs{"dmy_DateTime"},"Advent",$adv,1);
    #return 0;
    return $adv;
}

Die Zeile 231 ist:

Code Auswählen Erweitern

my $t = timelocal(0, 0, 0, 25, 12, $Jahr-$Dez);

[kumue]

schon versucht, 12 durch 11 zu ersetzen ?

Code Auswählen Erweitern

my $t = timelocal(0, 0, 0, 25, 11, $Jahr-$Dez); <<<<<<<<<<<<<<<<< hier
     my $wd = (localtime $t)[6];
     $t = timelocal(0, 0, 0, 25-$wd-1, 11, $Jahr-$Dez);<<<<<<<<<<<<<<<<<<< hier
        my $dt2 = DateTime->new(
                       year   => $Jahr-$Dez,
                       month  => 11, <<<<<<<<<<<<<<<<<<<<<<<<<<<< hier
                       day    => 25-$wd-1,
                       hour   => 0,
                       minute => 0,
                       second => 0
                     );


[Damian]

Fehler hat sich wieder mal geändert:

Code Auswählen Erweitern

z_advent DOIF: the at function "Advent" must return a timespec and not Month '12' out of range 0..11 at ./FHEM/99_myUtils.pm line 231.
.: {Advent}

Bevor ihr noch tagelang weiter experimentiert.

Wenn deine Adventfunktion deklariert ist und in der Kommandozeile der Aufruf {Advent} den richtigen Wert liefert.

Dann kannst du in der Bedingung von DOIF Advent als eine Perlfunktion direkt angeben in der Form:

(.... and Advent==1 ...) (set bla...)

Eckige oder geschweifte Klammern sind hier fehl am Platz.

Gruß

Damian

[raspklaus]

Hallo Damian,

wie meinst Du in der Kommandozeile den richtigen Wert liefert.

Sorry habe die Sub nicht selbst geschrieben.

Was muss ich eingeben damit ein Wert angezeigt wird ?

Danke für deine Hilfe

[Damian]

Hallo Damian,

wie meinst Du in der Kommandozeile den richtigen Wert liefert.

Sorry habe die Sub nicht selbst geschrieben.

Was muss ich eingeben damit ein Wert angezeigt wird ?

Danke für deine Hilfe

Habe ich doch geschrieben:

Code Auswählen Erweitern

{Advent}

[raspklaus]

Danke, wieder was gelernt

Wenn ich {Advent} eingebe bekomme ich folgende Meldung wie oben

Code Auswählen Erweitern

Month '12' out of range 0..11 at ./FHEM/99_myUtils.pm line 231.

[Damian]

Danke, wieder was gelernt

Wenn ich {Advent} eingebe bekomme ich folgende Meldung wie oben

Code Auswählen Erweitern

Month '12' out of range 0..11 at ./FHEM/99_myUtils.pm line 231.

Dann muss die Funktion der Autor korrigieren und dann kannst du sie in DOIF, wie geschrieben, verwenden

Gruß

Damian

[raspklaus]

Aber laut

http://forum.fhem.de/index.php/topic,42209.0.html

soll sie so funktionieren. Ich habe sie 1 zu 1 übernommen

[kumue]

Also wenn die sub Advent tut, dann müsste es doch so funktionieren..

Code Auswählen Erweitern

define z_advent DOIF ([06:00-22:00] and [BM_Bewegungsmelder:brightness] <= 100 and Advent==1) (set advent on)
DOELSEIF ([06:00-22:00] and [BM_Bewegungsmelder:brightness] > 100 and Advent==1) (set advent off)
DOELSEIF ([22:00]) (set advent off)

[raspklaus]

Die Sub tuts jetzt aber bei Deiner DOIF kommt die Meldung

Code Auswählen Erweitern

_advent DOIF: no left bracket of condition: DOIF ([06:00-22:00] and [BM_Bewegungsmelder:brightness] <= 100 and Advent==1) (set advent on)DOELSEIF ([06:00-22:00] and [BM_Bewegungsmelder:brightness] > 100 and Advent==1) (set advent off)DOELSEIF ([22:00]) (set advent off)

[kumue]

Fehlendes Leerzeichen vor DOELSEIF

Was hast du an der sub verändert ?

[Damian]

Also wenn die sub Advent tut, dann müsste es doch so funktionieren..

Code Auswählen Erweitern

define z_advent DOIF ([06:00-22:00] and [BM_Bewegungsmelder:brightness] <= 100 and Advent==1) (set advent on)
DOELSEIF ([06:00-22:00] and [BM_Bewegungsmelder:brightness] > 100 and Advent==1) (set advent off)
DOELSEIF ([22:00]) (set advent off)

Im DEF-Editor darf nur noch:

Code Auswählen Erweitern

([06:00-22:00] and [BM_Bewegungsmelder:brightness] <= 100 and Advent==1) (set advent on)
DOELSEIF ([06:00-22:00] and [BM_Bewegungsmelder:brightness] > 100 and Advent==1) (set advent off)
DOELSEIF ([22:00]) (set advent off)
stehen.

DOIF ist bereits der Modulname.

Gruß

Damian

[moonsorrox]

Im DEF-Editor darf nur noch:

Code Auswählen Erweitern

([06:00-22:00] and [BM_Bewegungsmelder:brightness] <= 100 and Advent==1) (set advent on)
DOELSEIF ([06:00-22:00] and [BM_Bewegungsmelder:brightness] > 100 and Advent==1) (set advent off)
DOELSEIF ([22:00]) (set advent off)
stehen.

DOIF ist bereits der Modulname.

Gruß

Damian

siehe hier

[Vize]

http://forum.fhem.de/index.php/topic,42209.0.html
Ich habe sie 1 zu 1 übernommen

Anscheinend nicht, denn auch dort steht, wie kumue schon schrieb, jeweils eine 11 und keine 12 drin...

Gruß
Andreas