构造异步 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并且仅适用于承诺（在回调中不起作用）。

你似乎混淆了两者。因此，我现在建议人们在引用异步函数时不要使用“async”一词，而仅使用它来引用关键字async。

您的函数不会返回承诺，因为您向它传递了回调。因此它不能与 一起使用await。因为您无法使用，await所以将函数标记为 是没有意义的async。我认为你的代码有缺陷，即使它有效 - 该机制仍然有效（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);
      }
  })

有承诺但没有承诺的代码的正确版本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 可以方便地支持 Promise 和回调，因此在这种特定情况下，您只需删除要使用的回调即可async/await。然而，并非所有图书馆都这样做。如果您需要将基于回调的函数转换为 Promise，则需要将其包装在 Promise 构造函数中：

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