Wypisywanie liczb nie zawsze działa js

pytanie zadane 16 stycznia 2016 w JavaScript przez rotip Nowicjusz (150 p.)

Witam, dopiero się uczę js-a i natrafiłem na dziwny (jak dla mnie) problem.

Podając liczby w formularzu otrzymuje ciąg liczb np. podając liczby 1 i 5 otrzymuje ciąg: 1 2 3 4 5. Natomiast podając liczby 5 i 10 otrzymuję tylko w ciągu tylko pierwszą liczbę czyli : 5. Dla innych niektórych liczb też nie działa poprawnie. Czy powie mi ktoś o co tu chodzi?

Podaje kod:

<html>
<head>
    <meta charset="utf-8">
    <script>
        function numbers()
        {
            var first = document.getElementById("firstNumber").value;
            var second = document.getElementById("secondNumber").value;
            if(!isNaN(first) && !isNaN(second) && (first!="") && (second!=""))
            {
                var liczby = "";

                if (first<=second) {
                    for (; first <= second; first++) {
                        liczby = liczby + " " + first;
                    }
                }
                else
                {
                    for (; first >= second; first--) {
                        liczby = liczby + " " + first;
                    }
                }

                document.getElementById("myNumbers").innerHTML = liczby;
            }
            else
                document.getElementById("myNumbers").innerHTML = "podane liczby są błędne";
        }
    </script>
</head>
<body>
<div id="numbers">
        Wypisywanie liczb<br>
        Podaj przedział liczb całkowitych które chcesz wypisać<br>
        Od<input type="text" id="firstNumber">
        do<input type="text" id="secondNumber">
        <input type="submit" value="Wypisz" onclick="numbers()">
        <div id="myNumbers"></div>
    </div>
</body>
</html>

3 odpowiedzi

odpowiedź 16 stycznia 2016 przez writen Nałogowiec (29,060 p.)
wybrane 17 stycznia 2016 przez rotip

odpowiedź 16 stycznia 2016 przez robert9620 Stary wyjadacz (11,640 p.)

<html>
<head>
    <meta charset="utf-8">
    <script>
        function numbers()
        {
            var first = parseInt( document.getElementById("firstNumber").value );
            var second = parseInt( document.getElementById("secondNumber").value );
            if(!isNaN(first) && !isNaN(second))
            {
                var liczby = "";
				alert(first);
				alert(second);
 
                if(first<=second){
					alert("if");
                    for (; first <= second; first++){
                        liczby = liczby + " " + first;
                    }
                }
                else
                {
					alert("else");
                    for (; first >= second; first--) {
                        liczby = liczby + " " + first;
                    }
                }
 
                document.getElementById("myNumbers").innerHTML = liczby;
                //document.getElementById("pictures").innerHTML = log;
            }
            else
                document.getElementById("myNumbers").innerHTML = "podane liczby są błędne";
        }
    </script>
</head>
<body>
<div id="numbers">
        Wypisywanie liczb<br>
        Podaj przedział liczb całkowitych które chcesz wypisać<br>
        Od<input type="text" id="firstNumber">
        do<input type="text" id="secondNumber">
        <input type="submit" value="Wypisz" onclick="numbers()">
        <div id="myNumbers"></div>
    </div>
</body>
</html>

Powyższy kod działa. Trzeba było zastosować parseInt. Dobrze jest też nauczyć się szukać, w którym miejscu jest błąd, dlatego specjalnie zostawiłem CI w kodzie alerty, żebyś mógł zobaczyć przykład jak wyszukać błąd.

komentarz 16 stycznia 2016 przez ScriptyChris Mędrzec (190,190 p.)

Przy debugowaniu lepiej stosować console.log() niż alert().

komentarz 17 stycznia 2016 przez robert9620 Stary wyjadacz (11,640 p.)
edycja 17 stycznia 2016 przez robert9620

Być może, ale w takich przypadkach dużego znaczenia to nie ma, a jakoś wolę alert. Mimo wszystko masz rację, lepszy byłby console.log.

odpowiedź 16 stycznia 2016 przez Mariusz O Mądrala (5,290 p.)

http://www.w3schools.com/jsref/jsref_parseint.asp

Spróbowałbym

first = parseInt( first );
second = parseInt( second );

po ifie.

komentarz 16 stycznia 2016 przez writen Nałogowiec (29,060 p.)

A dlaczego po ifie, a nie na przykład w dwóch pierwszych linijkach funkcji?

Twoja odpowiedź zawiera link, który jest już chyba zakazany na tym forum :p

komentarz 16 stycznia 2016 przez Mariusz O Mądrala (5,290 p.)

Ponieważ, w ifie sprawdzasz zawartość pól. Dlaczego miałbyś parsować to na int, skoro np. mogą być puste?

komentarz 16 stycznia 2016 przez Comandeer Guru (600,810 p.)

To zwróci Ci NaN, więc to nawet pożądane ;)

komentarz 16 stycznia 2016 przez writen Nałogowiec (29,060 p.)

A co, wybuchnie ci komputer jak sparsujesz puste zmienne? Wtedy można skrócić warunek.
Kod należy pisać tak, by zawierał jak najmniej zbędnych instrukcji.