In java, il valore restituito all'interno del blocco sincronizzato sembra un cattivo stile. Importa davvero?

Question

Ho una Collections.synchronizedList di WeakReference, _components;

ho scritto qualcosa di simile a quanto segue, aspettandosi il compilatore a lamentarsi:

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

Ma il compilatore è perfettamente soddisfatto. Notare che List.add() restituisce VERO. Quindi ok, qualsiasi uscita da un blocco sincronizzato rilascia il blocco, ma questo non sembra strano? È un po 'come un "buco" nel blocco, simile all'utilizzo del ritorno in loop.

Sarebbe felice mantenere il codice come questo?

Answer 1

È assolutamente soddisfacente, come il ritorno da un loop o da un blocco try con un blocco finally appropriato. Devi solo essere consapevole della semantica, a quel punto ha perfettamente senso.

E 'certamente il codice più semplice di introdurre una variabile locale per il gusto di farlo:

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

Non c'è niente di sbagliato nel tornare all'interno di un blocco synchronized. Il blocco verrà rilasciato correttamente.