UnhandledPromiseRejectionWarning：TypeError：无法读取属性

Question

我正在尝试创建一个简单的登录验证，但是当我通过邮递员故意在登录发布方法中输入错误的电子邮件时，我收到了这个错误。

UnhandledPromiseRejectionWarning: TypeError: Cannot read property 'password' of null at C:\Users\layug\Documents\work\zuitt\batch-125\s32 -36\course-booking-api\controllers\userControllers.js:54 :74 在 processTicksAndRejections (internal/process/task_queues.js:95:5)

当邮箱正确，密码错误时，我会收到错误的邮箱或密码信息错误信息。

问题是电子邮件不正确，或者在数据库中找不到电子邮件（通过“结果”变量）。在这些情况下，它开始在控制台中显示错误。

我使用的代码如下。

module.exports.login = (reqBody) => {
  // model method
  return User.findOne({email: reqBody.email}).then((result, error) => {
    // This is the line causing the error.
    const isPasswordCorrect = bcrypt.compareSync(reqBody.password, result.password);
    if (result == null || isPasswordCorrect == false) {
      return "Email or password is incorrect";
    } else {
      return {access: auth.createAccessToken(result.toObject())};
    }
  });
}

当我这样写条件时，一切正常，但这不是我登录验证的理想条件。

module.exports.login = (reqBody) => {
  return User.findOne({ email: reqBody.email }).then((result, error) => {
    if (result == null) {
      return "Email not registered";
    } else {
      const isPasswordCorrect = bcrypt.compareSync(reqBody.password, result.password);
      if (isPasswordCorrect === true) {
        return { access: auth.createAccessToken(result.toObject()) };
      } else {
        return "Wrong password";
      }
    }
  });
}

Answer 1

简单的解决方法是将尝试获取用户密码的函数包装在 try-catch 语句中。在这种情况下，您正在查找一个不存在的用户，该用户返回一个 null，然后您尝试读取该用户的密码，给您错误“无法读取 null 的密码”

//model method
return User.findOne({ email: reqBody.email }).then((result, error) => {
  //below is the line where I get the error
  try {
    const isPasswordCorrect = bcrypt.compareSync(
      reqBody.password,
      result.password
    );

    if (result == null || isPasswordCorrect == false) {
      return "Email or password is incorrect";
    } else {
      return { access: auth.createAccessToken(result.toObject()) };
    }
  } catch (err) {
    return "Email does not exist";
  }
});

顺便说一句，您的解决方案很好。