Как правильно реализовать отправку $_POST данных и возвращение ошибки?

Question

[deadmemoras]

В общем, есть маленькая mvc, пишу на ней.
Сейчас пишу регистрацию, и возникло "размышление":
Метод register принимает введенные данные с инпута - что мне дальше делать? Ну, т.е - проверить на корректность веденных данных и пустоту, и если все норм - передаю данные в метод(модель) reg и там уже делаю все что нужно(добавление в бд) или же, лучше передавать веденные данные в модель(сразу) и там уже проверять, если ошибка - возвращаю return false и проверяю if(!Auth::reg($_POST['login'].....) $this->error('Не верно введены данные') ?
И еще, как лучше будет делать условие if($_POST['submit']) ?

Answer 1

[Greg Popov]

Валидация производится на уровне Модели, так как данные её, валидация и ответственность за них несет модель.

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

Answer 2

[DVamp1r3]

if ( ($model->load($app->request->post()) !== false)
как-то так, например, делается создание новой модели с валидацией и прочей шелухой в yii.

за разграничение доступа к действиям над моделью отвечает контроллер. т.е. уже после проверки прав на действие идет работа с моделью.

Comments

[deadmemoras]

Знаю я про Yii, только там делается: if($model->load(\Yii::$app->request->post()) && $model->validate())
Не разу не видел конструкцию подобной вашей.
Насчет этого - да, знаю.
Уже реализовываю методы, просто, это мой первый опыт(в целом практически, только yii2 из всего пхп и умею юзать), поэтому и не думал даже про это.

[DVamp1r3]

Да, валидация там отдельным методом, впрочем уже выше ответили что валидация - обязанность модели. Как оно будет именоваться дело десятое. При ошибке рекомендую выбрасывать исключение, а не возвращать true/false.

[deadmemoras]

Dmitry Butenko: насчет валидации - у меня метод в главной модели(от которой все наследуются) которая возвращает булевое значение. (Надоумили на реализацию похожу в Yii)
Спасибо.
Если есть ошибки - я их вывожу и делаю return false;