Java重构示例三(转载)

关键字：Java 程序设计 重构 示例 原则 优化 技巧 方法

序言

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

11 使方法通用化

11.1 重构前

public DayPart[] sortDayPart() {   if (dayParts == null || dayParts.length == 0) {     return dayParts;   }   List<DayPart> dayPartList = Arrays.asList(dayParts);     Collections.sort(dayPartList, new Comparator<DayPart>() {     public int compare(DayPart o1, DayPart o2) {       if (o1.getIndex() < o2.getIndex()) {         return -1;       } else if (o1.getIndex() > o2.getIndex()) {         return 1;       } else {         return 0;       }     }   });     return dayPartList.toArray(new DayPart[dayPartList.size()]); }

11.2 重构后

public DayPart[] sortDayPart() {   return this.sortDayPart(SortMode.ASC); }   public DayPart[] sortDayPart(final SortMode sortMode) {   if (dayParts == null || dayParts.length == 0) {     return dayParts;   }   List<DayPart> dayPartList = Arrays.asList(dayParts);     Collections.sort(dayPartList, new Comparator<DayPart>() {     public int compare(DayPart o1, DayPart o2) {       if (o1.getIndex() < o2.getIndex()) {         return sortMode.isAsc() ? -1 : 1;       } else if (o1.getIndex() > o2.getIndex()) {         return sortMode.isAsc() ? 1 : -1;       } else {         return 0;       }     }   });     return dayPartList.toArray(new DayPart[dayPartList.size()]); }

12 避免空语句

12.1 重构前

public int getRemainMinutes(int hour, int minute) {   int startHour = fromHour;   int startMinute = fromMinute;   if (this.fromAfterEqual(hour, minute)) {     // ------from-------position------to--------     ;//use default init value   } else if (this.toAfterEqual(hour, minute)) {     // ------from-------position------to--------     startHour = hour;     startMinute = minute;   } else {     // --------from-------to--------position-------     startHour = toHour;     startMinute = toMinute;   }     return this.getMinutes(startHour, startMinute, toHour, toMinute); }

12.2 重构后

public int getRemainMinutes(int hour, int minute) {   // --------from-------to--------position-------   int startHour = toHour;   int startMinute = toMinute;     if (this.fromAfterEqual(hour, minute)) {     // ------position------from-------to--------     startHour = fromHour;     startMinute = fromMinute;   } else if (this.toAfterEqual(hour, minute)) {     // ------from-------position------to--------     startHour = hour;     startMinute = minute;   }     return this.getMinutes(startHour, startMinute, toHour, toMinute); }

13 公共逻辑后置

13.1 重构前

public int getRemainMinutes(int hour, int minute) {   if (this.fromAfterEqual(hour, minute)) {     // ------position------from-------to--------     return (toHour * 60 + toMinute) - (fromHour * 60 + fromMinute);   } else if (this.toAfterEqual(hour, minute)) {     // ------from-------position------to--------     return (toHour * 60 + toMinute) - (hour * 60 + minute);   }   else {     // --------from-------to--------position-------     return 0; //(toHour * 60 + toMinute) - (toHour * 60 + toMinute);   } }

13.2 重构后

private int getMinutes(int startHour, int startMinute, int endHour, int endMinute) {   int minutes = 0;   minutes = (endHour * 60 + endMinute) - (startHour * 60 + startMinute);     return minutes; }   public int getRemainMinutes(int hour, int minute) {   // --------from-------to--------position-------   int startHour = toHour;   int startMinute = toMinute;     if (this.fromAfterEqual(hour, minute)) {     // ------position------from-------to--------     startHour = fromHour;     startMinute = fromMinute;   } else if (this.toAfterEqual(hour, minute)) {     // ------from-------position------to--------     startHour = hour;     startMinute = minute;   }     return this.getMinutes(startHour, startMinute, toHour, toMinute); }

14 公共逻辑前置

14.1 重构前

private StringBuffer transformDialect(String intPart, String fracPart) {   StringBuffer cn = new StringBuffer();     if (isIntegerZero(intPart)) {     return cn;   }     if (this.isFractionZero(fracPart)) { // 无小数，增加附加信息(整)     cn.append("元").append("整");   } else { // 有小数，增加附加信息(零)     cn.append("元").append("零");   }     return cn; }

14.2 重构后

private StringBuffer transformDialect(String intPart, String fracPart) {   StringBuffer cn = new StringBuffer();     if (isIntegerZero(intPart)) {     return cn;   }     cn.append("元");   if (this.isFractionZero(fracPart)) { // 无小数，增加附加信息(整)     cn.append("整");   } else { // 有小数，增加附加信息(零)     cn.append("零");   }     return cn; }

15 使用清晰的布尔型变量替换逻辑表达式

15.1 重构前

public boolean tailGreatHead(int headHour, int headMinute, int tailHour, int tailMinute, boolean includeEqual) {     return ((headHour < tailHour) ||  ((headHour == tailHour) &&            ((headMinute < tailMinute) || includeEqual && (headMinute == tailMinute)))); }

 

15.2 重构后

public 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)); }

 

<!--EndFragment-->

 

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