java – 安全 – 数组直接存储

我甚至提到过: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;
}
上一篇:java – 来自“com.sun.*”和“sun.*”包的类不应该用于Jersey客户端的Sonar问题


下一篇:Shell命令-文件及内容处理之split、paste