The list being synchronized only means that add
, remove
etc. operations are synchronized and therefore atomic. Iteration however is not and if a thread adds
while another is iterating, you could get a ConcurrentModificationException.
By manually synchronizing your iteration block, you ensure that the list is not modified while iterating.
One alternative is to use a CopyOnWriteArrayList
which provides an iterator that iterates over the list as it was known when the iteration started, regardless of subsequent modifications. That collection is however not very efficient if you need to change the content of the list very often.