perf(useAsyncEffect): move isAsyncGenerator outside the hook (alibaba…

…#2186)
jiaoruogu · May 16, 2023 · a80979e · a80979e
1 parent b0a67b9
commit a80979e
Showing 1 changed file with 6 additions and 5 deletions.
diff --git a/packages/hooks/src/useAsyncEffect/index.ts b/packages/hooks/src/useAsyncEffect/index.ts
@@ -2,15 +2,16 @@ import type { DependencyList } from 'react';
 import { useEffect } from 'react';
 import { isFunction } from '../utils';
 
+function isAsyncGenerator(
+ val: AsyncGenerator<void, void, void> | Promise<void>,
+): val is AsyncGenerator<void, void, void> {
+ return isFunction(val[Symbol.asyncIterator]);
+}
+
 function useAsyncEffect(
  effect: () => AsyncGenerator<void, void, void> | Promise<void>,
  deps?: DependencyList,
 ) {
- function isAsyncGenerator(
- val: AsyncGenerator<void, void, void> | Promise<void>,
- ): val is AsyncGenerator<void, void, void> {
- return isFunction(val[Symbol.asyncIterator]);
- }
  useEffect(() => {
  const e = effect();
  let cancelled = false;