Доброго времени суток. Когда есть необходимость вывода особого сообщения в yii, например, об успешном заполнении формы, приходится совать setFlash в action контроллера, который обрабатывает форму. То есть вывод зашивается в контроллер. Если человек, который будет работать после меня захочет поменять это сообщение ему придется лезть в контроллер (если он вообще догадается). Вопрос: не противоречит ли это MVC?
-
лично я в первую очередь смотрю на контроллер, чтобы узнать логику вывода. Думаю ничего страшного нет.– IVsevolod5 сен 2013 в 8:18
-
2чем больше я узнаю о yii тем меньше оно мне нравится.– zb'5 сен 2013 в 8:21
-
А чем вы пользуетесь? Небольшое описание в сравнении с yii если можно. Был бы признателен.– Самат Жанбеков5 сен 2013 в 8:24
-
Это мне на будущее.– Самат Жанбеков5 сен 2013 в 8:26
-
2Yii никой то странный фраемворк. В котором что бы что то сделать иногда приходится такой венегред понаписать, что потом самому стыдно.– dlarchikov5 сен 2013 в 8:36
4 ответа
На askdev ответили почти также: Long story short: разделение MVC невозможно реализовать в полной мере в удобном виде. Валидация модели, которая должна работать на самом низком уровне и не иметь с выводом ничего общего, возвращает уже сгенерированные сообщения об ошибках. При желании можно сделать таблицу вывода и в setFlash записывать только код сообщения, но это будет только муторней.
В своих проектах я использую архитектуру с применением классов помощников Service{ClassName}.php
Для всех flash сообщений на проекте у меня есть отдельный класс ServiceFlash. Все сообщения находятся в этом файле, что упрощает поиск и редактирование этих сообщений. Выглядит он примерно так:
class ServiceFlash extends ServiceAbstract
{
public static function error()
{
Yii::app()->user->setFlash('errorFlashContainer', "Произошла непредвиденная ошибка. Попробуйте еще раз");
}
public static function addSuccessfull()
{
Yii::app()->user->setFlash('mainFlashContainer', "Запись успешно добавлена!");
}
public static function saveSuccessfull()
{
Yii::app()->user->setFlash('mainFlashContainer', "Изменения успешно сохранены!");
}
public static function deleteSuccessfull()
{
Yii::app()->user->setFlash('mainFlashContainer', "Запись успешно удалена!");
}
}
Соответственно в контроллере остается только вызывать эти статические методы
ServiceFlash::addSuccessfull();
У если вы хотите идеально все, то лучше вынести в файл. Но можно и так, так как на практике эти тексты меняются не чаще чем дизайн сайта.
Ну это не будет противоречить MVC никак.
Главное комментируй код, чтобы можно потом было все четко понять.
Такие сообщения можно делать и не через setFlash т.к. формы можно обрабатывать ajax'ом и в JS посылать данные, а получать сообщения, их и выводить и нет необходимости постоянно отправлять форму и перезагружать страницу.