初心者のJavaScript isNaNに問題はありますか？

Question

function IsValid(value){ 
    var valid = true; 

    if (isNaN(value) || value == ""){ 
     valid = false; 
    } 

    return value; 
} 

JavaScriptの初心者ですから、これは簡単な質問です。上記の関数を使用してパラメータをチェックし、エラーメッセージを生成しています。それはそれの "isNaN"の部分を除いて正常に動作します。

if (IsValid(wind)) { 
     wind = parseFloat(wind); 

     if (wind < MIN_WIND || wind > MAX_WIND) { 
      SetError("divWindError", "Wind must be between " + MIN_WIND + 
      " and " + MAX_WIND); 
      return; 
     } 
    } else { 
     SetError("divWindError", "Wind is not valid"); 
     return; 
    } 

Answer 1

いくつかのこと。まず、あなたが二、あなたのチェックは、同様nullとundefinedチェックを含むように更新する必要があり、間違った値を返すしている：

function IsValid(value){ 
    var valid = true; 

    if (!value || isNaN(value)){ // updated check to include NaN, "", undefined, and null 
     valid = false; 
    } 

    return valid; // correct return value is valid, not value 
} 

Answer 2

function IsValid(value){ 
 
    var valid = true; 
 

 
    if (isNaN(value) || value == ""){ 
 
     valid = false; 
 
    } 
 

 
    return valid; // you needed to return valid not the value 
 
}