java - "inconsistent synchronization"是什么意思？

Question

这是我的 Java 1.6 类:

public class Foo {
  private ArrayList<String> names;
  public void scan() {
    if (names == null) {
      synchronized (this) {
        this.names = new ArrayList<String>();
        // fill the array with data
      }
    }
  }
}

Findbugs 说:

Inconsistent synchronization of com.XXX.Foo.names; locked 40% of time

这是什么意思，我做错了什么？当两个或多个客户端同时调用 Foo.scan() 时，我试图避免出现问题。

Answer 1

这是因为您只在设置 names 变量时进行同步，而不是在读取它时进行同步。因此，在读取和写入之间可以执行另一个线程，您将创建两个 ArrayList 并用数据填充它们，创建的第一个将进行 GC。

您需要将 synchronized block 放在读取和写入周围，或者在方法中添加 synchronized 修饰符。

public class Foo {
  private ArrayList<String> names;
    public void scan() {
      synchronized (this)
        if (names == null) {
           this.names = new ArrayList<String>();
           // fill the array with data
         }
       }
     }
  }