11

我的代码有效，但我有一个最佳实践问题：我有一组处于状态的对象，用户交互一次将更改一个对象的值。据我所知，我不应该直接改变状态，我应该总是使用它setState。如果我想以任何代价避免这种情况，我将通过迭代深度克隆数组，并更改克隆。然后将状态设置为克隆。在我看来，避免改变我以后会改变的状态只会降低我的表现。

详细版本： this.state.data 是一个对象数组。它代表论坛中的主题列表，并且收藏按钮将切换，调用clickCollect()。由于我在状态中有一个数组，所以当我更改一项的 is_collected 属性时，我需要创建一个数组的副本以使用，并且在更改为新值后，我可以将其设置为状态。 

var data = this.state.data.slice(0);
data[index].is_collected = !data[index].is_collected;
this.setState({data: data});

var data = this.state.data：这会将指针复制到数组，并且 push()、shift() 等会直接改变状态。两者data都会this.state.data受到影响。

var data = this.state.data.slice(0)：这会进行浅克隆，push 和 shift 不会改变状态，但在我的克隆中，我仍然有指向状态数组元素的指针。所以如果我改变data[0].is_collected，this.state.data[0].is_collected也会改变。这发生在我打电话之前setState()。

通常我应该这样做：

var data = []; 
for (var i in this.state.data) {
    data.push(this.state.data[i]); 
}

然后我更改索引处的值，当它为假时将其设置为真，当它为真时将其设置为假：

data[index].is_collected = !data[index].is_collected;

并改变状态：

this.setState({data: data});

考虑到我的数组比较大或者非常大，我猜这个迭代会降低我的APP的性能。如果我出于任何原因知道这是正确的方法，我会支付这笔费用。然而，在这个函数clickCollect（在所有情况下，新值都会进入状态。实际上，我setState只要求 UI 再次呈现。所以问题是：

  1. 在这种情况下我是否必须创建深层克隆？( for var i in ...)
  2. .slice(0)如果不是，如果我的数组包含对象，那么进行浅克隆 () 是否有意义？正在对数组内部的对象进行更改，因此浅克隆仍然会更改我的状态，就像副本 ( data = this.state.data) 会做的那样。

为了简单起见，我的代码被简化了，API 调用也被删掉了。

这是一个初学者的问题，因此也欢迎采用完全不同的方法。或其他问答链接。

import React from 'react';

var ForumList = React.createClass({
  render: function() {
      return <div className="section-inner">
        {this.state.data.map(this.eachBox)}
      </div>
  },
  eachBox: function(box, i) {
    return <div key={i} className="box-door">
        <div className={"favorite " + (box.is_collected ? "on" : "off")} onTouchStart={this.clickCollect.bind(null, i)}>
          {box.id}
        </div>
    </div>
  },
  getInitialState: function() {
    return {data: [
      {
        id: 47,
        is_collected: false
      },
      {
        id: 23,
        is_collected: false
      },
      {
        id: 5,
        is_collected: true
      }
    ]};
  },
  clickCollect: function(index) {
    var data = this.state.data.slice(0);
    data[index].is_collected = !data[index].is_collected;
    this.setState({data: data});
  }
});

module.exports = ForumList;
4

4 回答 4

3

就我个人而言，我并不总是遵守规则，如果你真的明白你想要做什么，那么我认为这不是问题。 

var data = this.state.data.slice(0);
data[index].is_collected = !data[index].is_collected;
this.setState({data: data});

在这种情况下，像这样改变状态并setState再次调用就可以了

this.state.data[index].is_collected = !this.state.data[index].is_collected;
this.setState({data: this.state.data});

你应该避免改变你的状态的原因是，如果你有一个引用，并且多次this.state.data调用，你可能会丢失你的数据：setState

const myData = this.state.data
myData[0] = 'foo'
this.setState({ data: myData })
// do something...
// ...
const someNewData = someFunc()
this.setState({ data: someNewData })

myData[1] = 'bar' // myData is still referencing to the old state
this.setState({ data: myData }) // you lose everything of `someNewData`

如果你真的关心这个，就去immutable.js

于 2017-02-06T06:32:47.760 回答
2

静音状态直接破坏了 React 数据流的主要原则（它是单向的），使您的应用程序非常脆弱，并且基本上忽略了整个组件生命周期。

因此，虽然没有什么能真正阻止你在没有 setState({}) 的情况下改变组件状态，但如果你想真正利用 React，就必须不惜一切代价避免这种情况，否则你将超越库的核心功能之一。

于 2017-02-06T05:30:15.443 回答
1

如果您想遵循反应最佳实践，您应该在更改任何属性时对所有数组进行浅拷贝。请查看“不可变”库的实现。

但是，根据我的经验和我的观点，setState如果您有“shouldCompomenentUpdate”实现，则应该调用方法。如果你认为你的浅拷贝会消耗更多的资源，那么反应虚拟 dom 检查，你可以这样做：

this.state.data[0].property = !this.state.data[0].property;
this.forceUpdate();
于 2017-02-06T06:58:51.190 回答
0

如果我理解你的问题是正确的，你有一个对象数组，当数组中单个对象的属性发生变化时，

  1. 创建数组的深层克隆并传递给 setState
  2. 创建一个浅克隆并传递给 setState

我刚刚检查了redux 示例待办事项应用程序，如果对象的单个属性发生更改，您必须创建该单个对象的新副本，而不是整个数组。我建议您阅读redux并在可能的情况下使用来管理您的应用程序的状态。

于 2017-02-06T06:38:12.560 回答