Какие паттерны проектирования не нравятся

Singleton — часто злоупотребляют, превращая в глобальное хранилище состояния, что нарушает принципы тестируемости и инкапсуляции. Усложняет mock-инг в тестах.

Service Locator — скрывает зависимости, делая код менее прозрачным. Лучше использовать Dependency Injection.

// Плохо: Service Locator
class OrderService {
    void process() {
        Database db = ServiceLocator.getDatabase();
        db.save();
    }
}

// Лучше: DI
class OrderService {
    private final Database db;
    
    OrderService(Database db) {
        this.db = db;
    }
}

Factory Method — иногда приводит к избыточным иерархиям классов, когда можно обойтись простыми конструкторами или билдерами.