Syntaxproblem bei mehreren if-Abfragen

[Michi240281]

Hallo zusammen,

ich möchte gerne meinen Abfallkalendar per Sprachausgabe umsetzen. Habe das bislang mit Mailbenachrichtigung gemacht bzw. mit Nachrichten über das ENIGMA2-Modul.

Ich möchte nun jedoch, dass bei Müll x die entsprechende MP3 abgespielt wird. Irgendwo habe ich wohl noch einen Fehler drin, den ich aber nicht sehen kann.

Mit einer if-Abfrage funktioniert es:

Code Auswählen Erweitern

define events holiday
attr events group Events
attr events room Haus
define CheckEventHeute at *17:22:00 {\
my $Eventname;;\
my $EventHeute;;\
$EventHeute = fhem("get events today");;\
print $EventHeute;;\
if ($EventHeute == "Gelber Sack") {fhem("set Funk_Gong_Ton playTone 001");;\
}\
}

doch mit folgenden if-Abfragen will es nicht hinhauen.

Code Auswählen Erweitern

define events holiday
attr events group Events
attr events room Haus
define CheckEventHeute at *17:22:00 {\
my $Eventname;;\
my $EventHeute;;\
$EventHeute = fhem("get events today");;\
print $EventHeute;;\
if ($EventHeute == "Gelber Sack") {fhem("set Funk_Gong_Ton playTone 001");;\
if ($EventHeute == "Restmüll") {fhem("set Funk_Gong_Ton playTone 002");;\
if ($EventHeute == "Altpapier") {fhem("set Funk_Gong_Ton playTone 003");;\
f ($EventHeute == "Biomüll") {fhem("set Funk_Gong_Ton playTone 004");;\
}\
}

Könnte hier vllt mal jmd schauen, wo ich den Fehler habe?

Besten Dank und schönen Restsonntag allen!

[Puschel74]

Hallo,

ein einfaches copy&paste nach Notepad++ zeigt mir schon mindestens 3 fehlende }

Das sollte aber eigentlich auch eine Fehlermeldung im Logfile erzeugen.

Grüße

[Michi240281]

Ok danke!

Jetzt habe ich es so:

Code Auswählen Erweitern

define events holiday
attr events group Events
attr events room Haus
define CheckEventHeute at *17:52:00 {\
my $Eventname;;\
my $EventHeute;;\
$EventHeute = fhem("get events today");;\
print $EventHeute;;\
if ($EventHeute == "Gelber Sack") {fhem("set Funk_Gong_Ton playTone 001")};;\
if ($EventHeute == "Restmuell") {fhem("set Funk_Gong_Ton playTone 002")};;\
if ($EventHeute == "Altpapier") {fhem("set Funk_Gong_Ton playTone 003")};;\
if ($EventHeute == "Biomuell") {fhem("set Funk_Gong_Ton playTone 004")}}

Jetzt geht es, allerdings werden alle Töne abgespielt, also alle 4 "set" werden ausgeführt, obwohl die Prüfung nur bei "Gelber Sack" zutreffen müsste.

??

[Puschel74]

Hallo,

== wird mMn bei Zahlen verwendet
eq sollte das sein was du suchst.

Grüße

[Michi240281]

So:

Code Auswählen Erweitern

define events holiday
attr events group Events
attr events room Haus
define CheckEventHeute at *18:59:00 {\
my $Eventname;;\
my $EventHeute;;\
$EventHeute = fhem("get events today");;\
print $EventHeute;;\
if ($EventHeute eq "Gelber Sack") {fhem("set Funk_Gong_Ton playTone 001")};;\
if ($EventHeute eq "Restmuell") {fhem("set Funk_Gong_Ton playTone 002")};;\
if ($EventHeute eq "Altpapier") {fhem("set Funk_Gong_Ton playTone 003")};;\
if ($EventHeute eq "Biomuell") {fhem("set Funk_Gong_Ton playTone 004")}}

tut sich garnix!

[Puschel74]

Hallo,

dann lass dir mal einen Logfileeintrag erzeugen mit dem Inhalt von $EventHeute und schau mal was tatsächlich in der Variable steht.

Grüße

[UliM]

Hi,
bau doch mal ein

Code Auswählen Erweitern

Debug "$EventHeute";
ein und schau, was drin steht. Offenbar passt Dein eq nicht zu dem entahltenen Text.
Mglw musst Du umstellen von "equals" auf "contains":

Code Auswählen Erweitern

if ($EventHeute =~ m/Gelber Sack/)
=8-)

[Michi240281]

Code Auswählen Erweitern

2014.02.23 19:56:00 1: DEBUG>Gelber Sack   

[Rince]

Blöde Frage:

Code Auswählen Erweitern

set Funk_Gong_Ton playTone 001



direkt in der fhem Befehlszeile funktioniert?

[Michi240281]

Blöde Frage:

Code Auswählen Erweitern

set Funk_Gong_Ton playTone 001



direkt in der fhem Befehlszeile funktioniert?

KLar, das geht natürlich!

[Rince]

Sorry, war nur ne Frage.
Dann wäre jetzt imho der Code schlau, der deine Mülltüten feststellt

[Michi240281]

Moin,

mit folgendem Code hat es gestern, als $Eventheute "Gelber Sack" war, funktioniert:

Code Auswählen Erweitern

define CheckEventHeute at *08:18:00 {\
my $Eventname;;\
my $EventHeute;;\
$EventHeute = fhem("get events today");;\
print $EventHeute;;\
if ($EventHeute =~ m/Gelber Sack/) {fhem("set Funk_Gong_Ton playTone 001")};;\
if ($EventHeute =~ m/Restmuell/) {fhem("set Funk_Gong_Ton playTone 002")};;\
if ($EventHeute =~ m/Altpapier/) {fhem("set Funk_Gong_Ton playTone 003")};;\
if ($EventHeute =~ m/Biomuell/) {fhem("set Funk_Gong_Ton playTone 004")}}

Heute zeigt $Eventheute "Biomuell", doch nun bleibt der Gong stumm.

Also irgendwas scheint an der Syntax noch nicht zu stimmen.

[Michi240281]

Was macht eigentlich immer das "\" am Ende der Zeilen? Ist das dafür da, um einen Zeilenumbruch einzuleiten?

Habe den Code jetzt nämlich folgendermaßen umgestellt, und jetzt klappt alles!

Code Auswählen Erweitern

define CheckEventHeute at *08:52:00 {\
my $Eventname;;\
my $EventHeute;;\
$EventHeute = fhem("get events today");;\
print $EventHeute;;\
if ($EventHeute =~ m/Gelber Sack/) {fhem("set Funk_Gong_Ton playTone 001")};; if ($EventHeute =~ m/Restmuell/) {fhem("set Funk_Gong_Ton playTone 002")};; if ($EventHeute =~ m/Altpapier/) {fhem("set Funk_Gong_Ton playTone 003")};; if ($EventHeute =~ m/Biomuell/) {fhem("set Funk_Gong_Ton playTone 004")}}

EDIT: Nachdem nun die Ansteuerung des Gongs funktioniert wollte ich mal versuchen, 2 Befehle ausführen zu lassen, und zwar einmal den Gong und dann noch eine Anzeige auf der Dreambox. Folgenden Code habe ich dazu erstellt:

Code Auswählen Erweitern

define CheckEventHeute at *08:52:00 {\
my $Eventname;;\
my $EventHeute;;\
$EventHeute = fhem("get events today");;\
print $EventHeute;;\
if ($EventHeute =~ m/Gelber Sack/) {fhem("set Funk_Gong_Ton playTone 001")};; if ($EventHeute =~ m/Restmuell/) {fhem("set Funk_Gong_Ton playTone 002")};; if ($EventHeute =~ m/Altpapier/) {fhem("set Funk_Gong_Ton playTone 003")};; if ($EventHeute =~ m/Biomuell/) {fhem("set Funk_Gong_Ton playTone 004 [b];; set DM500HD msg message 60 $EventHeute[/b]")}}

Seltsamerweise wird nun NUR noch die Nachricht auf der DM500HD angezeigt und der Gong bleibt stumm. Jmd ne Idee? Gibts da irgendeine Art Priorisierung? Normalerweise müsste doch beides ausgeführt werden oder nicht? Ist da vllt die FB überfordert?

[Rince]

Das \ am Zeilenende ist genau dann nötig, wenn du den Code direkt in der fhem.cfg bastelst. So wie auch die ;;

Um das etwas übersichtlicher zu machen, würde ich dringend empfehlen das at NICHT über die cfg zu bearbeiten,  sondern über das DEF.
Da sieht man dann auch vernünftig, welche if jetzt welche Befehle ausführen. Und man muss nicht dauernd umdenken mit den Klammern etc.

[Damian]

Wenn ich sehe, wie du dich hier quälst - probier mal:

Code Auswählen Erweitern

define CheckEventHeute at *08:18:00
IF (fhem("get events today") =~ m/Gelber Sack/) (set Funk_Gong_Ton playTone 001);
IF (fhem("get events today") =~ m/Restmmuell/) (set Funk_Gong_Ton playTone 002);
IF (fhem("get events today") =~ m/Altpapier/) (set Funk_Gong_Ton playTone 003);
IF (fhem("get events today") =~ m/Biomuell/) (set Funk_Gong_Ton playTone 004)

Wenn man weniger tippen muss, kann man auch weniger falsch machen. 

siehe: http://forum.fhem.de/index.php/topic,17895.msg118503.html#msg118503

Gruß

Damian