违背hooks原则

[mankeheaven]

hooks是依赖顺序的，以下代码应该会报错，至于为啥违反了react原则又没报错，这个很有水准

export function useObservableInternal<TOutput, TInputs extends Readonly<any[]>>(
  useCustomEffect: typeof useEffect,
  init:
    | (() => Observable<TOutput>)
    | ((inputs$: Observable<[...TInputs]>) => Observable<TOutput>),
  inputs?: [...TInputs]
): Observable<TOutput> {
  if (!inputs) {
    return useRefFn(init as () => Observable<TOutput>).current
  }

  const inputs$Ref = useRefFn(() => new BehaviorSubject(inputs))
  const source$Ref = useRefFn(() => init(inputs$Ref.current))

  const firstEffectRef = useRef(true)
  useCustomEffect(() => {
    if (firstEffectRef.current) {
      firstEffectRef.current = false
      return
    }
    inputs$Ref.current.next(inputs)
  }, inputs)

  return source$Ref.current
}

[crimx]

https://observable-hooks.js.org/zh-cn/api/#useobservablestate
见文档说明，这是兼容了两个不一样的使用场景，所以不会冲突。但如果让我再设计一遍可能不会这么骚了嘻嘻😁。

[mankeheaven]

get

还是代码设计，强制了只能使用其中一种，不能出现切换为另一种，否则就会出现问题，是说怎么违背了react hooks原则，还能跑通的。。。

这两种方式底层使用不同的 hooks，故请勿选了一种方式之后又通过改变参数切换到另一种方式。

[mankeheaven]

通过这种参数来做事情，感觉始终不是一个明智之举，如果重新设计，可能还是分开多个api好，这个重载方式，跟react的hooks顺序问题，容易让人蒙蔽

因为我是在这个usexxx之前return,出现过bug的。所以对这个比较敏感