• Najnowsze pytania
  • Bez odpowiedzi
  • Zadaj pytanie
  • Kategorie
  • Tagi
  • Zdobyte punkty
  • Ekipa ninja
  • IRC
  • FAQ
  • Regulamin
  • Książki warte uwagi

Wypisywanie liczb nie zawsze działa js

Aruba Cloud VPS - 50% taniej przez 3 miesiące!
0 głosów
873 wizyt
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

+5 głosów
odpowiedź 16 stycznia 2016 przez writen Nałogowiec (29,060 p.)
wybrane 17 stycznia 2016 przez rotip
 
Najlepsza
Zapamiętajcie w końcu, że dane pobrane z formularza to są stringi! Jeżeli mają być wprowadzane liczby to trzeba to przekonwertować na typ liczbowy, np funkcją parseInt().
+2 głosów
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.
–1 głos
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 (605,680 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.
komentarz 16 stycznia 2016 przez Mariusz O Mądrala (5,290 p.)
Więc wykonasz o dwie za dużo niepotrzebnie. ;) Bo i tak chcesz je parsować tylko jak masz liczby.
komentarz 16 stycznia 2016 przez Comandeer Guru (605,680 p.)

Liczba instrukcji będzie taka sama, bo odpadną dwa warunki z if-a. I kod stanie się przejrzystszy.

Podobne pytania

0 głosów
3 odpowiedzi 525 wizyt
pytanie zadane 26 marca 2018 w Sieci komputerowe, internet przez xAstern Nowicjusz (180 p.)
0 głosów
2 odpowiedzi 229 wizyt
pytanie zadane 16 stycznia 2018 w C i C++ przez Hawkeye Początkujący (360 p.)
0 głosów
1 odpowiedź 243 wizyt
pytanie zadane 10 grudnia 2016 w C i C++ przez DODO Bywalec (2,950 p.)

93,093 zapytań

142,054 odpowiedzi

321,491 komentarzy

62,435 pasjonatów

Motyw:

Akcja Pajacyk

Pajacyk od wielu lat dożywia dzieci. Pomóż klikając w zielony brzuszek na stronie. Dziękujemy! ♡

Oto polecana książka warta uwagi.
Pełną listę książek znajdziesz tutaj

Wprowadzenie do ITsec, tom 1 Wprowadzenie do ITsec, tom 2

Można już zamawiać dwa tomy książek o ITsec pt. "Wprowadzenie do bezpieczeństwa IT" - mamy dla Was kod: pasja (użyjcie go w koszyku), dzięki któremu uzyskamy aż 15% zniżki! Dziękujemy ekipie Sekuraka za fajny rabat dla naszej Społeczności!

...