Doppelte Benachrichtigung bei Telegram

[Jan1]

@Crazy32
Da was er schreibt ist schon alles richtig und man baut das zur Sicherheit auch meist ein. Das schadet auf alle Fälle nie und das baust Du später auch mit ein wenn es ohne laufen würde, weil das einfach sicherer ist. Was aber auch richtig ist, bei Dir scheint noch was anderes nicht ganz zu passen und das sollte auch mal geprüft werden.
Setzt das am WE einfach mal um und schau was passiert, wenn noch Fragen offen sind geht es hier weiter.

[CruziX]

@Crazy32 sagte in Doppelte Benachrichtigung bei Telegram:

@CruziX

Damit habe ich aber nicht die Ursache beseitigt. Ich will ja wissen woran es liegt, damit ich in Zukunft die Scripte sauber schreiben kann.

Daher auch Workaround ^^

[Michi68]

Hallo ,
das erste was mir auffällt ist das du true und fals nimmst müsste aber 1 und 0 sein wenn ich das richtig in Erinnerung habe, geht auch aber wird vom Adapter später wieder zurück auf 0 oder 1 gesetzt. Würde für mich das doch recht späte zweite nachsenden erklären. Ist aber nur geraten.

Gruß Michael

[Jan1]

@Michi68
Ganz unrecht hast nicht, da ich aber auch eine DECT Dose habe, ein Script das damit läuft und ich ebenfalls zufällig in dem Script ein Telegram verschicke, hab ich mal nachgesehen was ich da drin habe und bei mir läuft das mit true und false, obwohl der Adapter in dem DP 1 oder 0 stehen hat. Somit ist das wohl egal.

[ticaki]

@Jan1
Das mit true und 1 liefert eine Grund warum der State 2 mal geändert wird.

Gibt es in Blockly nicht die Möglichkeit diesen Code umzusetzen?

if (!obj.state.val != !obj.oldState.val) { telegram verschicken}

die beiden "nicht" um den Wert in true/false umzuwandeln.

[Jan1]

@ticaki
Man kann auch einfach 1 und 0 schreiben, wobei das doppelte Senden zeitlich schon zu weit auseinander liegt um das als Grund fest zu machen und ein ähnliches Script läuft bei mir mit true und false.

Muss er mal testen, ob es mit 0 und 1 klappt, schaden tust auf alle Fälle auch nicht.

[Michi68]

wegen der langen zeit zwischen den beiden Nachrichten Tippe ich ja darauf, hab auch 2 AVM Steckdosen am laufen und weiß daher das die auch mit true oder false geschaltet werden können, hatte aber auch mal Stress damit weshalb ich das dann geändert habe, weiß aber nicht mehr wieso ich Stress hatte. Hatte auch schon mal das Problem das Telegram 2 Nachrichten hintereinander gesendet hat aber die kamen innerhalb von 1-2 Sekunden da hilf dann ein einfaches Timeout drum zusetzen von 3-4 Sekunden. Aber 1-3 Minuten ist schon sehr viel Zeit.

[Jan1]

@Michi68
Eben, im Sekunden Bereich würde ich da auch wegen true und false, oder eben prellen den Fehler suchen. Hier liegen Minuten dazwischen.

[ticaki]

@Jan1
Da hast du recht, aber... gehen wir das logisch an: Er setzt den Wert auf true und bekommt eine "true" Nachricht. Ein X Zeitspanne später bekommt er eine weitere true nachricht. Was nur geht wenn der Wert zwar geändert wurde aber noch immer true ist.

True = ist alles was nicht false ist
False = 0/false/''

Folglich sollte er den alten Übergabe wert in True oder False ändern und das mit dem neuen Wert, der ebenfalls in true oder false geändert wurde, überprüfen und nur auslösen, wenn die beiden sich unterscheiden.

Die einfachste Version einen Wert in Javascript in einen Logikwert zu ändern ist in "nicht" bzw. "!" zu benutzen.

nicht true == nicht "ich bin true" -> ist true
nicht true == nicht 1 -> ist true
nicht true == nicht true -> ist true

Es sollte zwar auch ohne das "nicht" gehen. Aber dann wäre 2 == 1 false wohingegen !(2) == !(1) true ergibt.

In einem Beispiel von dir fragst du im geposteten Code "state.val" ab... finde das leider in Blockly nicht. Der alte Wert wäre "oldState.val".

[Jan1]

@ticaki
Schauen wir mal was bei dem Umbau rauskommt. In Blockly gibts das und ist vorheriger Wert.

[Michi68]

so ich noch mal habe das jetzt mal getestet

mit true oder false wie man hier sieht schreib er auch true rein und die Steckdose geht an und die Nachricht wurde mit Telegram versendet.

Nach ca. 2-3 Minuten ändert fritzdect.0 den Wert true in 1 um aber es wurde keine zweite Nachricht verschickt so wie ich es vermutet hatte.

Also liegt da schon mal nicht der Fehler

[Jan1]

@Michi68
Eigentlich war es klar, da darauf auch nicht getriggert wird. Das =wahr ist unnötig da doppelt und im Bild hast nicht das umgesetzt was ich gepostet habe und als möglichen Fehler auf dem Schirm hatte. Nimm einfach Wert aus dem Trigger Bereich, weil das eben besser für die Laufzeit ist und sich auf den Wert des Trigger bezieht, hatte ich ogben schon erklärt.

[ticaki]

@Michi68
Ist alles möglich. Der Topicersteller sollte eine Logausgabe ins Skript einbauen und prüfen ob es 2 mal aufgerufen wird und am besten "state.from" ins log schreiben.

[Jan1]

@ticaki
Hab bei mir jetzt noch mal nach dem on/off DP geschaut und da stimmt vom Adapter schon was nicht, zumindest ist es verwirrend. Der DP ist als Logikwert deklariert und bietet beim manuellen setzten eben nur true und false an, was auch funktioniert. Das wird aber nach einer gewissen Zeit in 0 und 1 gewandelt, was zwar immer noch boolen ist, aber als Deklaration für den DP eher eine Zahl.
Das sollte aber auch nicht das Problem des doppelten Senden sein, sondern eher eine unschöne Eigenschaft des Adapter die aber trotzdem funktioniert.

[Crazy32]

Wow, so viele Antworten. Gestern Abend hatte ich das Phänomen, dass die Lampe ausgeschaltet wurde und die Meldung "Lampe aus" kam. Kurze Zeit später wieder die zweite Meldung, allerdings mit der Meldung "Lampe an", obwohl die Lampe aus war. Seltsam. Ich habe jetzt mal die Werte von "wahr" und "falsch" auf "1" und "0" geändert.
Das Script für die Lampenschaltung habe ich so gelassen, weil ich mit der Cron-Funktion leicht Probleme hatte, da hat sich die Lampe minütlich ein- und ausgeschaltet. Mit der Cron-Funktion komme ich noch nicht so ganz klar. Ich poste jetzt nochmal meine aktuelle Konfiguration und schaue was passiert.
@ticaki Das mit der Logausgabe ist eine gute Idee, da weiß ich aber auch noch nicht wie das geht...

[Jan1]

@Crazy32
Beim CRON hast wohl die Minuten drin und nicht nur die Wochentage, somit heißt Dein CRON jede Minute von Montag bis Freitag. Schau doch einfach was da steh und genau das macht er. 0 und 1 hatte ich ja getestet und da ist der Adapter etwas seltsam aufgebaut.
Wenn die Meldung auf Telegram kommt, dass die Lampe aus ist und diese aber an bleibt, dann stimmt bei Dir was ganz anderes nicht, denn das sollte nur kommen wenn die Lampe aus ist.

Die Log Meldungen bekommst mit dem Debugblock, wobei das auch nicht wirklich nötig ist, da ja ein Telegram im Fall des Trigger geschickt wird. Die Debug Blöcke kann man aber überall verteilen und sie mit nem eindeutigen Namen versehen, dann taucht genau der Name auch im Log auf und man weiß wann was passiert. Dein Blockly ist eigentlich wirklich ne sehr einfache Geschichte und deshalb wundert es mich, dass das so rum zickt.

[dslraser]

@Crazy32
bei den Wochentagen = 0/1/2/3/4/5
könntest Du das auch auf eine Abfrage reduzieren.
Wochentag < 6
(falls Wochentag als Nummer kleiner als 6)

[Crazy32]

@dslraser sagte in Doppelte Benachrichtigung bei Telegram:

@Crazy32
bei den Wochentagen = 0/1/2/3/4/5
könntest Du das auch auf eine Abfrage reduzieren.
Wochentag < 6
(falls Wochentag als Nummer kleiner als 6)

Ich habe es auf <=5 gesetzt, danke. Ist ja doch etwas übersichtlicher.

@Jan1 sagte in Doppelte Benachrichtigung bei Telegram:

@Crazy32
Kannst auch mal in der Fritz bei den Ereignissen schauen, ob der Trigger mehr fach auslöst, sprich die Steckdose den Zustand ungewollt wechselt, dann liegt es nämlich daran und nicht am Script, denn das macht nur was vorgegeben wird und ist selbst in Deiner Version nicht falsch.

Heute Morgen hatte ich wieder meine zwei Meldungen in Telegram, dass die Lampe ausgeschaltet wurde, die erste um 07.12 Uhr und die zweite um 07.16 Uhr. In den Ereignissen der Fritzbox ist um 07.12 Uhr die Meldung "Anmeldung an der FRITZ!Box Benutzeroberfläche von IP-Adresse 192.168.178.11". Die IP ist von meinem Raspi.

Gerade gesehen, im ioBroker Log ist eine Warnung von 07.12 Uhr zu sehen. Das würde mit der ersten Schaltzeit passen. Ich kann das aber nicht deuten.

[ticaki]

@Crazy32
lol...

Das bedeutet, das der Javascript-Adapter was dagegen hat das du die Werte im Skript mit 0 und 1 steuert. Hat aber keine Auswirkungen außer das es im Log mies aussieht.

Kann vielleicht jemand mit dieser Hardware ein Github Issue im entsprechenden Adapter anlegen, dass da true und false und nicht 0 und 1 gesetzt wird?

[Jan1]

@ticaki
Das hatte ich ja auch schon vermutete, dass der Adapter an der Stelle nicht sauber programmiert ist, weil true und false funktioniert und der Adapter schaltet das auf 1 und 0 um.