Findbugs : NP_LOAD_OF_KNOWN_NULL_VALUE - 싱글 톤 클래스 더블 체크

Question

나는 싱글 톤 클래스를 생성자에서 두 번 검사한다.

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이 표시됩니다. 따라서 우리는 런타임에 모든 요청 ID에 대해 단일 객체를 만들어야합니다.

감사합니다.

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 개) 요소

당신은 기본 size-based eviction을 위해 LRU 전략을 사용 Guava cache을 사용할 수 있습니다로 제한한다.

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 { 
    } 
}