Question

我甚至提到过：Sonar Violation: Security - Array is stored directly

我的代码是---＆gt;

    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

再次，谢谢你的呐喊和时间。

Answer 1

不确定Sonar的想法，但clone()的防御性浅层复制应该适用于数组，Arrays.copyOf和System.arrayCopy()也是如此。

另一方面，既然你已经在列表中调用了一个列表：selectedObjectsList，你也可以把它变成一个实际的列表并重构一下：

public final void setSelectedSchedules(List<ScheduleDTO> selectedSchedules) {
    this.selectedSchedules = selectedSchedules != null ? new ArrayList<ScheduleDTO>(selectedSchedules) : null;
}

安全性 - 数组直接存储

1 个答案: