简化条件判断式,少写IF/ELSE
有人说过:写太多if/else 的代码不是好代码。写了几年的代码,感觉这句话很有道理。如果一段代码有太多的if/else,起码有以下问题。第一阅读性差。可能自己当时看的懂,过了一段时间估计自己也看不懂了,更何况其他程序员来理解这段代码。第二维护性差:写太多的if/else,如果稍有不慎,出错几率很大,特别是当其他程序员来改这段代码,那么可能引起新的bug。第三,可能反映出你面向对象思想不深刻,有些IF/ELSE需要从整体设计上改善。
下面看两个例子,希望能给你带来一点启示。看看如何减少下面代码中的if/else?
一个计算每月天数的函数:
public int getDaysOfMonth(int month) {
//一个月的天数
int days=0;
if(month==1) {
days=31;
}else if(month==2) {
days=28;
}else if(month==3) {
days=31;
}else if(month==4) {
days=30;
}else if(month==5) {
days=31;
}else if(month==6) {
days=30;
//下面继续7-12个月的if/else...
}else {
days=-1;
}
return days;
}
第二个函数:
public int getNumber(int number) {
if(number <=18) {
return 18;
}
if(18<number && number<60) {
return number;
}
if(number >60) {
return 60;
}
return 0;
}
第一个求每月天数的代码,我们可以考虑写个长度为12的数组
int[] monthDay=new int[]{31,28,31,30,31,...,31};
这样我们以月为访问索引通过month[index]可以很快拿到每个月的天数。这种方式称作表驱动法。
第二个getNumber函数,我们可以考虑使用最值函数,方式如下:
max(min(60,number),18)。
在重构改善既有代码设计书上提到如何改善复杂的条件表达式:
如果你的if判断很复杂,那么你可以将部分判断抽出一个函数,取一个好的函数名,这样不仅简化了复杂的判断,也让代码更清晰。
如果代码中还存在嵌套循环,你可以把某些判断条件从嵌套中抽出来,在该条件为真时立刻返回结果。
改善代码质量不仅要做好设计,也要在实现细节上做好。如果你用心去改善你的代码,你肯定能写成质量更高的代码!