Как избавится от дублирования в контроллере?

Question

[Andrey]

Начал .NET CORE изучать и назрел вопрос с выборкой данных:
Мне для двух экшенов по сути необходимо сделать одни и те же действия, я думаю эти действия вынести в метод. Вопрос, куда поместить этот метод? Оставить тут же в контроллере или такое надо выносить в сервисы/еще куда либо?
Как правильно это делается с точки зрения NET CORE?

Answer 1

[Илья]

Зависит...
Много от чего
Надо понимать контекст, потому что спектр решений широк: от "оставить как есть" до "вынести в сервис", от "сделать сущность доменной области и в ней метод" до "вынести в метод расширения"

Comments

[Andrey]

я вытаскиваю сотрудников через ADO NET. Просто мне нужны сотрудники в нескольких экшенах

[Илья]

Andrey, в таком случае вам нужно реализовать шаблон Repository, либо, действительно, взять EntityFramework в котором шаблоны Repository и UnitOfWork реализованы из коробки

[Andrey]

Илья, Entity не подходит, т.к. все нужно делать без ОРМ

[Илья]

Andrey, ну тогда просто реализуйте паттерн репозиторий. Или паттернами тоже нельзя пользоваться?

[Василий Банников]

Andrey, Тогда выносите этот код в новый класс наподобие EmployeeRepository и в нём метод GetXXX

Answer 2

[Владимир Коротенко]

Если данные одни то что то пошло не так. Курите вашу архитектуру. Кстати сервисы по своему избыточны при использовании EF

Comments

[Илья]

Кстати сервисы по своему избыточны при использовании EF

Почему вы так считаете?

[Владимир Коротенко]

Илья, при правильной архитектуре вы по сути отдаёте запрос наружу. Сверху висят фильтры авторизации, максимум что вам нужно это automapper (м) отмепить сущности и при необходимости закешировать их. Все решается по месту

[Илья]

Владимир Коротенко, ну запросы понятно. Для того что не изменяет состояние системы, логично. Но для команд могут быть сложные сценарии, в которых куски кода могут дублироваться.

PS. Когда я пишу «запрос» и «команда» я имею ввиду термины CQRS

[Владимир Коротенко]

Илья, Так и выносите их. Рефакторинг в общем то для этого и придумали.
Причем это ложится в проектные методики нечеткого планирования.
Вы сделали на коленке, допустим фэйсбук, потом допилили.

[Илья]

Владимир Коротенко, ну так выносить куда? Как вы назовёте то, в чем будет обитать эта логика?

[Владимир Коротенко]

Илья, Вот сейчас пожалуй назову сервисами, раньше называл утилитами, хэлперами, какой то неведомой фигней