Контроллер не должен формировать срез данных или выполнять какую то логику, инициировать работу логики и получить из сформированного среза данных объект который нужно отдать пользователю это задача контроллера, т.е. отмапить полученный срез данных в dto который будет отдан пользователю, если срез данных получить не удалось, то обрабатываются ошибки и тоже формируются объекты для ошибок и отдаются в качестве результата наружу. Логика обрабатывается в более глубоком слое.
У вас если действия контроллеров 1 к 1 мапятся на методы сервисов, то у вас это не контроллер, а прокси. Вы функции контроллера вынесли и замешали с бизнес-логикой в "более глубоком слое".
Года два назад мне упал проект в котором было около 40 контроллеров, и в совокупности чуть более 500 действий контроллеров, которые выглядели так:
1.
2.
3.
4.
5.
6.
7.
8.
[HttpGet/Post("some")]
public async Task<IActionResult> SomeAction(SomeRequestModel request)
{
return Ok(
_mapper.Map<SomeResultModel>(
await _someManager.SomeActionAsync(
_mapper.Map<SomeRequest>(request)));
}
И таких методов, напомню, более 500 штук. И они продолжали плодиться.
Количество оверхеда зашкаливает.
Умные люди тупо отказывались работать на этом проекте, ибо говно.
Ибо кто-то начитался всякой херни в интернетах, неправильно понял, так и не понял концепции контроллера и заложил такую архитектуру, чтобы люди как можно больше занимались обезьяньим трудом.
И да, это всё было обильно покрыто тестами, которые по фидбеку от команды, никогда не ломаются. Никогда, какой бы рефакторинг не проводился, они абсолютно бессмысленные и бестолковые.
Вот такие упоротые "орхитекторы" гнездятся тут и там, не желая задействовать даже грамма своего мозга, прежде чем начать работать.
Сначала должен идти вызов необходимой логики и получение результата, далее если результат получен успешно, то он мапится на SomeModelDtoToReturn и после этого отдается return Ok(SomeModelDtoToReturn), если не получен то создается объект ошибки и выдается вместе со статусом ошибки.
А логика обработки запроса где? Правильно, она у вас переехала в глубокие недра бизнес-логики.
И вы ещё рассказываете про SRP.
