[URH-14] useTimer 신규

[jeongbaebang]

👾 Pull Request

• 티켓: useTimer 신규

• 상태: 신규

1️⃣ Spec

• timer 기능을 지원하는 hook
• 전달된 콜백에 대한 시작과 취소기능

2️⃣ 변경 사항

• 변경 사항 없음

3️⃣ 예시 코드

const Component = () => {
  const [count, setCount] = useState(0);
  const { start, cancel } = useTimer(() => {
    setCount(1234);
  }, delay);

  return (
    <div>
      <p aria-label="count-display">{count}</p>
      <button onClick={start}>start</button>
      <button onClick={cancel}>cancel</button>
    </div>
  );
};

4️⃣ 관련 문서 (선택 사항)

• 없음

[Choozii]

🥺 갬둥..

[Choozii]

🤩 천재만재

[Choozii]

Suggested change

	(async () => {
	cancelTokenRef.current = { isCancelled: false };
	timerRef.current = delayExecution(ms);
	await timerRef.current.start(cancelTokenRef.current);
	callbackRef.current();
	})();
	}, [ms]);
	cancelTokenRef.current = { isCancelled: false };
	timerRef.current = delayExecution(ms);
	timerRef.current.start(cancelTokenRef.current).then(() =>{
	callbackRef.current();
	});

[Choozii]

[P4] 아니믄

cosnt startTimer = async () => {
clearActiveTimer(); // 기존 타이머 취소
      cancelTokenRef.current = { isCancelled: false };
      timerRef.current = delayExecution(ms);

      await timerRef.current.start(cancelTokenRef.current);
      callbackRef.current();
}

  const startHandler = useCallback(() => {
    startTimer();
    },[ms]);

이렇게는 어떠세요?
IIFE가 있는게 가독성이 안좋은거 같아서요!

[jeongbaebang]

짧은 IIFE는 오히려 쉽게 로직이 읽을 수 있을 거 같기도 한데 다른 분들 의견도 궁금하네요.

[bicochan]

useTimer가 머지되면 useDelayFlag에도 적용해 봐야겠어요!🔥