Findbugs: NP_LOAD_OF_KNOWN_NULL_VALUE - Singleton-Klasse Doppelte Überprüfung

Question

Ich habe eine Singleton-Klasse mit Doppelprüfung im Konstruktor.

Findbugs meldet den folgenden Fehler.

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); 
    } 
} 

ich mit einem statischen Objektverweis gehen, aber die Voraussetzung ist, ein einzelnes Objekt für einen einzigartigen Request-IDs zu erstellen.

Sagen wir, Request ID 1 wird für mehrere Anfragen da sein. Daher müssen wir zur Laufzeit ein einzelnes Objekt für alle Request-IDs erstellen.

Danke,

Answer 1

Sie einen ConcurrentMap anstelle des Doppelrückschlagsperrmuster verwendet werden soll. Ihre Methode ist in Bezug auf die Sichtbarkeit von Elementen der Karte nicht korrekt (weitere Informationen zur Sichtbarkeit finden Sie unter 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 {} 
} 

bearbeiten zu 10 Elementen mit LRU-Ersetzungsstrategie begrenzen

Sie Guava cache verwenden können, die LRU-Strategie size-based eviction für Standard verwendet.

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