Столкнулся с ситуацией. На новом проекте сделал вот такой сервис, который выполняет несколько сложных запросов к бд и возвращает определенный результат в контроллер, который к этому сервису обращается. Сказали на проекте, что так неправильно делать. Сказали, что это нужно в модели писать. Есть в этом правда? Ну, и еще сказали, что название плохое. Что будто бы люди, кто будет в дальнейшем заниматься поддержкой, не поймут, что там внутри происходит
Если у вас в проекте принято такое писать в моделях, то пиши в моделях
не нужно такое писать в модели, но если такие правила приняты в команде, то почему бы и нет, только это все наверное через scope() нужно реализовать
Название — да, не очень, сложно понять, что он там на самом деле делает. По идее, если у тебя там только мега-запросы, то это прям та самая Q из CQRS
если не в модели, то куда лучше записать? я же формально всё верно сделал? только, наверное, да, само название сервиса не совсем правильное
вот таких пару запросов в этом сервисе. это легаси проект, его рефакторим
в вот $user = Auth::user() так не надо, как минимум он должен прийти как параметр функции getPar...Balances(User $user) и обращение к сессии почему здесь, а не как параметр метода
Можно записать так, как я предложил
Обсуждают сегодня