本文总结 Java 常见代码坏味道包括长方法、重复代码、过深嵌套、魔法值、上帝类并给出 10 种可落地的重构手法。正文1. 什么是代码坏味道坏味道是代码中潜在问题的信号不一定报错但影响可读性、可维护性和扩展性。2. 坏味道一长方法方法超过 50 行职责不清。重构提取方法。java复制下载// 重构前 public void processOrder(Order order) { // 校验 // 计算价格 // 扣库存 // 发消息 } // 重构后 public void processOrder(Order order) { validate(order); BigDecimal price calculatePrice(order); deductStock(order); sendMessage(order); }3. 坏味道二重复代码相同逻辑多处出现。重构提取方法、父类、工具类或策略模式。4. 坏味道三过深嵌套java复制下载if (a) { if (b) { if (c) { // ... } } }重构卫语句提前返回。java复制下载if (!a) return; if (!b) return; if (!c) return; // ...5. 坏味道四魔法值java复制下载if (status 1) { ... }重构枚举或常量。java复制下载public enum OrderStatus { CREATED(1), PAID(2), SHIPPED(3); private final int code; }6. 坏味道五上帝类一个类几千行承担过多职责。重构按职责拆分单一职责原则。7. 坏味道六过长参数列表java复制下载public void create(String name, int age, String city, String email, String phone) {}重构参数对象。java复制下载public void create(UserCreateReq req) {}8. 坏味道七switch/if-else 泛滥java复制下载if (type.equals(A)) { ... } else if (type.equals(B)) { ... } else if (type.equals(C)) { ... }重构策略模式 工厂。java复制下载MapString, Handler handlers; handlers.get(type).handle();9. 坏味道八注释过多好代码自解释。注释应说明为什么而非是什么。10. 坏味道九可变全局状态静态可变字段导致并发问题和测试困难。重构依赖注入、不可变对象。11. 坏味道十异常吞掉java复制下载try { // ... } catch (Exception e) { // 什么都不做 }重构记录日志、抛出、或明确处理。12. 重构手法清单提取方法。内联方法。提取变量。引入参数对象。以多态替换条件。以策略替换 if-else。卫语句替换嵌套。枚举替换魔法值。拆分大类。引入设计模式。13. 重构步骤保证有测试覆盖。小步修改频繁提交。每次只做一种重构。重构与功能开发分开。14. 工具支持IDEA 重构菜单。SonarQube 检测坏味道。PMD、Checkstyle。单元测试保障。15. 总结重构不是重写而是持续改善。识别坏味道掌握常用手法配合测试和工具让代码保持整洁可维护。