Nie wiem co robie źle w tym kodzie

pytanie zadane 27 lutego 2020 w JavaScript przez cez-kot Nowicjusz (120 p.)

Niedawno zacząłem uczyć się JS ale mi nie wychodzi
Chciałem zrobić minutnik który odlicza 3 min
Ale gdy przychodzi co do czego to nie pokazuje się na stronie
Pomocy.

function IleZostalo() {
  if(minuta<=0&&sekunda<=0)
	{ 
		document.getElementById('sekundnik').innerHTML='Koniec czasu ';
    }

	sekunda=sekunda-1;
	if (sekunda<0) {sekunda=59; minuta=minuta-1;}
	if (minuta<0) {minuta=59;}

	if(minuta>=0&&sekunda>=0)
	{
		document.getElementById('sekundnik').innerHTML='Czas jaki pozostał do końca; '+minuta+' min '+sekunda+' sek' ;
		
		setTimeout("odliczanie()",1000);
	} 

}

<!DOCTYPE html>
<html lang="en">

<head>
    <meta charset="UTF-8">
    <title>Document</title>

    <script type="text/javascript" src="IleZostalo.js"> </script>

</head>

<body onload="IleZostalo();">
    
    <div id="sekundnik"></div>

</body>

</html>

Co tu jest źle?

1 odpowiedź

odpowiedź 27 lutego 2020 przez Mariusz08 Maniak (62,300 p.)

https://jsfiddle.net/r108ykje/

minuta is undefined

komentarz 27 lutego 2020 przez cez-kot Nowicjusz (120 p.)

Zaktualizowałem kod i dalej nie działą
Zdefiniowałem minute i sekunde

function IleZostalo() {
  var minuta = 2;
  var sekunda = 60;
  if (minuta <= 0 && sekunda <= 0) {
    document.getElementById('sekundnik').innerHTML = 'Koniec czasu ';
  }

  sekunda = sekunda - 1;

  if (sekunda < 0) {
    sekunda = 60;
    minuta = minuta - 1;
  }
  if (minuta < 0) {
    minuta = 2;
  }

  if (minuta >= 0 && sekunda >= 0) {
    document.getElementById('sekundnik').innerHTML = 'Czas jaki pozostał do końca; ' + minuta + ' min ' + sekunda + ' sek';

    setTimeout("IleZostalo()", 1000);
  }

}

komentarz 27 lutego 2020 przez ScriptyChris Mędrzec (190,190 p.)

Przenieś deklaracje zmiennych minuta i sekunda (linie 2 i 3) nad funkcję (czyli na początek skryptu), bo teraz przy każdym odpaleniu funkcji te zmienne są tworzone od nowa i w ostatnim ifie (pozostałe nie są spełniane) na okrągło pokazywany jest ten sam tekst w <div>.