optimize: 优化代码

Author: Ohh-889

src/features/menu/MenuProvider.tsx

@@ -5,7 +5,7 @@ import { selectActiveFirstLevelMenuKey, setActiveFirstLevelMenuKey } from '@/fea
 import { useLang } from '../lang';
 import { useRoute, useRouter } from '../router';
 
-import { filterRoutesToMenus, getActiveFirstLevelMenuKey } from './MenuUtil';
+import { filterRoutesToMenus, getActiveFirstLevelMenuKey, getSelectKey } from './MenuUtil';
 import { MixMenuContext } from './menuContext';
 
 const MenuProvider: FC<PropsWithChildren> = ({ children }) => {
@@ -17,6 +17,8 @@ const MenuProvider: FC<PropsWithChildren> = ({ children }) => {
 
   const { locale } = useLang();
 
+  const activeFirstLevelMenuKey = useAppSelector(selectActiveFirstLevelMenuKey);
+
   const menus = useMemo(
     () => filterRoutesToMenus(router.reactRouter.routes.find(r => r.id === '(base)')?.children || []),
     // eslint-disable-next-line react-hooks/exhaustive-deps
@@ -28,33 +30,17 @@ const MenuProvider: FC<PropsWithChildren> = ({ children }) => {
     return rest;
   }) as App.Global.Menu[];
 
-  const activeFirstLevelMenuKey = useAppSelector(selectActiveFirstLevelMenuKey);
-
   const childLevelMenus = menus.find(menu => menu.key === activeFirstLevelMenuKey)?.children as App.Global.Menu[];
 
-  const selectKey = (() => {
-    const { activeMenu, hideInMenu } = route.handle;
-
-    const name = route.pathname as string;
-
-    const routeName = (hideInMenu ? activeMenu : name) || name;
-
-    return [routeName];
-  })();
+  const selectKey = getSelectKey(route);
 
   /** - 可以手动指定菜单或者是默认当前路由的一级菜单 */
   function changeActiveFirstLevelMenuKey(key?: string) {
-    let routeKey = key;
-
-    if (!routeKey) {
-      routeKey = getActiveFirstLevelMenuKey(route);
-    }
+    const routeKey = key || getActiveFirstLevelMenuKey(route);
 
     dispatch(setActiveFirstLevelMenuKey(routeKey || ''));
   }
 
-  console.log('menus', menus, childLevelMenus, activeFirstLevelMenuKey);
-
   const mixMenuContext = {
     activeFirstLevelMenuKey,
     allMenus: menus,

src/features/menu/MenuUtil.tsx

@@ -120,3 +120,13 @@ export function mergeMenus(menus: App.Global.Menu[], newMenus: App.Global.Menu[]
 
   return menus;
 }
+
+export function getSelectKey(route: Router.Route) {
+  const { activeMenu, hideInMenu } = route.handle;
+
+  const name = route.pathname as string;
+
+  const routeName = (hideInMenu ? activeMenu : name) || name;
+
+  return [routeName];
+}