驯服野兽：我如何重构凌乱的 React 组件-js教程-PHP中文网

驯服野兽：我如何重构凌乱的 React 组件

WBOY

发布： 2024-08-19 17:13:33

原创

1134 人浏览过

Taming the Beast: How I Refactored a Messy React Component

我们都去过那里。你打开几个月前编写的 React 组件，感觉就像你正在看一个匆忙的人编写的代码 - 因为你可能很匆忙。截止日期迫在眉睫，功能需要交付。快进到今天，是时候重构那个混乱的组件了。

所以，这就是我解决这个问题的方法。

最初的恐怖

我注意到的第一件事是该组件变得太大了。它尝试做所有事情，比如处理状态、进行 API 调用、管理复杂的 UI 逻辑，甚至直接应用样式。这是一个超过 540 行的文件，通读它就像在没有地图的丛林中漫步一样。

第一步是接受现实：这段代码不再可维护。如果我（写这篇文章的人）几乎无法理解正在发生的事情，那么其他人就没有机会。所以，我决定把它分解。

分解它

我首先确定了组件的不同职责。共有三个明确的区域：

状态管理：处理组件的状态与 UI 逻辑交织在一起。
API 调用：获取数据并处理加载状态。
渲染 UI：以稍微复杂的 UI 结构显示数据。
这些职责中的每一个都需要分开。

提取状态和 API 逻辑的钩子

我做的第一件事是将状态管理和 API 逻辑提取到自定义挂钩中。这不仅清理了组件，还使在其他地方测试和重用逻辑变得更加容易。

这里提到一些代码（不是原始代码）：

function useDataFetching(apiEndpoint) {
  const [data, setData] = useState(null);
  const [loading, setLoading] = useState(true);
  const [error, setError] = useState(null);

  useEffect(() => {
    async function fetchData() {
      try {
        let response = await fetch(apiEndpoint);
        let result = await response.json();
        setData(result);
      } catch (err) {
        setError(err);
      } finally {
        setLoading(false);
      }
    }

    fetchData();
  }, [apiEndpoint]);

  return { data, loading, error };
}

登录后复制

通过useDataFetching，我提取了API调用逻辑并处理了加载和错误状态。现在，组件只需要调用这个钩子并获取必要的数据，干净简单。

简化 UI 逻辑
接下来，我查看了渲染逻辑。以前，我在渲染函数中检查加载、错误和数据，这使得它很难理解。我把这个逻辑分成小的、独立的函数，就像这样（当然不是原来的；）

function renderLoading() {
  return <p>Loading...</p>;
}

function renderError(error) {
  return <p>Error: {error.message}</p>;
}

function renderData(data) {
  return <div>{/* Complex UI logic here */}</div>;
}
//After that, component is ni much pretty shape

function MyComponent() {
  const { data, loading, error } = useDataFetching('/api/data-endpoint');

  if (loading) return renderLoading();
  if (error) return renderError(error);
  if (data) return renderData(data);

  return null;
}

登录后复制