将函数放在 useEffect 中是一种好习惯吗？

Question

问题是：

假设我在组件中有相当广泛的逻辑。我应该将所有这些逻辑放在useEffects 中，还是应该将这些函数移到useEffect 之外并使用useCallback？

我找不到任何经过同行评审的方法来解决这个主题。

下面是我的意思的一个例子：

1. useEffect 方法中的函数

  useEffect(() => {
    const fun1 = () => {
      /**
       * 50 lines of code
       */
    }
    fun1()
  }, [var1, var2])

  useEffect(() => {
    const fun2 = () => {
      /**
       * 50 lines of code
       */
    }
    fun2()
  }, [var3, var4])

1. useEffect 方法之外的函数

  const fun1 = useCallback(() => {
    /**
     * 50 lines of code
     */
  }, [var1, var2])

  const fun2 = useCallback(() => {
    /**
     * 50 lines of code
     */
  }, [var3, var4])

  useEffect(() => {
    fun1()
  }, [fun1])

  useEffect(() => {
    fun2()
  }, [fun2])

让我们假设它更复杂。首选哪种方法？

Answer 1

有很多方法可以在effect hook 中重构closure。因为你还没有展示你的函数代码，所以考虑一下计数器组件的常见示例：

鉴于以下效用函数在范围内...

function isSingular (n: number): boolean {
  return Math.abs(n) === 1;
}

function Example (): React.ReactElement {
  const [count, setCount] = useState(0);
  const [message, setMessage] = useState('');

  useEffect(() => {
    const updateMessage = () => {
      setMessage(`You clicked ${count} time${isSingular(count) ? '' : 's'}`);
    };
    updateMessage();
  }, [count, setMessage]);

  return (
    <div>
      <div>{message}</div>
      <button onClick={() => setCount(n => n + 1)}>Increment</button>
    </div>
  );
}

该组件使用两个状态变量，count 和 message（根据 count 计算得出）。闭包updateMessage 定义在效果挂钩的回调中，它封装了count 和setMessage。有很多方法可以重构它：

重要提示：无论您使用哪种方法，如果涉及使用dependency array，请确保依赖项列表正确且详尽。

---

您可以在效果挂钩之外定义闭包，并将其作为回调直接传递（这是您询问的那个）：

function Example (): React.ReactElement {
  const [count, setCount] = useState(0);
  const [message, setMessage] = useState('');

  const updateMessage = () => {
    setMessage(`You clicked ${count} time${isSingular(count) ? '' : 's'}`);
  };

  useEffect(updateMessage, [count, setMessage]);

  return // the same JSX...
}

<div id="root"></div><script src="https://unpkg.com/react@17.0.2/umd/react.development.js"></script><script src="https://unpkg.com/react-dom@17.0.2/umd/react-dom.development.js"></script><script src="https://unpkg.com/@babel/standalone@7.16.6/babel.min.js"></script><script>Babel.registerPreset('tsx', {presets: [[Babel.availablePresets['typescript'], {allExtensions: true, isTSX: true}]]});</script>
<script type="text/babel" data-type="module" data-presets="tsx,react">

// You'd use ESM:
// import ReactDOM from 'react-dom';
// import {default as React, useEffect, useState} from 'react';

// This snippet uses UMD:
const {useEffect, useState} = React;

function isSingular (n: number): boolean {
  return Math.abs(n) === 1;
}

function Example (): React.ReactElement {
  const [count, setCount] = useState(0);
  const [message, setMessage] = useState('');

  const updateMessage = () => {
    setMessage(`You clicked ${count} time${isSingular(count) ? '' : 's'}`);
  };

  useEffect(updateMessage, [count, setMessage]);

  return (
    <div>
      <div>{message}</div>
      <button onClick={() => setCount(n => n + 1)}>Increment</button>
    </div>
  );
}

ReactDOM.render(<Example />, document.getElementById('root'));

</script>

---

您可以将其定义为 impure function 而不是闭包：

function updateMessage (
  count: number,
  setString: React.Dispatch<React.SetStateAction<string>>,
): void {
  setString(`You clicked ${count} time${isSingular(count) ? '' : 's'}`);
};

function Example (): React.ReactElement {
  const [count, setCount] = useState(0);
  const [message, setMessage] = useState('');

  useEffect(() => updateMessage(count, setMessage), [count, setMessage]);

  return // the same JSX...
}

---

您可以将与message 状态相关的所有内容提取到custom hook：

function useMessage (count: number): string {
  return `You clicked ${count} time${isSingular(count) ? '' : 's'}`;
};

function Example (): React.ReactElement {
  const [count, setCount] = useState(0);
  const message = useMessage(count);

  return // the same JSX...
}

---

您可以使用memo hook:

function Example (): React.ReactElement {
  const [count, setCount] = useState(0);
  const message = useMemo(() => `You clicked ${count} time${isSingular(count) ? '' : 's'}`, [count]);

  return // the same JSX...
}

---

希望这能给您一些观点，然后您可以将其应用于您的组件代码。

Answer 2

假设我在组件中有相当广泛的逻辑。

您可能应该将逻辑从组件中移出并放入钩子中，因为这是钩子尝试solve 的主要问题之一。

除此之外，您的第二种方法引入了useCallback 的附加依赖项。你这样做有什么收获？不多。但是丢失了很多东西（更复杂的加上额外的依赖）。