[問題] 想請問一段多執行緒的程式碼

[coolcomm (coolcomm)]

以下程式碼節錄自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)
→ tkcn:因為 register 是 static method，沒有 this
→ tkcn:這寫法在 static method 是很常見的作法 http://goo.gl/ioAvZ
→ coolcomm:所以和static void synchronized register完全一樣囉?
→ tkcn:是，但你上面的例子還有個 getRegistry 在外面，所以不同
→ coolcomm:嗯 這部分了解了
※ 編輯: coolcomm        來自: 1.163.82.56          (01/24 23:24)
→ n3oanderson:This is check then act scenario, which is not
→ n3oanderson:thred safe.
囧 我發現我忘了貼getRegistry()的定義

static Set<IDKey> getRegistry() { return REGISTRY.get(); }

就這樣一行

※ 編輯: coolcomm        來自: 114.43.180.83        (01/27 09:11)
※ coolcomm:轉錄至看板 Programming                                 02/05 19:37