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 doc，使用 Collections.synchronizedList 时需要同步对列表的每次访问。我不确定我的同步块是否会这样做？

1. 我对同步块的使用是否等同于

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

2. 我应该使用这样的 ReentrantLock

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

Answer 1

您不需要同步 简单操作，例如 add 或 remove，因为这是由实现在内部处理的，这正是原因您使用它们：避免自己处理同步

但对于 复合操作，如 迭代 或 多次删除 超出内部同步范围，您必须提供自己的锁定机制.

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() 初始化的列表的方法是线程安全的，无需使用进一步的synchronized 或其他锁定机制。其余的请参见@Pragmateek 的回答。