Liczby z przedziału(2 formularze); puste pola traktowane jak zera

pytanie zadane 21 września 2017 w JavaScript przez Karoldev Początkujący (300 p.)

Z drugiego odc. kursu MZ. Jedyna rzecz, której nie ujarzmiłem to puste pola, które pozostają zerami, niewzruszone moimi próbami wpłynięcia na nie. Próbowałem podejść do tego m. in. tak: if (firstNum.length == 0 || secondNum.length == 0) {
blank = "Oba pola muszą być uzupełnione!"
}

Ale na nic. Cały kod:

<body>

    <input type="text" id="first" />
    <input type="text" id="second" />
    <input type="submit" value="Pokaż" onclick="showNumbers()" />
    <div id="range"></div>
    <script>
        function showNumbers() {
            var firstNum = Number(document.getElementById("first").value);
            var secondNum = Number(document.getElementById("second").value);
            var blank = "";
            if (firstNum.length == 0 || secondNum.length == 0) {
                blank = "Oba pola muszą być uzupełnione!" } // To nie działa
                for (var i = firstNum; i <= secondNum; i++) {
                    blank = blank + i + " ";
                }
            } else if (firstNum > secondNum) {
                for (var j = firstNum; j >= secondNum; j--) {
                    blank = blank + j + " ";
                }
            } else if (firstNum == secondNum) {
                blank = "Liczby mają identyczną wartość!"
            }
           
            else if (isNaN(firstNum) || isNaN(secondNum)) {
                if (isNaN(firstNum) && !isNaN(secondNum)) blank = "Wartość w 1. polu nie jest liczbą"
                else if (isNaN(secondNum) && !isNaN(firstNum)) blank = "Wartość w 2. polu nie jest liczbą"
                else blank = "Wartości w obu polach  nie są liczbami"
               
            }
            document.getElementById("range").innerHTML = blank;
        }

    </script>
</body>

2 odpowiedzi

odpowiedź 21 września 2017 przez Jedras Maniak (54,860 p.)

Generalnie coś w rodzaju:

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


if(Number.isInteger(liczba1) && Number.isInteger(liczba2)){
 // jest ok				
}else{
 //jest problem
}

powinno rozwiązać Twój problem. Teraz tylko odpowiednio zmień swój kod.

komentarz 22 września 2017 przez Karoldev Początkujący (300 p.)

Zrobiłem to w taki sposób i nie wiem czemu teraz konsola stwierdza, że showNumbers isn't defined.

<input type="text" id="first" />
    <input type="text" id="second" />
    <input type="submit" value="Pokaż" onclick="showNumbers()" />
    <div id="range"></div>
    <script>
        function showNumbers() {
            var liczba1 = parseInt(document.getElementById("first").value);
            var liczba2 = parseInt(document.getElementById("second").value);

            var blank = "";

            if (Number.isInteger(liczba1) && Number.isInteger(liczba2)) {
                if (liczba1 < liczba2) {
                    for (var i = liczba1; i <= liczba2; i++) {
                        blank = blank + i + " ";
                    }
                } else if (liczba1 > liczba2) {
                    for (var i = 0; liczba1 >= liczba2; i--) {
                        blank = blank + i + " ";
                    }
                } else if (liczba1 == liczba2) {
                    blank = "Liczby mają identyczną wartość!";
                }
            } else {
                if (isNaN(liczba1) && isNaN(liczba2)) {
                    blank = "To nie są liczby!";
                } else if (isNan(liczba1) && !isNaN(liczba2)) {
                    blank = "Wartość w 1. polu nie jest liczbą";
                } else if (!isNan(liczba1) && isNaN(liczba2)) {
                    blank = "Wartość w 2. polu nie jest liczbą";
                }
            }
            document.getElementById("range").innerHTML = blank;
        }

odpowiedź 8 października 2017 przez Karoldev Początkujący (300 p.)

Dzisiaj ponownie w tym grzebię i dalej konsola stwierdza showFunctions is not defined. Pozmieniałem to trochę po bożemu, zrobiłem listenera, to teraz w ogóle nie czyta tej pierwszej zmiennej, stwierdzając, że not defined.

 <input type="text" id="first" />
    <input type="text" id="second" />
    <button type="button" id="show-btn">Pokaż</button>
    <div id="range"></div>
    <script>
        var shower = document.getElementById("show-btn");
        shower.addEventListener("click", function() {
            var liczba1 = parseInt(document.getElementById("first").value);
            var liczba2 = parseInt(document.getElementById("second").value);

            var blank = "";

            if (Number.isInteger(liczba1) && Number.isInteger(liczba2)) {
                if (liczba1 < liczba2) {
                    for (var i = liczba1; i <= liczba2; i++) {
                        blank = blank + i + " ";
                    }
                } else if (liczba1 > liczba2) {
                    for (var i = 0; liczba1 >= liczba2; i--) {
                        blank = blank + i + " ";
                    }
                } else if (liczba1 == liczba2) {
                    blank = "Liczby mają identyczną wartość!";
                }
            } else {
                if (isNaN(liczba1) && isNaN(liczba2)) {
                    blank = "To nie są liczby!";
                } else if (isNan(liczba1) && !isNaN(liczba2)) {
                    blank = "Wartość w 1. polu nie jest liczbą";
                } else if (!isNan(liczba1) && isNaN(liczba2)) {
                    blank = "Wartość w 2. polu nie jest liczbą";
                }
            }
            document.getElementById("range").innerHTML = blank;
        });