Question

我的代码如下。我明白了

public MyClass{

    private static DataSource dataSource = null;

    private static DataSource getDataSource(){
        if (dataSource == null) {
            try {
                dataSource = // something.
            } catch (Exception e) {
                // some exception.
            }
        }

        return dataSource;
    }

    public List doSomething(){

        // ...

        if(dataSource == null){
            dataSource = getDataSource();
        }

        dataSource.getConnection();
        // ...

    }
}

我在声纳anaylsis中看到以下消息。

Dodgy - Write to static field from instance method

This instance method writes to a static field. This is tricky to get correct if multiple instances are being manipulated, and generally bad practice.
findbugs:ST_WRITE_TO_STATIC_FROM_INSTANCE_METHOD Sep12 Reliability > Architecture

除了我们正在改变doSomething方法中的静态变量之外，我在这个实现中看到一切都没问题。我们如何解决这个问题？

Answer 1

不确定静态分析工具的工作原理，但是 -

尝试通过静态设置器写入您的值：

private synchronized static void setDataSource(DataSource ds) {
    dataSource = ds;
}

这样你就可以了

   if(dataSource == null){
        setDataSource(getDataSource());
   }

Answer 2

您必须在两个解决方案之间进行选择，首先在您的Datasource类中添加getDatasource()方法并同步您实例化静态字段的块：

    private static DataSource getDataSource(){
     synchronize(DataSource.class){
            if (dataSource == null) {
                try {
                    dataSource = // something.
                } catch (Exception e) {
                    // some exception.
                }
            }
       }
        return dataSource;
      }

然后您将从doSomething()方法

调用此方法

public void doSomething(){
//
DataSource dataSource=DataSource.getDataSource();
//
}

第二个解决方案是使用静态块或直接从声明中在类加载时实例化您的字段。

static {
 dataSource = // something.
}

从实例方法写入静态字段

2 个答案: