我甚至提到过:Sonar Violation: Security – Array is stored directly
我的代码是—>
public final void setSelectedObjectsList(final ScheduleDTO[] selectedObjectsList)
// Security - Array is stored directly
//The user-supplied array 'selectedObjectsList' is stored directly.
{
if (selectedObjectsList != null) {
this.selectedObjectsList = selectedObjectsList.clone();
} else {
this.selectedObjectsList = null;
}
}
这已经在处理防御性复制,为什么声纳会在功能参数上对我大吼大叫.
这不是重复的
Sonar Violation: Security – Array is stored directly
再次,谢谢你的呐喊和时间.
解决方法:
不确定Sonar在想什么,但使用clone()进行防御性浅层复制应该可以正常使用数组,就像Arrays.copyOf和System.arrayCopy()一样.
另一方面,由于您已经在调用数组列表:selectedObjectsList,您还可以将其设为实际列表并重构一下:
public final void setSelectedSchedules(List<ScheduleDTO> selectedSchedules) {
this.selectedSchedules = selectedSchedules != null ? new ArrayList<ScheduleDTO>(selectedSchedules) : null;
}