使用Collection自身调用的removeAll（）的潜在错误

Question

在使用Sonar进行代码审查期间，发现以下代码是错误的代码：

ArrayList<String> ops = new ArrayList<String>();
ops.add("test");
ops.removeAll(ops);

Sonar抱怨集合本身调用了removeAll。

我同意这很丑陋，但这会引入错误吗？

注意：这不是我的代码，我正在对其进行审查。

Answer 1

问题是，是否ConcurrentModificationException可能导致，或列表损坏，循环不断，删除条目失败或类似原因。

ArrayList，尤其是在Oracle的JDK8中，似乎不会发生这些问题。

那是否表示该代码还可以呢？

不，这不好。

该代码：

• 依靠列表的实现removeAll足够聪明以处理一个非常奇怪的用例

• 不必要地复杂阅读和理解，从而造成维护问题

• 正在做不必要的工作，因此完成工作所需的时间超过了需要的时间（不是，这可能很重要）

您是在代码审查的背景下这么说的。我会对其进行标记，并与他们讨论为什么使用它，并从可靠性，维护和（非常次要）性能的角度解释为什么ops.clear();或ops = new ArrayList<String>();（取决于上下文）几乎肯定是一个更好的选择。