React - 未安装组件上的 setState()

IT技术 javascript ajax reactjs state
2021-03-03 17:39:42

在我的react组件中，我试图在 ajax 请求正在进行时实现一个简单的微调器 - 我使用状态来存储加载状态。

出于某种原因，我的 React 组件中的这段代码会引发此错误

只能更新已安装或正在安装的组件。这通常意味着您在未安装的组件上调用了 setState()。这是一个无操作。请检查未定义组件的代码。

如果我摆脱了第一个 setState 调用，错误就会消失。

constructor(props) {
  super(props);
  this.loadSearches = this.loadSearches.bind(this);

  this.state = {
    loading: false
  }
}

loadSearches() {

  this.setState({
    loading: true,
    searches: []
  });

  console.log('Loading Searches..');

  $.ajax({
    url: this.props.source + '?projectId=' + this.props.projectId,
    dataType: 'json',
    crossDomain: true,
    success: function(data) {
      this.setState({
        loading: false
      });
    }.bind(this),
    error: function(xhr, status, err) {
      console.error(this.props.url, status, err.toString());
      this.setState({
        loading: false
      });
    }.bind(this)
  });
}

componentDidMount() {
  setInterval(this.loadSearches, this.props.pollInterval);
}

render() {

    let searches = this.state.searches || [];


    return (<div>
          <Table striped bordered condensed hover>
          <thead>
            <tr>
              <th>Name</th>
              <th>Submit Date</th>
              <th>Dataset &amp; Datatype</th>
              <th>Results</th>
              <th>Last Downloaded</th>
            </tr>
          </thead>
          {
          searches.map(function(search) {

                let createdDate = moment(search.createdDate, 'X').format("YYYY-MM-DD");
                let downloadedDate = moment(search.downloadedDate, 'X').format("YYYY-MM-DD");
                let records = 0;
                let status = search.status ? search.status.toLowerCase() : ''

                return (
                <tbody key={search.id}>
                  <tr>
                    <td>{search.name}</td>
                    <td>{createdDate}</td>
                    <td>{search.dataset}</td>
                    <td>{records}</td>
                    <td>{downloadedDate}</td>
                  </tr>
                </tbody>
              );
          }
          </Table >
          </div>
      );
  }

问题是为什么当组件应该已经安装时我会收到这个错误（因为它是从 componentDidMount 调用的）我认为一旦安装了组件就可以安全地设置状态？

6个回答

没有看到渲染功能有点困难。虽然已经可以发现您应该做的事情，但每次使用间隔时，您都必须在卸载时清除它。所以：

componentDidMount() {
    this.loadInterval = setInterval(this.loadSearches, this.props.pollInterval);
}

componentWillUnmount () {
    this.loadInterval && clearInterval(this.loadInterval);
    this.loadInterval = false;
}

由于卸载后可能仍会调用那些成功和错误回调，因此您可以使用间隔变量来检查它是否已安装。

this.loadInterval && this.setState({
    loading: false
});

希望这会有所帮助，如果这不起作用，请提供渲染功能。

干杯

或者干脆： componentWillUnmount() { clearInterval(this.loadInterval); }
2021-04-30 17:39:42
@GregHerbowicz 如果您使用计时器卸载和安装组件，即使您进行简单的清除，它仍然可以被触发。
2021-05-16 17:39:42
布鲁诺，你不能只是测试“这个”上下文的存在.. ala this && this.setState .....
2021-05-20 17:39:42

问题是为什么当组件应该已经安装时我会收到这个错误（因为它是从 componentDidMount 调用的）我认为一旦安装了组件就可以安全地设置状态？

它不是从 调用的componentDidMount。您会componentDidMount生成一个回调函数，该函数将在计时器处理程序的堆栈中执行，而不是在componentDidMount. 显然，当您的回调 ( this.loadSearches) 执行时，组件已卸载。

因此，接受的答案将保护您。如果您使用的其他异步 API 不允许您取消异步函数（已提交给某个处理程序），您可以执行以下操作：

if (this.isMounted())
     this.setState(...

这将消除您在所有情况下报告的错误消息，尽管它确实让人感觉像是在掩饰一切，特别是如果您的 API 提供了取消功能（与setInterval一样clearInterval）。

isMounted是 facebook 建议不要使用的反模式：facebook.github.io/react/blog/2015/12/16/...
2021-04-26 17:39:42
是的，我确实说“感觉就像在地毯下扫东西”。
2021-05-14 17:39:42

对于需要另一种选择的人，ref 属性的回调方法可以是一种解决方法。handleRef 的参数是对 div DOM 元素的引用。

有关 refs 和 DOM 的详细信息：https : //facebook.github.io/react/docs/refs-and-the-dom.html

handleRef = (divElement) => {
 if(divElement){
  //set state here
 }
}

render(){
 return (
  <div ref={this.handleRef}>
  </div>
 )
}
使用 ref 有效地“isMounted”与仅使用 isMounted 完全相同，但不太清楚。isMounted 不是反模式，因为它的名字，而是因为它是一个反模式，用于保存对未安装组件的引用。
2021-05-07 17:39:42 
class myClass extends Component {
  _isMounted = false;

  constructor(props) {
    super(props);

    this.state = {
      data: [],
    };
  }

  componentDidMount() {
    this._isMounted = true;
    this._getData();
  }

  componentWillUnmount() {
    this._isMounted = false;
  }

  _getData() {
    axios.get('https://example.com')
      .then(data => {
        if (this._isMounted) {
          this.setState({ data })
        }
      });
  }


  render() {
    ...
  }
}
对于函数组件，我会使用 ref: const _isMounted = useRef(false); @Tamjid
2021-04-22 17:39:42
有没有办法为功能组件实现这一点？@john_per
2021-04-27 17:39:42

分享一个由react hooks启用的解决方案。

React.useEffect(() => {
  let isSubscribed = true

  callApi(...)
    .catch(err => isSubscribed ? this.setState(...) : Promise.reject({ isSubscribed, ...err }))
    .then(res => isSubscribed ? this.setState(...) : Promise.reject({ isSubscribed }))
    .catch(({ isSubscribed, ...err }) => console.error('request cancelled:', !isSubscribed))

  return () => (isSubscribed = false)
}, [])

可以将相同的解决方案扩展到任何时候您想取消之前对 fetch id 更改的请求，否则多个进行中的请求之间会出现竞争条件（this.setState称为乱序）。

React.useEffect(() => {
  let isCancelled = false

  callApi(id).then(...).catch(...) // similar to above

  return () => (isCancelled = true)
}, [id])

这要归功于javascript 中的闭包。

总的来说，上面的想法接近于react doc 推荐的makeCancelable 方法，它明确指出

isMounted 是一个反模式

信用

https://juliangaramendy.dev/use-promise-subscription/

其它你可能感兴趣的问题
上一篇如何使用 querySelector 按名称选择输入元素？ 下一篇如何在javascript中将String转换为long？