sincronizados y locales copias de las variables

Question

que estoy buscando en algún código heredado que tiene la siguiente expresión idiomática:

Map<String, Boolean> myMap = someGlobalInstance.getMap(); 
synchronized (myMap) { 
    item = myMap.get(myKey); 
} 

La advertencia que recibo de inspecciones de código de Intelli-J es:

Synchronization on local variable 'myMap' 

Es esta es la sincronización adecuada y por qué?

Map<String, Boolean> myMap = someGlobalInstance.getMap(); 
synchronized (someGlobalInstance.getMap()) { 
    item = myMap.get(myKey); 
} 

Answer 1

La razón por la que esto se señala como un problema es porque la sincronización en variables locales suele ser una mala idea.

Si el objeto devuelto por someGlobalInstance.getMap() es siempre el mismo, entonces el bloque sincronizado de hecho utilizar el Monitor de que los objetos cuasi-global y el código produce el resultado esperado.

También estoy de acuerdo con la sugerencia de utilizar un contenedor sincronizado, si solo necesita sincronizar las llamadas get()/put() y no tiene bloques sincronizados más grandes. Pero asegúrese de que el mapa esté solo accedido a través del contenedor o tendrá otra oportunidad para errores.

También tenga en cuenta que si someGlobalInstance.getMap() hace no devuelven el mismo objeto todo el tiempo, entonces incluso el segundo ejemplo de código no funcionará correctamente, incluso podría ser peor que el código original ya que es posible sincronizar en un objeto diferente de al que llamas get() en.

Answer 2

creo que sería mejor usar synchronized wrapper para su mapa

Answer 3

Creo que el código podría estar en buen estado, dependiendo de lo que hace el método getMap(). Si mantiene una referencia a una instancia que debe compartirse entre hilos, tiene sentido. La advertencia es irrelevante ya que la variable local no se inicializa localmente.

Answer 4

Alex tiene razón en que agregar un contenedor sincronizado llamando al Collections.synchronizedMap(Map) es un enfoque típico aquí. Sin embargo, si toma este enfoque, puede haber situaciones en las que necesite sincronizarse en el bloqueo Map; p.ej. al iterar sobre el mapa.

Map<String, String> syncMap = Collections.synchronizedMap(new HashMap<String, String>()); 

// Synchronized on map to prevent ConcurrentModificationException whilst iterating. 
synchronized (syncMap) { 
    for (Map.Entry<String, String> entry : syncMap.entrySet()) { 
    // Do work 
    } 
} 

En su ejemplo, la advertencia de la idea puede ser ignorado, ya que es evidente que su variable local: map se recupera desde otro lugar (someGlobalInstance) en lugar de crearse dentro del método, y por consiguiente puede acceder potencialmente de otros hilos.