Problem z "timerem"

Question

Witam serdecznie, zgłaszam się tutaj ponieważ mam problem którego nie rozumiem. Mam taki oto kod 

function Start(time){
	var LocalTime = time; // Zmieniamy zmienną która nam przyszła na zmienną lokalną.
	
	document.getElementById("menu").innerHTML = LocalTime;
	if(LocalTime > 0)
	{
		LocalTime--;
		setTimeout(Start(LocalTime),10000);
	}
	
}

function SetAll()
{
	var TimeOut = 10; 
	//var Slide = true; 
	
	Start(TimeOut);
	
}

no i za każdym razem wyświetla mi 0, nie wiem czemu. Gdy skasuje ifa, który ustawia timeouta wyświetla mi pierwszą przypisaną wartość 

Answer 1

setTimeout(Start(LocalTime),10000);

W tym miejscu do setTimeout przekazujesz wynik wywołania funkcji Start. Dlatego setTimeout  nie działa tak, jak tego oczekujesz. Powinieneś albo przekazać callbacka, w którym dopiero wywołasz swoją funkcję (lepszy sposób), albo przesłać tam referencję ze zbindowanym parametrem LocalTime (gorszy sposób). Czyli

setTimeout(function () {
    Start(LocalTime);
},10000);

, albo:

setTimeout(Start.bind(null, LocalTime),10000);

 

Comments

A jakbym chciał to zrobić w jednej funkcji, to jak zrobić  to by przypisanie zmiennej TimeOut zostało tylko raz? Zamiast bawić się w odsyłanie z 1szej funkcji do drugiej i z drugiej do pierwszej może to jakiś sposób by był, by po prostu wszystko wykonać tam.

---

Umieść tą zmienną w wyższym scope:

var time = 10;

function Start() { 
    document.getElementById("menu").innerHTML = time;

    if(time > 0) {
        time--;
        setTimeout(Start,2000);
    }
     
}
 
function SetAll() {      
    Start();     
}