useInViewport中的demo

[JeromeD3]

如果threshold是数组的话，由于deps这里是浅比较，会在useEffectWithTarget中引起effect()的重新执行。这种情况下不考虑优化一下么？还是说这点性能可以忽略不计。🤔小白求解答

[JeromeD3]

我觉得可以给threshold套一个useRef，但是整体代码看起来不太美观

[liuyib]

类似于 useInViewport 中使用的 useEffectWithTarget 这个东西，也就是 createEffectWithTarget，其内部使用了 depsAreSame 对比依赖，depsAreSame 算法如下：

hooks/packages/hooks/src/utils/depsAreSame.ts

Lines 1 to 9 in 2ff3ed9

	import type { DependencyList } from 'react';
	export default function depsAreSame(oldDeps: DependencyList, deps: DependencyList): boolean {
	if (oldDeps === deps) return true;
	for (let i = 0; i < oldDeps.length; i++) {
	if (!Object.is(oldDeps[i], deps[i])) return false;
	}
	return true;
	}

可以看到，对于数组类型，做了一层深比较。所以，直接写数组没有性能问题