¿Qué significa "sincronización inconsistente"?

Question

Ésta es mi clase de 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 dice:

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

¿Qué significa y qué estoy haciendo mal? Estoy tratando de evitar problemas cuando dos o más clientes llaman al Foo.scan() al mismo tiempo.

Answer 1

Es por culpa sólo se está sincronizando cuando se establece la variable names y no cuando lo lea. Así que entre la lectura y la escritura podría ejecutarse otro subproceso y usted crearía dos ArrayLists y los llenaría con datos, el primero creado obtendría GC.

Es necesario poner el bloque sincronizado en torno a la lectura y la escritura o añadir el modificador sincronizado con el método.

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 
     } 
     } 
    } 
    } 

Answer 2

La primera vez que se hace referencia en el interior namesscan está fuera de synchronized bloque.
por ejemplo, si scan se llama dos veces a partir de dos hilos diferentes y names es nulo, puede ir así

1. if (names == null) desde el primer hilo es procesado (a true).
2. if (names == null) desde el segundo hilo se procesa (a true).
3. Primer hilo entra en el bloque synchronized, asigna names y sale del bloque synchronized.
4. El segundo hilo entra en el bloque synchronized, asigna names y sale del bloque synchronized.

Ahora, names se inicializa dos veces. Y este es solo un escenario posible en el que se obtienen resultados inesperados.