安全上移重复代码的核心是识别稳定逻辑与共性结构并保留子类差异。应上移模板流程、通用工具方法、共享资源及合理默认实现;避免语义不同的方法合并;推荐模板方法模式,用abstract定义变化点、hook预留扩展;工具方法设为protected,字段封装为private+getter/setter;重构后须验证行为一致性、异常路径和测试覆盖。

把多个子类中重复的代码安全地上移到抽象父类,核心是识别“稳定不变的行为逻辑”和“可被统一管理的共性结构”,同时保留子类必要的差异化实现。关键不在于“能移多少”,而在于“该不该移、怎么移得干净、后续是否好维护”。
识别真正可上移的冗余代码
不是所有重复都适合上移。重点找以下几类:
- 完全一致的模板方法逻辑:比如“校验参数 → 执行主流程 → 记录日志 → 返回结果”,各子类步骤顺序和骨架相同,仅中间某步不同;
- 通用工具型方法:如格式化时间、拼接 URL、解析 JSON 字段(不依赖子类特有字段);
- 受保护的共享状态或资源:如共用的 HttpClient、配置对象、缓存容器(需注意线程安全);
- 默认实现合理且可被覆盖的方法:比如 getTimeout() 默认返回 5000,子类按需重写。
避免上移:仅实现细节不同但语义不同的方法(如 sendEmail() 和 sendSms() 看似都“发送”,但协议、参数、异常处理完全不同,不应强行合并)。
用模板方法模式封装变化点
这是最安全、最常用的重构方式。把不变的部分写死在抽象类里,把变的部分定义为 abstract 或 hook 方法,由子类实现:
public abstract class DataProcessor {
// 模板方法:定义执行流程
public final Result process(Request req) {
validate(req); // 共用校验
Result result = doProcess(req); // 子类实现差异逻辑
logResult(result); // 共用日志
return result;
}
<pre class="brush:php;toolbar:false;">private void validate(Request req) { /* 通用校验 */ }
private void logResult(Result r) { /* 通用日志 */ }
// 声明变化点 —— 必须由子类提供
protected abstract Result doProcess(Request req);
// 可选钩子方法 —— 子类可选择性覆盖
protected void beforeProcess(Request req) {}}
子类只需关注 doProcess,其他逻辑天然复用,且无法破坏流程顺序。
谨慎提取 protected 工具方法和字段
把子类反复写的工具逻辑(非业务核心)抽成 protected 方法,比 public 更安全(仅对子类可见):
- 字段尽量设为 private + protected getter/setter,避免子类直接修改内部状态;
- 共用对象(如 ObjectMapper、RestTemplate)建议通过构造函数注入或延迟初始化,而非静态单例,方便测试和替换;
- 若子类需扩展某工具方法行为,可用“策略+钩子”组合:提供默认实现 + 预留 before/after 钩子。
重构后必须验证的三件事
上移不是结束,而是新责任的开始:
- 运行时行为一致性:所有子类在相同输入下,输出和副作用(日志、网络调用、DB 写入)必须与重构前一致;
- 异常路径全覆盖:检查抽象类中抛出的异常是否被子类正确捕获或声明,避免编译失败或静默吞异常;
- 单元测试迁移:原属子类的重复测试用例,应转为抽象类的测试(用 Mock 子类验证模板流程),再保留少量子类特例测试。
不复杂但容易忽略。安全上移的本质,是让抽象类承担“契约与骨架”,让子类专注“实现与定制”。
Java免费学习笔记:立即使用
解锁 Java 大师之旅:从入门到精通的终极指南











