javascript - ログインjsファイル（log.js）をこのhtmlコードにリンクしていますが、変更はありません

Question

これは私の .html 本文コンテンツです:

<form name="login"> 
    <table border="0" cellpadding="2.0" cellspacing="2.0" align="left"> 
        <tr>
            <td>Username&nbsp;</td>
            <td>
                <input name="username" type="text" id="username" maxlength="15"></input>
            </td> 
        </tr> 
        <tr>
            <td>Password&nbsp;</td>
            <td>
                <input name="password" type="password" id="password" maxlength="10"></input>       
            </td> 
        </tr> 
        <tr>
            <td>&nbsp;</td> 
            <td>&nbsp;</td> 
        </tr> 
        <tr>
            <td>&nbsp;</td>
            <td>
                <strong>
                    <input name="Login" type="button" value="Submit" onclick="Vlogin()"></input>
                </strong>
            </td> 
        </tr> 
    </table> 
</form>

...そしてJavaScriptコード：

function Vlogin()
{
    var user=document.login.username;
    var pass=document.login.password;
    if (user.value=="account1" && pass.value=="backup1") 
    { 
        window.location= "index.jsp";
    }
    else
        alert("Invalid username or password");
        user.focus();
    }
}

Answer 1

function Vlogin()
{
    var user=document.login.username;
    var pass=document.login.password;
    if (user.value == "account1" && pass.value == "backup1") 
    { 
        window.location= "index.jsp";
    }
    else{
        alert("Invalid username or password");
        user.focus();
    }
}

「else」の後の中括弧

Answer 2

使用しているメソッドで DOM 内の要素にアクセスすることはできません.. document.login など

生の JS を使用する場合は、getElementsByTagName() https://developer.mozilla.org/en-US/docs/DOM/document.getElementsByTagNameを使用できます。

しかし、JQuery はほぼ標準であるため、JQuery を使用することをお勧めします。

function Vlogin() {
    var login = $('form[name=login]')
    var user= $('[name=username]', login).val()
    var pass= $('[name=password]', login).val()
    if (user === "account1" && pass ==="backup1") { 
        window.location= "index.jsp";
    } else {
        alert("Invalid username or password");
        user.focus();
    }
}

また、ソース内ではすべて平易な英語であるため、検証コードは簡単にハッキングされる可能性があることも指摘したいと思います:)