JS条件判断疑问：输入正确账号密码为何同时执行if和else？

嘿，我来帮你捋清楚这个问题：你遇到的核心问题是循环内部的if/else逻辑设计错误，导致哪怕输入正确的账号密码，也会触发错误提示。

问题根源 #

你的signIn函数会遍历数据库里的每一个用户对象：

• 当遍历到匹配的用户时，会执行if分支打印newsfeed
• 但遍历到不匹配的用户时，会立刻执行else分支打印"Error"

举个例子：如果你输入的是第一个用户的账号密码，循环第一次会匹配并打印newsfeed，但接下来还要遍历第二个、第三个用户——这两个不匹配，所以会各打印一次"Error"，最终结果就是newsfeed和Error同时出现。

修复方案 #

我们需要调整逻辑：先遍历寻找匹配的用户，找到后就终止循环；如果遍历完所有用户都没找到匹配，再打印错误信息。

修改后的完整代码：

var database = [ 
  { username: "john83reuben", password: "12345" }, 
  { username: "jdominic", password: "12345" }, 
  { username: "johnreuban", password: "12345" } 
];
var newsfeed =[ 
  { username: "john83reuben", timeline: "Psalm 118:19. I will extol the Lord at all times; His praise will always be on my lips"}, 
  { username: "jdominic", timeline: "Psalm 34:1-3. I thank you, Lord, with all my heart; I sing praise to you before the gods."}, 
  { username: "johnreuban", timeline: "Psalm 138: 1-2. I will give thanks to the Lord because of his righteousness;"} 
];

function signIn(user, pass){
  // 标记是否找到匹配的用户
  let isUserFound = false;
  for(let i=0; i < database.length ; i++){
    if(database[i].username === user && database[i].password === pass ){
      console.log(newsfeed);
      isUserFound = true;
      // 找到匹配后立刻退出循环，避免多余遍历
      break;
    }
  }
  // 只有遍历完所有用户都没匹配，才输出错误
  if(!isUserFound){
    console.log("Error");
  }
}

var usernamePrompt = prompt("What is your username?");
var passwordPrompt = prompt("What is your password?");
signIn(usernamePrompt,passwordPrompt);

更简洁的优化写法 #

你也可以用数组的find方法来简化代码，可读性更高：

function signIn(user, pass){
  // 直接查找匹配的用户
  const matchedUser = database.find(item => item.username === user && item.password === pass);
  if(matchedUser){
    console.log(newsfeed);
  } else {
    console.log("Error");
  }
}

find方法会遍历数组，找到第一个符合条件的元素就返回，找不到则返回undefined，刚好适配我们的验证逻辑。

内容的提问来源于stack exchange，提问作者johnny_linux