keltanas
marty cats
Absinthe, ESI, я говорю про ESI
Smarty & загрузка и старт PHP контроллера
- Автор темы ggfdsfds
- Дата начала
А сколько их передавать из контроллера допустимо?
В моем примере мы обращаемся к модели из вида. Это нормально. Этим данным в контроллере делать нечего.
Как ты предлагаешь их получать в виде? Виджет делать?
Мне, если честно, ни то, ни другое не нравится.
Вот только что подумал - а если так?
PHP:
return new View('file.tpl', [
'newsCollection' => function() { return NewsCollection::all(); }
// ...
]);
...
{% if weWannaSeeNews %}
{% for newsItem in newsCollection %}
...
{% endfor %}
{% endif %}
Вот видишь, о чём и речь, чтобы не писать одну строчку подтягивающую данные, ты предлагаешь городить ESI и разбросать код по куче мест. Напоминает шутку про регулярку, когда решение добавляет кучу других проблем.
Вурдалак
Продвинутый новичок
AmdY, а как ты выводишь блоки, которые не имеют никакого отношения к данному контроллеру?
Я делаю запрос «/user/42.json» — получаю юзера.
Я делаю запрос «/user/42.html» — получаю юзера и ещё всякую хрень типа последних новостей на сайте. News::lastNews()?
Ну или вот, например, пользователь видит список рекомендаций на каждой странице сайта-магазина, для каждого юзера логика своя (что он в последний раз покупал, может быть какая-то инфа от стронних сервисов и т.д.). Где ты будешь инкапсулировать такую логику?
Я делаю запрос «/user/42.json» — получаю юзера.
Я делаю запрос «/user/42.html» — получаю юзера и ещё всякую хрень типа последних новостей на сайте. News::lastNews()?
Ну или вот, например, пользователь видит список рекомендаций на каждой странице сайта-магазина, для каждого юзера логика своя (что он в последний раз покупал, может быть какая-то инфа от стронних сервисов и т.д.). Где ты будешь инкапсулировать такую логику?
Последнее редактирование:
Absinthe
жожо
Как раз идеально ложится на шаблон такой вызов модели. И в json как раз этих данных не будет.
Прекрасный пример в поддержку этого способа
В модели.
keltanas, бля, ну ESI это лигшь красивое название для таких же хаков. Просто обозвали красивым словом, так же как HMVC и является продолжением SSI. Вот код по твоей ссылке
PHP:
<?php echo $view['actions']->render(
new \Symfony\Component\HttpKernel\Controller\ControllerReference('...:news', array('max' => 5)),
array('strategy' => 'esi'))
?>
<?php echo $view['actions']->render(
$view['router']->generate('latest_news', array('max' => 5), true),
array('strategy' => 'esi'),
) ?>keltanas
marty cats
AmdY, ну и что. Зато код все равно получается чище и управляемее, чем дерганье модели из шаблона.
И кстати так лучше
{{ render_esi(controller('...:news', { 'max': 5 })) }}
И кстати так лучше
{{ render_esi(controller('...:news', { 'max': 5 })) }}
ggfdsfds
Новичок
В общем прихожу к выводу что это допустипо - в шаблоне в отдельных местах делать "дергание данных". Но без использования логики - когда дергаешь модель - т.е. не нужно передавать там кучу параметров 5, здесь 10 и т.д. А то это уже получается в шаблон вписываеш то, по идее за что должен отвечать контроллер или модель. Просто название класса - и вызов функции.
Но в любом случае должен быть MVC - и тогда все будет более правильно и аккуратно.
Но в любом случае должен быть MVC - и тогда все будет более правильно и аккуратно.
Absinthe
жожо
Через дополнительный костыль чище, чем напрямую?
Вид может получать данные из модели, никакого нарушения MVC тут нет. Почему тебе кажется, что этот код грязный?
ggfdsfds
Новичок
Ну хотя бы потому, что я открою файлик php-где описана модель и буду править логику модели. А так получается, что еще нужно и в шаблон залазить.
Для банальных header , footer - или главной - где необходимо получить - например список последних новостей - это еще куда не шло. Но все равно - мне кажется, что лучше все это загружать в модель.
В общем думаю, что оставлю и то (сама логика старта Smarty объекта) и то (плагин {dataGet model="className" func="main"}.
Absinthe
жожо
Не нужно. Шаблон получает данные, которые отдала модель. Достаточно исправить модель.
ggfdsfds
Новичок
Хотя что интересно - вот первый сайт - который сделан на вот таких {dataGet} - мне понравился с точки зрения логики меньше (их просто куча в шаблоне), чем тот сайт, где все крутится вокруг контроллера.
ggfdsfds
Новичок
Имею в виду вот так:
Когда еще в вот эти параметры запихивают {assign} чего-то там и особенно из GET-запросов.
Вурдалак
Продвинутый новичок
Я как бэ про это:
Полное днище. А не про 100500 переменных в return View().
Если ты захочешь сделать нормально, как белый человек, протащив нужные зависимости для виджета новостей или как его там, то тебе придётся писать что-то типа того же ESI, во всяком случае не далеко от него по трудоёмкости (функцию для Twig, кеширование, etc.).
Но мне ESI не очень нравится идеологически.
Вурдалак
Продвинутый новичок
Получать, конечно, может. Но не из воздуха.