VS
Ну так может в скоупы пихать что-то часто используемое, а в query_object какую-то сложную логику и жить счастливо
Size: a a a
2020 April 06
r
мы просто выносили в отдельные папки
r
все скоупы
r
все валидации
VS
ну типа where(deleted: false) => not_deleted, это простой скоуп
VS
и скорее всего много где будет вызван
M
По поводу сложных скоупов - согласен, их имеет смысл вынести куда-либо.
M
мы просто выносили в отдельные папки
Не возникало неудобств? Я про то, что переключаться постоянно по разным файлам.
M
Я вообшем-то согласен что в идеале, скоупов в модели лучше не иметь. Лучше иметь query objects / finders /fetch services. Модель она модель, а не сборник запросов. Тут srp работает как и везде.
Вот мы написали такой гем в конторе:
https://github.com/abak-press/findit/blob/master/README.md
Вот мы написали такой гем в конторе:
https://github.com/abak-press/findit/blob/master/README.md
Спасибо, взгляну
M
Viacheslav Stepanov
Ну так может в скоупы пихать что-то часто используемое, а в query_object какую-то сложную логику и жить счастливо
Так и поступаем сейчас, просто дискуссия возникла, стоит ли продолжать )
r
Не возникало неудобств? Я про то, что переключаться постоянно по разным файлам.
нет
VS
Так и поступаем сейчас, просто дискуссия возникла, стоит ли продолжать )
ну тут спрашивать в чатах нет смысла, имхо, вам нужно решить удобно ли вам это в вашем проекте, не вызывает ли боли, не вызовет ли боли в дальнейшем
AN
Viacheslav Stepanov
Ну так может в скоупы пихать что-то часто используемое, а в query_object какую-то сложную логику и жить счастливо
Скорее так да, без перегибов. Основные вещи наверно могут быть в модели.
M
Viacheslav Stepanov
ну тут спрашивать в чатах нет смысла, имхо, вам нужно решить удобно ли вам это в вашем проекте, не вызывает ли боли, не вызовет ли боли в дальнейшем
Смысл в том, чтобы вдруг кто пошарит аргумент или статью, которую мы не учли и это повлияет на решение )
AA
А какие преимущества, если задуматься, дает вам модель без скоупов? Кроме осознания морального превосходства
AN
мы просто выносили в отдельные папки
Имхо это тоже самое что на модули бить, как бы модель не толстая, но это обман, ещё и по файлам переключаться. Была где то статья про это. Но да, наверно, это лучшем чем модель на 1к строк.
AN
Alex A
А какие преимущества, если задуматься, дает вам модель без скоупов? Кроме осознания морального превосходства
Такие же как и srp и порядок везде. Проще понять, проще тестировать, нет вероятности что разрастается до огромных размеров.
M
Имхо это тоже самое что на модули бить, как бы модель не толстая, но это обман, ещё и по файлам переключаться. Была где то статья про это. Но да, наверно, это лучшем чем модель на 1к строк.
С точки зрения архитектуры - разницы никакой, с вынесением даже хуже ИМХО. Но это вкусовщина. А вот с точки зрения читабельности - довольно классно и удобно было выносить ту же стейт машину в отдельный консерн
M
Так что тут бабка надвое )
VS
вынесите скоупы в concern