Javascriptの検証コントロール

Question

私は自分のフォームにいくつかの簡単なフィールドを検証するためのjavascriptを書いています。私の問題は、フォームにエラーがない場合でもコードが機能することです。

は、具体的には、if(openHour > closeHour)

でエラーがスローされます。これは、コード

function checkForm() 
{ 
    var openHour; 
    var closeHour; 
    var i; 

    for(i=1;i<8;i++) 
    { 
    openHour = document.getElementById("openHours" + i).value; 
    closeHour= document.getElementById("closeHours" + i).value; 

    if(openHour > closeHour) 
    { 
    document.getElementById('error').innerHTML= "Opening Error at " + i; 
    return false; 
    } 
    if(openHour == "0" && closeHour > 0) 
    { 
    document.getElementById('error').innerHTML= "Closing Error at " + i; 
    return false; 
    } 


    } 
} 

Answer 1

openHour = parseInt(document.getElementById("openHours" + i).value); 
closeHour = parseInt(document.getElementById("closeHours" + i).value); 

私はちょうど解析するのに必要それらの値を比較できるようにするために、値を整数に変換します。すべて

Answer 2

は常にopenHoursとcloseHoursフィールドのデフォルト値があるのですか？

例えば、いくつかのデフォルト値を設定してみてください：

var openHour = 0; 
var closeHour = 0; 

その後openHoursとclosedHoursがそれらを割り当てる前に、それらの値を持っているかどうかを確認します。

if (document.getElementById("openHours" + i).value != null && document.getElementById("openHours" + i).value != "") 
    openHour = document.getElementById("openHours" + i).value; 


if (document.getElementById("closeHours" + i).value != null && document.getElementById("closeHours" + i).value != "") 
    closeHour = document.getElementById("closeHours" + i).value;