From 21debf9e346f544e38e5e237fcd40a130fb3f260 Mon Sep 17 00:00:00 2001
From: lizijie <lzjiiie@163.com>
Date: Wed, 25 Aug 2021 08:51:04 +0800
Subject: [PATCH] 删除冗余代码
---
screen-manage/src/main/java/com/moral/api/service/impl/MenuServiceImpl.java | 43 +++++++++++++++++++++++--------------------
1 files changed, 23 insertions(+), 20 deletions(-)
diff --git a/screen-manage/src/main/java/com/moral/api/service/impl/MenuServiceImpl.java b/screen-manage/src/main/java/com/moral/api/service/impl/MenuServiceImpl.java
index 137a78a..204f902 100644
--- a/screen-manage/src/main/java/com/moral/api/service/impl/MenuServiceImpl.java
+++ b/screen-manage/src/main/java/com/moral/api/service/impl/MenuServiceImpl.java
@@ -48,6 +48,7 @@
@Autowired
GroupMenuMapper groupMenuMapper;
+
@Override
public MenuQueryDTO queryAllMenus() {
//������������������
@@ -86,7 +87,7 @@
List<Menu> ownMenus = menuMapper.getMenusByGroupId(group.getId());
/*������������������������������������������������������������
- * ���������������������������������������������������������������������������������������������������������*/
+ * ���������������������������������������������������������������������������������������������������������*/
removeMenuWithChildren(ownMenus);
//������������������
@@ -160,10 +161,12 @@
UpdateWrapper deleteGroupMenuWrapper = new UpdateWrapper();
List<Integer> allMenuIds = new ArrayList<>();
allMenuIds.add(id);
- allMenuIds.addAll(childrenIds);
- deleteGroupMenuWrapper.in("menu_id",allMenuIds);
- deleteGroupMenuWrapper.set("is_delete",Constants.DELETE);
- groupMenuMapper.update(null,deleteGroupMenuWrapper);
+ if (!ObjectUtils.isEmpty(childrenIds)) {
+ allMenuIds.addAll(childrenIds);
+ }
+ deleteGroupMenuWrapper.in("menu_id", allMenuIds);
+ deleteGroupMenuWrapper.set("is_delete", Constants.DELETE);
+ groupMenuMapper.update(null, deleteGroupMenuWrapper);
//������������������
dto.setCode(ResponseCodeEnum.SUCCESS.getCode());
dto.setMsg(ResponseCodeEnum.SUCCESS.getMsg());
@@ -243,13 +246,13 @@
/**
- * @Description: ������������������������������������������������������
- * @Param: [menus]
- * @return: java.util.List<com.moral.api.entity.Menu>
- * @Author: ���������
- * @Date: 2021/5/13
- */
- public void combinationParentChildrenMenus(List<Menu> menus){
+ * @Description: ������������������������������������������������������
+ * @Param: [menus]
+ * @return: java.util.List<com.moral.api.entity.Menu>
+ * @Author: ���������
+ * @Date: 2021/5/13
+ */
+ public void combinationParentChildrenMenus(List<Menu> menus) {
//������menu������������
Map<Integer, Menu> menuMap = new HashMap<>();
for (Menu menu : menus) {
@@ -349,13 +352,13 @@
}
/**
- * @Description: ������������������������������
- * @Param: [menus]
- * @return: java.util.List<com.moral.api.entity.Menu>
- * @Author: ���������
- * @Date: 2021/5/28
- */
- private void removeMenuWithChildren(List<Menu> menus){
+ * @Description: ������������������������������
+ * @Param: [menus]
+ * @return: java.util.List<com.moral.api.entity.Menu>
+ * @Author: ���������
+ * @Date: 2021/5/28
+ */
+ private void removeMenuWithChildren(List<Menu> menus) {
//������������������
QueryWrapper wrapper = new QueryWrapper();
wrapper.eq("is_delete", Constants.NOT_DELETE);
@@ -373,7 +376,7 @@
menus.removeIf(new Predicate<Menu>() {
@Override
public boolean test(Menu menu) {
- if(ObjectUtils.isEmpty(menuMap.get(menu.getId()).getChildren()))
+ if (ObjectUtils.isEmpty(menuMap.get(menu.getId()).getChildren()))
return false;
return true;
}
--
Gitblit v1.8.0