优化此代码 const openKeys = ref<string[]>([]) const selectedKeys = ref<string[]>([]) const { currentMenu, currentMenuTree, currentMenuList } = storeToRefs( useLayoutStore(), ) const rootSubmenuKeys = currentMenuList.value.filter((v: any) => { if (v.type === 0) { return v.parentId } }) watch( () => currentMenu, () => { openKeys.value = [currentMenu.value?.parentId] selectedKeys.value = [currentMenu.value?.id] }, { immediate: true }, ) const router = useRouter() /** * 点击事件 * @param e 事件对象 */ const handleClick = (e: any) => { const item = currentMenuList.value.find((_) => _.id === e.key) if (item) { router.push(item.path) } } /** * SubMenu 展开/关闭的回调 * @param e 展开的openKeys */ const onOpenChange = (e: any) => { const latestOpenKey = e.find((key: any) => openKeys.value.indexOf(key) === -1) if (rootSubmenuKeys.indexOf(latestOpenKey) === -1) { openKeys.value = e } else { openKeys.value = latestOpenKey ? [latestOpenKey] : [] } }
时间: 2024-03-21 15:42:21 浏览: 94
C语言中const定义常量
There are a few optimizations that could be made to this code:
1. Instead of using `ref` for `openKeys` and `selectedKeys`, you can use `reactive` to make the code more concise:
```
const state = reactive({
openKeys: [],
selectedKeys: [],
})
```
2. Instead of using `storeToRefs` to convert the store state to refs, you can use the `toRefs` function, which is shorter and more concise:
```
const { currentMenu, currentMenuTree, currentMenuList } = toRefs(useLayoutStore())
```
3. Instead of using `watch` to watch the `currentMenu` state changes, you can use a computed property to update the `openKeys` and `selectedKeys` arrays:
```
const selectedMenu = computed(() => {
const item = currentMenuList.value.find((_) => _.id === currentMenu.value?.id)
return [item?.id] || []
})
const parentMenu = computed(() => {
const item = currentMenuList.value.find((_) => _.id === currentMenu.value?.parentId)
return [item?.id] || []
})
watch([selectedMenu, parentMenu], ([selected, parent]) => {
state.selectedKeys = selected
state.openKeys = parent
})
```
4. Instead of using `router.push` in the `handleClick` function, you can use the `router.push` method directly in the template:
```
<Menu.Item :key="item.id" :to="item.path">{{ item.name }}</Menu.Item>
```
5. Finally, instead of using `rootSubmenuKeys` to filter the list of menu items, you can use a computed property to filter the list of menu items based on their type:
```
const subMenuItems = computed(() => {
return currentMenuList.value.filter((v: any) => v.type === 0 && v.parentId)
})
```
By applying these optimizations, the code can be simplified and made more concise.
阅读全文