Правильно ли размещать логику валидации в классе Request?

Question

[hrustbb2]

Собственно суть вопроса в заголовке. В многочисленных примерах к Laravel, в качестве лучших практик , рекомендуется размещать логику валидации запроса в классе-наследнике Request. Не нарушает ли это принципа единой ответственности, особенно если эта логика выходит за рамки обычной проверки соответствия параметров некоему регулярному выражению?

Comments

[JhaoDa]

А какая у этого класс ещё есть ответственность?

[Максим Грищенко]

Не в классе наследнике Request, а наследнике RequestForm, если мне память не изменяет

[hrustbb2]

Максим Федоров, Ага, оно самое :)

Answer 1

[Максим Грищенко]

Ну да, не по SOLID код :) Если вы об этом
https://github.com/laravel/framework/blob/7.x/src/...

Нарушено сильно пару принципов:

• Single Responsibility Principle
  
• Liskov Substitution Principle - нельзя подменить FormRequest классом Request, там где идет завязка на индивидуальный для FormRequest интерфейс
  

Но, к слову, это довольно удобный способ валидации :)

Comments

[JhaoDa]

Зачем ссылка на форк?

[Максим Грищенко]

JhaoDa, упс, не углядел

[JhaoDa]

Но вопрос-то бы не про сам FormRequest, а про валидацию в нём. Я так и не понял связи ответа с вопросом.

[Максим Грищенко]

JhaoDa, такс, я видимо не так понял вопрос?

[JhaoDa]

Максим Федоров,

Правильно ли размещать логику валидации в классе Request?

Размещение валидации в форм реквесте как-то зависит от того, что внутренности форм реквеста не SOLID'ны? Не думаю.

[Максим Грищенко]

JhaoDa, эксплуатация не SOLID'ного кода, как ни крути :)
Я же не ругаю, вопрос был "по солиду ли", я ответил

[kafkiansky]

Solid Responsibility Principle?

[Максим Грищенко]

kafkiansky, ну опечатался... с кем не бывает

[hrustbb2]

Но, к слову, это довольно удобный способ валидации :)

Надеюсь не самый удобный из всех возможных? :)

[Максим Грищенко]

hrustbb2,самый

[Максим Грищенко]

hrustbb2, код не солидный, но архитектурно удобно сделано

[hrustbb2]

Максим Федоров, это печально

[hrustbb2]

Максим Федоров, А где удобней организована валидация? в Yii2 или в Laravel?

[Максим Грищенко]

hrustbb2, в Yii2 тоже не по солиду, там она (насколько я помню) сделана в моделях актив-рекорд

[hrustbb2]

Максим Федоров, Не совсем. Валидация в Model, а ActiveRecord наследуется от Model (если мой склероз мне не изменяет). Не, ну всетаки, раз уж солид мы пока отодвинули, где удобней? (У нас тут батл с коллегами просто)

[Максим Грищенко]

hrustbb2,

а ActiveRecord наследуется от Model

и в этот момент она становится ActiveRecord Model :)