在React.useEffect挂钩中未调用clearInterval

问题描述

我的React游戏有一个<Clock/>组件来跟踪时间。

暂停游戏后计时器应该停止。

我正在使用Redux来管理播放/暂停状态以及经过的时间。

const initialState = { inProgress: false,timeElapsed: 0 }

inProgress状态由另一个组件上的按钮处理，该组件调度一个操作以更新存储（仅用于inProgress值）。

<Clock/>组件在其timeElapsed钩中以useEffect递增setInterval。尚不清楚。

import React from 'react';
import { connect } from 'react-redux';

const Clock = ({ dispatch,inProgress,ticksElapsed }) => {

    React.useEffect(() => {

        const progresstimer = setInterval(function(){
            inProgress ? dispatch({ type: "CLOCK_RUN" }) : clearInterval(progresstimer);
        },1000)

    },[inProgress]);

    return (
        <></>
    )
};

let mapStatetoProps = ( state ) => {
    let { inProgress,ticksElapsed } = state.gameState;
    return { inProgress,ticksElapsed };
}

export default connect(
    mapStatetoProps,null,)(Clock);

在setInterval内部，当inProgress为false时，我希望clearInterval(progresstimer)停止计时。

还有另一个问题，在[inProgress]钩中遗漏useEffect会导致计时器以不可思议的速率递增，导致应用崩溃。

谢谢。

解决方法

inProgress是传递给setInterval的函数的stale closure。

您可以通过清除清除功能中的间隔来解决它：

const Clock = ({ dispatch,inProgress,ticksElapsed }) => {
  React.useEffect(() => {
    const progressTimer = setInterval(function () {
      inProgress && dispatch({ type: 'CLOCK_RUN' });
    },500);
    return () =>
      //inprogress is stale so when it WAS true
      //  it must now be false for the cleanup to
      //  be called
      inProgress && clearInterval(progressTimer);
  },[dispatch,inProgress]);

  return <h1>{ticksElapsed}</h1>;
};

const App = () => {
  const [inProgress,setInProgress] = React.useState(false);
  const [ticksElapsed,setTicksElapsed] = React.useState(0);
  const dispatch = React.useCallback(
    () => setTicksElapsed((t) => t + 1),[]
  );
  return (
    <div>
      <button onClick={() => setInProgress((p) => !p)}>
        {inProgress ? 'stop' : 'start'}
      </button>
      <Clock
        inProgress={inProgress}
        dispatch={dispatch}
        ticksElapsed={ticksElapsed}
      />
    </div>
  );
};
ReactDOM.render(<App />,document.getElementById('root'));

<script src="https://cdnjs.cloudflare.com/ajax/libs/react/16.8.4/umd/react.production.min.js"></script>
<script src="https://cdnjs.cloudflare.com/ajax/libs/react-dom/16.8.4/umd/react-dom.production.min.js"></script>
<div id="root"></div>