Przykład z wypisywaniem liczb z kursu z yt

Question

Witam, na wstępie pragnę zaznaczyć że jestem świeży i na forum i w technologii JS. Do rzeczy, robiłem sobie kurs pana Mirosława (Kurs JavaScript odc.2 na yt) i napotkałem się na problem przy drugim przykładzie.

//HTML

<input type="text" id="liczba1"/>
<input type="text" id="liczba2"/>
<input type="submit" value="Pokaż!" onclick="pokaz()"/>
<div id="pojemnik"></div>

//JS

function pokaz()
{
	var pole1 = document.getElementById("liczba1").value;
	var pole2 = document.getElementById("liczba2").value;
	var napis = "";
	
	if(pole1<pole2)for(i=pole1;i<=pole2;i++){napis = napis + i +" ";};
	if(pole1>pole2)for(i=pole1;i>=pole2;i--){napis = napis + i +" ";};
	if(pole1==pole2)document.getElementById("pojemnik").innerHTML = "Pola mają taką samą wartość";
	
	document.getElementById("pojemnik").innerHTML = napis;
}

1. Czemu przy pojawieniu się liczby 10 w pierwszym bądź w drugim polu i jakiejkolwiek innej liczby poza liczbami ujemnymi, 0 i 1 w drugim polu funkcja nie wypisuje liczb?

2.Czemu gdy pojawiają się takie same liczy funkcja nie wypisuje mi polecenia o takich samych liczbach?

Pozdrawiam, Przemek.

Answer 1

Witam, w tej postaci kod twój we wszystkich if'ach porównuje typ danych String, który powoduje te błędy.

Jeżeli w trakcie pobierania danych z formularza, poinformujesz JS o tym że to są liczby to będzie działać.

Aby to wykonać możesz np. dodać funkcję parseInt(), która zamieni te stringi na typ Integer:

zamiast

var pole1 = document.getElementById("liczba1").value;
var pole2 = document.getElementById("liczba2").value;
 

 

daj

var pole1 = parseInt(document.getElementById("liczba1").value);
var pole2 = parseInt(document.getElementById("liczba2").value);
 

 

Comments

Jest ok, ale nadal nie rozwiązuje problemu przy tych samych cyfrach. Nie zwraca mi komentarza "Pola mają taką samą wartość".

Answer 2

Zapisz tak:

if(pole1==pole2) { napis = "Pola mają taką samą wartość";}

Twoje rozwiązanie powodowało, że polecenie 

document.getElementById("pojemnik").innerHTML = napis;

nadpisywało to co miało być wyświetlone z 

if(pole1==pole2)document.getElementById("pojemnik").innerHTML = "Pola mają taką samą wartość";

 

Comments

Ok. Działa, dzięki. Jeszcze dużo nauki przede mną.

Answer 3

Odpowiedź została usunięta!