jsの1つの関数で複数のフィールドを検証する方法

Question

複数のテキストフィールドを検証する必要があるが正しいとは思えない関数があります。たとえば、$("#textfield").blur(validation);を使用して、各フィールドがフォーカスを失ったときに検証関数を実行します（3つのフィールドすべてがblurを使用して検証関数を呼び出します）。私はifステートメントを正しく注文していないと思う。また、thisを使用して検証を行う簡単な方法がわかっている場合は、それも素晴らしいでしょう。ありがとうございました。

function validation(height,width,color){ 

if (height.length == 0 || height == null){ //check for empty field 
    $("#msg").html("This field cannot be empty"); 
} 
else if (isNaN(height)){ //check that height is a number 
    $("#msg").html("The height must be a number"); 
}else{ 
    $("#msg").html("The height is " + height); //if height is ok shows this message 
} 
if (width.length == 0 || width== null){ //same for width 
    $("#msg").html("This field cannot be empty"); 
} 
else if (isNaN(width)){ 
    $("#msg").html("The width must be a number"); 
}else{ 
    $("#msg").html("The width is " + width); 
} 
if (color.length == 0 || color == null){ 
    $("#msg").html("This field cannot be empty"); 
} 
else if (color == "white" || color == "grey"){ //check that color is one of these two options 
    $("#msg").html("The color is " + color); 
}else{ 
    $("#msg").html("The color must be white or grey"); 
} 

} 

Answer 1

それはあなたが再利用可能なisNullOrEmpty機能を作ることができる少なくとも第1 .length === 0

続いnullをチェックすることをお勧めします -

-

function isNullOrEmpty (obj) { 
     return !((obj !== undefined) && (obj !== null) && (obj .length === 0)); 
    } 

を次に、あなたのように一般化することができます

function validateNumber(obj ,fieldName) { 
    if(isNaN(obj)) { 
     return 'The ' + fieldName + ' must be number.' 
    } 
    return ''; 
} 

Answer 2

if(!height) { $("#msg").text("Height is not defined") }これはfalse、未定義、NaN、ゼロ、文字列の空の値を表します