function show_time()
{
setTimeout("show_time()",1000);
setTimeout("sale()",1000);
}
Tak się nie przekazuje funkcji. Powinno być:
function show_time()
{
setTimeout(show_time,1000);
setTimeout(sale,1000);
}
Moim zdanie lepiej użyć setInterval zamiast rekurencji, ponieważ możesz wtedy łatwo zatrzymać odliczanie używając clearInterval , a tak twoja pętla cały czas działa
Twój licznik idzie dalej ponieważ drugi warunek (godzina < 0) nie jest spełniony