чаще всего бесполезная трата времени. аргумент про смену бд неубедителен. вещь полезная когда заведомо известно что данные будут браться откуда-то с другого места но временно давайте брать из бд. например пока из бд потом из эластика.
и потом при работе с репо в конце надо собрать в итоге модель, которая может быть со сложными связями или какие-то хитрые выборки. и вот там замучаешся гонять туда сюда прослойку. так что тулить его везде это может быть как выстрел в ногу
если код используется лишь однажды, не лучше ли держать его поближе к тому месту, где он нужен? ладно б ещё если было там мощное переиспользование, но тут — зачем наворачивать кучу лишних классов?
Я попросил уточнить, как автор ответа определил, что приведенный кусок кода находится в контролере, а не в классе отвечающим за валидацию, т.е. FormRequest
и да, конечно он мог выделить валидацию в другой метод, но где? в контроллере?) А если создал отдельный класс, то зачем? если для этого уже есть возможность в ларе) Чтобы другой разработчик, при встрече кода искал этот злоебучий класс?)