В MVC данные не должны проверяться в модели?

День добрый!
На просторах интернета прочитал, что при создании приложения MVC, данные не должны проверяться в модели, а проходить через модель должны только уже проверенные данные, которые должны проверяться в контроллере.
Что по этому поводу считает официальная наука?

0

в mvc нет строгой реализации бизнес-логики. где то в контроллере, где то в моделях, где то в немодельных структурах. все остальное зависит от архитектуры и "фанатиков личного подхода к реализации".

1

Сообщение от Fatmarmelad Что по этому поводу считает официальная наука?

В общем правильно. Модели составляют функции, реализующие основной функционал. Нагружать их чрезмерными проверками нерационально.

1

Fatmarmelad, С точки зрения науки операция описывается в отдельном классе, и вне зависимости от того, какие данные в нее передают, она должна всегда отрабатывать корректно - либо выполняется полностью, либо кидает исключение. Всевозможные проверки описывают в value-object-ах и запросами к БД. При таком подходе получается что валидация в контроллере на самом деле необязательна.

В контроллере делаются простые проверки ввода, в случае если валидация не прошла - отдается список ошибок со статусом 422. При неудачной выполнении операции перехватывают исключение из предметной области (DomainException) отдают сообщение со статусом 409.

Можно идти на компромиссы и опускать некоторые проверки, учитывая что они были уже произведены в контроллере.

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

2

Сообщение от Fatmarmelad что при создании приложения MVC, данные не должны проверяться в модели, а проходить через модель должны только уже проверенные данные, которые должны проверяться в контроллере.

Формулировка не очень хорошая. Вот что значит непроверенные данные? Допустим я могу легко проверить в контроллере что в переменной лежит число, а не строка. А если нужно проверить что элемент с таким ID существует в БД то как я это из контроллера сделаю? Если из контролера вызвать метод модели, который проверяет наличие записи в БД, это считается проверкой в контроллере или в моделе?
В общем много вопросов.

Я вижу это так. Кейс с регистрацией пользователей. Юзер может зарегистрироваться сам, а также его может создать админ. В коде это может быть два контроллера и модель. В каждом из этих частей кода будут свои проверки. Например для саморегистрации проверка капчи, для админа наличие роли админ, а в моделе проверка что юзера с такой почтой в базе нет.

1

Сообщение от tarasalk Если из контролера вызвать метод модели, который проверяет наличие записи в БД, это считается проверкой в контроллере или в моделе?

Нужно понимать, что тут идет речь о слоях.

Слой контроллера парсит запрос, к нему относится и роутер и все-возможные посредники. Проверка прав доступа, и первичная валидация ввода данных. После того, как он точно определил какую операцию пользователь пытается выполнить управление передается в модель.

Модель - это все, что касается бизнес логики. Это отражение реального мира, но не полностью - а ровно на столько, чтоб программа решала задачу. Именно поэтому она называется моделью - моделью предметной области.

После выполнения операции мы уже формируем ответ.

Контроллер относится лишь к пользовательскому интерфейсу. И несмотря на то, что поток исполнения идет от контроллера к модели. Модель важнее и не должна зависеть от контроллера, более того должна защищаться от него.

Если вы делаете проверку такого рода прямо в контроллере.

То получается, что часть бизнес логики вытекает наружу - в контроллер. И даже в этом случае не имеет значения где вы это написали и какой метод вызвали. Проверка на уникальность определенно относится к бизнес логике. Отношение того или иного участка кода к тому или иному слою не зависит от того где вы его написали, но лучше это разделение выражать более явно.

0