作者coolcomm (coolcomm)
看板java
標題[問題] 想請問一段多執行緒的程式碼
時間Thu Jan 24 22:06:44 2013
以下程式碼節錄自org.apache.commons.lang3.builder.HashCodeBuilder
private static final ThreadLocal<Set<IDKey>> REGISTRY = new ThreadLocal<>();
static void register(Object value) {
synchronized(HashCodeBuilder.class) {
if (getRegistry() == null) { REGISTRY.set(new HashSet<IDKey>()); }
}
getRegistry().add(new IDKey(value));
}
想請問
ThreadLocal不是thread-safe嗎 為甚麼還要加上synchronized關鍵字?
--
※ 發信站: 批踢踢實業坊(ptt.cc)
◆ From: 1.163.82.56
另外 synchronized(HashCodeBuilder.class) 有什麼特別含意嗎
為什麼不直接寫 synchronized(this)...
※ 編輯: coolcomm 來自: 1.163.82.56 (01/24 22:12)
1F：→ tkcn:因為 register 是 static method，沒有 this 01/24 22:24
3F：→ coolcomm:所以和static void synchronized register完全一樣囉? 01/24 22:29
4F：→ tkcn:是，但你上面的例子還有個 getRegistry 在外面，所以不同 01/24 23:16
5F：→ coolcomm:嗯 這部分了解了 01/24 23:21
※ 編輯: coolcomm 來自: 1.163.82.56 (01/24 23:24)
6F：→ n3oanderson:This is check then act scenario, which is not 01/26 15:37
7F：→ n3oanderson:thred safe. 01/26 15:39
囧 我發現我忘了貼getRegistry()的定義
static Set<IDKey> getRegistry() { return REGISTRY.get(); }
就這樣一行
※ 編輯: coolcomm 來自: 114.43.180.83 (01/27 09:11)
※ coolcomm:轉錄至看板 Programming 02/05 19:37