umi4 useModel如果返回改变state的方法，为什么这个方法在定义的时候必须要用useCallback包裹，不包裹就会造成性能优化无效，每次都render

[buquan]

// 定义 obj2.ts
import { useState, useCallback } from 'react';
export default () => {
  const [obj, setObj] = useState({a: 0, b: 0, person: {son: {}}, worker: {x: 1}});
  const changeObj = () => {
    setObj((pre) => {
      return {
        a: 1,
        b: Math.random(),
        person: {
          name: 'name',
          son: {
            name: 'son'
          }
        },
        worker: pre.worker
      }
    });
  };
  return { obj, setObj, changeObj };
};

// 使用
import {useInterval} from 'ahooks'
const Component =  () => {
   const { son, worker, setObj, changeObj } = useModel('obj2', (model) => ({
      changeObj: model.changeObj,
      son: model.obj.person.son,
      worker: model.obj.worker,
    }));
  
    useInterval(
    () => {
        changeObj();
    },
    2000,
    {
      immediate: true,
    }
  );

  useEffect(() => {
    console.log('son', son);  // 每次都执行(如果不返回changeObj和使用changeObj，只使用setObj直接调用，那么此处深比较就不会每次都执行)
  }, [son]);

  return <></>
}

[buquan]

终于找到问题了！这里面由于返回的是

{
      changeObj: model.changeObj,
      son: model.obj.person.son,
      worker: model.obj.worker,
 }

其中changeObj函数是新的函数，fast-deep-equal深比较所以触发了更新，那请问比较这块要不要把函数过滤掉呢，🤔

[fz6m]

理论上你在 react 里不用 useCallback 去包裹一个函数，不光是在 model 里，在任何一个 react 组件或者 hooks 里，每次渲染就会重新创建，所以我感觉和在不在 model 里无关，另外现在更多人不去写 useCallback 了，大量的重新创建函数是家常便饭，不差多创建几次。

或者此处考虑把数据 state 全管理在 model 里，函数操作全管理在 hooks 里？

[buquan]

多谢指点，model场景的更新函数我都用useCallback处理了，区别于react-redux那套通常只浅比较的是data，而自定义hook确实挺容易造成处理不当导致优化无效