简化条件判断式，少写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判断很复杂，那么你可以将部分判断抽出一个函数，取一个好的函数名，这样不仅简化了复杂的判断，也让代码更清晰。

如果代码中还存在嵌套循环，你可以把某些判断条件从嵌套中抽出来，在该条件为真时立刻返回结果。

改善代码质量不仅要做好设计，也要在实现细节上做好。如果你用心去改善你的代码，你肯定能写成质量更高的代码!