MongoDB数据获取与验证异常：始终返回‘invalid email’错误

核心问题点 #

• 未处理用户不存在的场景：当db.Register.findOne({email: em})找不到匹配用户时，useremail为null，此时访问useremail.pass会直接抛出错误，触发catch块返回"invalid email"。
• 变量名错误：代码里定义的密码变量是pass，但判断时误用了未定义的pw，导致逻辑失效。
• 空else分支：密码不匹配时没有返回响应，会让请求一直处于挂起状态。

修复后的代码 #

app.post('/templates/views/login', async(req,res) => {
    try {
      const em = req.body.email;
      const pass = req.body.pw;

      // 先校验请求参数完整性
      if (!em || !pass) {
        return res.status(400).send("邮箱或密码不能为空");
      }

      const useremail = await db.Register.findOne({ email: em });
      
      // 先判断用户是否存在
      if (!useremail) {
        return res.status(400).send("邮箱未注册");
      }

      // 修正变量名，用定义好的pass做比对
      if (useremail.pass === pass) {
        res.status(200).render("index");
      } else {
        res.status(400).send("密码错误");
      }

    } catch (error) {
      // 打印错误日志方便排查，返回通用服务器错误
      console.error("登录异常:", error);
      res.status(500).send("服务器内部错误");
    }
  });

关键修复说明 #

1. 增加参数校验：提前检查邮箱和密码是否为空，避免无效的数据库查询。
2. 区分用户不存在与密码错误：先判断useremail是否为null，单独返回"邮箱未注册"，避免和密码错误混淆。
3. 修正变量名错误：将pw改为定义好的pass，确保比对逻辑正确执行。
4. 完善else分支逻辑：密码不匹配时返回明确的错误信息，避免请求挂起。
5. 优化异常处理：catch块中打印错误日志，同时返回服务器内部错误，既方便排查问题，也避免暴露敏感的数据库错误信息。

额外建议 #

不要明文存储用户密码，建议用bcrypt等哈希算法加密存储，比对时用哈希校验：

// 注册时加密密码
const hashedPass = await bcrypt.hash(pass, 10);
// 登录时校验密码
const isMatch = await bcrypt.compare(pass, useremail.pass);
if (isMatch) { /* 登录成功逻辑 */ }

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