代码重构

1. 重构原则

1.1 什么是重构

重构是在不改变代码外在行为的前提下，对代码内部结构进行调整，使其更易理解和修改。

1.2 重构时机

1.3 重构与测试

重构循环:
1. 确保有测试覆盖
2. 小步重构
3. 每步后运行测试
4. 测试通过则继续，失败则回退

2. 代码坏味道

2.1 常见坏味道

3. 常用重构手法

3.1 提取方法

// 重构前
void printOwing() {
    // 打印横幅
    System.out.println("***********");
    System.out.println("** Owing **");
    System.out.println("***********");
    // 计算并打印金额
    double outstanding = 0.0;
    for (Order o : orders) {
        outstanding += o.getAmount();
    }
    System.out.println("name: " + name);
    System.out.println("amount: " + outstanding);
}

// 重构后
void printOwing() {
    printBanner();
    double outstanding = getOutstanding();
    printDetails(outstanding);
}

3.2 以查询替代临时变量

// 重构前
double getPrice() {
    double basePrice = quantity * itemPrice;
    if (basePrice > 1000) {
        return basePrice * 0.95;
    }
    return basePrice * 0.98;
}

// 重构后
double getPrice() {
    if (basePrice() > 1000) {
        return basePrice() * 0.95;
    }
    return basePrice() * 0.98;
}
double basePrice() { return quantity * itemPrice; }

3.3 以多态替代条件表达式

// 重构前
double getSpeed() {
    switch (type) {
        case EUROPEAN: return getBaseSpeed();
        case AFRICAN: return getBaseSpeed() - getLoadFactor() * numberOfCoconuts;
        case NORWEGIAN_BLUE: return isNailed ? 0 : getBaseSpeed(voltage);
    }
}

// 重构后
abstract class Bird {
    abstract double getSpeed();
}
class EuropeanBird extends Bird {
    double getSpeed() { return getBaseSpeed(); }
}
class AfricanBird extends Bird {
    double getSpeed() { return getBaseSpeed() - getLoadFactor() * numberOfCoconuts; }
}

3.4 其他常用手法

4. 重构安全策略

4.1 小步重构

每次只做一种重构 → 运行测试 → 确认通过 → 下一步

4.2 重构检查清单