JavaScript: Czy można wywołać funkcję cleartimeout przed rozliczeniem?

Question

Mam funkcję, która ustawia licznik czasu i wywołuje się z powrotem po wygaśnięciu licznika czasu.

Chciałbym wiedzieć, czy złą praktyką jest czyszczenie timera na górze funkcji.

Powodem tego jest to, że będę wywoływał tę funkcję asynchronicznie od czasu do czasu, a jeśli nie usunę najpierw licznika czasu, będę miał dwa działające jednocześnie.

Zdaję sobie sprawę, że mogę wyczyścić licznik czasu, zanim wykonam drugie wywołanie funkcji, ale zastanawiam się, czy spowoduje to problemy w dowolnej przeglądarce, jeśli po prostu zatrzymam wywołanie funkcji cleartimeout wewnątrz funkcji zawierającej timer .

Jedna inna myśl - czy mogę przetestować zmienną licznika przed wykonaniem połączenia bezczeszczenia, aby sprawdzić, czy jest to timer?

Oto przykładowy kod:

function onAir(){ 

    // reset timer 
    clearTimeout(timer); 

    $.getJSON("http://mywebsite.com?format=json&callback=?",function(data){ 
     if(data.result == '1'){ 
      do stuff here 
     } 
     else{ 
      do other stuff here 
     } 
    }); 

    // start timer 
    timer = setTimeout("onAir()",60000); 
} 

Dzięki za podzielenie się ze mną mózg!

Kenny

Answer 1

Tak, w porządku. Ponadto, należy zadzwonić "setTimeout()" tak:

timer = setTimeout(onAir, 60000); 

Answer 2

Tak można nazwać clearTimeout o zmiennej wartości null, a świat nie będzie implozji.

Answer 3

Tak można nazwać clearTimeout o zmiennej null.

Także proponuję zmienić swoje setTimeout więc nie użyje eval:

timer = setTimeout(onAir,60000); 

Answer 4

Usuwanie Timeout nie budzi wątpliwości do mnie (ale nie jestem javascript guru).

Btw, można znaleźć rzeczy ciekawych (sprawdzanie istniejącego timeout) na tym wątku: Check if a timeout has been cleared?

Answer 5

Tak można nazwać clearTimeout(timer), ale istnieją pewne przypadki brzegowe gdzie może to spowodować problemy.

Jeśli wcześniej ustawiono timer z inną wartością całkowitą, możliwe, że zabijamy całkowicie niepowiązany zegar.

setTimeout po prostu zwraca wartość całkowitą dla licznika czasu. Jeśli nie jesteś pewien, czy zegar został ustawiony poprzednio, można dodać sprawdzenie przed wywołaniem clearTimeout:

if (window.timer) 
{ 
    clearTimeout(timer); 
} 
... 
timer = setTimeout(onAir, duration); 

Rozwiązaniem ewentualnego zanieczyszczenia zmiennej timera jest użycie Zamknięcie:

(function(){ 
    var timer, 
    duration; 
    duration = 60000; 
    window.onAir = function onAir(){ 
    ...code... 
    if (timer){ 
     clearTimeout(timer); 
    } 
    timer = setTimeout(onAir,duration); 
    }; 
});