Question

I’ve got a form where the user inputs 3 values, which are then calculated. The outputs are displayed again within the form in some “readonly” output boxes. For each input, I want to validate if they are a number, if not, instead of the form showing “NaN” I want to display an error saying, “Please enter a number” (or something like that). Below is the code I am using, which is executed “onkeyup”:

function checkforNumber() 
{
    if (isNaN(sInput || dInput || pInput) == true) {
        alert("You entered an invalid character. Please reset the form.");
    }
    else {
        return(false);
    }
} 

Am I using this function incorrectly? Is there something wrong with the syntax?

Thanks

Answer 1

if (isNaN(sInput) || isNaN(dInput) || isNaN(pInput)) {
    alert("You entered an invalid character. Please reset the form.");
}

also make sure that those 3 variables sInput, dInput and pInput are not strings but were obtained by using parseFloat or parseInt methods.

var sInput = parseFloat(document.getElementById('sinput').value);
var dInput = parseFloat(document.getElementById('dinput').value);
var pInput = parseFloat(document.getElementById('pinput').value);