Findbugs-調用java.util.concurrent.ConcurrentHashMap的順序可能不是原子的

Question

我的項目中有以下代碼

// Declaration

private ConcurrentHashMap<Long, ConcurrentHashMap<String, Object>> instanceSpecificBeanMap = new ConcurrentHashMap<>();

地圖的使用

ConcurrentHashMap<String, Object> beanMapForInstance = instanceSpecificBeanMap.get(currentInstanceId);
    Object beanObject = null;
    if(beanMapForInstance == null){
        synchronized (initLockObject) {
            beanMapForInstance = instanceSpecificBeanMap.get(currentInstanceId);
            if(beanMapForInstance == null){
                beanMapForInstance = new ConcurrentHashMap<>();
                instanceSpecificBeanMap.put(currentInstanceId, beanMapForInstance);
            }
        }
    }

我認為上述操作是原子的，盡管findbugs將其顯示為問題？ 我在這里想念什么嗎？

編輯

beanMapForInstance僅可從此類訪問。 沒有其他類可以訪問此變量。

另外，目前我們正在使用Java 7

Answer 1

使用putIfAbsent方法。 如果給定鍵不存在新值，則恰恰是打算在映射中自動添加一個新值。

ConcurrentHashMap<String, Object> newMap = new ConcurrentHashMap<>();
ConcurrentHashMap<String, Object> beanMapForInstance = instanceSpecificBeanMap.putIfAbsent(currentInstanceId, newMap);
if (beanMapForInstance == null) {
    beanMapForInstance = newMap;
}
(... work with beanMapForInstance ...)

Answer 2

您可以使用Java 8中的computeIfAbsent簡化此過程並解決問題：

ConcurrentHashMap<String, Object> beanMapForInstance = 
        instanceSpecificBeanMap.computeIfAbsent(
            currentInstanceId, 
            k -> new ConcurrentHashMap<>()  // might need explicit generics
        );