声纳在两个稍微相似的场景下的不同行为

Question

考虑以下案例

public void setRules(MyRule[] rules) {
    this.rules=rules;
}

Sonar 抱怨说，

接收数组的构造函数和方法应该克隆对象和 存储副本。这可以防止用户未来的更改影响 内部功能。

这是真的，当有人在设置后更改array 的值时，我明白了这一点，该更改可能会影响内部函数。如果我们克隆该数组更改将不会生效。

现在考虑另一种情况

public void setExecutions(List<MyRule> ruleList) {
    this.ruleList= ruleList;
}

现在Sonar 没有告诉任何问题。为什么这里不需要克隆？

Answer 1

事实上，这个 PMD 规则背后只有一个动机：原语是按值传递的，而对象是按引用传递的。数组呢？数组是 Java 中的对象，因此也是按引用传递的，但一些初级开发人员可能没有意识到这一点，并且可能认为整个数组是按值传递的。我个人认为这条规则毫无价值。