Утверждение не null объекта Singleton

Question

Нужно ли Assert.notNull Singleton Object?

У меня есть класс:

public class ComponentFactory { 
    private static LibraryFrame libraryFrame; 

    public static synchronized LibraryFrame getLibraryFrame() { 
     if (libraryFrame == null) { 
      libraryFrame = new LibraryFrame(); 
     } 
     return libraryFrame; 
    } 
} 

Теперь он нужен для использования в качестве:

LibraryFrame libraryFrame = ComponentFactory.getLibraryFrame(); 
Assert.notNull(libraryFrame); 
// other part 

Здесь Утверждают класс org.springframework.util.Assert.

Если сообщение Assertion failed is anyway вызывает System.exit (0) после сбоя?

Answer 1

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

Answer 2

Согласно the documentation for that class, он действительно предназначен для использования методом для обоснования своих аргументов. (То есть он предназначен для принудительного использования предварительных условий.) У вас должны быть модульные тесты для ComponentFactory, и они должны утверждать, что его статический метод getLibraryFrame() не возвращает null, но не стоит утверждать это во время выполнения ,

Answer 3

Ну, когда вы используете класс Assert из JUnit, любой метод там есть все void, но он использует фреймворк JUnit, чтобы сообщить вам, прошел тест или не прошел тест.

Если это метод модульного тестирования, то вы выполнили свой тест, но если это было связано с временем выполнения, используйте условие if.

Answer 4

Я рекомендую использовать initialization-on-demand pattern (т. Е. Ленивый загрузочный синглтон) для вашего синглтона. В этом шаблоне, если построение экземпляра не выполняется, вы получите исключение, иначе вы получите объект.

Это улучшает ваше решение двумя способами: он не имеет накладных расходов на синхронизацию и у него нет служебных данных Assert. Оба метода генерируют исключение, если объект singleton не создан - в коде вы получите IllegalArgumentException из assert.

Использование инициализации по требованию ваш код будет:

public class ComponentFactory 
{ 
    private ComponentFactory() 
    { 
    } 

    private static class LazyHolder 
    { 
    public static final LibraryFrame INSTANCE = new LibraryFrame(); 
    } 

    public static LibraryFrame getLibraryFrame() 
    { 
    return LazyHolder.INSTANCE; 
    } 
} 

Чтобы использовать все равно будет:

LibraryFrame libraryFrame = ComponentFactory.getLibraryFrame(); 

... кроме вас больше не нужно ни Assert, ни Synchronized ,