Что хотят услышать интервьюеры:

Ручная реализация Singleton обычно плоха тем, что она легко ломается в многопоточной среде и требует дополнительной проверки на корректность. Такой код часто усложняет тестирование и повышает связанность компонентов. Кроме того, Singleton может скрыто превращаться в глобальное состояние, что делает систему менее предсказуемой.

Определение:

Ручной Singleton — это шаблон, в котором разработчик сам пишет логику ограничения создания объекта до одного экземпляра. Недостаток такого подхода в том, что легко допустить ошибки в синхронизации, инициализации и управлении жизненным циклом объекта. В результате код становится хрупким, плохо расширяемым и неудобным для тестов.

Пример использования:

Типичный пример — класс конфигурации или менеджер подключения, где кажется, что нужен один общий экземпляр на приложение.

public class Singleton {
    private static Singleton instance;

    private Singleton() {
    }

    public static Singleton getInstance() {
        if (instance == null) {
            instance = new Singleton();
        }
        return instance;
    }
}

Проблема здесь в том, что при одновременном вызове getInstance() из нескольких потоков могут создаться разные объекты.

Пояснение кода:

Код показывает наивную реализацию Singleton без синхронизации. Сначала проверяется, создан ли экземпляр. Если нет — создаётся новый объект и сохраняется в статическое поле. Если экземпляр уже есть — возвращается он же.

Пошагово это работает так:

Первый вызов getInstance() видит instance == null.
Создаётся объект new Singleton().
Ссылка сохраняется в instance.
Следующие вызовы получают уже созданный объект.

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

Ключевые моменты:

Ручной Singleton легко сделать неверно, особенно в многопоточном коде.
Основной риск — создание нескольких экземпляров вместо одного.
Такой подход часто приводит к скрытому глобальному состоянию.
Singleton усложняет модульное тестирование и замену зависимостей.
Из-за этого в современных проектах предпочтительнее использовать DI-контейнеры или более безопасные способы инициализации.

В чем заключается недостаток реализации Singleton через ручной код и почему это считается плохой практикой?