在 Async/Await 下留下无用的同步代码是一种反模式吗？

编程问答 2022-06-01

问题描述

据我所知，await 背后的要点是“等待”对 promise 的已解析值采取行动，直到它作为微任务遇到，正如 Jake Archibald 所解释的 here。

This video by LLJS 表明 async-await 本质上是生成器运行器/解释器函数的语法糖，它产生等待的位置并将承诺的固定值传递给 .next() 方法。这意味着当发生等待时，运行者对 .next() 的执行将作为微任务排入队列。

实际上，await 下的所有代码只会在下一个微任务检查点执行。如果不需要承诺的等待值的代码位于其下方，这可能是一个问题，这正是异步 IIFE 的问题。

async function ping() {
  for (let i = 0; i < 5; i++) {
    let result = await Promise.resolve("ping");
    console.log(result);
  }

  console.log("Why am I even here?");
}
    
async function pong() {
  for (let i = 0; i < 5; i++) {
    let result = await Promise.resolve("pong");
    console.log(result);
  }

  console.log("I have nothing to do with any of this");
}
    
console.log("Let the games begin!");
ping();
pong();
console.log("Placeholder for code that is not related to ping pong");

在这个例子中，外部日志首先作为运行脚本任务的一部分被记录，然后按照它们在微任务队列中排队的顺序记录已解决的承诺的值。在这整个过程中，留在 for 循环下面的日志与循环无关，并且会被不必要地暂停，直到它们各自函数体中的最后一个微任务出队列。

这正是我们使用 async 函数作为 IIFE 时发生的情况。如果您在 await 下有打算同步执行的代码，则它必须不必要地等待，直到它上面的所有等待都已从微任务队列中检出。

如果有人盲目地将他们的整个快速路由包装在 async 函数中，我可以看到这是一个问题，他们将不必要地await 解决某些承诺，例如数据库操作、发送电子邮件、阅读文件等...，那么为什么人们仍然这样做？

app.post('/forgotPwd',async (req,res) => {
  const {email,username} = req.body;

  if (!email) {
    res.status(400).json({error: "No username entered"});
    return;
  }

  if (!username) {
    res.status(400).json({error: "No email entered"});
    return;
  }

  const db = client.db();
  
  const user = await db.collection("Users").findOne({username: username,"userInfo.email": email});

  if (!user) {
    res.status(400).json({error: "Account not found"});
    return;
  }

  const authToken = await getAuthToken({id: user._id.toHexString()},"15m");

  // Would probably send a more verbose email
  await sgMail.send({
    from: process.env.EMAIL,to: email,subject: 'Forgot Password',text: `Use this url to reset your password: http://localhost:5000/confirmation/passConf/${authToken}`,});

  res.json({error: ""});
});

解决方法

如果您希望 async 函数中的某些内容同步运行，请确保它位于函数中的第一个 await 之前。

那么为什么人们仍然这样做？

这对于 SO 来说可能是一个题外问题，因为它主要需要基于意见的答案，但很可能是 A) 因为他们不希望该代码运行直到上面的代码已经完成了，或者 B) 因为他们不理解 async 函数。

,

使用 async/await 的目的是让异步代码看起来是同步的，因为它更容易阅读。事实上，它是一个语法糖，在下面隐藏了回调地狱。您无需处理回调即可处理异步操作。

就您而言，如果您认为 for 循环之后的代码与等待的操作无关，则不应将其放在 await 之后。或者您应该重构代码，使其不使用 await（回调）。

至于人们为什么这样做的问题。好吧，你能说出为什么人们使用 .map() 代替 .forEach() 吗？或者你能说出他们为什么不处理异常吗？他们可能并不完全理解它，或者（正如 T.J. Crowder 提到的）他们确实希望代码在等待的操作之后运行。就这么简单。

async-awaitevent-loopjavascript