[JavaScript] Kod raz działa poprawnie a raz nie

Question

Witam,

Zrobiłem swoją wersję kodu z kursu P. Mirka o JS odc. 2. Kod działa prawidłowo, tzn. jeśli dwie liczby to wyświetla mi je rosnąco lub malejąco w divie. Problem pojawia się, gdy mam już wprowadzone dwie liczby, wyświetle wynik, dokonam ich modyfikacji i wówczas raz wypisuje się zakres liczb a raz tylko jedna pojedyncza liczba. Dlaczego tak się dzieje?

function sprawdz()
		{
			var liczba1 = document.getElementById("pole1").value;
			var liczba2 = document.getElementById("pole2").value;
			var wynik = "";
			var NaN = "Nie wprowadzono liczb!";
			var equal = "Liczby są takie same";
			
			
			if(liczba1 < liczba2)
			{
				for(i=liczba1; i<=liczba2; i++)
				{
					wynik += i + " ";
				}
				document.getElementById("wynik").innerHTML = wynik;
			}
			else if(liczba1 > liczba2)
			{
				for(i=liczba1; i>=liczba2; i--)
				{
					wynik += i + " ";
				}
				document.getElementById("wynik").innerHTML = wynik;
			}
			else document.getElementById("wynik").innerHTML = equal;
		}

 

Answer 1

function sprawdz()
{
	var liczba1 = Number(document.getElementById("pole1").value);
	var liczba2 = Number(document.getElementById("pole2").value);
	var wynik = "";
	var NaN = "Nie wprowadzono liczb!";
	var equal = "Liczby są takie same";
	 
	 
	if(liczba1 < liczba2)
	{
		for(i=liczba1; i<=liczba2; i++)
		{
			wynik += i + " ";
		}
		document.getElementById("wynik").innerHTML = wynik;
	}
	else if(parseInt(liczba1) > parseInt(liczba2))
	{
		for(i=liczba1; i>=liczba2; i--)
		{
			wynik += i + " ";
		}
		document.getElementById("wynik").innerHTML = wynik;
	}
	else document.getElementById("wynik").innerHTML = equal;
}

Sprawdź to. Ja miałem problem jak brałem liczby 2-9 i 10+.

Comments

Teraz działa za każdym razem. Dzięki!

Czyli rozumiem, że najpierw, przed pobraniem wartości z htmla, trzeba przy użyciu obiektu "Number" dokonać konwersji danych na wartości liczbowe i dopiero później te zmienne porównywać?

BTW. W pętli else if nie jest potrzebne już rzutowanie przy użyciu "parseInt()" ;)

---

Jak porównywałem bez Number to "2" było wieksze od "10". 

Testowałam różne rzeczy i zapomniałem usunąć parseInt. 

Answer 2

sprawdz(document.getElementById("pole1").value, document.getElementById("pole2").value);
function sprawdz(liczba1, liczba2)
        {
            var wynik = "";
            var NaN = "Nie wprowadzono liczb!";
            var equal = "Liczby są takie same";
             
             
            if(liczba1 < liczba2)
            {
                for(i=liczba1; i<=liczba2; i++)
                {
                    wynik += i + " ";
                }
                document.getElementById("wynik").innerHTML = wynik;
            }
            else if(liczba1 > liczba2)
            {
                for(i=liczba1; i>=liczba2; i--)
                {
                    wynik += i + " ";
                }
                document.getElementById("wynik").innerHTML = wynik;
            }
            else document.getElementById("wynik").innerHTML = equal;
        }

Spróbuj tak.

Comments

W ogóle nie działa, przechodzi od razu do ostatniego elsa.

Ktoś? Coś?

Answer 3

Debugowałeś krok po kroku? Wyskakuje jakiś błąd w konsoli?