8

在 React 中更新有状态组件时，如果组件使用当前状态来更新新状态，这被认为是一种不好的做法。

例如，如果我有一个类来存储过滤器是否处于打开状态，那么这些用于更新状态的选项之一在性能方面是否比另一个更可取？

选项1：

class Container extends Component {
    state = {
        show: false
    }

    show = () => this.setState({ show: true })

    hide = () => this.setState({ show: false })

    render() {
        <ExternalComponent
            show={this.show}
            hide={this.hide}
        />
    }
}

选项 2：

class Container extends Component {
    state = {
        show: false
    }

    toggleVisibility = () => this.setState({ show: !this.state.show })

    render() {
        <ExternalComponent
            toggleVisibility={this.toggleVisibility}
        />
    }
}

选项 3：

class Container extends Component {
    state = {
        show: false
    }

    setShow = (newVal) => this.setState({ show: newVal })

    render() {
        <ExternalComponent
            setShow={this.setShow}
        />
    }
}
4

2 回答 2

9

组件访问自己的状态并没有错。只写状态不会非常有用！但是，在将组件状态或状态更改方法暴露给其他组件时，您应该非常小心。组件状态是内部的，只能通过经过深思熟虑的界面从外部触摸，以防止您的组件变得纠缠不清。

事实上，在 React 文档中有一个与您的示例 #2 类似的示例：

class Toggle extends React.Component {
  constructor(props) {
    super(props);
    this.state = {isToggleOn: true};

    // This binding is necessary to make `this` work in the callback
    this.handleClick = this.handleClick.bind(this);
  }

  handleClick() {
    this.setState(prevState => ({
      isToggleOn: !prevState.isToggleOn
    }));
  }

  render() {
    return (
      <button onClick={this.handleClick}>
        {this.state.isToggleOn ? 'ON' : 'OFF'}
      </button>
    );
  }
}

ReactDOM.render(
  <Toggle />,
  document.getElementById('root')
);

但是，请注意与您的示例有所不同。toggle 方法需要在构造函数中绑定，以确保这this意味着您所期望的含义。

如果包装组件是跟踪 child 可见性的一个组件ExternalComponent，那么与其将切换方法传递给子组件，我希望包装器呈现某种隐藏/显示功能，然后传递当前将子组件作为道具的可见性或选择性地渲染它（请注意，选择性渲染将导致整个子组件在再次启用时重新安装，这可能会很昂贵；您最好隐藏它而不是将其拆除并重新创建它）。这使得关注点的划分变得清晰：包装器知道可见性，子组件不需要知道如何或为什么做出该决定，也不需要接触包装器的内部状态。

于 2017-02-27T21:53:01.003 回答
1

使用当前状态的值来确定新的状态值并没有错。

选项 2 的代码更少，这对我很有吸引力。但是，有时我在使用第三方组件（例如 Semantic UI React 模态）时可能不得不使用选项 1，并且它具有我们必须定义的显示和隐藏处理程序。

选项 3 也可以；我会将它用于除此显示/隐藏之外的其他应用程序（事实上，几乎一直使用该应用程序，尤其是当您控制输入组件时）。

于 2017-02-27T23:25:14.697 回答