javascript - await async - ESLint 中的竞争条件错误 require-atomic-updates

Question

下面的代码在使用 ESLint 检查时会导致竞争条件:

  let match = false

  try {
    match = await something()
  } catch (err) {
    // do something
  }
  if (match === false) {
    // do something
  }

编写这段代码的更好方法是什么？

编辑:

  let request = ctx.request.body || {}
  let password = request.password
  let match = false

  try {
    match = await bcrypt.compare(password, 'xxxxx')
  } catch (err) {
    ctx.throw(401, err)
  }
  if (match === false) {
    ctx.throw(401, 'invalid password')
  }

  ctx.body = {
    message: 'logged in ok'
  }

来自 ESLint 的错误:

Possible race condition: ctx.body might be reassigned based on an outdated value of ctx.body require-atomic-updates

Answer 1

您可以安全地忽略警告:)

ESLint 旨在捕捉这样的事情:

 let value = 0;

async function race() {
  value += await Promise.resolve(1);
  console.log(value);
}

race(); race();

在这种情况下，race memoizes value 在堆栈上，await 一个 tick，然后写回 value。当其他代码同时运行时，value 可能已更改，然后更新可能会关闭……这不是原子的。

然而，在您的情况下，您从 ctx.request.body 读取并写入 ctx.body，因此没有非原子更新。此外，可能没有其他中间件同时访问相同的 ctx，因此不可能有任何并发​​修改。因此，在您的情况下，这是一个误报，甚至怀疑这在任何方面都是积极的(它可能是 ESLint 中的错误)。