Trong java, giá trị trả về trong khối đồng bộ có vẻ giống như kiểu xấu. Thật sự nó có ảnh hưởng sao?

Question

Tôi có một Collections.synchronizedList của WeakReference, _components;

tôi đã viết một cái gì đó như sau, mong các trình biên dịch để phàn nàn:

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

Nhưng trình biên dịch là hoàn toàn hài lòng. Lưu ý rằng List.add() trả về TRUE. Vì vậy, ok, bất kỳ lối ra từ một khối đồng bộ phát hành khóa, nhưng điều này không lạ? Nó giống như một "lỗ hổng" trong khối, tương tự như sử dụng trở lại trong một vòng lặp.

Bạn có hài lòng khi duy trì mã như thế này không?

Answer 1

Hoàn toàn ổn - khi đang trở về từ vòng lặp hoặc từ khối try có khối finally thích hợp. Bạn chỉ cần nhận thức được ngữ nghĩa, tại thời điểm đó nó có ý nghĩa hoàn hảo.

Đó chắc chắn đoạn code đơn giản hơn so với việc giới thiệu một biến địa phương vì lợi ích của nó:

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

Không có gì sai khi trả lại bên trong khối synchronized. Khóa sẽ được phát hành chính xác.