代码异味（Code Smell）识别与重构指南

1、引言：什么是“代码异味”？

在软件开发中，“代码异味（Code Smell）”是指那些虽然不会导致程序编译失败或运行错误，但暗示着潜在设计缺陷或可维护性问题的代码结构。它们是代码演进过程中的“信号灯”，提示我们某段代码可能需要优化。

1.1 ✅ 为什么关注代码异味？

• 预示潜在风险：可能是未来 bug 的温床
• 阻碍重构：让代码难以理解、测试和修改
• 影响协作效率：阅读成本高，新人上手难
• 降低系统稳定性：频繁修改易引入错误

1.2 🧩 代码异味 ≠ Bug

特征	异味	Bug
是否编译通过	是	否（可能）
是否运行正确	是	否
是否需要修复？	建议尽早重构	必须修复

2、常见的代码异味类型及其危害

本节将介绍 10 种典型的代码异味，并提供丰富代码示例说明其特征及重构建议。

2.1. 长函数（Long Method）

🔍 特征：

• 方法体超过 50 行
• 包含多个职责逻辑
• 条件判断嵌套深

🧨 危害：

• 可读性差
• 测试困难
• 修改容易引入副作用

💡 示例：

public void processOrder(Order order) {
   
    if (order == null) throw new IllegalArgumentException();
    if (order.getItems().isEmpty()) return;

    double total = 0;
    for (Item item : order.getItems()) {
   
        if (item.getQuantity() > 10) {
   
            total += item.getPrice() * item.getQuantity() * 0.9;
        } else {
   
            total += item.getPrice() * item.getQuantity();
        }
    }

    if (order.getCoupon() != null) {
   
        total *= order.getCoupon().getDiscountRate();
    }

    order.setTotal(total);
    order.setStatus("PAID");
    orderRepository.save(order);
}

✅ 重构建议：

• 提取子方法，按职责拆分逻辑
• 使用命名清晰的方法名表达意图

private double calculateSubtotal(Order order) {
   
    return order.getItems().stream()
        .mapToDouble(this::calculateItemPrice)