Правильно ли это использование двухэлементной проверки?

Question

Я в основном ищу механизм кэширования для пользователей HelperWrapper. Правильно ли это использование двухместного синглета?

final class HelperWrapper { 
  private static volatile Helper helper = null; 
  
  public static Helper getHelper() { 
    if (helper == null) { 
      synchronized (HelperWrapper.class) { 
        if (helper == null) { 
          helper = new Helper(); 
        } 
      } 
    } 
    return helper; 
  } 
} 

Я знаю, что это не помешало бы кому-либо создавать свои собственные объекты Helper, но это не цель.

Дополнительная информация:

• Я не могу изменить класс Helper, чтобы конструктор приватным. Я тоже этого не хочу.
• требования диктуют, что помощник не должен быть охотно инстанцирован

Answer 1

По состоянию на Java 5, то реализации is safe. Тем не менее, рекомендуется использовать внутренний класс вместо:

final class HelperWrapper { 
    private static final class DeferredLoader { 
    public static final Helper HELPER_INSTANCE = new Helper(); 
    } 

    public static Helper getHelper() { 
    return DeferredLoader.HELPER_INSTANCE; 
    } 
} 

Это и обеспечивает безопасность потоки и ленивые инициализации, используя при этом гарантии безопасности нити инициализации класса, чтобы избежать летучих или синхронизации ключевых слов на каждый вызов.