ReactJS setState很慢

时间:2014-11-27 18:18:26

标签: reactjs

我正在尝试使用ReactJS,但遇到困难时将其合并到我的表单中。我正在ReactJS中构建一个自动建议表单。在我的onChangeQuery中,我正在设置状态,然后调用AJAX函数从我的服务器端代码中获取建议;但是,我注意到了 loadGroupsFromServer函数未获得最新状态...这是一个关键的中风太慢。我知道setState不是即时的,但为什么我会将setState用于我的表格?使用refs更好吗?最好从e.target.value获得价值?

帮助!

  var GroupForm = React.createClass({
    getInitialState: function() {
      return {query: "", groups: []}
    },
    componentDidMount: function () {
      this.loadGroupsFromServer();
    },
    loadGroupsFromServer: function () {
      $.ajax({
        type: "GET",
        url: this.props.url,
        data: {q: this.state.query},
        dataType: 'json',
        success: function (groups) {
          this.setState({groups: groups});
        }.bind(this),
        error: function (xhr, status, err) {
          console.error(this.props.url, status, err.toString());
        }.bind(this)
      });
    },
    onChangeQuery: function(e) {
      this.setState({query: e.target.value})
      this.loadGroupsFromServer();
    },
    render: function() {
      return (
        <div>
          <form class="form form-horizontal">
            <div class="col-lg-12">
              <input type="text" className="form-control" value={this.state.query} placeholder="Search for groups" onChange={this.onChangeQuery} />
            </div>
          </form>
          <GroupList groups={this.state.groups} />
        </div>
      )
    }
  })

5 个答案:

答案 0 :(得分:43)

setState会进行回调,以确保在this.loadGroupsFromQuery()更新后才会调用this.state

this.setState({...}, function() {
  this.loadGroupsFromQuery();
});

答案 1 :(得分:11)

setState最佳实践

<强> ES5

this.setState(function(state, props){
   return {
      query: e.target.value
   }
}, function()=>{
   //after callback 
});

<强> ES6

 this.setState((state, props) => ({
    query: e.target.value
 }), ()=>{
   //after callback 
 });

答案 2 :(得分:6)

loadGroupsFromServer应该采用查询参数。

componentDidMount: function () {
  this.loadGroupsFromServer(this.state.query);
},
loadGroupsFromServer: function (query) {
  $.ajax({
    /* ... */
    data: {q: query},
    /* ... */
  });
},
onChangeQuery: function(e) {
  this.setState({query: e.target.value})
  this.loadGroupsFromServer(e.target.value);
},

您通常应该尝试减少对状态的访问,因为它为您提供了更大的灵活性。

根据具体情况,您可能希望在执行某些操作之前等待组件更新:

onChangeQuery: function(e) {
  this.setState({query: e.target.value}, function(){
      this.loadGroupsFromServer(this.state.query)
  }.bind(this));
},

或者你可以像这样重写它,在我看来这更干净。

onChangeQuery: function(e) {
  this.setState({query: e.target.value});
},
componentWillUpdate: function(nextProps, nextState){
  // optionally perform better heuristics, but this prevents them pressing space
  // from triggering an update
  if (nextState.query.trim() !== this.state.query.trim()) {
      this.loadGroupsFromServer(nextState.query);
  }
}

答案 3 :(得分:4)

单字符关闭问题的根本原因是您在设置之前正在读取状态。 setState很快,但它不是即时和同步的。以前的答案为您提供了几种重构方法。

答案 4 :(得分:1)

setState并不慢,但它是异步的。因此,根据上面的答案,它将在您的状态设置时同步工作。

根据最佳实践状态,应该在render方法的外部设置。