Sprawdzanie czy wartość w polu jest liczbą i wyświetlanie odpowiedniej wiadomości JS

Question

Witam otóż mam taki program,który pobiera wartości z pól i po kliknięciu przycisku wyświetla wartości między tymi liczbami.W programie jest kilka "ifów",które zapobiegają spieprzeniu sprawy przez usera.Mam natomiast problem z napisaniem sprawdzenia czy user wpisuje liczbę,napisałem takie 2 funkcje ale są one jakby kompletnie ignorowane.Chodzi o funkcję isNumber.

Cała funkcja:

function pokaz(){
    var wynik="Liczby to: ";
    var num1 = document.getElementById("pole1").value;
    var num2 = document.getElementById("pole2").value;
    if(isNumber(num1)){
        document.getElementById("wyswietlLiczby").innerHTML="Wartość w lewym polu nie jest liczbą";
    }
    else if(isNumber(num2)){
        document.getElementById("wyswietlLiczby").innerHTML="Wartość w prawym polu nie jest liczbą";
    }
    if(num1>num2)
    {
       for(i=num1;i>=num2;i--){
        wynik+= i+",";
    }
      document.getElementById("wyswietlLiczby").innerHTML = wynik;
    }
    else if(num1==num2)
        {
            document.getElementById("wyswietlLiczby").innerHTML="Liczby są takie same!";
        }
    else{
    for(i=num1;i<=num2;i++){
        wynik+= i+",";
    }
    document.getElementById("wyswietlLiczby").innerHTML= wynik;
        }
    } 

Funkcja która powinna sprawdzać czy wartość wpsiana jest liczbą:

function isNumber(num1,num2){
    if(isNaN(num1)){ 
    return true;
    }
    else if(isNaN(num2)){ 
    return true;
    }
    else return false;
}

Tak wiem można to zoptymalizować lepiej ale najpierw chcę żeby wszystko chociaż działało.

Comments

A nie prościej <input type="number"> ?

Answer 1

Pytanie podstawowe - co rozumiesz przez poprawną dla tego przykładu liczbę?

Pisałem jakiś czas temu posta na blogu o problemach przy konwersji string do number: http://poradnik.drogimex.pl/2018/04/12/problemy-przy-zamianie-string-na-number-w-javascript/

Jest tam też mały codepen pokazujący w czym tkwi problem.

Zerknij sobie i myślę, że powinieneś znaleźć tam odpowiedź jak przekonwertować stringa do number.