błąd w kodzie JS

pytanie zadane 13 sierpnia 2018 w JavaScript przez Adamek185wp Obywatel (1,280 p.)

<!DOCTYPE html>
<html lang="pl">
<head>
	<meta charset="utf-8">
	<title>strona</title>
	<script src="js.js"></script>
</head>
<body>

	<input id="pole1" type="text" >
	<input id="pole2" type="text" >
	<input type="submit" value="przycisk" onclick="wypisz()">
	
	<div id="wynik"></div>

</body>
</html>

function wypisz()
{
	var liczba1 = document.getElementById("pole1").value;
	var liczba2 = document.getElementById("pole2").value;
	var napis = "";

	if (liczba1>liczba2) 
	{
		for (i=liczba1; i<=liczba2; i++)
		{
			napis = napis + i + " ";
		}
		document.getElementById("wynik").innerHTML = napis;
	}

	else if (liczba2>liczba1) 
	{
		for (i=liczba2; i<=liczba1; i++)
		{
			napis = napis + i + " ";
		}
		document.getElementById("wynik").innerHTML = napis;
	}
}

Czemu tak nie działa?

2 odpowiedzi

odpowiedź 13 sierpnia 2018 przez Tomek Sochacki Ekspert (227,510 p.)

Strasznie mieszasz typy string i number... W JS jest to dopuszczalne, ale moim zdaniem lepiej (i sam też się tej zasady stosuję) jawnie konwertować typy np. ze string do number i odwrotnie i dopiero wtedy wykonywać odpowiednie operacje matematyczne (już na typie number) albo konkatenacje znaków (na typie string).

Jakiś czas temu pisałem artykuł o konwersjach ze string do number, może Ci się przyda: http://poradnik.drogimex.pl/2018/04/12/problemy-przy-zamianie-string-na-number-w-javascript/