web-dev-qa-db-ja.com

Reactフックを使用して、クリック時にIntervalをクリアするにはどうすればよいですか？

フックに反応するようにコードをリファクタリングしようとしていますが、正しく実行しているかどうかわかりません。 setInterval/setTimoutコードをコピーしてフックに貼り付けようとしましたが、意図したとおりに機能しませんでした。さまざまなことを試した後、それを機能させることができましたが、これが最善の方法かどうかはわかりません。

UseEffectを使用してアンマウントの間隔をクリアできることはわかっていますが、アンマウントする前にクリアしたいと思います。

次のグッドプラクティスはありますか？そうでない場合は、マウントを解除する前にsetInterval/setTimoutをクリアするためのより良い方法は何ですか？

ありがとう、

seTimeout

import { useState, useEffect } from 'react';

let timer = null;

const useTimeout = () => {
    const [count, setCount] = useState(0);
    const [timerOn, setTimerOn] = useState(false);

    useEffect(() => {
        if (timerOn) {
            console.log("timerOn ", timerOn);

            timer = setInterval(() => {
                setCount((prev) => prev + 1)
            }, 1000);

        } else {
            console.log("timerOn ", timerOn);
            clearInterval(timer);
            setCount(0);
        }
    return () => {
        clearInterval(timer);
    }
    }, [timerOn])

    return [count, setCount, setTimerOn];
}

export default useTimeout;

コンポーネント

import React from 'react';
import useTimeout from './useTimeout';

const UseStateExample = () => {
    const [count, setCount, setTimerOn] = useTimeout()
    return (
        <div>
            <h2>Notes:</h2>
            <p>New function are created on each render</p>
            <br />
            <h2>count = {count}</h2>
            <button onClick={() => setCount(prev => prev + 1)}>Increment</button>
            <br />
            <button onClick={() => setCount(prev => prev - 1)}>Decrement</button>
            <br />
            <button onClick={() => setTimerOn(true)}>Set Interval</button>
            <br />
            <button onClick={() => setTimerOn(false)}>Stop Interval</button>
            <br />

        </div>
    );
}

export default UseStateExample;
reactjsreact-hooks
4
Ar-51

---追加@ 2019-02-11 15:58 ---

HooksAPIでsetIntervalを使用するのに適したパターン：

https://overreacted.io/making-setinterval-declarative-with-react-hooks/

---起源の答え---

いくつかの問題：

  1. モジュールのグローバルスコープで非定数変数を使用しないでください。このモジュールの2つのインスタンスを1つのページで使用する場合、それらはそれらのグローバル変数を共有します。

  2. timerOnがtrueからfalseに変わると、return関数が実行されるため、「else」ブランチのタイマーをクリアする必要はありません。

私の考えのより良い方法：

import { useState, useEffect } from 'react';

export default (handler, interval) => {
  const [intervalId, setIntervalId] = useState();
  useEffect(() => {
    const id = setInterval(handler, interval);
    setIntervalId(id);
    return () => clearInterval(id);
  }, []);
  return () => clearInterval(intervalId);
};

ここで実行例：

https://codesandbox.io/embed/52o442wq8l?codemirror=1

1
李元秋

この例では、いくつか追加します...

  1. タイムアウトのオン/オフスイッチ（「実行中」の引数）。完全にオンまたはオフに切り替えます。

  2. いつでもタイムアウトを0に戻すことができるリセット機能：

    実行中に呼び出されると、実行は継続されますが、0に戻ります。実行されていないときに呼び出されると、開始されます。

const useTimeout = (callback, delay, running = true) => {
  // save id in a ref so we make sure we're always clearing the latest timeout
  const timeoutId = useRef('');

  // save callback as a ref so we can update the timeout callback without resetting it
  const savedCallback = useRef();
  useEffect(
    () => {
      savedCallback.current = callback;
    },
    [callback],
  );

  // clear the timeout and start a new one, updating the timeoutId ref
  const reset = useCallback(
    () => {
      clearTimeout(timeoutId.current);

      const id = setTimeout(savedCallback.current, delay);
      timeoutId.current = id;
    },
    [delay],
  );

  // keep the timeout dynamic by resetting it whenever its' deps change
  useEffect(
    () => {
      if (running && delay !== null) {
        reset();

        return () => clearTimeout(timeoutId.current);
      }
    },
    [delay, running, reset],
  );

  return { reset };
};

したがって、上記の例では、次のように使用できます...

const UseStateExample = ({delay}) => {
    // count logic
    const initCount = 0
    const [count, setCount] = useState(initCount)
    
    const incrementCount = () => setCount(prev => prev + 1)
    const decrementCount = () => setCount(prev => prev - 1)
    const resetCount = () => setCount(initCount)

    // timer logic
    const [timerOn, setTimerOn] = useState(false)
    const {reset} = useTimeout(incrementCount, delay, timerOn)

    const startTimer = () => setTimerOn(true)
    const stopTimer = () => setTimerOn(false)
    
    return (
        <div>
            <h2>Notes:</h2>
            <p>New function are created on each render</p>
            <br />
            <h2>count = {count}</h2>
            <button onClick={incrementCount}>Increment</button>
            <br />
            <button onClick={decrementCount}>Decrement</button>
            <br />
            <button onClick={startTimer}>Set Interval</button>
            <br />
            <button onClick={stopTimer}>Stop Interval</button>
            <br />
            <button onClick={reset}>Start Interval Again</button>
            <br />

        </div>
    );
}
0
Matt Wills