Таймер не заканчивается, когда он достигает нуля

Question

Я работаю над этим таймером javascript и не могу понять, почему он не останавливается, когда часы, минуты и секунды равны нулю.

Код:

var s= 18000; 
var h= Math.floor(s/3600); 
s-= h*3600; 
var m= Math.floor(s/60); 
s -= m*60; 
var counter=setInterval(timer, 1000); //1000 will run it every 1 second 

function timer() 
{ 
    if(s == 0){ 
    if(m == 0){ 
     h=h-1; 
     s=59; 
     m=59; 
     if(h == 0){ 
     clearInterval(counter); 
     } 
    } else { 
     m=m-1; 
     s=59; 
    } 
    //Do code for showing the number of seconds here 
    } else { 
    s = s - 1; 
    } 
    document.getElementById("timer").innerHTML=h+'hrs '+m+'min '+s+'secs '; 
} 

Answer 1

Проблема, как представляется, логика в этом разделе здесь ...

if(s == 0){ 
    if(m == 0){ 
     h=h-1; // what if h is also 0 here?? this would set it negative 
     s=59; 
     m=59; 
     if(h == 0){ 
     clearInterval(counter); 
     } 

Я думаю, что вы должны делать свой if(h==0) чек раньше ... перед тем, как декремент ценности. Таким образом, вы можете запустить свой блок if else с помощью if (s == 0 && m == 00 && h == 0) и использовать его для очистки интервала. Если все уже равно нулю, вы не хотите изменять больше значений.

Answer 2

Проблема, которую я вижу в том, что вы будете уменьшать h переменные на 1, когда все переменное время - и временных переменных Я имею в виду h, m и s устанавливаются в ноль:

if(s == 0) { 
    if(m == 0) { 
     h=h-1; 
     s=59; 
     m=59; 
// more code goes here 

Так h будет -1, и таймер никогда не остановится.
Лучшее, что я могу предложить, - полностью переписать свой таймер и использовать здесь только секунды. Каждый раз, когда вызывается функция timer, вы проверяете, равен ли s нулю - если это так, вы останавливаете таймер. В противном случае вы уменьшите s на 1. Чтобы обновить внутренний HTML-код #timer, вы можете пересчитать количество часов, минут и секунд при каждом вызове метода timer - это решение будет намного проще понимать и поддерживать, чем цепочка вложенных условных операторов.