代码重构，消灭if
最近工作中，开发一个功能，可以参照已有类似的功能。二话不说，CtrlCV，修修补补，完事。
写完后，看着这段代码，陷入了沉思。我写了这段代码，得到了什么。一、完成工作任务。二、。。。（想不出来了）。这不是工具人吗？
随后重新审视了这段代码。一、用到了WebService。二、代码重复率高。三、代码臃肿，没有区分主干。
那，优化？最近手上的挺多的，而且需要花费半天时间，还得重新测试。
那，不优化？不提升，永远写烂代码。遇到不会的知识不去学习，永远都不会。
答案很明显了，干！
- 借此学习下WebService
- 抽取公共方法
- 抽取方法，体现红花绿叶
- 解决Sonar问题
期间还遇到个有意思的地方，以前写的是
if(!func(A)){
result="参数A不合法"
return result
}
if(!func(B)){
result="参数B不合法"
return result
}
if(!dunc(C)){
result="参数C不合法"
return result
}
dothing
return result虽然卫语句能减少else，使逻辑清晰。但是这儿太多重复代码了。修改如下：
if(!valided(result,A)
|| !valided(result,B)
|| !valided(result,C)){
return result
}else{
dothing
return result
}
boolean valided(result,param){
if(!func(param)){
result="参数"+param+“不合法”
return false
}
return true
}这里的条件看着真不舒服，而且不管条件真假，都要return result啊，再改改。
大家还记得这个公式吗？!(!A | !b) = A & B
if(valided(result,A)
&& valided(result,B)
&& valided(result,C)){
dothing
}
return result
boolean valided(result,param){
if(!func(param)){
result="参数"+param+“不合法”
return false
}
return true
}到此，这段代码优雅了许多。一、区分出主干，体现红花绿叶。二、减少了2个if，即使后续增加条件，也不会像之前那么臃肿
本文参与 腾讯云自媒体同步曝光计划，分享自作者个人站点/博客。
原始发表：2021-3-09 1，如有侵权请联系 cloudcommunity@tencent.com 删除
评论
登录后参与评论
推荐阅读
腾讯云开发者
