DB取得したメニュー情報をセッションに詰めるとうまくループで表示できない

Question

railsでアプリケーションを作っており、今、以下のイメージでメニューを生成しようと考えています。

１：DBに設定されているメニュー情報(メニュー名、メニューパス）を取得し、session[:menus]に設定
２：html.erbでループ表示

という実装をしたいのですが、うまく実装できません。

DBに取得して、セッションに詰めるところまではデバッグで確認できたのですが、
その後でhtml.erbに表示する部分ができません。

単純にerb側の構文の間違いのような気もするのですが、どのようにしたら正しく表示されるでしょうか？

①：以下の方法ではできませんでした。

**homeController.rb**

    #ユーザーがサインイン済みであれば、以下の処理を行う
    if user_signed_in?
      #メニューを取得
      session[:menus] = Menu.all 

    end

**application.html.rb**

 <% session[:menus].each do |menu| %>
    <li><%= link_to menu.menuNm, menu.menuPath %></li>
 <% end %>

②：以下の方法でもできませんでした。

**homeController.rb**

    #ユーザーがサインイン済みであれば、以下の処理を行う
    if user_signed_in?
      #メニューを取得
      session[:menus] = Menu.all
      $menus = session[:menus]

    end

**application.html.rb**

 <% $menus.each do |menu| %>
    <li><%= link_to menu.menuNm, menu.menuPath %></li>
 <% end %>

③：以下の方法だとなぜかできます。

**homeController.rb**

    #ユーザーがサインイン済みであれば、以下の処理を行う
    if user_signed_in?
      #メニューを取得
      
      $menus = Menu.all

    end

**application.html.rb**

 <% $menus.each do |menu| %>
    <li><%= link_to menu.menuNm, menu.menuPath %></li>
 <% end %>

型の問題等があるのでしょうか？
知識不足で大変申し訳無いのですが、お答えいただけると助かります。
よろしくお願いいたします。

Answer 1

Rails的には毎回Menuを読み込むべきです

module ApplicationHelper
  def user_menus
    return [] unless user_signed_in?
    Menu.all
  end
end

**application.html.rb**

 <% user_menus.each do |menu| %>
    <li><%= link_to menu.menuNm, menu.menuPath %></li>
 <% end %>

補足

■ session

sessionに認証以外のデータ、特に複雑なオブジェクトを入れる考えは止めましょう
VersionUpや機能拡張等でsession内と実装の乖離が生まれると動かなくなる危険性があります

■ $ はグローバル変数です

動きはしますが、controller/view以外の全てにも影響するので、基本的に使うべきではありません

Comments

[ManaKuri09]

回答有り難うございます！
毎回取るようにしたほうが良かったのですね。
現場でjavaをやっていたときには共通で持ち回るものは基本的にセッションに詰めていたので、その方が良いのかと思ってました。
ご教示いただいた内容を参考に修正してみようと思います。ありがとうございます！

[Ighrs]

無駄に通信したくないのであれば、描画内容のキャッシュを行うなど、それを目的とした仕組みで専用に工夫すべきでしょう
言語やフレームワークにもよるとは思いますが、少なくともRailsでsessionに過剰な責務を持たせるのは、独立性や保守性も下がりますしアンチパターンとして扱われています