From 5a9d31ad1ebebe1ee85b61ff7da1d49df0fa71ab Mon Sep 17 00:00:00 2001 From: "refactoringjanitor[bot]" <125389361+refactoringjanitor[bot]@users.noreply.github.com> Date: Thu, 25 May 2023 18:20:43 +0000 Subject: [PATCH] Applied CollapseNestedIfStatements to 4 files --- .../main/java/me/zhengjie/utils/FileUtil.java | 6 ++---- .../modules/system/rest/RoleController.java | 6 ++---- .../system/service/impl/MenuServiceImpl.java | 18 ++++++------------ .../me/zhengjie/config/MultipartConfig.java | 6 ++---- 4 files changed, 12 insertions(+), 24 deletions(-) diff --git a/eladmin-common/src/main/java/me/zhengjie/utils/FileUtil.java b/eladmin-common/src/main/java/me/zhengjie/utils/FileUtil.java index f73998c4..9f0a2da1 100644 --- a/eladmin-common/src/main/java/me/zhengjie/utils/FileUtil.java +++ b/eladmin-common/src/main/java/me/zhengjie/utils/FileUtil.java @@ -198,10 +198,8 @@ public class FileUtil extends cn.hutool.core.io.FileUtil { // getCanonicalFile 可解析正确各种路径 File dest = new File(path).getCanonicalFile(); // 检测是否存在目录 - if (!dest.getParentFile().exists()) { - if (!dest.getParentFile().mkdirs()) { - System.out.println("was not successful."); - } + if (!dest.getParentFile().exists() && !dest.getParentFile().mkdirs()) { + System.out.println("was not successful."); } // 文件写入 file.transferTo(dest); diff --git a/eladmin-system/src/main/java/me/zhengjie/modules/system/rest/RoleController.java b/eladmin-system/src/main/java/me/zhengjie/modules/system/rest/RoleController.java index 8655d24d..559f2f50 100644 --- a/eladmin-system/src/main/java/me/zhengjie/modules/system/rest/RoleController.java +++ b/eladmin-system/src/main/java/me/zhengjie/modules/system/rest/RoleController.java @@ -144,10 +144,8 @@ public class RoleController { private int getLevels(Integer level) { List levels = roleService.findByUsersId(SecurityUtils.getCurrentUserId()).stream().map(RoleSmallDto::getLevel).collect(Collectors.toList()); int min = Collections.min(levels); - if (level != null) { - if (level < min) { - throw new BadRequestException("权限不足,你的角色级别:" + min + ",低于操作的角色级别:" + level); - } + if (level != null && level < min) { + throw new BadRequestException("权限不足,你的角色级别:" + min + ",低于操作的角色级别:" + level); } return min; } diff --git a/eladmin-system/src/main/java/me/zhengjie/modules/system/service/impl/MenuServiceImpl.java b/eladmin-system/src/main/java/me/zhengjie/modules/system/service/impl/MenuServiceImpl.java index 0d307a6e..3fa978ae 100644 --- a/eladmin-system/src/main/java/me/zhengjie/modules/system/service/impl/MenuServiceImpl.java +++ b/eladmin-system/src/main/java/me/zhengjie/modules/system/service/impl/MenuServiceImpl.java @@ -123,18 +123,14 @@ public class MenuServiceImpl implements MenuService { if (menuRepository.findByTitle(resources.getTitle()) != null) { throw new EntityExistException(Menu.class, "title", resources.getTitle()); } - if (StringUtils.isNotBlank(resources.getComponentName())) { - if (menuRepository.findByComponentName(resources.getComponentName()) != null) { - throw new EntityExistException(Menu.class, "componentName", resources.getComponentName()); - } + if (StringUtils.isNotBlank(resources.getComponentName()) && menuRepository.findByComponentName(resources.getComponentName()) != null) { + throw new EntityExistException(Menu.class, "componentName", resources.getComponentName()); } if (Long.valueOf(0L).equals(resources.getPid())) { resources.setPid(null); } - if (resources.getIFrame()) { - if (!(resources.getPath().toLowerCase().startsWith(HTTP_PRE) || resources.getPath().toLowerCase().startsWith(HTTPS_PRE))) { - throw new BadRequestException(BAD_REQUEST); - } + if (resources.getIFrame() && !(resources.getPath().toLowerCase().startsWith(HTTP_PRE) || resources.getPath().toLowerCase().startsWith(HTTPS_PRE))) { + throw new BadRequestException(BAD_REQUEST); } menuRepository.save(resources); // 计算子节点数目 @@ -151,10 +147,8 @@ public class MenuServiceImpl implements MenuService { } Menu menu = menuRepository.findById(resources.getId()).orElseGet(Menu::new); ValidationUtil.isNull(menu.getId(), "Permission", "id", resources.getId()); - if (resources.getIFrame()) { - if (!(resources.getPath().toLowerCase().startsWith(HTTP_PRE) || resources.getPath().toLowerCase().startsWith(HTTPS_PRE))) { - throw new BadRequestException(BAD_REQUEST); - } + if (resources.getIFrame() && !(resources.getPath().toLowerCase().startsWith(HTTP_PRE) || resources.getPath().toLowerCase().startsWith(HTTPS_PRE))) { + throw new BadRequestException(BAD_REQUEST); } Menu menu1 = menuRepository.findByTitle(resources.getTitle()); if (menu1 != null && !menu1.getId().equals(menu.getId())) { diff --git a/eladmin-tools/src/main/java/me/zhengjie/config/MultipartConfig.java b/eladmin-tools/src/main/java/me/zhengjie/config/MultipartConfig.java index d1580126..095eab29 100644 --- a/eladmin-tools/src/main/java/me/zhengjie/config/MultipartConfig.java +++ b/eladmin-tools/src/main/java/me/zhengjie/config/MultipartConfig.java @@ -36,10 +36,8 @@ public class MultipartConfig { MultipartConfigFactory factory = new MultipartConfigFactory(); String location = System.getProperty("user.home") + "/.eladmin/file/tmp"; File tmpFile = new File(location); - if (!tmpFile.exists()) { - if (!tmpFile.mkdirs()) { - System.out.println("create was not successful."); - } + if (!tmpFile.exists() && !tmpFile.mkdirs()) { + System.out.println("create was not successful."); } factory.setLocation(location); return factory.createMultipartConfig();