Findbugs：NP_LOAD_OF_KNOWN_NULL_VALUE - Singleton類雙重檢查

Question

我有一個Singleton類，在構造函數中進行雙重檢查。

Findbugs正在報告以下錯誤。

Load of known null value in ... NP_LOAD_OF_KNOWN_NULL_VALUE

class SomeClass { 
    private Object lock = new Object(); 
    private Map<String,Resource> resourceMap = new HashMap<>(); 

    public Resource getResource(String resourceId) { 
    if (resourceMap.get(resourceId) == null) { 
     synchronized(lock) { 
     if (resourceMap.get(resourceId) == null) 
      Resource resource = new Resource(); 
      resourceMap.put(resourceId,resource); 
     } 
    } 
    return resourceMap.get(resourceId); 
    } 
} 

我可以用一個靜態對象的引用去，但要求是創建一個單一對象的唯一請求ID。

說，請求ID 1將在那裏多個請求。所以，我們將不得不在運行時爲此爲所有請求標識創建單個對象。

謝謝，

Answer 1

您應該使用ConcurrentMap而不是雙重檢查鎖定模式。您的方法在地圖元素的可見性方面不正確（有關可見性的更多詳細信息，請參閱以下question）。

class SomeClass { 
    private final ConcurrentMap<String, Resource> resourceMap = new ConcurrentHashMap<>(); 

    public Resource getResource(String resourceId) { 
     return resourceMap.computeIfAbsent(resourceId, r -> new Resource()); 
    } 

    private static class Resource {} 
} 

編輯限制與LRU替換策略10元

您可以使用Guava cache使用LRU策略默認size-based eviction。

import com.google.common.cache.*; 

class SomeClass { 
    private final LoadingCache<String, Resource> resourceMap = 
      CacheBuilder.newBuilder() 
        .maximumSize(10L) 
        .build(new CacheLoader<String, Resource>() { 
         @Override 
         public Resource load(String key) { 
          return new Resource(); 
         } 
        }); 

    public Resource getResource(String resourceId) { 
     return resourceMap.getUnchecked(resourceId); 
    } 

    private static class Resource { 
    } 
}