mirror of
				https://github.com/continew-org/continew-admin.git
				synced 2025-10-31 10:57:13 +08:00 
			
		
		
		
	优化:基于 ESLint 和阿里编码规约插件对部分代码规范进行优化
This commit is contained in:
		| @@ -52,16 +52,16 @@ public class DeptDO extends BaseDO { | ||||
|      */ | ||||
|     private Long parentId; | ||||
|  | ||||
|     /** | ||||
|      * 部门排序 | ||||
|      */ | ||||
|     private Integer deptSort; | ||||
|  | ||||
|     /** | ||||
|      * 描述 | ||||
|      */ | ||||
|     private String description; | ||||
|  | ||||
|     /** | ||||
|      * 部门排序 | ||||
|      */ | ||||
|     private Integer deptSort; | ||||
|  | ||||
|     /** | ||||
|      * 状态(1启用 2禁用) | ||||
|      */ | ||||
|   | ||||
| @@ -58,7 +58,7 @@ public class RoleDO extends BaseDO { | ||||
|     private String roleCode; | ||||
|  | ||||
|     /** | ||||
|      * 数据权限(1全部数据权限 2本部门及以下数据权限 3本部门数据权限 4仅本人数据权限 5自定数据权限) | ||||
|      * 数据权限(1全部数据权限 2本部门及以下数据权限 3本部门数据权限 4仅本人数据权限 5自定义数据权限) | ||||
|      */ | ||||
|     private DataScopeEnum dataScope; | ||||
|  | ||||
|   | ||||
| @@ -66,9 +66,9 @@ public class RoleRequest extends BaseRequest { | ||||
|     private String roleCode; | ||||
|  | ||||
|     /** | ||||
|      * 数据权限(1全部数据权限 2本部门及以下数据权限 3本部门数据权限 4仅本人数据权限 5自定数据权限) | ||||
|      * 数据权限(1全部数据权限 2本部门及以下数据权限 3本部门数据权限 4仅本人数据权限 5自定义数据权限) | ||||
|      */ | ||||
|     @Schema(description = "数据权限(1全部数据权限 2本部门及以下数据权限 3本部门数据权限 4仅本人数据权限 5自定数据权限)", type = "Integer", | ||||
|     @Schema(description = "数据权限(1全部数据权限 2本部门及以下数据权限 3本部门数据权限 4仅本人数据权限 5自定义数据权限)", type = "Integer", | ||||
|         allowableValues = {"1", "2", "3", "4", "5"}) | ||||
|     private DataScopeEnum dataScope; | ||||
|  | ||||
|   | ||||
| @@ -61,12 +61,6 @@ public class DeptDetailVO extends BaseDetailVO { | ||||
|     @Schema(description = "上级部门 ID") | ||||
|     private Long parentId; | ||||
|  | ||||
|     /** | ||||
|      * 部门排序 | ||||
|      */ | ||||
|     @Schema(description = "部门排序") | ||||
|     private Integer deptSort; | ||||
|  | ||||
|     /** | ||||
|      * 描述 | ||||
|      */ | ||||
| @@ -74,6 +68,12 @@ public class DeptDetailVO extends BaseDetailVO { | ||||
|     @ExcelProperty(value = "描述") | ||||
|     private String description; | ||||
|  | ||||
|     /** | ||||
|      * 部门排序 | ||||
|      */ | ||||
|     @Schema(description = "部门排序") | ||||
|     private Integer deptSort; | ||||
|  | ||||
|     /** | ||||
|      * 状态(1启用 2禁用) | ||||
|      */ | ||||
|   | ||||
| @@ -57,18 +57,18 @@ public class DeptVO extends BaseVO { | ||||
|     @Schema(description = "上级部门 ID") | ||||
|     private Long parentId; | ||||
|  | ||||
|     /** | ||||
|      * 部门排序 | ||||
|      */ | ||||
|     @Schema(description = "部门排序") | ||||
|     private Integer deptSort; | ||||
|  | ||||
|     /** | ||||
|      * 描述 | ||||
|      */ | ||||
|     @Schema(description = "描述") | ||||
|     private String description; | ||||
|  | ||||
|     /** | ||||
|      * 部门排序 | ||||
|      */ | ||||
|     @Schema(description = "部门排序") | ||||
|     private Integer deptSort; | ||||
|  | ||||
|     /** | ||||
|      * 状态(1启用 2禁用) | ||||
|      */ | ||||
|   | ||||
| @@ -65,9 +65,9 @@ public class RoleDetailVO extends BaseDetailVO { | ||||
|     private String roleCode; | ||||
|  | ||||
|     /** | ||||
|      * 数据权限(1全部数据权限 2本部门及以下数据权限 3本部门数据权限 4仅本人数据权限 5自定数据权限) | ||||
|      * 数据权限(1全部数据权限 2本部门及以下数据权限 3本部门数据权限 4仅本人数据权限 5自定义数据权限) | ||||
|      */ | ||||
|     @Schema(description = "数据权限(1全部数据权限 2本部门及以下数据权限 3本部门数据权限 4仅本人数据权限 5自定数据权限)") | ||||
|     @Schema(description = "数据权限(1全部数据权限 2本部门及以下数据权限 3本部门数据权限 4仅本人数据权限 5自定义数据权限)") | ||||
|     @ExcelProperty(value = "数据权限", converter = ExcelBaseEnumConverter.class) | ||||
|     private DataScopeEnum dataScope; | ||||
|  | ||||
|   | ||||
| @@ -62,9 +62,9 @@ public class RoleVO extends BaseVO { | ||||
|     private String roleCode; | ||||
|  | ||||
|     /** | ||||
|      * 数据权限(1全部数据权限 2本部门及以下数据权限 3本部门数据权限 4仅本人数据权限 5自定数据权限) | ||||
|      * 数据权限(1全部数据权限 2本部门及以下数据权限 3本部门数据权限 4仅本人数据权限 5自定义数据权限) | ||||
|      */ | ||||
|     @Schema(description = "数据权限(1全部数据权限 2本部门及以下数据权限 3本部门数据权限 4仅本人数据权限 5自定数据权限)") | ||||
|     @Schema(description = "数据权限(1全部数据权限 2本部门及以下数据权限 3本部门数据权限 4仅本人数据权限 5自定义数据权限)") | ||||
|     private DataScopeEnum dataScope; | ||||
|  | ||||
|     /** | ||||
|   | ||||
| @@ -54,19 +54,6 @@ public interface DeptService extends BaseService<DeptVO, DeptDetailVO, DeptQuery | ||||
|      */ | ||||
|     List<Tree<Long>> buildTree(List<DeptVO> list); | ||||
|  | ||||
|     /** | ||||
|      * 检查部门名称是否存在 | ||||
|      * | ||||
|      * @param deptName | ||||
|      *            部门名称 | ||||
|      * @param parentId | ||||
|      *            上级部门 ID | ||||
|      * @param deptId | ||||
|      *            部门 ID | ||||
|      * @return 是否存在 | ||||
|      */ | ||||
|     boolean checkDeptNameExist(String deptName, Long parentId, Long deptId); | ||||
|  | ||||
|     /** | ||||
|      * 导出 | ||||
|      * | ||||
|   | ||||
| @@ -100,7 +100,7 @@ public class DeptServiceImpl extends BaseServiceImpl<DeptMapper, DeptDO, DeptVO, | ||||
|     @Transactional(rollbackFor = Exception.class) | ||||
|     public Long create(DeptRequest request) { | ||||
|         String deptName = request.getDeptName(); | ||||
|         boolean isExist = this.checkDeptNameExist(deptName, request.getParentId(), null); | ||||
|         boolean isExist = this.checkNameExist(deptName, request.getParentId(), request.getDeptId()); | ||||
|         CheckUtils.throwIf(() -> isExist, String.format("新增失败,'%s'已存在", deptName)); | ||||
|  | ||||
|         // 保存部门信息 | ||||
| @@ -110,6 +110,16 @@ public class DeptServiceImpl extends BaseServiceImpl<DeptMapper, DeptDO, DeptVO, | ||||
|         return deptDO.getDeptId(); | ||||
|     } | ||||
|  | ||||
|     @Override | ||||
|     @Transactional(rollbackFor = Exception.class) | ||||
|     public void update(DeptRequest request) { | ||||
|         String deptName = request.getDeptName(); | ||||
|         boolean isExist = this.checkNameExist(deptName, request.getParentId(), request.getDeptId()); | ||||
|         CheckUtils.throwIf(() -> isExist, String.format("新增失败,'%s'已存在", deptName)); | ||||
|  | ||||
|         super.update(request); | ||||
|     } | ||||
|  | ||||
|     @Override | ||||
|     @Transactional(rollbackFor = Exception.class) | ||||
|     public void delete(List<Long> ids) { | ||||
| @@ -179,12 +189,6 @@ public class DeptServiceImpl extends BaseServiceImpl<DeptMapper, DeptDO, DeptVO, | ||||
|         }); | ||||
|     } | ||||
|  | ||||
|     @Override | ||||
|     public boolean checkDeptNameExist(String deptName, Long parentId, Long deptId) { | ||||
|         return baseMapper.exists(Wrappers.<DeptDO>lambdaQuery().eq(DeptDO::getDeptName, deptName) | ||||
|             .eq(DeptDO::getParentId, parentId).ne(deptId != null, DeptDO::getDeptId, deptId)); | ||||
|     } | ||||
|  | ||||
|     @Override | ||||
|     public void export(DeptQuery query, HttpServletResponse response) { | ||||
|         List<DeptDO> deptList = this.listDept(query); | ||||
| @@ -193,6 +197,22 @@ public class DeptServiceImpl extends BaseServiceImpl<DeptMapper, DeptDO, DeptVO, | ||||
|         ExcelUtils.export(list, "部门数据", DeptDetailVO.class, response); | ||||
|     } | ||||
|  | ||||
|     /** | ||||
|      * 检查名称是否存在 | ||||
|      * | ||||
|      * @param deptName | ||||
|      *            部门名称 | ||||
|      * @param parentId | ||||
|      *            上级部门 ID | ||||
|      * @param deptId | ||||
|      *            部门 ID | ||||
|      * @return 是否存在 | ||||
|      */ | ||||
|     private boolean checkNameExist(String deptName, Long parentId, Long deptId) { | ||||
|         return baseMapper.exists(Wrappers.<DeptDO>lambdaQuery().eq(DeptDO::getDeptName, deptName) | ||||
|             .eq(DeptDO::getParentId, parentId).ne(deptId != null, DeptDO::getDeptId, deptId)); | ||||
|     } | ||||
|  | ||||
|     /** | ||||
|      * 填充数据 | ||||
|      * | ||||
|   | ||||
| @@ -0,0 +1,4 @@ | ||||
| <?xml version="1.0" encoding="UTF-8" ?> | ||||
| <!DOCTYPE mapper PUBLIC "-//mybatis.org//DTD Mapper 3.0//EN" "http://mybatis.org/dtd/mybatis-3-mapper.dtd" > | ||||
| <mapper namespace="top.charles7c.cnadmin.system.mapper.DeptMapper"> | ||||
| </mapper> | ||||
		Reference in New Issue
	
	Block a user