Rails: Как добавить динамический контент в Layout?

ну если делать рендер шапки, то получается мне надо во всех экшинах прописывать метод, который будет собирать в @переменную список городов и потом во вьюхе все генерить

я пришел к выводу, что правильней будет использовать хелпер.

Можно в ApplicationController сделать before_action, который будет загружать города в @переменную.

Спасибо. Понял.

Jeiwan: Похоже я поторопился. Я в контроллере сделал
def city_list
City.all
end
и пытаюсь во вьюхе slim сделать
- for item in city_list do
li=item.name

undefined local variable or method `city_list' for

maxyc_webber: или тут можно только через переменные?

maxyc_webber: только переменные. Причем именно инстансные (с @ в начале).

Jeiwan: а запись вида - for item in controller.city_list do плохая практика?

maxyc_webber: да, плохая. Вообще, в контроллере публичными должны быть только экшены, а все остальные методы — либо приватные, либо protected. Ну и при таком вызове метода получается, что мы прямо в представлении делаем запрос к бд для загрузки данных. Не комильфо. Да и не к чему такие выкрутасы, всё делается проще.

Jeiwan: В таком случае у меня (я с пхп пришел) ломаются шаблоны проектирования.
Допустим. у меня 100 контроллеров. Наследуются от одного Application. Страница, помимо основного контента, содержит несколько динамичных элементов: список городов, список специализаций, список последних тем с форума.
Получается все 100 контроллеров будут делать запросы к бд и формировать переменные, которые потом двинутся в сторону view... а если я для части контроллеров сменил лэйаут и мне там не нужны эти 20 переменных, нужны другие 30?

я гипотетически )

maxyc_webber: ну так контроллеры не одновременно же выполняются. Такая цепочка: запрос на урл → экшн в контроллере, который собирает данные из бд → рендер вьюхи. Если у тебя в экшене грузятся 20 разных типов данных (сущностей), то это, скорее, ошибка проектирования. Интерфейс просто превратиться в кашу.
Если города нужно будет грузить не для всех экшенов, то это ограничение можно сделать в before_action.
Да, всё зависит от конкретной задачи. Например, список городов можно выводить не в лэйауте, а подключать через паршиал на тех страницах, где он нужен. Но если таких страниц большинство, или вообще этот список должен быть на все страницах, то тогда нужно делать через before_action.
Единственное, что по-моему точно не правильно, так это грузить их через хэлперы. Хэлперы не должны делать запросов к БД, а они всего-лишь помогают обрабатывать данные для вьюх. А данные предоставляет контроллер, который берет их из БД.

Jeiwan: спасибо. стало яснее.