常见代码重构技巧( 五 ) _重构

将查询与修改分离
任何有返回值的方法，都不应该有副作用
移除不必要临时变量
临时变量仅使用一次或者取值逻辑成本很低的情况下
引入解释性变量
将复杂表达式（或其中一部分）的结果放进一个临时变量，以此变量名称来解释表达式用途
if ((platform.toUpperCase().indexOf("MAC") > -1) && (browser.toUpperCase().indexOf("IE") > -1) && wasInitialized() && resize > 0) {// do something } final boolean isMacOs = platform.toUpperCase().indexOf("MAC") > -1; final boolean isIEBrowser = browser.toUpperCase().indexOf("IE") > -1; final boolean wasResized = resize > 0; if (isMacOs && isIEBrowser && wasInitialized() && wasResized) {// do something }复制代码
使用卫语句替代嵌套条件判断
把复杂的条件表达式拆分成多个条件表达式，减少嵌套。嵌套了好几层的if - then-else语句，转换为多个if语句
//未使用卫语句public void getHello(int type) {if (type == 1) {return;} else {if (type == 2) {return;} else {if (type == 3) {return;} else {setHello();}}}} //使用卫语句public void getHello(int type) {if (type == 1) {return;}if (type == 2) {return;}if (type == 3) {return;}setHello();}复制代码
使用多态替代条件判断断
当存在这样一类条件表达式，它根据对象类型的不同选择不同的行为。可以将这种表达式的每个分支放进一个子类内的复写函数中，然后将原始函数声明为抽象函数。
public int calculate(int a, int b, String operator) {int result = Integer.MIN_VALUE;if ("add".equals(operator)) {result = a + b;} else if ("multiply".equals(operator)) {result = a * b;} else if ("divide".equals(operator)) {result = a / b;} else if ("subtract".equals(operator)) {result = a - b;}return result;}复制代码
当出现大量类型检查和判断时，if else（或）语句的体积会比较臃肿，这无疑降低了代码的可读性。另外，if else（或）本身就是一个“变化点”，当需要扩展新的类型时，我们不得不追加if else（或）语句块，以及相应的逻辑，这无疑降低了程序的可扩展性，也违反了面向对象的开闭原则。
基于这种场景，我们可以考虑使用“多态”来代替冗长的条件判断，将if else（或）中的“变化点”封装到子类中。这样，就不需要使用if else（或）语句了，取而代之的是子类多态的实例，从而使得提高代码的可读性和可扩展性。很多设计模式使用都是这种套路，比如策略模式、状态模式。
public interface Operation { int apply(int a, int b); }public class Addition implements Operation { @Override public int apply(int a, int b) { return a + b; } }public class OperatorFactory {private final static Map operationMap = new HashMap<>();static {operationMap.put("add", new Addition());operationMap.put("divide", new Division());// more operators}public static Operation getOperation(String operator) {return operationMap.get(operator);}}public int calculate(int a, int b, String operator) {if (OperatorFactory .getOperation == null) {throw new IllegalArgumentException("Invalid Operator");}return OperatorFactory .getOperation(operator).apply(a, b);}复制代码
使用异常替代返回错误码
非正常业务状态的处理，使用抛出异常的方式代替返回错误码
尽量使用标准异常避免在语句块中抛出异常
//使用错误码public boolean withdraw(int amount) {if (balance < amount) {return false;} else {balance -= amount;return true;}}//使用异常public void withdraw(int amount) {if (amount > balance) {throw new IllegalArgumentException("amount too large");}balance -= amount;}复制代码
引入断言
某一段代码需要对程序状态做出某种假设，以断言明确表现这种假设。
引入Null对象或特殊对象