2016-09-09 7 views
0

Ich bin ziemlich neu in der Codierung, so dass dieser Block Code wirklich schlecht ist.JavaScript funktioniert nicht auf der Website

Ich mache ein einfaches Login-System wirklich schnell, aber jetzt bin ich eingesaugt worden und kann nicht herausfinden, was mit meinem Code falsch ist. Hilfe wird sehr geschätzt.

Der Benutzername und das Passwort ist "admin" (ich weiß, es ist schlecht, aber diese Website wird nicht öffentlich sein).

HTML:

function loginUser() { 
 
    var usernameId; 
 
    usernameId = document.getElementById("username1").value; 
 
    if (usernameId == "admin") { 
 
    window.location = "home.html"; 
 
    } 
 
} 
 
function loginPassword() { 
 
    var passwordId; 
 
    passwordId = document.getElementById("password1").value; 
 
    if (passwordId == "admin") { 
 
    loginUser(); 
 
    } 
 
}
<form class="login-form"> 
 
    <input type="text" name="username1" placeholder="username"/> 
 
    <input type="password" name="password1" placeholder="password"/> 
 
    <button onClick="loginPassword()">login</button>

Antwort

0

Sie verwenden name="username1" und name="password1", aber Sie sind dann die Elemente mit getElementById() bekommen. Wenn ja, müssen Sie das id Attribut verwenden, anstatt des name Attribut:

function loginUser() { 
 
    var usernameId; 
 
    usernameId = document.getElementById("username1").value; 
 
    if (usernameId == "admin") { 
 
    window.location = "home.html"; 
 
    } 
 
} 
 
function loginPassword() { 
 
    var passwordId; 
 
    passwordId = document.getElementById("password1").value; 
 
    if (passwordId == "admin") { 
 
    loginUser(); 
 
    } 
 
}
<form class="login-form"> 
 
    <!--    v---- changed here  ----> 
 
    <input type="text" id="username1" placeholder="username"/> 
 
    <!--     v---- and here  ----> 
 
    <input type="password" id="password1" placeholder="password"/> 
 
    <button onClick="loginPassword()">login</button> 
 
</form>

+0

@Vatsal Das ist eigentlich nicht wahr; HTML-Attribute (wie in diesem Fall 'onclick') unterscheiden nicht zwischen Groß- und Kleinschreibung, also sind' onclick' und 'onClick' gleichwertig. – Frxstrem

+0

Sie haben Recht :) – Vatsal

+0

Das funktioniert und die if-Anweisung funktioniert, aber die 'window.location' funktioniert nicht. Trotzdem danke! :) –

Verwandte Themen