2 回答

TA貢獻1796條經驗 獲得超7個贊
首先要明白,synchronize加鎖的一般都是對某個對象而言的(也可以對類進行加鎖)。
1中非線程安全的synchronized的加鎖對象其實是ListHelper<E>實例化的對象,而不是list,其他的線程無法再對該ListHelper<E>實例化的對象進行操作,對于該例,就是無法再進行putIfAbsent()方法的使用,但是其中的list是public的,所以可以直接對list進行操作,比如list.add()等操作,進而造成線程不安全

TA貢獻1864條經驗 獲得超6個贊
大概是由于public List<E> list = Collections.synchronizedList(new ArrayList<E>());
的這個list是public的,可以被直接修改的。
假如有多個線程直接修改這個list。
方法1雖然把putIfAbsent(E x)
這個方法加鎖了,但是其他線程依然可以修改由于public暴露出來的list。
比如線程A:listHelper.list.add(x)
的時候,線程B剛好在執行boolean absent = !list.contains(x);
,有可能線程A把x添加進去了,但是線程B的absent也判斷為true,然后線程B就再次add了x。
方法2是根據list加鎖,所以只有持有list的鎖才能進入判斷和添加,當線程A在listHelper.list.add(x)
的時候,線程B是不能進入到synchronized (list)
方法內的
添加回答
舉報