Правильно ли сделано кэширование?

Question

[vadimstroganov]

Здравствуйте!
Есть партиал с меню.
Хочу этот фрагмент закешировать, вроде бы получилось, но при обновлении записи в меню, партиал остается тем же, кэш не сбрасывается, не пойму что не так.

_menu.slim

- menu = render_menu(code)

ul class="nav navbar-nav navbar-right"
  = cache ['menu', Menu.latest]
    - menu.each do |item|
      = cache item do
        li
          = (item.page.template.name == "index") ? link_to(item.page.title, "/") : link_to(item.page.title, item.page.slug)

В модели Menu.rb

def self.latest
  MenuTranslation.order(:updated_at).last
end

application_helper.rb:

def render_menu(code)
  menu = Menu.find_by :code => code
  menu.children
end

Полей created_at, updated_at в таблице menus нет, они есть в menu_translations (там хранятся переводы пункта меню)

И еще вопрос, сама ссылка на пункт меню хранится в таблице menus, то есть мне и туда нужно добавлять updated_at, и потом следить за updated_at в двух таблицах?

Answer 1

[Запорожченко Олег]

Функция cache может принимать или уже готовый ключ для хранения, или, как в Вашем случае, экземпляр модели. В этом случае ключ будет построен из id и updated_at, поэтому updated_at должен быть во всех моделях.

Еще в Вашем коде
- menu = render_menu(code)

вызывается до первого кеша, в этом случае в первом вызове кеша нет никакого смысла

Проще всего посмотреть логи консоли, в случае с кеширование рельса пишет какие фрагменты использует

Comments

[vadimstroganov]

А как правильней тогда сделать получение menu ?

[Запорожченко Олег]

Strollager: я не вижу всей вашей структуры базы, и код совсем не очевиден. Киньте модельки Menu и MenuTranslation

[vadimstroganov]

Олег: Как я понимаю, такой вариант должен быть верным?
Menu.rb

class Menu < ActiveRecord::Base
  translates :title
  accepts_nested_attributes_for :translations, allow_destroy: true

  acts_as_nested_set

  belongs_to :page

  def self.latest
    MenuTranslation.order(:updated_at).last
  end

end

MenuTranslation пуста, используется только для того чтобы дернуть данные из таблицы

application_helper.rb

module ApplicationHelper

  def render_menu(code, partial)
    menu = Menu.find_by :code => code
    render :partial => partial, :locals => { :menu => menu.children }
  end

end

в партиал _header.slim я подключаю меню так:
= render_menu("header_menu", "common/menu")

Вроде все работает, вопрос только все ли правильно сделал

[vadimstroganov]

_menu.slim

ul class="nav navbar-nav navbar-right"
  = cache ['menu', Menu.latest]
    - menu.each do |item|
      = cache item do
        li
          = (item.page.template.name == "index") ? link_to(item.title, "/") : link_to(item.title, item.page.slug)

[Запорожченко Олег]

Strollager: Давайте поясню момент, который меня смущает. Мне кажется, что либо тут не все гладко, либо я не доконца понимаю Вашу структуру

= cache ['menu', Menu.latest]
    - menu.each do |item|

Вызов этого кеша нужен для того, чтобы избежать запроса

Menu.find_by(:code => code).children

Т.е кешируется вызов к модели Menu, но ключ кеша, по которому определяется актуальнен кеш или нет строится по другой модели -

MenuTranslation.order(:updated_at).last

И еще в этом кеше никак не учавствует код меню - "header_menu". Вы пробовали одновременно выводить 2 закешированнных меню? Если у вас для другого меню другой паршиал, то лучше не делать вот так render_menu("header_menu", "common/menu") а создать по хелперу для каждого меню.