Вопрос по MVC. Где выполнять проверку?

Question

[Hint]

Интернет-магазин. Есть контроллер cart (корзина), есть модели cart и product. Пользователь жмет «добавить в корзину», вызывается действие add контроллера cart, которое вызывает метод addProduct модели cart. В каком месте правильнее выполнять проверку существования и наличия товара (методы модели product)? В контроллере или в моделе (или и там, и там)? При этом надо вывести пользователю информативное сообщение об ошибке.

Answer 1

[Виталий Юшкевич]

Все что касается валидации / проверки данных должно уходить в модель. Если же вам нужно проверять, валидная корзина (на верхнем более абстрактном уровне), то выполняйте эту проверку в контроллере.

А насчет различных сообщений — возвращайте из модели различные сообщения в случае неудачи и дальше их обрабатывайте в контроллере, делайте удобные проброски на вьюхи, где уже выводите в зависимости от этого различные наборы красоты :)

PS удачи и успехов в вашем не легком деле :)

Comments

[egorinsk]

> Если же вам нужно проверять, валидная корзина (на верхнем более абстрактном уровне), то выполняйте эту проверку в контроллере.

Не могу с вами согласиться. Гораздо логичнее сделать для этого метод в классе Корзина: в таком случае его можно будет вызывать откуда угодно, а код в контроллере повторно использовать не получится.

Answer 2

[Yuri Shikanov]

Существует подход thin controllers — that models, согласно которому в контроллере нужно, как можно меньше всего выполнять. В идеале чтобы контроллеры не больше 10 строк были.

Comments

[Yuri Shikanov]

thin controllers — fat models

Answer 3

[int03e]

В контроллере смотреть на результат выполнения cart.addProduct (true/false) и выводить ошибку при рендеринге вьюхи, если есть такая необходимость.

Comments

[Hint]

Т. е. проверять в моделе. А если я захочу выводить разные сообщения (например, «товар не найден» и «товар закончился»), то придется еще возвращать код ошибки (или выкидывать исключения)?

[int03e]

Подождите, ну при добавлении товара в корзину что происходит — он записывается в БД? Это ответственность модели. Может, вы его в сессию записываете? Тогда это контроллеры/хелперы. Если первый случай — модель может отдавать различный код ошибки, да. И в любом случае, в классическом MVC модель НЕ МОЖЕТ напрямую взаимодействовать с view, в качестве прослойки используется контроллер.

[Hint]

Записывается в БД. Но ведь по сути моделе «корзина» все равно, есть товар на складе или его нет. И модель не должна теоритически знать, как пользователю красиво что-то показать (например, предложить альтернативный товар в случае отсутствия текущего).

[furyk]

А что если разделить на 2 метода?

if cart.checkProduct
    cart.addProduct
else
    // выводим ошибку

[Hint]

furyk, по сути метод checkProduct должен находиться в моделе «товар»: product->isAvailable(). Так вот и вопрос, где вызывать этот метод, в моделе или контроллере.

[int03e]

Я обычно делал как-то так, псевдокод:

# carts_controller.rb

def add_to_cart
  @product = Product.find_by_name(params[:product][:name])
  redirect_to products_path, message: "Товар не найден!" if @product.nil?
  if @product.present? && @product.count.zero?
    @alternative_product = Product.alternative(@product)
    redirect_to products_path, message: "Товар закончился, есть альтернативный"
  else
   Cart.add_product(@product)
   redirect_to carts_path, message: "Товар добавлен в корзину"
end

[Сергей]

Но ведь по сути моделе «корзина» все равно, есть товар на складе или его нет.

Как так? если нет товара, значит и добавить в корзину его нельзя. По крайней мере, я думаю, что должна быть именно такая логика для реальных товаров.

И модель не должна теоритически знать, как пользователю красиво что-то показать

Она и не будет знать, она просто вернет код ошибки. А что с этим делать, решит контроллер.