多线程正确性-同步不一致

这有什么问题...？

public final void setListValid(final List<ValidRes> listValidRes) {
    this.listValidRes = listValidRes;
}

声纳对我大喊大叫：

Inconsistent synchronization of xxx.listValidRes; locked 50% of time

有人知道我需要做什么吗？

---

解决方案

问题中给出的代码没有同步。我假设您在代码中的其他位置同步了this.listValidRes。而这正是Sonar告诉您的：如果您对某个资源进行同步，请对所有使用进行同步，或者根本不进行同步并让其他人处理它。

基本上这是一个设计决定：

• 您可以选择不同步并让客户端处理它。这样做的好处是，如果没有同步，它的速度会快得多。因此，如果您的类是在单线程设置中使用的，那么最好放弃同步。但将其清楚地记录为而不是线程安全或客户端将以多线程方式使用它并抱怨奇怪的错误...

• 如果选择(或必须)同步，则对关键资源的每次使用执行同步。有不同的方法可以实现这一点。也许您想要显示您实际上已同步的资源的使用情况。也许我或其他人可以在这方面给你一些好的建议。