エラー：Javascriptは常にtrueを返します

Question

このスクリプトをjspファイルに追加します。

この関数は、式と一致するものがないなど、失敗したシナリオでも常にtrueを返します。

なぜそれが起こるのですか？これを解決するにはどうすればよいですか？

<script type="text/javascript"> 
function validate() 
{ 
    document.getElementById("user").innerHTML=""; 
    document.getElementById("pass").innerHTML=""; 
    document.getElementById("mail").innerHTML=""; 
    document.getElementById("number").innerHTML=""; 

    var expression = /^[a-zA-Z_.0-9]+$/; 
    var mailExp = /^[\w\-\.\+]+\@[a-zA-Z0-9\.\-]+\.[a-zA-z0-9]{2,4}+$/; 
    var numberExp = /^[0-9]+$/; 
    if(!document.getElementById('username').value.match(expression)) 
    { 
     document.getElementById('username').focus(); 
     document.getElementById("user").innerHTML=" Enter valid user name"; 
     return false; 
    } 
    if(document.getElementById('password').value.length == 0) 
    { 
     document.getElementById('password').focus(); 
     document.getElementById("pass").innerHTML = "Fill Password field"; 
     document.getElementById("pass").style.visibility = "visible"; 
     return false; 
    } 
    if(document.getElementById('cnfpassword').value.length == 0) 
    { 
     document.getElementById('cnfpassword').focus(); 
     document.getElementById("pass").innerHTML = "Fill confirm Password field"; 
     document.getElementById("pass").style.visibility = "visible"; 
     return false; 
    } 
    if(document.getElementById('password').value != document.getElementById('cnfpassword').value) 
    { 
     document.getElementById('password').focus(); 
     document.getElementById("pass").innerHTML = "Password and confirm password Not match"; 
     document.getElementById("pass").style.visibility = "visible"; 
     return false; 
    } 
    if(!document.getElementById('emailid').value.match(expression)) 
    { 
     document.getElementById('emailid').focus(); 
     document.getElementById("mail").innerHTML=" Enter valid E-mailID"; 
     document.getElementById("mail").style.visibility = "visible"; 
     return false; 
    } 
    if(!document.getElementById('number').value.match(numberExp)) 
    { 
     document.getElementById('number').focus(); 
     document.getElementById("number").innerHTML=" Enter only numbers"; 
     document.getElementById("number").style.visibility = "visible"; 
     return false; 
    } 
    else 
    { 
     return true; 
    } 
} 

Answer 1

リファクタリングのアドバイス以外に、最も簡単なことは、Firebugを使用して何が間違っているのかをデバッグすることです。それはあなたに頭痛の多くを保存します

Answer 2

変更 document.getElementById('password').value.length == 0

document.getElementById('password').value.length === 0

JavaScriptで

にすべて以下

はtrueを返します

0==""  
0== "0" 
false== "0" 
null== undefined 
0==' \t\r\n ' 

ソリューションすべて以下

を返します偽

0===""  
0=== "0" 
false=== "0" 
null=== undefined 
0===' \t\r\n ' 

"=="とは "！="オペレータは、強制 2つの値に変換しようとします比較が行われる前の同じタイプ。

「===」と「！==」オペレータは、上記に述べてきた何

Answer 3

があり、比較の時にタイプと値を確認します（また、アイデンティティ演算子として知っています）正しい、elseは最後のifにのみ対応していました。

また、==は===と同じではありません。

また、関数内にreturn文を1つだけ持つ方がよい場合もあります。