构造异步 Express GET 请求的最佳方式

Question

我一直在寻求通过结合异步/等待处理来更新我的 Express 技能，并有一个简短的问题。

从我在网上看到的示例来看，大多数请求都封装在 try/catch 块中，并在处理之前将任何等待任务添加到变量中。

app.post('/signup', async(req, res) => {
  const { email, firstName } = req.body
  const user = new User({ email, firstName })
  const ret = await user.save()
  res.json(ret)
}) 

我的代码如下所示：

app.route("/articles")
  // GET: articles
  .get(async (req, res) => {
    await Article.find((err, results) => {
      if (!err) {
        res.json(results);
      } else {
        res.send(err);
      };
    });
  })

我应该将 Mongoose 的响应分配给一个变量作为第一个代码块示例并在 try/catch 中处理，还是我的代码本质上做同样的事情，并且以最佳实践的方式？

提前致谢！

干杯， 詹姆斯

Answer 1

请注意，async 在 javascript 中有两种截然不同的用法。第一个是异步函数的一般概念，它可以用任何语言实现，并广泛用于具有各种设计模式（回调、承诺、async/await）的 javascript 中。第二个是async 关键字，用于允许使用await，并且只适用于promise（不适用于回调）。

您似乎混淆了两者。因此，我现在建议人们在提及异步函数时不要使用“async”这个词，而只使用它来指代 async 关键字。

你的函数没有返回一个承诺，因为你向它传递了一个回调。因此，它不能与await 一起使用。因为您不能使用await，所以将函数标记为async 是没有意义的。我认为您的代码 buggy 即使它有效 - 该机制仍然有效（await 关键字方便地忽略像您这样的非 Promise 函数并且不会产生错误）但是它不会传达您的意图很好，并且会混淆您代码的未来维护者。

恕我直言，您的代码的正确版本应该是：

app.route("/articles")
  // GET: articles
  .get((req, res) => {
    Article.find((err, results) => {
      if (!err) {
        res.json(results);
      } else {
        res.send(err);
      };
    });
  })

async/await 代码的正确版本是：

app.route("/articles")
  // GET: articles
  .get(async (req, res) => {
      try {
          res.json(await Article.find());
      }
      catch (err) {
          res.send(err);
      }
  })

带有 promise 但没有 async/await 的代码的正确版本应该是：

app.route("/articles")
  // GET: articles
  .get((req, res) => {
      Article.find()
          .then(result => res.json(result))
          .catch(err => res.send(err));
  })

---

以上当然只是我的意见，但我强烈建议您将其视为指导方针。上述三种形式中的任何一种对于大多数 javascript 程序员来说都是完全可以接受的，并且大多数人认为使用哪种形式是个人喜好问题。就我个人而言，我更喜欢没有await 的简单承诺，但async/await 在您有一些棘手的流控制逻辑时很有用。

---

请注意，Mongoose 方便地支持承诺和回调，因此在这种特定情况下，您可以删除回调以使用 async/await。然而，并非所有图书馆都这样做。如果需要将基于回调的函数转换为 Promise，则需要将其包装在 Promise 构造函数中：

function convertedToPromise () {
    return new Promise((resolve,reject) => {
        callbackBasedFunction((err,result) => {
            if (err) {
                reject(err)
            }
            else {
                resolve(result)
            }
        });
    });
}