ThreadLocal初始化上的FindBugs警告

Question

我有ThreadLocal实例，该实例已使用重写的initValue方法初始化。 我也用@edu.umd.cs.findbugs.annotations.SuppressWarnings("SIC_INNER_SHOULD_BE_STATIC_ANON")进行了注释，如下所示。

@edu.umd.cs.findbugs.annotations.SuppressWarnings("SIC_INNER_SHOULD_BE_STATIC_ANON")
private ThreadLocal<Integer> dbSwitchCount=new ThreadLocal<Integer>() {
    @Override 
    protected Integer initialValue() {
        return 0;
    }
};

声纳报告仍然抱怨“性能-可以分解为命名的静态内部类”。 我如何确保上述抱怨被忽略，或者如何避免该抱怨？

Answer 1

进行Sonar建议的操作：“可以将其分解为命名的静态内部类”。

命名为静态内部类：

class MyClass {
    static class MyThreadLocal extends ThreadLocal<Integer> {
        @Override 
        protected Integer initialValue() {
            return 0;
        }
    }
    private ThreadLocal<Integer> dbSwitchCount = new MyThreadLocal();
}

我认为Sonar认为这是“性能”改进的原因是因为匿名类是非静态的，而使其变为静态可改善内存管理。

Answer 2

注释不起作用，因为您正在注释dbSwitchCount字段，而不是匿名类，并且错误报告未绑定到该字段。 错误报告在XML中的样子如下：

<BugInstance type="SIC_INNER_SHOULD_BE_STATIC_ANON" priority="3" rank="20" abbrev="SIC" 
             category="PERFORMANCE" first="1">
  <Class classname="MyClass$1">
    <SourceLine classname="MyClass$1" start="1" end="10" 
                sourcefile="MyClass.java" sourcepath="MyClass.java"/>
  </Class>
  <SourceLine classname="MyClass$1" start="7" end="7" startBytecode="0" endBytecode="0" 
              sourcefile="MyClass.java" sourcepath="MyClass.java"/>
</BugInstance>

dbSwitchCount有关dbSwitchCount字段的任何内容。 因此，当抑制过滤器工作时，它不知道dbSwitchCount批注以某种方式与此错误报告相关联。 不幸的是，我没有办法注释匿名类本身。 在不更改实际代码的情况下，唯一可以禁止显示此警告的方法是注释外部类：

@SuppressFBWarnings("SIC_INNER_SHOULD_BE_STATIC_ANON")
public class MyClass {

    private final ThreadLocal<Integer> dbSwitchCount=new ThreadLocal<Integer>() {
        @Override 
        protected Integer initialValue() {
            return 0;
        }
    };
}

这样，警告消失了（建议使用@SuppressFBWarnings注释代替）。

通常，实例绑定（非静态）线程本地是可疑的。 参见，例如， 这个问题 。 因此，可能最初的问题是dbSwitchCount应该声明为静态（这样SIC_INNER_SHOULD_BE_STATIC_ANON警告也将消失）。

更新我检查了此检测器的FindBugs代码。 看起来有可能添加缺少的bug注释，从而可以消除警告注释字段或封闭方法的警告。 我已经在我们的错误跟踪器中创建了一个票证 。

UPDATE-2已 在FindBugs主干中修复 。