Fabryczna metoda tworzenia instancji singleton

Question

Napisałem poniżej kod w statycznej metodzie fabrycznej, aby zwrócić pojedynczą instancję DefaultCache.

public static ICache getInstance() { 
    if (cacheInstance == null) { 
     synchronized (ICache.class) { 
     if (cacheInstance == null) { 
      cacheInstance = new DefaultCache(); 
     } 
     } 
    } 
    return cacheInstance; 
} 

Czy naprawdę potrzebujemy 2nd czek null dla cacheInstance wewnątrz bloku zsynchronizowanego?

Answer 1

Posiadać prywatnego konstruktora, aby uniknąć wszystkich tych kontroli.

class Singleton { 
    private static final Singleton instance = new Singleton(); 

    private Singleton() { 
    } 

    public static final Singleton getInstance() { 
     return instance; 
    } 
} 

Answer 2

Zakładając cacheInstance jest statyczny należy rozważyć dwie możliwe strategie inicjalizacji:

• Lazy initilization: Pole Singleton jest wypełniana instancji w momencie, kiedy to po raz pierwszy użyty (jak na swoje przykład).

• Inicjalizacja (Standarad): Pole singleton jest wypełnione instancją zaraz po załadowaniu klasy przez maszynę JVM.

Więc jeśli zamierzasz używać leniwej inicjalizacji, powinieneś sprawdzić, czy nie ma to wartości zerowej, w przeciwnym razie po prostu zwrócisz wskaźnik zerowy.

Alternatywnie można również zainicjować pole w bloku statycznym.

//class declaration 
static { 
    cacheInstance = new DefaultCache(); 
} 

lub w tym samym wierszu, w którym deklarują, że

private static final DefaultCache cacheInstance = new DefaultCache(); 

Answer 3

chciałbym uniknąć konieczności sprawdzania rywalizacją używając leniwie zainicjowany pojedyncza:

public class Singleton { 
    public static ICache getInstance() { 
     return LazyCacheInitializer.INSTANCE; 
    } 

    private class LazyCacheInitializer { 
     private static final ICache INSTANCE = new DefaultCache(); 
    } 
} 

Answer 4

Musisz drugi czek ponieważ wartość mogła zostać ustawiona przez inny wątek podczas próby uzyskania blokady. W rzeczywistości nie masz bezpiecznego widoku tej wartości, dopóki nie znajdziesz się w zsynchronizowanym bloku. Mogło to zostać ustawione przez inny wątek jakiś czas temu.

Najprostszym leniwy Singleton jest użycie enum

public enum DefaultCache implements ICache { 
    INSTANCE 
} 

Zakładam, że nie robią tego tak można zmienić realizacji.

BTW: Sugeruję, aby w miarę możliwości używać tylko bezstanowych singletonów i używać iniekcji dependanecy dla wszystkich obiektów stanowych tak bardzo, jak to możliwe.

Answer 5

Tak, w przeciwnym razie można utworzyć więcej niż dwie instancje. Załóżmy, że masz więcej niż jeden wątek. Pierwszy test jest wykonywany w warunkach ras, tzn. Niektóre wątki widzą zmienną równą zeru i próbują ustawić instancję. Bez drugiej kontroli każdy wątek utworzy nową instancję.

Answer 6

zawsze można użyć w oparciu wdrożenie Singleton z enum jak Peter pisał, ale należy pamiętać, że wszystkie implementacje są samotnymi w klasie ładowarka.

Należy również zwrócić uwagę na interfejs Cloneable.