N
В контроллере не должно быть логики, которую можно переиспользоыать в сервисах)
тут просто есть желание вынести вспомогательную логику из контроллера. этот код врятли будет использоваться где то помимо исходного контроллера...
Size: a a a
2020 July 08
А
Твой сервис должен быть написан так, что если выдернуть его из фреймворка, он будет продолжать работать. Твой не будет.
А
тут просто есть желание вынести вспомогательную логику из контроллера. этот код врятли будет использоваться где то помимо исходного контроллера...
Ну так и оставляй в контроллере, в чем проблема?
MI
тут просто есть желание вынести вспомогательную логику из контроллера. этот код врятли будет использоваться где то помимо исходного контроллера...
Если ты хочешь заинжектить контроллер в сервис - значит у тебя есть логика, которую ты написал в контроллере и хочешь использовать в сервисе. Не надо так - перенеси её в сервис и используй в контроллере метод сервиса
N
MI
N
Ну так и оставляй в контроллере, в чем проблема?
проблема что количество методов используемых в маршрутизации иногда меньше методов хелперов
А
А
N
Да, нужно называть вещи своими именами) и вообще писать какую-то логику в контроллере - харам
ок. а если там забористая логика по сбору данных с бд. это же еще не бизнес логика и в сервис ее пихать не зачем. или сложная процедура сохранения загруженного из формы файла. это все таки выносить в сервис или в экшен? или другое?)
MI
ок. а если там забористая логика по сбору данных с бд. это же еще не бизнес логика и в сервис ее пихать не зачем. или сложная процедура сохранения загруженного из формы файла. это все таки выносить в сервис или в экшен? или другое?)
бизнес-логике вообще не место в контроллере. Всю логику нужно уносить в сервисы
А
ок. а если там забористая логика по сбору данных с бд. это же еще не бизнес логика и в сервис ее пихать не зачем. или сложная процедура сохранения загруженного из формы файла. это все таки выносить в сервис или в экшен? или другое?)
Почему сбор данных с бд - не бизнес логика? Тебе же не пофиг, как эти данные собираются. Значит, есть требования бизнеса.
MI
А для работы с БД и запросами вообще лучше использовать репозитории
А
ок. а если там забористая логика по сбору данных с бд. это же еще не бизнес логика и в сервис ее пихать не зачем. или сложная процедура сохранения загруженного из формы файла. это все таки выносить в сервис или в экшен? или другое?)
> или сложная процедура сохранения загруженного из формы файла
В аплоадер выноси, не прогадаешь
В аплоадер выноси, не прогадаешь
N
Почему сбор данных с бд - не бизнес логика? Тебе же не пофиг, как эти данные собираются. Значит, есть требования бизнеса.
ну как бы да, но ведь там нет решения каких либо задач. просто для конкретно этой странички решили взять по чуть-чуть сущностей из нескольких таблиц. и более, нигде эта логика использоваться не будет. странно выносить это в отдельный сервис. или нет?
MI
Нет, не странно. А как раз логично
N
> или сложная процедура сохранения загруженного из формы файла
В аплоадер выноси, не прогадаешь
В аплоадер выноси, не прогадаешь
UploaderService?А
UploaderService?А без постфикса непонятно, что это сервис?
N
Нет, не странно. А как раз логично
о. для меня эт прям откровение)
N
всё как то сомневался и даже решил это заменить напилом экшенов)