Funkcja nie wykonuje się

pytanie zadane 26 maja 2017 w JavaScript przez kami123m Nowicjusz (150 p.)

Funkcja sprawdz() niewykonuje się. Niewiem gdzie znajduje się błąd prosze o pomoc

<!DOCTYPE HTML>
<html lang="pl">
<head>
  <meta charset="utf-8"/>
  <title></title>
  
  <script type="text/javascript" >
  
	    var numer = ( Math.round(Math.random()*3));
		var liczba=0;
		
     function zmienkamieng()
	   { 	
		 var plik = "<img src=\"a/obraz1.png\"/>";	 
		 document.getElementById("pole1").innerHTML = plik;	
         var liczba=1;		 
	   }
	   function zmienpapierg()
	   { 	
		 var plik = "<img src=\"a/obraz2.png\"/>";	 
		 document.getElementById("pole1").innerHTML = plik;	
         var liczba=2;		 
	   }
	   function zmiennozyceg()
	   { 	
		 var plik = "<img src=\"a/obraz3.png\"/>";	 
		 document.getElementById("pole1").innerHTML = plik;
         var liczba=3 ;		 
	   } 
	   function zmienobrazk()
	   { 	
	        if(numer==0)
			{
			  numer=2;
			}
			
			var plikk = "<img src=\"a/obraz"+numer+".png\"/>";	 
		 document.getElementById("pole2").innerHTML = plikk;	
	   } 
	   function sprawdz()
	   {
	     if(liczba==1)
		 {
		   if(numer==3)
		   {
		     alert('Wygrałeś! Aby zagrać jeszcze raz otwórz strone ponownie');
		   }
		   else
		   {
		    alert('Przegrałeś! Aby zagrać jeszcze raz otwórz strone ponownie');
		   }
		 }
		 else if(liczba==2)
		 {
		   if(numer==1)
		   {
		     alert('Wygrałeś! Aby zagrać jeszcze raz otwórz strone ponownie');
		   }
		   else
		   {
		    alert('Przegrałeś! Aby zagrać jeszcze raz otwórz strone ponownie');
		   }
		 }
		 else if(liczba==3)
		 {
		   if(numer==2)
		   {
		     alert('Wygrałeś! Aby zagrać jeszcze raz otwórz strone ponownie');
		   }
		   else
		   {
		    alert('Przegrałeś! Aby zagrać jeszcze raz otwórz strone ponownie');
		   }
		 }
	   }
	   
	   
  </script>
 
</head>
<body>
   <div id="pole1"></div>
   <input type="submit" value="Kamień" onclick="zmienkamieng()" style="position: absolute; top: 110px">
   <input type="submit" value="Papier" onclick="zmienpapierg()" style="position: absolute; top: 110px; left: 90px">
   <input type="submit" value="Nożyce" onclick="zmiennozyceg()" style="position: absolute; top: 110px; left: 180px">
   <div id="pole2" style="position: absolute; right: 1100px; top: 5px"></div>
   <input type="submit" value="Sprawdź" onclick="zmienobrazk(), sprawdz()" style="position: absolute; top: 145px; left: 85px">
   
   
</body>
</html>

2 odpowiedzi

odpowiedź 26 maja 2017 przez Assasz Nałogowiec (30,460 p.)
wybrane 26 maja 2017 przez kami123m

Najlepsza

Jeśli do onclicka dodajesz dwie funkcje, to musisz je kończyć znakiem ;. Proponuję jednak korzystać z eventListenerów lub przerzucić się na jQuery - życie staje się prostsze.

komentarz 26 maja 2017 przez kap Stary wyjadacz (11,620 p.)

Zaciąganie całej biblioteki by zaoszczędzić kilka znaków w tym przypadku to słaby pomysł.

komentarz 26 maja 2017 przez Assasz Nałogowiec (30,460 p.)

Załączenie jednego linka to dobry pomysł, by ułatwić sobie pisanie całych skryptów. A znajomość jQuery na pewno przyda się w przyszłości, jeśli nie teraz.