как бы
Я решил вынести эту логику в отдельный сервис
Ну а какую бизнес задачу решает? так и назови
Это понятно Я хочу вынести в этот сервис CRUD логику А "CrudBaseService" звучит тоже не айс
Ну это понятно, что делает это круд? Создаёт ботов? BotService или там CountryService и тд
Базовый сервис, от которого потом будут наследоваться всякие UserService и ClientService Поэтому и затык с нэймингом базового
если у тебя круд с данными никак не работает и просто их пишет это у тебя не сервис а репозиторий, такие обстрактные вещи лучше обсудить с командой как вы будете это называть
просто не очень понятно что ты будешь делать когда поменяется логика или что то добавят, ты будешь менять базовую логику во всех потомках а это больно
В том то и соль, что я пока один над проектом работаю) Оно то да Но другого варианта избежать дублирования одного и того же кода я пока что не вижу Опять таки, для большинства потомков будет достаточно того, что есть у родителя Если чего-то нет - дописываем, что-то не подходит у родителя - меняем Я думал сделать это трейтами, но это тоже так себе выход, вроде как
ну я бы наверное сделал обстрактный репозиторий и от него бы отнаследовался и все если цель конечно сделать универсальный круд
а вероятность того, что с наследованием потом будет больше проблем, чем пользы, рассматривается? В сервисе будет что, методы по одной строчке? $model->update($validated) Model::create($validated) и тд? Если да - то проще написать крудогенератор. Копипаста в данном случае будет меньшей из проблем. Если операции сложнее - их всё равно придётся расписывать индивидуально, и смысл от единого сервиса исчезнет (см. пункт выше)
Рассматривается) Знаю, что это не совсем хорошая практика Просто какая штука выходит Дублирование - плохо, нужно стараться соблюдать принцип DRY Наследование - тоже плохо, т.к. может потянуть за собой в будущем болезненный рефакторинг, как минимум И тут вопрос в том, что хуже Судя по тому, что я смотрю, несоблюдение DRY походу менее страшно Тем более, что касается простых методов в до десятка строчек
Обсуждают сегодня