Может ли следующий Singleton быть небезопасным в многопоточной среде?

Question

Я хочу быть уверенным, что мой экземпляр Singleton доступен безопасно и с минимальной синхронизацией, но у меня есть сомнение относительно предложения first if вне синхронизированного блока. Возможно ли, чтобы INSTANCE имел непустое значение, когда оно не было полностью построено? Если да, то как я могу решить проблему.

Я думаю, что в том числе весь блок get() уменьшит эффективность, потому что будет так много переменных конфигурации, которые необходимо читать тысячи раз в секунду из разных частей программы с помощью этого метода get().

public class ConfsDBLoader { 

    private static ConfsDBLoader INSTANCE = null; 
    private static final Object lock = new Object(); 

    private ConfsDBLoader() { //Codes loading the db objects 
    } 

    public static ConfsDBLoader get(){ 
     if(INSTANCE != null){ 
      return INSTANCE; 
     } else { 
      synchronized(lock){ 
       if(INSTANCE == null){ 
        INSTANCE = new ConfsDBLoader(); 
       } 
       return INSTANCE; 
      } 
     } 
    } 

} 

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

Answer 1

Нет. Синхронизация недостаточно, чтобы убедиться, что вы видите правильное значение на INSTANCE. Вы можете увидеть непустой, но поврежденный экземпляр, если ваш ConfsDBLoader, потому что он может быть неправильно сконструирован к тому моменту, когда другой поток вызывает getInstance().

У вас есть 3 варианта:

• нетерпеливого Initialize и сделать окончательный
• Синхронизировать весь метод
• Сделать INSTANCE volatile