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

如何解决构造异步Express GET请求的最佳方法

我一直在寻求通过结合异步/等待处理来更新我的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中进行处理，还是我的代码本质上做相同的事情，并且采用的是最佳实践？

谢谢！

干杯， 詹姆斯

解决方法

请注意，JavaScript中async一词有两种截然不同的用法。第一个是异步功能的一般概念，该功能可以用任何语言实现，并广泛用于具有各种设计模式（回调，promise，async/await）的javascript中。第二个是async关键字，它用于允许使用await，并且仅在promise上起作用（在回调中不起作用）。

您似乎混淆了两者。因此，我现在建议人们在引用异步函数时不要使用“异步”一词，而应仅使用它来引用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);
      }
  })

带有承诺但没有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)
            }
        });
    });
}