Java同步語句

Question

我需要像這樣的線程安全列表。

public class BookingList { 

    private List<Booking> bookings; 

    public BookingList() { 
     bookings = Collections.synchronizedList(new ArrayList<Booking>()); 
    } 

    @Override 
    public void addBooking(Booking booking) 
    { 
    synchronized (bookings) { 
     bookings.add(booking); 
    } 
    } 

    @Override 
    public void removeBooking(Booking booking) 
    { 
    synchronized (bookings) { 
     bookings.remove(booking); 
    } 
    } 
} 

根據java文檔，當使用Collections.synchronizedList時，需要同步每個訪問列表。我不確定我的同步塊是否會這樣做？

1. 是我使用synchronized塊相當於

   ... 
   public synchronized void addBooking(Booking booking) { 
       bookings.add(booking); 
   } 
   

2. 我應該使用這樣

   private Lock lock = new ReentrantLock(); 
   public void addBooking(Booking booking) { 
       try { 
       lock.lock; 
       bookings.add(booking); 
       } finally { 
       lock.release(); 
       } 
   } 
   

Answer 1

一個ReentrantLock的你並不需要同步簡單的操作 like add或刪除，因爲這是由執行內部處理，這也正是爲什麼你使用它們：爲了避免處理同步自己

但複合操作像迭代或多個移除是出在內部同步的範圍內，您必須提供您自己的鎖定機制。

Answer 2

回答您的問題：

1：

public synchronized void addBooking(Booking booking) { 
    bookings.add(booking); 
} 

相當於

public void addBooking(Booking booking) { 
    synchronized (this){ 
     bookings.add(booking); 
    } 
} 

2：你的榜樣，你不應該使用ReentrantLock。調用已用Collections.synchronizedList()初始化的列表的方法是線程安全的，不需要使用其他鎖定機制。其餘請參閱@ Pragmateek的答案。