这段代码有优化空间吗final ListInteger roles = ListsneWArrayList;final Boolean existAppraise = activityAppraiseeServiceexistByEmpCodeempCodeif BooleanUtisisTrueexistAppraise roles addUserTypeEnumAPPRAISEE getVal
是的,这段代码有一些优化空间。
首先,可以将final List<Integer> roles = Lists.neWArrayList()这一行的代码简化为final List<Integer> roles = new ArrayList<>()。
然后,可以将if (BooleanUtis.isTrue(existAppraise))和if (BooleanUtils.isTrue(existExaminer))这两个判断条件合并为一个条件判断。
最后,可以将roles.add(UserTypeEnum.APPRAISEE.getValue())和roles.add(UserTypeEnum.EXAMINER.getValue())这两行代码提取出来,避免重复。
下面是优化后的代码:
final List<Integer> roles = new ArrayList<>();
if (BooleanUtis.isTrue(activityAppraiseeService.existByEmpCode(empCode))) {
roles.add(UserTypeEnum.APPRAISEE.getValue());
}
if (BooleanUtils.isTrue(activityExaminerService.existByEmpCode(empCode))) {
roles.add(UserTypeEnum.EXAMINER.getValue());
}
return roles;
``
原文地址: https://www.cveoy.top/t/topic/hBxm 著作权归作者所有。请勿转载和采集!