Java重构示例一(转载)

Java重构示例一

本文通过Java示例代码片段展示了常用重构原则和技巧，供初级开发人员参考。精致的代码能够清楚传达作者的意图，精致的代码是最好的注释，精致的代码非常容易维护和扩展。程序员阅读精致的代码如同大众欣赏优美的散文一样享受。

1 尽量简洁

1.1 重构前

if ( flag == 1 ){     return true; } else{     return false; }

1.2 重构后

return flag == 1;

2 使用三位运算符

2.1 重构前

if ( "Male".equals(gender) ) {     return "Mr."; } else{     return "Mrs."; }

2.2 重构后

return "Male".equals(gender) ? "Mr." : "Mrs.";

3 使用常量替换硬编码

3.1 重构前

for (int i = 0; i < 12; i++) {     this.getDays(i);     //skip... }

3.2 重构后

int YEAR_MONTHS = 12; for (int month = 0; month < Month.YEAR_MONTHS; month++) {     this.getDays(month);     //skip... }

4 使用方法替换反复出现的表达式

4.1 重构前

public boolean isStartAfter(Date date) {     Calendar calendar = BusinessCalendar.getCalendar();     calendar.setTime(date);     int hour = calendar.get(Calendar.HOUR_OF_DAY);     int minute = calendar.get(Calendar.MINUTE);          return ( (hour<fromHour)              || ( (hour==fromHour)                   && (minute<=fromMinute)                  )             );   }       public boolean includes(Date date) {     Calendar calendar = BusinessCalendar.getCalendar();     calendar.setTime(date);     int hour = calendar.get(Calendar.HOUR_OF_DAY);     int minute = calendar.get(Calendar.MINUTE);          return ( ( (fromHour<hour)                || ( (fromHour==hour)                    && (fromMinute<=minute)                   )              ) &&              ( (hour<toHour)                || ( (hour==toHour)                     && (minute<=toMinute)                    )              )            );   }

4.2 重构后

private boolean tailGreatHead(int headHour, int headMinute, int tailHour, int tailMinute, boolean includeEqual) {     boolean tailGreatHeadHour = (headHour < tailHour);     boolean tailEqualHeadHour = (headHour == tailHour);     boolean tailGreatHeadMinute = (headMinute < tailMinute);     boolean tailEqualHeadMinute = (headMinute == tailMinute);       boolean tailGreatEqualHeadMinute = tailGreatHeadMinute || includeEqual && tailEqualHeadMinute;       return (tailGreatHeadHour || (tailEqualHeadHour && tailGreatEqualHeadMinute));   }     private boolean tailGreatHead(int headHour, int headMinute, int tailHour, int tailMinute) {     return tailGreatHead(headHour, headMinute, tailHour, tailMinute, false);   }     private boolean tailGreatEqualHead(int headHour, int headMinute, int tailHour, int tailMinute) {     return tailGreatHead(headHour, headMinute, tailHour, tailMinute, true);   }     public boolean isStartAfter(Date date) {     Calendar calendar = BusinessCalendar.getCalendar();     calendar.setTime(date);     int hour = calendar.get(Calendar.HOUR_OF_DAY);     int minute = calendar.get(Calendar.MINUTE);          return this.tailGreatEqualHead(hour, minute, fromHour, fromMinute);   }       public boolean includes(Date date) {     Calendar calendar = BusinessCalendar.getCalendar();     calendar.setTime(date);     int hour = calendar.get(Calendar.HOUR_OF_DAY);     int minute = calendar.get(Calendar.MINUTE);          return this.tailGreatEqualHead(fromHour, fromMinute, hour, minute) &&              this.tailGreatEqualHead(hour, minute, toHour, toMinute);   }

5 及早结束非正常逻辑

5.1 重构前

int count = 0; if(taskList != null && !taskList.isEmpty()){     //skip...     return count; } else {   return count; }

 

5.2 重构后

int count = 0; if(taskList == null || taskList.isEmpty()){   return 0; }   //skip...   return count;

 

<!--EndFragment-->

 

源地址:http://passport.baidu.com/?business&aid=6&un=%B2%DC%CB%C9%C7%E0#7