1

我无法让循​​环在我的简单 js 登录脚本中工作。当我尝试使用数组中最后一个(user3 和 pass3)以外的任何登录名登录时,它返回 false。

我究竟做错了什么?

我都试过了=====

var userLogins = [{user:"user1", password:"pass1"},{user:"user2", password:"pass2"},{user:"user3", password:"pass3"}]
var success = null;
function logon(user, pass) {
    userok = false;
    for (i = 0; i < userLogins.length; i++)
    { 
        if(pass == userLogins[i].password && user == userLogins[i].user )
        {
            success = true;
        }
        else
        {
            success = false;
        }
    }
    secret(success);
}

function getData() {
    var user = document.getElementById("userid").value;
    var password = document.getElementById("password").value;
    logon(user, password);
}

function secret(auth){
    if(auth)
    {
        show('success');
        hide('login');
    }
    else
    {
        show('error');
        hide('login');
    }
}

function show(show) {
    document.getElementById(show).className = "show";
}
function hide(hide) {
    document.getElementById(hide).className = "hide";
}

4

2 回答 2

3

break找到时使用。否则下一个循环会将成功设置为假。

for (var i = 0; i < userLogins.length; i++)
{ 
    if(pass == userLogins[i].password && user == userLogins[i].user )
    {
        success = true;
        break;
    }
    else
    {
        success = false;
    }
}
secret(success);
于 2013-01-31T02:51:10.917 回答
3
for (i = 0; i < userLogins.length; i++)
{ 
    if(pass == userLogins[i].password && user == userLogins[i].user )
    {
        success = true;
    }
    else
    {
        success = false;
    }
}

您需要 a breakin there,否则您的true值 for只会在下一次迭代中success被覆盖...除了最后可能的 credentials,没有“下一次”迭代。false

完成此操作后,您实际上根本不需要else分支:

var success = false;
for (i = 0; i < userLogins.length; i++) { 
    if (pass == userLogins[i].password && user == userLogins[i].user) {
        success = true;
        break;
    }
}
于 2013-01-31T03:23:44.240 回答