javascript - ReactJS 的 render() 中 setInterval 计时器无法正常工作

Question

我在 React 组件中有一个倒计时器，计时 10 秒。如果在这 10 秒内组件接收到后端数据，则计时器将停止。如果没有，就会倒数到0，此时刷新页面，再次倒数，以此类推，直到接收到数据。我的代码如下:

constructor() {
    ...
    this.counter = 10;
    ...
}

render() {
    const interval = setInterval(() => {
        const result = this.state.data;
        if (result) {
            this.setState({
                resultsReceived: true
            });
            clearInterval(interval);
        } else {
            this.setState({
                resultsReceived: false
            });

            this.counter = this.counter - 1;
            if (this.counter === 0) {
                window.location.reload();
            }
        }
    }, 1000);

问题是，计时器似乎并没有每秒递减。相反，行为是不稳定的 - 从 10 到可能 7，速度非常快，然后它可能会挂起，甚至在 0 之后继续，即负数。页面正在重新加载。这段代码是错误的，还是与 React 状态相关的问题？任何帮助都会很棒!

更新:

我将代码更改为:

constructor() {
    this.state = {
        ...
        interval: '',
        counter: 10
    };
}

componentWillMount() {
    $.ajax({
        url: this.props.api,
        datatype: 'json',
        cache: false,
        success: (data) => {
            this.setState({ data });
        }
    });
    const interval = setInterval(() => {
        const results = this.state.data;
        if (results) {
            this.setState({
                resultsReceived: true
            });
            clearInterval(this.state.interval);
        } else {
            this.setState({
                resultsReceived: false
            });
            let counter;
            counter = this.state.counter - 1;
            this.setState({ counter });
            if (this.state.counter === 0) {
                window.location.reload();
            }
        }
    }, 1000);
    this.setState({ interval });
}

componentWillUnmount() {
    clearInterval(this.state.interval);  
}

它比以前更好，因为计时器第一次工作正常 - 每 1 秒从 10 到 0。然而，即使 10 秒后数据还没有加载，计时器又从 10 开始，到达 9 然后停止，直到数据最终加载。页面也变得非常慢。

Answer 1

我不确定为什么会发生这种情况，但我确信您的代码有些非常奇怪。

1. 首先，将时间间隔保持在状态是非常奇怪的。相反，请将其保留在组件实例中，以便您可以轻松清除它

2. 不要设置数据并间隔检查。您可以在 api 请求完成后释放间隔并将数据设置为状态。

3. 在组件被关闭之前始终清除间隔。

这就是我的建议。

constructor() {
  ...
  this.interval = null
  this.state = { counter: 10 }
  ...
}

componentWillMount() {
  $.ajax({
    url: this.props.api,
    datatype: 'json',
    cache: false,
    success: (data) => {
      this.setState({ data })
      clearInterval(this.interval);
    },
  });

  this.interval = setInterval(() => {
    if (counter <= 0) {
      clearInterval(this.interval);
      location.reload();
      return;
    }

    this.setState({
      counter: this.state.counter - 1,
    });
  }, 1000);
}

componentWillUnmount() {
  clearInterval(this.interval)
}

实际上，如果你不使用interval和setTimeout，如果你的state.counter存在只是为了检查已经过去了多少秒，那就更好了。

constructor() {
  ...
  this.timeout = null
  ...
}

componentWillMount() {
  $.ajax({
    url: this.props.api,
    datatype: 'json',
    cache: false,
    success: (data) => {
      this.setState({ data })
      clearTimeout(this.timeout);
    }
  });

  this.timeout = setTimeout(() => {
    clearTimeout(this.timeout);
    location.reload();
  }, 1000 * 10);
}

componentWillUnmount() {
  clearTimeout(this.timeout)
}

setInterval 很昂贵。不要将其用于此类工作。