Wartung am 15.11. – Forum ab 22:00 Uhr nicht erreichbar

[dtp]

Hallo,

ich bin gerade etwas am verzweifeln, da ich nicht so recht weiter komme. Mit folgendem Skript starte ich einen Countdown in Abhängigkeit eines CUxD-Timers, der nach dem Runterzählen automatisch endet.

// Countdown für Alarmaktivierung nach Abwesenheit

var count, working;

createState('AlarmCountdown', 0, {
  read: true,
  write: true,
  desc: 'Countdown für Alarmaktivierung nach Abwesenheit',
  def: ''
});

function countdown(count, working){
  var counter = setInterval(timer, 1000);

  function timer(){
    log(working);
    if((count === 0) || (working === false)){
      clearInterval(counter);
      return;
    }
    setState('AlarmCountdown', count);
    count=count-1;
  }
}

on({id: "hm-rpc.1.CUX2800001.3.WORKING", change: "ne"}, function(obj){
  countdown(getState('hm-rpc.1.CUX2800001.3.TIMER_SET').val, obj.state.val);
});

Das funktioniert auch soweit problemlos. Nun möchte ich den Countdown aber auch abbrechen können, wenn der Zustand WORKING des Timers auf false gesetzt wird. Doch leider reagiert der Timer darauf nicht. Die Variable "working" geht zwar einmalig auf "false", danach nimmt sie aber wieder den Wert "true" an, obwohl "hm-rpc.1.CUX2800001.3.WORKING" tatsächlich auf false steht, und der Timer zählt weiter fleißig bis 0 zurück. Wo liegt mein Denkfehler?

Gruß,

Thorsten

[paul53]

Du hast "working" global deklariert und außerdem als Übergabeparameter verwendet. Ändere mal:

function countdown(count, work){
  working = work;
  ...

[paul53]

Ich würde es so machen:

// Countdown für Alarmaktivierung nach Abwesenheit

var count,timer;

createState('AlarmCountdown', 0, {
  read: true,
  write: true,
  desc: 'Countdown für Alarmaktivierung nach Abwesenheit',
  def: ''
});

function countdown(){
  if(count > 0) {
    count--;
    setState('AlarmCountdown', count);
  } else clearInterval(timer);
}

on({id: "hm-rpc.1.CUX2800001.3.WORKING", change: "ne"}, function(obj){
  if(obj.state.val) { 
    count = getState('hm-rpc.1.CUX2800001.3.TIMER_SET').val;
    timer = setInterval(countdown, 1000);
  } else clearInterval(timer);
});

[dtp]

Klasse. Funktioniert genau so, wie gewünscht. Vielen, vielen Dank.

Gruß,

Thorsten