⚙️ Базовый класс в C#: когда чистая архитектура становится проблемой

Всё начинается разумно. В нескольких сервисах повторяется одна логика — контекст БД, аудит, логирование. Выносишь в базовый класс. Код короче, дублирования нет.

Проходит полгода. Простой юнит-тест поднимает глобальную шину событий и запускает рефлексию. Никто не просил. Просто создали экземпляр репозитория.

Что происходит в конструкторе:

public abstract class RepositoryBase<TEntity>
{
    protected RepositoryBase(DbContext context)
    {
        Context = context;
        ApplyEntityConfiguration(); // рефлексия
        SubscribeToAuditEvents();   // подписка на события
    }
}

Производный класс просто вызвал base() — и получил в нагрузку поведение, которое не запрашивал. Это невидимая связанность: поведение определяется не сигнатурой, а тем, что спрятано в цепочке наследования.

Особый случай — глобальное состояние

Статический инициализатор внутри базового класса меняет сериализацию для всего приложения при первой загрузке любого наследника. Не локально — глобально. Молча.

Наследование vs композиция

Глубокое дерево: RepositoryBase → MongoBase → AuditableBase → ProductRepository. Четыре уровня. Поведение на каждом. Ничего не видно на месте вызова.

Декоратор решает то же самое явно:

public class AuditingRepository : IProductRepository
{
    private readonly IProductRepository _inner;
    private readonly IAuditService _audit;

    public async Task<Product> GetByIdAsync(Guid id)
    {
        await _audit.LogAccessAsync(id);
        return await _inner.GetByIdAsync(id);
    }
}

Каждый слой делает одну вещь. Зависимости видны. Убрать логирование можно не трогая репозиторий.

Глобальная конфигурация — в точку запуска

Если что-то влияет на всё приложение, оно должно жить в Program.cs, а не в базовом классе, который загружается неизвестно когда.

Базовый класс становится проблемой, когда берёт на себя слишком много. В этот момент он перестаёт быть абстракцией и превращается в объект бога.

📍 Навигация: Вакансии • Задачи • Собесы

🐸 Библиотека шарписта

#sharp_view