UPDATE 31.10.: Amazon Alexa - ioBroker Skill läuft aus ?

[Caspar]

Moin,

erstmal vielen Dank für die vielen und guten Lösungsansätze die man hier, dank euch allen, findet

Nun zu meinem Problem, welches ich bisweilen nicht lösen konnte trotz intensiver suche….

Folgendes Script wartet zwar, wie gewollt, 20 Sekunden ab bevor es die Pushover-Nachricht versendet, aber das if Statement wird ignoriert.

.......
     if (status === true && alarmaktiv === true && haustuercheck === "TFK Caspar Haustuer:1") {

          setTimeout(function(){

          if (alarmaktiv === true) {
         textALARM.push(obj.common.name.substr(0, obj.common.name.indexOf(':')));
         sendTo('pushover.0', 'send', {
             title: 'EINBRUCH CASPAR',
            message: 'Einbruchsalarm Caspar' + '\n' + 'Ausloesender Melder: ' + obj.common.name.substr(0, obj.common.name.indexOf('.')),
            sound: 'updown',
            priority: 2,
            retry: 60,
            expire: 3600
         }); 
        }

         },20000) ;

      }
        else 
.......

Es handelt sich nur um einen Auszug des Scriptes da alles andere funktioniert.

Wenn das gesamte Script notwendig ist, liefere ich das gerne nach.

Ziel sollte es sein das nach Ablauf der 20 Sekunden die Variable alarmaktiv geprüft wird und nur wenn diese (immer noch) true ist, dann soll die Pushover-Nachricht versendet werden, ansonsten soll keine Nachricht versendet werden und der Rest des Scripts ausgeführt werden.

Ich wäre äußerst dankbar wenn mir jemand meinen Fehler erläutern könnte damit ich diesen verstehe und beheben kann.

JavaScript sitzt leider noch nicht so gut das ich das Problem alleine erkenne.

Grüße Caspar

[BuZZy]

Pack den kompletten Code aus dem setTimeout in eine Funktion z. B. function alarmMessage.. und führe dann nur die Funktion alarmMessage im setTimeout aus.

Gruß

[paul53]

@Caspar:

das if Statement wird ignoriert. `
Sind die Skriptvariablen status und a__larmaktiv__ global (außerhalb von Funktionen) deklariert ?

[Caspar]

Moin zusammen,

@BuZZy:

Pack den kompletten Code aus dem setTimeout in eine Funktion z. B. function alarmMessage.. und führe dann nur die Funktion alarmMessage im setTimeout aus. `

das hat geholfen, vielen Dank dafür!

@paul53:

Sind die Skriptvariablen status und a__larmaktiv__ global (außerhalb von Funktionen) deklariert ? `

Nein, deswegen habe ich sie der "neuen" Funktion mit übergeben.

Das fertige und funktionierende Ergebnis :

....
     if (status === true && alarmaktiv === true && haustuercheck === "TFK Caspar Haustuer:1") {
          setTimeout(function(){PushoverWait(textALARM,obj)},20000);
      }
        else 
        .......

Die Funktion PushoverWait :

function PushoverWait(textALARM,obj) 
{
         var alarmaktiv2 = getState("system.adapter.javascript.0.AlarmanlageCaspar").val;
     if (alarmaktiv2 === true) {
         textALARM.push(obj.common.name.substr(0, obj.common.name.indexOf(':')));
         sendTo('pushover.0', 'send', {
             title: 'EINBRUCH CASPAR',
            message: 'Einbruchsalarm Caspar' + '\n' + 'Ausloesender Melder: ' + obj.common.name.substr(0, obj.common.name.indexOf('.')),
            sound: 'updown',
            priority: 2,
            retry: 60,
            expire: 3600
         }); 
        }
}

Anfänglich hatte das nicht funktioniert weil die Variable alarmaktiv true war als ich diese an die Funktion PushoverWait übergeben habe.

Deswegen habe ich eine neue Variable alarmaktiv2 geschaffen die innerhalb der neuen Funktion den Status prüft.

Ich danke euch beiden für die Lösung