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;
}