无法将代码从 .then 更改为 async/await

Question

试图改变.then，像这样：

User.prototype.login = () => {
  return new Promise((resolve, reject) => {
    this.cleanup();
    usersCollection
      .findOne({ username: this.data.username })
      .then((attemptedUser) => {
        if (attemptedUser && attemptedUser.password == this.data.password) {
          resolve("logged in");
        } else {
          reject("invalid something");
        }
      })
      .catch(() => {
        reject("Please, try again later");
      });
  });

第一个效果很好，但是当我尝试将其更改为 async/await 时，如下所示：

User.prototype.login = () => {
  return new Promise(async (resolve, reject) => {
    this.cleanup();
    try {
      const attemptedUser = await usersCollection.findOne({ username: this.data.username });
      if (attemptedUser && attemptedUser.password == this.data.password) {
        resolve("logged in");
      } else {
        reject("invalid something");
      }
    } catch {
      reject("Please, try again later");
    }
  });
};

它给我一个错误，this.cleanup() 不是一个函数，经过几次尝试，我意识到异步以某种方式改变了“this”。

你能帮帮我吗，我在哪里出错了？

Answer 1

您的代码中有 2 个问题：

1. 在prototype 对象上定义的函数应该是常规函数而不是箭头函数，因为在对象上定义的函数被用作方法，即它们内部的 this 值是指构造函数或类的实例。使用箭头函数会设置不正确的 this 值，这很可能是您的代码出错的原因。

2. 您将 promise-chaining 与 async-await 语法混合在一起。async 函数总是返回 Promise，因此不要显式返回 Promise 对象，而是将 login 函数设为 async 函数并返回来自函数的字符串，这与使用字符串调用 resolve() 函数相同。要拒绝承诺，请从 async 函数中抛出错误。

这就是你应该如何定义login函数

User.prototype.login = async function() {
    this.cleanup();

    try {
      const attemptedUser = await usersCollection.findOne({username: this.data.username});

      if (attemptedUser && attemptedUser.password == this.data.password) {
        return "logged in";
      }

      throw new Error("invalid something");

    } catch (error) {
      console.log("Please, try again later");
      throw error;
    }
};