ak/zn-cloud
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

refactor(system): 优化项目管理相关代码

Browse Source 
- 注释掉 ProjectController 中的冗余代码
- 优化 ProjectMapper 中的查询条件，增加空值检查
This commit is contained in:
furongxin 2024-12-05 10:36:00 +08:00
parent b29eb854c1
commit bf21eb7042
2 changed files with 29 additions and 41 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

64
yudao-module-system/yudao-module-system-biz/src/main/java/cn/iocoder/yudao/module/system/controller/admin/project/ProjectController.java
View File

@ -77,44 +77,32 @@ public class ProjectController {
public CommonResult<ProjectRespVO> getProject(@RequestParam("id") Long id) { public CommonResult<ProjectRespVO> getProject(@RequestParam("id") Long id) {
ProjectDO project = projectService.getProject(id); ProjectDO project = projectService.getProject(id);
ProjectRespVO respVO = BeanUtils.toBean(project, ProjectRespVO.class); ProjectRespVO respVO = BeanUtils.toBean(project, ProjectRespVO.class);
if (respVO != null) { // if (respVO != null) {
// 获取项目中所有人员的用户编号列表 // // 获取项目中所有人员的用户编号列表
Set<Long> userIds = respVO.getStaff() == null ? new HashSet<>() : respVO.getStaff(); // Set<Long> userIds = respVO.getStaff() == null ? new HashSet<>() : respVO.getStaff();
userIds.add(respVO.getDirectorUserId()); // userIds.add(respVO.getDirectorUserId());
//
// 获取项目中所有部门编号列表 // // 获取项目中所有部门编号列表
Set<Long> deptIds = respVO.getParticipationDept() == null ? new HashSet<>() : respVO.getParticipationDept(); // Set<Long> deptIds = respVO.getParticipationDept() == null ? new HashSet<>() : respVO.getParticipationDept();
deptIds.add(respVO.getResponsibleDept()); // deptIds.add(respVO.getResponsibleDept());
//
// 获取所有用户信息 // // 获取所有用户信息
List<AdminUserDO> userDOS = userService.getUserList(userIds); // Map<Long, AdminUserDO> userMap = userService.getUserMap(userIds);
Map<Long, AdminUserDO> userMap = convertMap(userDOS, AdminUserDO::getId); // // 获取所有部门信息
// 获取所有部门信息 // Map<Long, DeptDO> deptMap = deptService.getDeptMap(deptIds);
List<DeptDO> deptDOS = deptService.getDeptList(deptIds); //
Map<Long, DeptDO> deptMap = convertMap(deptDOS, DeptDO::getId); // // 设置责任人名称
// respVO.setDirectorUserName(userMap.get(respVO.getDirectorUserId()).getNickname());
// 筛选出不是责任人的用户信息 // // 拼接项目人员名称
List<AdminUserDO> userVOs = userDOS.stream() // userMap.remove(respVO.getDirectorUserId());
.filter(user -> !user.getId().equals(respVO.getDirectorUserId())) // respVO.setStaffName(userMap.values().stream().map(AdminUserDO::getNickname).collect(Collectors.joining("、")));
.distinct() //
.collect(Collectors.toList()); // // 设置责任部门名称
// respVO.setResponsibleDeptName(deptMap.get(respVO.getResponsibleDept()).getName());
// 设置责任人名称 // // 拼接参与部门名称
respVO.setDirectorUserName(userMap.get(respVO.getDirectorUserId()).getNickname()); // deptMap.remove(respVO.getResponsibleDept());
// 拼接项目人员名称 // respVO.setParticipationDeptName(deptMap.values().stream().map(DeptDO::getName).collect(Collectors.joining("、")));
respVO.setStaffName(userVOs.stream().map(AdminUserDO::getNickname).collect(Collectors.joining("、"))); // }
// 筛选出不是责任人部门的部门信息
List<DeptDO> deptVOs = deptDOS.stream()
.filter(dept -> !dept.getId().equals(respVO.getResponsibleDept()))
.distinct()
.collect(Collectors.toList());
// 设置责任部门名称
respVO.setResponsibleDeptName(deptMap.get(respVO.getResponsibleDept()).getName());
// 拼接参与部门名称
respVO.setParticipationDeptName(deptVOs.stream().map(DeptDO::getName).collect(Collectors.joining("、")));
}
return success(respVO); return success(respVO);
} }

6
yudao-module-system/yudao-module-system-biz/src/main/java/cn/iocoder/yudao/module/system/dal/mysql/project/ProjectMapper.java
View File

@ -29,11 +29,11 @@ public interface ProjectMapper extends BaseMapperX<ProjectDO> {
.betweenIfPresent(ProjectDO::getEndDate, reqVO.getEndDate()) .betweenIfPresent(ProjectDO::getEndDate, reqVO.getEndDate())
.eqIfPresent(ProjectDO::getIsLongTerm, reqVO.getIsLongTerm()) .eqIfPresent(ProjectDO::getIsLongTerm, reqVO.getIsLongTerm())
.eqIfPresent(ProjectDO::getStatus, reqVO.getStatus()) .eqIfPresent(ProjectDO::getStatus, reqVO.getStatus())
.likeLeft(ProjectDO::getParticipationDept, "-" + reqVO.getParticipationDept()) .likeLeft(Objects.nonNull(reqVO.getParticipationDept()), ProjectDO::getParticipationDept, "-" + reqVO.getParticipationDept())
.or() .or()
.likeRight(ProjectDO::getParticipationDept, reqVO.getParticipationDept() + "-") .likeRight(Objects.nonNull(reqVO.getParticipationDept()), ProjectDO::getParticipationDept, reqVO.getParticipationDept() + "-")
.or() .or()
.like(ProjectDO::getParticipationDept, "-" + reqVO.getParticipationDept() + "-") .like(Objects.nonNull(reqVO.getParticipationDept()), ProjectDO::getParticipationDept, "-" + reqVO.getParticipationDept() + "-")
.orderByDesc(ProjectDO::getId)); .orderByDesc(ProjectDO::getId));
} }
} }