我甚至提到过: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,findbugs,sonarqube 来源: https://codeday.me/bug/20190529/1178085.html
本站声明: 1. iCode9 技术分享网(下文简称本站)提供的所有内容,仅供技术学习、探讨和分享; 2. 关于本站的所有留言、评论、转载及引用,纯属内容发起人的个人观点,与本站观点和立场无关; 3. 关于本站的所有言论和文字,纯属内容发起人的个人观点,与本站观点和立场无关; 4. 本站文章均是网友提供,不完全保证技术分享内容的完整性、准确性、时效性、风险性和版权归属;如您发现该文章侵犯了您的权益,可联系我们第一时间进行删除; 5. 本站为非盈利性的个人网站,所有内容不会用来进行牟利,也不会利用任何形式的广告来间接获益,纯粹是为了广大技术爱好者提供技术内容和技术思想的分享性交流网站。