Java synchronized statement

Question

Мне нужен потолочный arraylist вот так.

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. Является ли мое использование синхронизированных блоков эквивалентно

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

Вам не нужно синхронизировать простых операцийдобавить или удалить, поскольку это обрабатывается внутри реализации и именно поэтому вы используете их: чтобы избежать обработки синхронизации себя

Но для составных операций как итераций или множественного удаления, которые находятся вне объема внутренней синхронизации вы должны предоставить свой собственный механизм блокировки.

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.