> 代码重构,消灭if - Yuyy
Yuyy
Yuyy
代码重构,消灭if

最近工作中,开发一个功能,可以参照已有类似的功能。二话不说,CtrlCV,修修补补,完事。

写完后,看着这段代码,陷入了沉思。我写了这段代码,得到了什么。一、完成工作任务。二、。。。(想不出来了)。这不是工具人吗?

随后重新审视了这段代码。一、用到了WebService。二、代码重复率高。三、代码臃肿,没有区分主干。

那,优化?最近手上的挺多的,而且需要花费半天时间,还得重新测试。

那,不优化?不提升,永远写烂代码。遇到不会的知识不去学习,永远都不会。

答案很明显了,干!

  1. 借此学习下WebService

  2. 抽取公共方法

  3. 抽取方法,体现红花绿叶

  4. 解决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,即使后续增加条件,也不会像之前那么臃肿

发表评论

textsms
account_circle
email

Yuyy

代码重构,消灭if
最近工作中,开发一个功能,可以参照已有类似的功能。二话不说,CtrlCV,修修补补,完事。 写完后,看着这段代码,陷入了沉思。我写了这段代码,得到了什么。一、完成工作任务。二、。。…
扫描二维码继续阅读
2021-03-09
友情链接
标签
归档
近期文章
分类
近期文章