使用List参数仍触发OCP_OVERLY_CONCRETE_PARAMETER错误求助

问题描述 #

我在类中编写了一个接收List类型参数的transform方法，运行SpotBugs检查时，返回了“过于具体的参数（OCP_OVERLY_CONCRETE_PARAMETER）”错误。该方法仅对参数进行遍历操作，但SpotBugs提示：

.TestClass.transform(List): 1st parameter 'dtos' could be declared as java.lang.Iterable instead
[com.TestClass] At TestClass.java:[line 146] OCP_OVERLY_CONCRETE_PARAMETER

方法代码如下：

public void transform(final List<Dto1> dtos) { //line 145
  for (final Dto1 dto : dtos) {                //line 146
    Dto2 dto2 = convert(dto);
    ..
  }
}

想请教我忽略了什么，为何会触发该错误？

错误原因解析 #

这个错误触发的核心原因是违反了开闭原则（Open/Closed Principle，OCP），同时也是为了提升方法的通用性：

• 你的transform方法只用到了List的遍历能力，而Iterable是所有可遍历集合类型的父接口（包括List、Set、Queue甚至自定义的可遍历容器）。
• 把参数类型声明为更抽象的Iterable<Dto1>，不会影响现有List类型的调用方，同时能让更多类型的可遍历集合传入这个方法，扩展了方法的适用范围。
• SpotBugs通过这个提示，提醒你不要给方法参数绑定不必要的具体类型约束——既然只需要遍历能力，就没必要限定死必须是List。

简单来说，你忽略了方法参数应优先使用抽象类型而非具体类型的设计原则，这样能让代码更灵活、更符合面向对象的设计思想。

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