必要に応じてキーを追加するときにConcurrentMapを同期する必要はありますか？

Question

私はConcurrentMap < String、SomeObject >オブジェクトを持っています。 SomeObjectが存在する場合はそれを返すメソッドを作成するか、新しいSomeObjectを作成してマップに入れ、存在しない場合はそれを返します。

理想的には、ConcurrentMapのputIfAbsent(key, new SomeObject(key))を使用することができますが、そのたびに新しいSomeObject（キー）が作成されることになり、非常に無駄に思えます。

は、だから私は、次のコードに頼っが、それはこれを処理するための最良の方法だということを確認していない：

public SomeValue getSomevalue(String key){ 

    SomeValue result = concurrentMap.get(key); 

    if (result != null) 
    return result; 

    synchronized(concurrentMap){ 

    SomeValue result = concurrentMap.get(key); 

    if (result == null){ 
     result = new SomeValue(key); 
     concurrentMap.put(key, result); 
    } 

    return result; 
    } 
} 

Answer 1

理想的には、私はのConcurrentMapのputIfAbsent（キー、新しいSomeObjectの（キー））を使用することができますしかし、それは毎回新しいSomeObject（キー）を作成することを意味し、非常に無駄に思える。

次にcomputeIfAbsentを使用：

concurrentMap.computeIfAbsent(key, SomeObject::new); 

のConcurrentMapと​​​​を使用ブロックの中央にマップに対して操作を実行することから、他のスレッドを妨げません。 ConcurrentMapは同期のためにマップのモニターを使用することを約束しておらず、ConcurrentHashMapとConcurrentSkipListMapはマップ・オブジェクト上で同期しません。

ConcurrentMapインターフェイスでは、値が1回だけ計算されること、またはキーが既に存在する場合に値が計算されないことを保証しないことに注意してください。 ConcurrentHashMapはこれらの約束をしますが、ConcurrentSkipListMapは行いません。