为什么在下面的代码中它是竞态条件？

Question

我有以下代码ES Lint标记作为竞态条件：

const fillPersonPositions = async (person) => {
    person.positions = await getPositions(person.id);
};

完全错误：

Possible race condition: `person.positions` might be reassigned based on an outdated value of `person.positions`.eslint(require-atomic-updates)

如果我将"person.id“改为与person对象无关的值，比如"null"，错误就会消失。这里的问题是什么?如何避免它？(我的意思是修复代码，如果这是一个真正的竞态条件问题，或者告诉林特，如果不是问题)。

Answer 1

如果您多次调用fillPersonPositions，同时人员的id发生更改，则此代码可能会出现问题。例如：

person.id = 1;
const p1 = fillPersonPositions(person);
person.id = 3;
const p2 = fillPersonPositions(person);
await Promise.all([p1, p2]);

在这里，不能保证首先解析第一个fillPersonPositions，所以最终您可能会得到ID为3的person，但是对应于ID 1的positions属性。

当然，上面的内容是人为设计的，但ESLint并不知道这一点。

一种可能的修复方法是避免突变，而是返回(并使用)一个新对象：

const fillPersonPositions = async (person) => {
    const positions = await getPositions(person.id);
    return { ...person, positions };
};

(为了更容易理解的代码，在可能的情况下避免变化通常是一个好主意-出于类似的原因，当你有选择的时候，const是比let更好的选择)

Answer 2

代码被标记是因为person不是原子的，因为理论上它可以在异步操作期间在其他地方更改。

您可以通过更改.eslintrc.json文件中的以下条目来启用此规则：

"require-atomic-updates": "off"