Collections.synchronizedList引發的線程安全問題
有些容器是線程安全的(Vector,ConcurrentLinkedQueue等),有些則不是(list等),利用類
似 private static List<Task> taskQueue = Collections.synchronizedList(new
LinkedList<Task>());的方法可以得到本身不是線程安全的容易的線程安全的狀態,但是要注意
的是線程安全僅僅指的是如果直接使用它提供的函數,比如:queue.add(obj); 或者
queue.poll(obj);,這樣我們自己不需要做任何同步。
但如果是非原子操作,比如:
1. if(!queue.isEmpty()) {
2. queue.poll(obj);
3. }
我們很難保證,在調用了isEmpty()之後,poll()之前,這個queue沒有被其他線程修改。
所以對於這種情況,我們還是需要自己同步:
1. synchronized(queue) {
2. if(!queue.isEmpty()) {
3. queue.poll(obj);
4. }
5. }
對於Collections.synchronizedList(new LinkedList<Task>()),
在返回的列表上進行迭代時,用戶必須手工在返回的列表上進行同步:
List list = Collections.synchronizedList(new ArrayList()); ... synchronized(list) {
Iterator i = list.iterator(); // Must be in synchronized block while (i.hasNext())
foo(i.next()); }