重构 = 修改代码结构 + 不改变行为 + 通过测试验证

准备开发 → 看代码结构 → 发现设计有问题 → 重构 → 添加功能

发现 bug → 看代码 → 难以理解 → 重构 → bug 显露 → 修复

测试覆盖 → 小步修改 → 运行测试 → 确认通过 → 继续下一步

重构安全链：小步修改 → 频繁运行测试 → 发现问题立即回退

// ❌ 反例 - 命名不清晰
int d;  // 消逝的时间
List<Element> elemList;

// ✅ 正例 - 命名清晰
int elapsedTimeInDays;
List<Element> elements;

// ❌ 反例 - 重复代码
if (order.getAmount() > 1000) {
    discount = order.getAmount() * 0.9;
}
if (invoice.getAmount() > 1000) {
    discount = invoice.getAmount() * 0.9;
}

// ✅ 正例 - 提取公共方法
private BigDecimal calculateDiscount(BigDecimal amount) {
    return amount.compareTo(BigDecimal.valueOf(1000)) > 0
        ? amount.multiply(BigDecimal.valueOf(0.9))
        : amount;
}

// ❌ 反例 - 函数过长
public void printOwing() {
    Enumeration e = _orders.elements();
    double outstanding = 0.0;

    // 打印 banner
    System.out.println("*************************");
    System.out.println("***** Customer Owes ******");
    System.out.println("*************************");

    // 计算金额
    while (e.hasMoreElements()) {
        Order each = (Order) e.nextElement();
        outstanding += each.getAmount();
    }

    // 打印详情
    System.out.println("name: " + _name);
    System.out.println("amount: " + outstanding);
}

// ✅ 正例 - 拆分为小函数
public void printOwing() {
    printBanner();
    double outstanding = calculateOutstanding();
    printDetails(outstanding);
}

private void printBanner() {
    System.out.println("*************************");
    System.out.println("***** Customer Owes ******");
    System.out.println("*************************");
}

private double calculateOutstanding() {
    Enumeration e = _orders.elements();
    double result = 0.0;
    while (e.hasMoreElements()) {
        Order each = (Order) e.nextElement();
        result += each.getAmount();
    }
    return result;
}

// ❌ 反例 - 参数过多
void createOrder(String customerId, String productCode, int quantity,
                 String address, String phone, String email) {
    // ...
}

// ✅ 正例 - 使用参数对象
void createOrder(CreateOrderRequest request) {
    // ...
}

@Data
class CreateOrderRequest {
    private String customerId;
    private String productCode;
    private int quantity;
    private Address address;
    private Contact contact;
}

// ❌ 反例 - 过度访问其他对象
class Order {
    void calculatePrice(Product product) {
        double basePrice = product.getBasePrice();
        double discount = product.getDiscount();
        // ...
    }
}

// ✅ 正例 - 让对象自己处理
class Order {
    void calculatePrice(Product product) {
        return product.calculatePrice();
    }
}

// ❌ 反例 - 数据总是成对出现
void printInfo(String firstName, String lastName, String street,
                String city, String zipCode) {
    // ...
}

// ✅ 正例 - 组织成对象
void printInfo(Person person, Address address) {
    // ...
}

// ❌ 反例 - 使用基本类型
void connect(String host, int port, String protocol, boolean secure) {
    // ...
}

// ✅ 正例 - 使用对象
void connect(ConnectionConfig config) {
    // ...
}

class ConnectionConfig {
    private String host;
    private int port;
    private Protocol protocol;
    private boolean secure;
}