, senkronize blok içerisindeki dönüş değeri kötü tarzı gibi görünüyor. Gerçekten önemli mi?

Question

I WeakReference, _components bir Collections.synchronizedList sahiptir;

Ben derleyici şikayet bekliyor, aşağıdaki gibi bir şeyler yazdı:

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

Ama derleyici mükemmel memnun olduğunu. List.add() öğesinin TRUE değerini döndürdüğünü unutmayın. Tamam, senkronize edilmiş bir bloktan herhangi bir çıkış, kilidi serbest bırakır, ancak bu LOOK garip değil mi? Bu, bir döngüde geri dönüşün kullanılmasına benzeyen, blokta bir "delik" gibi bir şey.

Böyle kodu muhafaza mutlu olabilir mi? Bir synchronized bloğun içine dönen yanlış bir şey yoktur

Answer 1

Bir döngüden döndüğünden veya uygun bir finally bloğuna sahip bir try satırından kesinlikle iyi. Sadece anlambilimden haberdar olmanız gerekir, bu noktada tam anlamıyla mantıklıdır.

Kesinlikle onun uğruna bir yerel değişken tanıtan daha basit kod: çok zaten artık ben genellikle çok bloğun içine dönmek iken

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

. Kilit doğru şekilde bırakılacaktır.