Size: a a a

2020 November 02

S

Sergey in symfony
Mikhail Melnikov
Всем привет, подскажите пожалуйста нормально ли, если когда допустим я пишу фильтр, к примеру по акциям, для его реализации я создам всего один метод в репозитории, и он у меня в зависимости от того какой прилетел запрос используя 4 конструкции  if, не вложенные, а друг за другом, будет формировать запрос  $qb... выполнять его и отдавать список акций или же лучше написать 4 запроса отличающиеся по условиям
ну хотя бы свитч надо использовать
источник

SK

Serge K in symfony
Mikhail Melnikov
Всем привет, подскажите пожалуйста нормально ли, если когда допустим я пишу фильтр, к примеру по акциям, для его реализации я создам всего один метод в репозитории, и он у меня в зависимости от того какой прилетел запрос используя 4 конструкции  if, не вложенные, а друг за другом, будет формировать запрос  $qb... выполнять его и отдавать список акций или же лучше написать 4 запроса отличающиеся по условиям
Привет. Делал такую же штуку, приватный метод принимал query builder и фильтр, на условиях строил запросы. Норм штука. Чуть сложнее когда много условий и надо делать джоины. Можно посмотреть еще в сторону doctrine criteria
источник

MM

Mikhail Melnikov in symfony
А тут не нарушается случайно S olid?
источник

MM

Mikhail Melnikov in symfony
Просто интересно мнение, у меня сомнения по такому подходу
источник

VK

Vladyslav Kopaihorod... in symfony
Sergey
ну хотя бы свитч надо использовать
А разница?
источник

VK

Vladyslav Kopaihorod... in symfony
Mikhail Melnikov
А тут не нарушается случайно S olid?
Дажн если нарушается, что с того, если ті не шаришь ?)
источник

VK

Vladyslav Kopaihorod... in symfony
Забей
источник

VK

Vladyslav Kopaihorod... in symfony
Солид это не про фильтр т if else
источник

VK

Vladyslav Kopaihorod... in symfony
Метод в репе норм
источник

MM

Mikhail Melnikov in symfony
Ну S это как бы вроде принцип единой ответственности, или это только про классы?
источник

MM

Mikhail Melnikov in symfony
Почему не создать 4 метода, которые будут проще читаться и из названия сразу понятно что они одают, а рулить ими в контроллере...
источник

MM

Mikhail Melnikov in symfony
2 подхода, какой будет верный или оба?
источник

MM

Mikhail Melnikov in symfony
Или с 4 мя запросами не верный подход?
источник

AK

Alexander Kaluzhsky in symfony
Если запрос выполняет 1 функцию, даже джойня несколько таблиц, то является нарушает ли он S?
Вроде как нет. Если он производит модификацию и выборку, то стоит задуматься
источник

AK

Alexander Kaluzhsky in symfony
SOLID вроде про принципы, а не про реализацию
источник

MM

Mikhail Melnikov in symfony
Хорошо если if стало 10
источник

MM

Mikhail Melnikov in symfony
И запрос в репе уже 50 строк
источник

AK

Alexander Kaluzhsky in symfony
либо это что-то очень сложное, либо где-то поворот не туда произошел
источник

MM

Mikhail Melnikov in symfony
Ну допустим 10 категорий, это не сложно
источник

MM

Mikhail Melnikov in symfony
Хотя не фигню сказал
источник