Java返回Parent/Child列表触发Sonar RSPEC-1452的合规修复方案

我有一个Parent类，以及继承它的Child类。当方法返回单个Parent或Child实例时，直接返回Parent类型完全可行：

public Parent getStuff(boolean p) {
  return p ? new Parent() : new Child();
}

但当方法需要返回List<Parent>或List<Child>时，我用了List<? extends Parent>作为返回类型：

public List<? extends Parent> getListOfStuff(boolean p) {
  return p
    ? myOtherMethodThatReturnsListOfParents()
    : myOtherMethodThatReturnsListOfChilds();
}

public List<Parent> myOtherMethodThatReturnsListOfParents() {
  return new ArrayList<Parent>();
}

public List<Child> myOtherMethodThatReturnsListOfChilds() {
  return new ArrayList<Child>();
}

这时Sonar报出严重问题：Generic wildcard types should not be used in return types（RSPEC-1452规则）。

Sonar的这条规则禁止在返回类型中使用通配符，核心原因是会给调用方带来不便——调用方无法向返回的集合中添加任何Parent或其子类实例（编译器会报错），同时也会让API的语义变得模糊。以下是几种正确的修复方式：

方案1：统一返回List<Parent>（简洁转换方案） #

直接将List<Child>转换为List<Parent>返回，利用ArrayList的构造方法接收子类型集合完成类型转换：

public List<Parent> getListOfStuff(boolean p) {
  return p
    ? myOtherMethodThatReturnsListOfParents()
    : new ArrayList<Parent>(myOtherMethodThatReturnsListOfChilds());
}

这种方式简单直接，调用方可以正常操作返回的集合（添加、删除元素不受限）；缺点是会创建新集合副本，若原集合较大存在一定性能开销。

方案2：返回不可变的List<Parent> #

如果业务场景允许集合只读，可以用Collections.unmodifiableList包装，避免复制集合元素：

import java.util.Collections;

public List<Parent> getListOfStuff(boolean p) {
  return p
    ? myOtherMethodThatReturnsListOfParents()
    : Collections.unmodifiableList(myOtherMethodThatReturnsListOfChilds());
}

此方式性能更优，但调用方无法修改返回的集合（修改会抛出UnsupportedOperationException），适合只读场景。

方案3：调整内部方法返回类型 #

如果业务允许，可将myOtherMethodThatReturnsListOfChilds的返回类型改为List<Parent>，这样getListOfStuff可直接返回无需转换：

public List<Parent> myOtherMethodThatReturnsListOfChilds() {
  List<Parent> list = new ArrayList<>();
  // 向list中添加Child实例
  return list;
}

public List<Parent> getListOfStuff(boolean p) {
  return p
    ? myOtherMethodThatReturnsListOfParents()
    : myOtherMethodThatReturnsListOfChilds();
}

这种方式最优雅，但需要修改内部方法签名，适合内部方法仅在此场景下使用的情况。

内容的提问来源于stack exchange，提问作者elcye