node.js - 构建异步 Express GET 请求的最佳方式

Question

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

从我在网上看到的示例来看，大多数请求都结构化地包装在一个 try/catch block 中，并在处理它们之前将任何 await 任务添加到一个变量中。

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 中有两种截然不同的用法。第一个是异步函数的一般概念，它可以用任何语言实现，并在具有各种设计模式(回调、 promise 、async/await)的 javascript 中广泛使用。第二个是 async 关键字，它用于允许使用 await 并且仅适用于 promise (不适用于回调)。

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

您的函数没有返回 promise ，因为您向它传递了回调。因此它不能与await一起使用。因为您不能使用 await，所以将函数标记为 async 是没有意义的。我认为您的代码 有问题 即使它可以工作 - 该机制仍然有效( await 关键字方便地忽略像您这样的非 Promise 函数并且不会产生错误)但是它不会很好地传达您的意图，这会让您的代码的 future 维护者感到困惑。

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

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 的简单 promise ，但是当您有一些棘手的流程控制逻辑时，async/await 很有用。

---

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

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