最近工作中，开发一个功能，可以参照已有类似的功能。二话不说，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，即使后续增加条件，也不会像之前那么臃肿