En Java, el valor de retorno dentro del bloque sincronizado parece un mal estilo. ¿Realmente importa?

Question

Tengo una Collections.synchronizedList de WeakReference, _components;

escribí algo como lo siguiente, esperando que el compilador para quejarse:

public boolean addComponent2(Component e) { 
    synchronized (_components) { 
     return _components.add(new WeakReference<Component>(e)); 
    }   
} 

Pero el compilador es perfectamente satisfecho. Tenga en cuenta que List.add() devuelve TRUE. Entonces, cualquier salida de un bloque sincronizado libera el bloqueo, ¿pero esto no es extraño? Es como un "agujero" en el bloque, similar al uso de retorno en un bucle.

¿Sería feliz manteniendo un código como este?

Answer 1

Está absolutamente bien, ya que regresa de un bucle, o de un bloque try que tiene un bloque finally apropiado. Solo necesitas ser consciente de la semántica, momento en el que tiene perfecto sentido.

Es ciertamente código más simple que la introducción de una variable local para el bien de ella:

// Ick - method body is now more complicated, with no benefit 
public boolean addComponent2(Component e) { 
    boolean ret; 
    synchronized (_components) { 
     ret = _components.add(new WeakReference<Component>(e)); 
    } 
    return ret; 
} 

Answer 2

No hay nada de malo en regresar dentro de un bloque synchronized. El bloqueo se lanzará correctamente.