React Hooks useEffect,添加依赖项触发无限循环

时间:2019-06-11 12:08:12

标签: javascript reactjs react-hooks use-effect

除了useEffect之外,我还有一个props依赖项(setIsValid)。当我将此依赖关系添加到useEffect时,它会陷入无限循环。

调用子组件时的父项:

const setIsValid = (bool) => {
  const tmpStateCopy = Object.assign({}, state);
  tmpStateCopy.isValid = bool;

  setState(tmpStateCopy);
};

return <Child
  setIsValid={setIsValid}
/>

在子组件中:

const { setIsValid } = props;

const [state, setState] = useState({
  transformations: [],
  duplicateIndexes: []
});

const { transformations, duplicateIndexes } = state;

useEffect(() => {
  const invalids = transformations.find(x => (x.value === '' || x.replaceWith === ''));
  const hasDuplicates = duplicateIndexes.length > 0;
  const isValid = ((invalids === undefined) && (transformations.length > 0) && !hasDuplicates);

  setIsValid(isValid)

  console.log('got triggered');
}, [state]);

这样,代码可以工作,但我总是收到警告。

我想要的是,始终在状态中的值之一发生更改(转换/重复索引)时触发验证。

通过从道具中添加setIsValid()函数,它可以无限运行。

警告看起来像这样:

./src/components/UI/integrationBuilder/layoutElements/transformer/modules/ifModules/ifModule.js
  Line 103:  React Hook useEffect has missing dependencies: 'duplicateIndexes.length', 'setIsValid', and 'transformations'. Either include them or remove the dependency array  react-hooks/exhaustive-deps

我的问题是,如何在不收到此警告的情况下保持相同的逻辑?

1 个答案:

答案 0 :(得分:1)

因此,当状态更改时,您将调用效果。转换和重复索引已经被考虑。为避免警告,您可以在destructure内移动useEffect

const { setIsValid } = props;

const [state, setState] = useState({
  transformations: [],
  duplicateIndexes: []
});



useEffect(() => {
  const { transformations, duplicateIndexes } = state;
  const invalids = transformations.find(x => (x.value === '' || x.replaceWith === ''));
  const hasDuplicates = duplicateIndexes.length > 0;
  const isValid = ((invalids === undefined) && (transformations.length > 0) && !hasDuplicates);

  setIsValid(isValid)

  console.log('got triggered');
}, [state]);

还将setIsValid作为useEffect的依赖项,您也不能这样做,因为在每个渲染器上都会为其创建一个新函数,这将导致useEffect一次又一次地运行,直到您稍微重构代码。

const setIsValid = useCallback((bool) => {
  setState(prev =>  Object.assign({}, prev, {isValid: bool});
}, []);

现在您可以将setIsValid设置为依赖项。

相关问题