Входит ли пользовательский ввод в контроллер или модель?

Сейчас моя модель разделена, но мой контроллер и представления все еще объединены в 12-строчный файл. Я искал для этого настоящую систему MVC, разделяя представления, но, ища вещи для разделения, я заметил, что мой контроллер выполняет много работы, которая может принадлежать модели.

Например, скажем, у меня есть...

if (isset($_POST['write'])) {
    $obj = $objManager->get($_POST['id']);
    $obj->setFoo($_POST['foo'])
        ->setBar($_POST['bar']);
    $objManager->write($obj);
    echo ... 
}

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

if (isset($_POST['write'])) {
    $objManager->update($_POST);
    echo ...
}

... где update() делает в основном то же самое, устанавливает переменные и сохраняет данные.

Источник

3 ответа

Решение

Ваш первый пример выглядит как лучшее решение для меня. Он правильно отделяет задачи принятия пользовательского ввода от доступа к данным в вашей модели. Другими словами, модель не заботится о том факте, что ввод поступает из параметров POST.

Во втором примере вы тесно связываете модель с контроллером, потому что модель ожидает, что ей будет передан список параметров POST для обновления объекта. Модель не должна заботиться о том, откуда берутся переменные "id", "foo" и "bar".

Второй пример выглядит немного более элегантно, но он не так удобен для модульного тестирования. Чтобы выполнить его модульное тестирование, вам нужно будет передать ему ассоциативный массив, ключи которого соответствуют именам параметров POST. Это не так уж важно в PHP, так как все динамически типизировано, но это еще одна вещь, о которой вам нужно беспокоиться.

Источник

Весьма распространено, чтобы контроллер заботился обо всем пользовательском вводе и держался в стороне от переменной $_POST внутри модели, а контроллер заполнял модель. Чтобы узнать больше о подходе MVC, вы можете посмотреть, как это делают другие фреймворки, например Zend Framework.

Источник

Хороший способ подумать о MVC - это спросить себя: "Если бы я изменил X на уровне вида, что бы мне пришлось изменить на уровнях Controller или Model?"

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

Контроллер - это то, что принимает внешний ввод (например, от пользователя), вызывает на уровне модели, чтобы обработать его, а затем определяет, какое представление предоставить в качестве ответа.

В вашем примере $_POST содержит необработанный ввод, а ключи в массиве $_POST определяются тем, как они кодируются в ваших представлениях в основном HTML. Вы не должны ожидать, что ваша модель будет знать, какие действительные ключи находятся в $_POST или нужно ли значения, которые необходимо обработать, преобразовать и т. Д., Оставьте это задание контроллеру, что должно гарантировать, что значения, которые он передает модели, будут соответствовать условия, которые ожидают классы / функции уровня модели.

Источник
Другие вопросы по тегам