有违反声纳的行为:
违反声纳:安全性-阵列直接存储
public void setMyArray(String[] myArray) { this.myArray = myArray; }
解:
public void setMyArray(String[] newMyArray) { if(newMyArray == null) { this.myArray = new String[0]; } else { this.myArray = Arrays.copyOf(newMyArray, newMyArray.length); } }
但是我想知道为什么吗?
它的抱怨,你存储阵列是由主叫方持有相同的数组。也就是说,如果调用者随后修改了此数组,则存储在对象中的数组(以及对象本身)将发生变化。
解决方案是在传递对象时在对象内进行复制。这称为 防御性复制 。收集的后续修改不会影响存储在所述对象内的阵列。
在返回集合时(例如在相应的getMyArray()调用中)通常也要这样做。否则,接收者可能会执行修改并影响存储的实例。
getMyArray()
注意,这显然适用于所有可变集合(实际上是所有可变对象),而不仅仅是数组。还要注意,这会对性能产生影响,需要与其他问题一起进行评估。