React切换列出竞争条件?

时间:2018-04-10 01:18:19

标签: javascript reactjs race-condition deep-copy

我正在尝试实现一个队列,其中用户可以将项目从一个列表切换到另一个列表。即从“可用”到“与客户端”,其中队列的状态保存在根React组件中,如下所示:

this.state = {
  queue: {
    available: [{ name: "example", id: 1 }, ...],
    withClient: [],
    unavailable: []   
  }
};

然而我的移动功能被破坏了:

move(id, from, to) {
  let newQueue = deepCopy(this.state.queue);
  console.log("NEW QUEUE FROM MOVE", newQueue); // { [from]: [], [to]: [undefined] }
  console.log("QUEUE IN STATE FROM MOVE", this.state.queue); // { [from]: [{...}], [to]: [] }
  let temp = newQueue[from].find(x => x.id === id);
  newQueue[from] = this.state.queue[from].filter(x =>
    x.id !== id
  );
  newQueue[to] = this.state.queue[to].concat(temp);
  this.setState({
    queue: newQueue
  });
}

我期待2个console.logs是相同的。这里似乎发生了某种竞争状况,我不理解。这会导致错误Cannot read property 'id' of undefined

现在,从“可用”列表中的每个项目中包含的HelpedButton组件触发移动的唯一方法是传递道具:

class HelpedButton extends React.Component {
  constructor() {
    super();
    this.clickWrapper = this.clickWrapper.bind(this);
  }

  clickWrapper() {
    console.log("I have id right?", this.props.id); //outputs as expected
    this.props.move(this.props.id, "available", "withClient");
  }

  render() {
    return (
      <span style={this.props.style}>
        <button onClick={this.clickWrapper}>
          <strong>Helped A Customer</strong>
        </button>
      </span>
    );
  }
}

export default HelpedButton;

我认为deepCopy函数没有任何问题,但是这里是来自node_modules的导入文件:

"use strict";

function deepCopy(obj) {
  return JSON.parse(JSON.stringify(obj));
}

module.exports = deepCopy;

1 个答案:

答案 0 :(得分:1)

react documentation建议的依赖于先前状态的setState的方法是使用看起来像setState((prevState,prevProp)=>{})的更新程序表单。使用此方法,您的move函数将如下所示。

move(id, from, to) {

  let temp = newQueue[from].find(x => x === x.id);
  newQueue[from] = this.state.queue[from].filter(x =>
    x.id !== id
  );
  newQueue[to] = this.state.queue[to].concat(temp);
  this.setState((prevState)=>{
    queue: {
      ...prevState.queue,
      [from]: prevState.queue[from](o => x.id !== id),
      [to]: [from]: prevState.queue[from].concat(prevState.queue[from].find(x => x === x.id))
    }
  });
}

我相信您看到不同输出的原因是console.log输出一个实时对象,这意味着如果您运行console.log(obj)并稍后更改obj参数,则显示的参数是更改。尝试使用console.log("obj: " + JSON.strignify(obj))进行调试。
以下是关于在依赖react docs

中的先前状态时应调用setState的异步方法的原因