嵌套条件if/else重构
//重构前
if(StringUtils.isNotEmpty(value)){
int result = service.deleteYy(value,type);
if(result > 0){
return MsgUtil.success();
}else{
return MsgUtil.fail();
}
}else{
return MsgUtil.fail();
}
//重构后
if(StringUtils.isNotEmpty(value)){
if(service.deleteYy(value,type) > 0){
return MsgUtil.success();
}
}
return MsgUtil.fail();
总结
- 取消只使用一次的临时变量
- 将重复代码移动到新的方法中,使用重复的方法里调用该新方法
- 嵌套条件优化,将不满足条件的语句放在方法后面,前提是if中要有满足返回值