2017-02-25 9 views
0

So habe ich ein Login-Formular im machen, und ich möchte testen, ob die Eingänge leer sind oder nicht. Ich bekomme keinen Fehler, aber wenn ich es einreiche ignoriert es einfach die onsubmit. Hilfe?Formular onsubmit funktioniert nicht richtig

das ist meine Form:

<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> 

und meine 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

Sie vergleichen DOM-Elemente mit Strings, das macht keinen Sinn. Sie müssen auf die Eigenschaft 'value' der Eingabeelemente zugreifen. – CBroe

+0

OHHH ja danke ich habe vergessen ...... – guy

+0

das war ein dummer Fehler: D – guy

Antwort

1

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

Sie vergleichen DOM-Elemente in Strings hier ;-)

Sie müssen den Wert Eigenschaft des Eingangs zuzugreifen Elemente.

+0

Bereits getan. Vielen Dank! – guy

1

Wechsel:

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

dazu:

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

ja ich schon, danke – guy

Verwandte Themen