Node.js+MySQL登录API空值检查失效问题求助

嘿，我一眼就瞅出问题所在了——你的条件判断逻辑完全写反啦！

你现在的代码里，当用户名或密码为空的时候，反而跑去执行登录验证的逻辑了，这可不就等于跳过了空值检查嘛😅。咱们来把逻辑掰正：

错误点分析 #

你这段代码的逻辑是：如果用户名/密码为空 → 去查数据库验证登录，这完全和你想要的“空值时返回错误”相反。正确的逻辑应该是：如果用户名/密码为空 → 直接返回错误；否则才去执行登录验证。

修复后的完整代码示例 #

module.exports.login = (req, res) => {
  const username = req.body.username;
  const password = req.body.password;

  // 先做空值检查，不符合直接返回错误
  if (!username || !password) {
    return res.status(400).json({
      status: 400,
      message: '用户名和密码不能为空'
    });
  }

  // 参数合法，再执行登录验证逻辑
  helper.userLogin(username)
    .then((data) => {
      if (data) {
        // 注意：这里强烈建议不要直接明文比较密码！用bcrypt哈希验证更安全
        if (data.password === password) {
          return res.status(200).json({
            status: 200,
            message: '登录成功'
          });
        } else {
          return res.status(401).json({
            status: 401,
            message: '密码错误'
          });
        }
      } else {
        return res.status(404).json({
          status: 404,
          message: '用户不存在'
        });
      }
    })
    .catch((err) => {
      // 别忘了捕获数据库查询的异常
      return res.status(500).json({
        status: 500,
        message: '服务器内部错误',
        error: err.message
      });
    });
};

几个关键优化点 #

• 用return来终止后续代码执行，避免逻辑混乱；
• 补充了各种异常场景的返回（用户不存在、密码错误、服务器异常）；
• 强烈提醒：不要直接明文存储和比较密码！一定要用bcrypt这类库对密码进行哈希处理后再存储，验证时用哈希比对，避免安全风险；
• 可以把空值检查做得更严谨，比如判断是否是空白字符串：!username.trim() || !password.trim()，防止用户输入全空格的情况。

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