Java重构示例二(转载)

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

6 满足条件立即跳出循环

6.1 重构前

 

public boolean contain(int year, Month month, int day) {

  boolean found = false;

  for (IPolyDate date : dateList) {

    if (date.same(year, month.getMonth(), day)) {

      found = true;

      break;

    }

  }

 

  return found;

}

 

6.2 重构后

 

public boolean contain(int year, Month month, int day) {

  for (IPolyDate date : dateList) {

    if (date.same(year, month.getMonth(), day)) {

      return true;

    }

  }

 

  return false;

}

 

7 使用表结构分离公共逻辑，避免重复（坚持DRY原则）

7.1 重构前

 

public void testGetIntPart() throws Exception {

  assertEquals("0", digitTransform.getIntPart("0.01");

  assertEquals("1", digitTransform.getIntPart("1.2");

  assertEquals("1234", digitTransform.getIntPart("1234");

  assertEquals("1", digitTransform.getIntPart("1.01");

  assertEquals("0", digitTransform.getIntPart("0.01");

  assertEquals("11111", digitTransform.getIntPart("11111");

  assertEquals("1000", digitTransform.getIntPart("1000.11");

}

 

7.2 重构后

 

public void testGetIntPart() throws Exception {

String[][] cases = new String[][] { { "0.01", "0" }, { "1.2", "1" },

{ "1234", "1234" }, { "1.01", "1" }, { "0", "0" },

{ "11111", "11111" }, { "1000.11", "1000" } };

 

for (int i = 0, len = cases.length; i < len; i++) {

assertEquals(cases[i][1], digitTransform.getIntPart(cases[i][0]));

}

}

 

8 分离变化参数，使用格式化方法

8.1 重构前

 

public class ExceedMaxWeekIndexOfMonthException extends

IndexOutOfBoundsException {

 

  private static final long serialVersionUID = 1L;

 

  public ExceedMaxWeekIndexOfMonthException(String message) {

    super(message);

  }

 

}

 

8.2 重构后

 

public class ExceedMaxWeekIndexOfMonthException extends

IndexOutOfBoundsException {

 

  private static final long serialVersionUID = 1L;

 

  public ExceedMaxWeekIndexOfMonthException(int index, int maxCountOfWeekDay,

Month month) {

    super(formatMessage(index, maxCountOfWeekDay, month));

  }

 

  private static String formatMessage(int index, int maxCountOfWeekDay,

Month month) {

    return "Arguement index[" + index + "] exceeds max week index["

+ maxCountOfWeekDay + "] of month[" + month.toString() + "].";

  }

 

}

 

9 使用方法抽取公共逻辑

9.1 重构前

  

public int getYear() {

  Calendar date = this.getCalendarDate();

 

  return date.get(Calendar.YEAR);

}

 

public int getMonth() {

  Calendar date = this.getCalendarDate();

 

  return date.get(Calendar.MONTH);

}

 

public int getDay() {

  Calendar date = this.getCalendarDate();

 

  return date.get(Calendar.DAY_OF_MONTH);

}

 

public int getHour() {

  Calendar date = this.getCalendarDate();

 

  return date.get(Calendar.HOUR_OF_DAY);

}

 

public int getMinute() {

  Calendar date = this.getCalendarDate();

 

  return date.get(Calendar.MINUTE);

}

 

public int getSecond() {

  Calendar date = this.getCalendarDate();

 

  return date.get(Calendar.SECOND);

}

 

public int getMillisSecond() {

  Calendar date = this.getCalendarDate();

 

  return date.get(Calendar.MILLISECOND);

  }

 

9.2 重构后

 

  private int get(int field) {

    Calendar date = this.getCalendarDate();

 

    return date.get(field);

  }

 

  public int getYear() {

    return this.get(Calendar.YEAR);

  }

 

  public int getMonth() {

    return this.get(Calendar.MONTH);

  }

 

  public int getDay() {

    return this.get(Calendar.DAY_OF_MONTH);

  }

 

  public int getHour() {

    return this.get(Calendar.HOUR_OF_DAY);

  }

 

  public int getMinute() {

    return this.get(Calendar.MINUTE);

  }

 

  public int getSecond() {

    return this.get(Calendar.SECOND);

  }

 

  public int getMillisSecond() {

    return this.get(Calendar.MILLISECOND);

  }

 

10 尽量使用已有的Java API

10.1 重构前

 

protected String[] getConfigLocations() {

    String[] baseCfgs = this.getBaseCfgs();

    String[] extra = this.getExtraCfgs();

    if (extra != null && extra.length > 0) {

      int baseCfgLen = baseCfgs.length;

      int extraLen = extra.length;

      String[] cfgLocations = new String[baseCfgLen + extraLen];

      for(int i = 0; i < baseCfgLen; i++){

        cfgLocations[i] = baseCfgs[i];

      }

 

      for(int i = 0; i < extraLen; i++){

        cfgLocations[baseCfgLen + i] = extra[i];

      }

     

      return cfgLocations;

    } else {

      return baseCfgs;

    }

 }

 

10.2 重构后

 

  protected String[] getConfigLocations() {

    String[] baseCfgs = this.getBaseCfgs();

    String[] extra = this.getExtraCfgs();

if (extra != null && extra.length > 0) {

      int baseCfgLen = baseCfgs.length;

      int extraLen = extra.length;

      String[] cfgLocations = new String[baseCfgLen + extraLen;

      System.arraycopy(baseCfgs, 0, cfgLocations, 0, baseCfgLen);

      System.arraycopy(extra, 0, cfgLocations, baseCfgLen,

extraLen;

     

      return cfgLocations;

    } else {

      return baseCfgs;

    }

 }

 

 

<!--EndFragment-->

 

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