
本文介绍如何利用 java 泛型与接口设计重构高度相似的过滤器构建方法,避免因继承导致的代码冗余和脆弱基类问题,同时提升可维护性与类型安全性。
本文介绍如何利用 java 泛型与接口设计重构高度相似的过滤器构建方法,避免因继承导致的代码冗余和脆弱基类问题,同时提升可维护性与类型安全性。
在实际开发中,我们常遇到这样一种场景:两个功能几乎完全一致的方法,仅因参数类型(如 WarehouseSimpleSearchFilter 与它的子类 WarehouseSearchFilter)和返回类型略有不同,就不得不复制粘贴整段逻辑——这不仅违反 DRY(Don’t Repeat Yourself)原则,更埋下了维护隐患。
最直接且类型安全的解决方案是使用有界泛型(Bounded Generics),将共用逻辑抽取为单一方法:
private <t extends warehousesimplesearchfilter> T getWarehouseSearchFilter(
Set<long> carrierIds,
Set<long> shipperIds,
Authorization authorization,
T searchFilter) {
if (CollectionUtils.isEmpty(carrierIds) && CollectionUtils.isEmpty(shipperIds)) {
final Set<long> newCarrierIds = getCompanyIds(WarehousePermission.READ_AVAILABLE, authorization);
final Set<long> newShipperIds = getCompanyIds(WarehousePermission.READ_COMPANY, authorization);
searchFilter.setCarrierIds(newCarrierIds);
searchFilter.setShipperIds(newShipperIds);
return searchFilter;
}
if (!CollectionUtils.isEmpty(carrierIds)) {
checkPermission(WarehousePermission.READ_AVAILABLE, authorization);
searchFilter.setCarrierIds(Collections.unmodifiableSet(carrierIds));
}
if (!CollectionUtils.isEmpty(shipperIds)) {
checkPermission(WarehousePermission.READ_COMPANY, authorization);
searchFilter.setShipperIds(Collections.unmodifiableSet(shipperIds));
}
return searchFilter;
}</long></long></long></long></t>
✅ 优势说明:
- 方法签名清晰表达“接受任意 WarehouseSimpleSearchFilter 子类型,并原样返回该类型”,编译期即可保证类型安全;
- 调用方无需修改:getWarehouseSearchFilter(..., new WarehouseSimpleSearchFilter()) 与 getWarehouseSearchFilter(..., new WarehouseSearchFilter()) 均可无缝工作;
- 后续新增子类(如 WarehouseAdvancedSearchFilter)也自动兼容,无需新增方法。
⚠️ 但需注意两个关键设计问题:
-
返回值语义冗余:当前方法始终返回入参 searchFilter 的引用(即“就地修改 + 返回”)。若非构建流式 API(Fluent Interface),这种返回并无实际意义。更简洁、更符合直觉的设计是改为 void 方法:
private <t extends warehousesimplesearchfilter> void populateWarehouseSearchFilter( Set<long> carrierIds, Set<long> shipperIds, Authorization authorization, T searchFilter) { ... }</long></long></t>这样语义更明确——“填充已有对象”,避免调用方误以为返回了新实例。
继承关系存疑:WarehouseSearchFilter extends WarehouseSimpleSearchFilter 表面看是“增强版”,但严格来说,“增强搜索过滤器”并不是一种“简单搜索过滤器”——它可能拥有额外字段(如 search, isLocked)、不同校验逻辑或生命周期行为。强行继承易导致脆弱基类问题(Fragile Base Class):未来对 WarehouseSimpleSearchFilter 的修改(如添加字段或重写方法)可能意外破坏子类行为。
? 推荐重构路径(面向接口编程):
定义统一契约接口,解耦实现与行为:
public interface WarehouseSearchFilter {
void setCarrierIds(Set<long> ids);
void setShipperIds(Set<long> ids);
// 可选:声明通用查询方法,如 toQuery()
}</long></long>
让两类实现独立实现该接口(不再继承):
public class WarehouseSimpleSearchFilter implements WarehouseSearchFilter { /* 字段 + 实现 */ }
public class WarehouseSearchFilter implements WarehouseSearchFilter { /* 独立字段 + 实现 */ }
此时泛型方法可进一步升级为面向接口:
private <t extends warehousesearchfilter> void populateSearchFilter(
Set<long> carrierIds,
Set<long> shipperIds,
Authorization authorization,
T filter) {
// 逻辑完全复用,且不再依赖继承层次
}</long></long></t>
? 总结:
- ✅ 优先用泛型 + 接口替代“为复用而继承”的反模式;
- ✅ 避免无意义的返回值,用 void 明确表达副作用操作;
- ✅ 将公共逻辑下沉至工具类静态方法或接口默认方法(若逻辑不依赖具体字段);
- ✅ 持续审视继承关系是否真实反映“is-a”,而非仅为代码复用妥协。
如此重构后,代码更健壮、可测试性更高,也为未来扩展(如支持 JSON 序列化策略、审计日志注入等)预留清晰扩展点。
Java免费学习笔记:立即使用
解锁 Java 大师之旅:从入门到精通的终极指南











