2017-02-25 7 views
0

doesntの仕事をONSUBMITだから私は、ログインフォームイム作りを持っている、と私は入力が空かそうでないかどうかをテストします。 私はエラーを取得していないが、私が提出したときにそれだけをonSubmitを無視します。 ?フォームが正しく

これは私のフォームです:

<form onsubmit="return ifEmpty();" action="login1.php" method="post" id="loginform" accept-charset="UTF-8"> 
        <div style="vertical-align: top;" dir="rtl"> 
         <label dir="rtl" class="labels">שם משתמש/אימייל</label> 
         <br> 
         <br> 
         <input name="unameemail" id="username" type="text"> 
        </div> 
        <div style="display: block;vertical-align:top;" dir="rtl"> 
         <br> 
         <label dir="rtl" class="labels">סיסמה</label> 
         <br> 
         <br> 
         <input name="password" id="passwordin" type="password"> 
        </div> 
        <div> 
        <br> 
         <br> 
         <br> 
         <input class="btn" type="submit" name="submitone" onclick="" value="login"> 
        </div> 
       </form> 
      </div> 

と私のJS:

<script type="text/javascript"> 
      function ifEmpty() { 

       if (document.getElementById("username") == "") { 

        document.getElementById("username").style.backgroundColor = "red"; 
        document.getElementById("username").style.color = "white"; 
        document.getElementById("username").className += " invalidform"; 
        return false; 
       } else if (document.getElementById("passwordin") == "") { 
        document.getElementById("username").style.backgroundColor = "white"; 
        document.getElementById("username").style.color = "black"; 
        document.getElementById("password").style.backgroundColor = "red"; 
        document.getElementById("password").style.color = "white"; 
        document.getElementById("password").className += " invalidform"; 
        return false; 
       } 

      } 

     </script> 
+3

あなたは意味をなさない文字列にDOM要素を比較しています。入力要素の 'value'プロパティにアクセスする必要があります。 – CBroe

+0

はいあー......私は忘れてしまった愚かな間違いだった – guy

+0

をありがとう:私はそれはそれだけでアウトエラーが発生した、をonSubmit無視しませんでした公式の答え – guy

答えて

1

場合(のdocument.getElementById( "ユーザ名")== ""){

あなたがここに文字列あなたが入力の値プロパティにアクセスする必要があります;-)

にDOM要素を比較しています要素。

+0

はすでにやった – guy

1

変更:これに

if (document.getElementById("username") == "") { 

if (document.getElementById("username").value == "") { 
+0

MB。ありがとう! – guy

関連する問題