refactor: inline_プレフィックスパターンでflashメッセージの表示位置を制御

yasulab · yasulab · commit 4ed4e33badf8 · 2025-08-08T14:51:04.000+09:00
## 実装内容
- 'inline_' プレフィックスがついたflashメッセージは、デフォルト位置（ページ上部）では表示しない
- 各ビュー内でカスタム位置に表示できるようにヘルパーメソッドを追加
- /dojos ページの年フィルタリングのアラートとインフォメッセージに適用

## メリット
- flashメッセージの2重表示問題を解決
- inline_alert → alert-alert のようにBootstrapのCSSクラスを自動適用
- 将来的に inline_warning, inline_success なども同じパターンで使用可能
- コードがシンプルになり、条件分岐が不要に

## 変更箇所
- flash[:alert] → flash[:inline_alert] に変更（エラー時）
- flash.now[:inline_info] を追加（成功時）
- render_inline_flash_messages ヘルパーメソッドを追加
- テストを inline_alert に対応
diff --git a/app/controllers/dojos_controller.rb b/app/controllers/dojos_controller.rb
@@ -8,7 +8,7 @@ def index
         year = params[:year].to_i
         # 有効な年の範囲をチェック
         unless year.between?(2012, Date.current.year)
-          flash[:alert] = "指定された年は無効です。2012年から#{Date.current.year}年の間で指定してください。"
+          flash[:inline_alert] = "指定された年は無効です。2012年から#{Date.current.year}年の間で指定してください。"
           return redirect_to dojos_path(anchor: 'table')
         end
         
@@ -18,8 +18,11 @@ def index
         # その年末時点でアクティブだった道場を取得
         dojos_scope = Dojo.active_at(end_of_year)
         @page_title = "#{@selected_year}年末時点のCoderDojo一覧"
+        
+        # 成功メッセージもinline_で表示
+        flash.now[:inline_info] = "#{@selected_year}年末時点のアクティブな道場を表示中"
       rescue ArgumentError
-        flash[:alert] = "無効な年が指定されました"
+        flash[:inline_alert] = "無効な年が指定されました"
         return redirect_to dojos_path(anchor: 'table')
       end
     else
diff --git a/app/helpers/application_helper.rb b/app/helpers/application_helper.rb
@@ -48,6 +48,15 @@ def page_lang(lang)
     lang.empty? ? 'ja' : lang
   end
 
+  # 'inline_' プレフィックスがついたflashメッセージをビュー内で表示するヘルパー
+  # inline_alert → alert, inline_warning → warning のように変換してBootstrapのCSSクラスを適用
+  def render_inline_flash_messages
+    flash.select { |type, _| type.to_s.start_with?('inline_') }.map do |type, message|
+      css_class = type.to_s.gsub('inline_', '')
+      content_tag(:div, message, class: "alert alert-#{css_class}", style: "margin-bottom: 15px;")
+    end.join.html_safe
+  end
+
   def kata_description
     "道場で役立つ資料やコンテスト情報、立ち上げ方や各種支援をまとめています。"
   end
diff --git a/app/views/dojos/index.html.erb b/app/views/dojos/index.html.erb
@@ -33,15 +33,7 @@
       年次データのフィルタリング
     </h3>
     
-    <% if flash[:alert] %>
-      <div style="padding: 10px; background: #f8d7da; border: 1px solid #f5c6cb; border-radius: 4px; margin-bottom: 15px; color: #721c24;">
-        <%= flash[:alert] %>
-      </div>
-    <% elsif @selected_year %>
-      <div style="padding: 10px; background: #d1ecf1; border: 1px solid #bee5eb; border-radius: 4px; margin-bottom: 15px;">
-        <strong><%= @selected_year %>年末時点</strong>のアクティブな道場を表示中
-      </div>
-    <% end %>
+    <%= render_inline_flash_messages %>
     
     <div style="display: flex; align-items: center; justify-content: center; gap: 10px; margin-top: 15px;">
       <%= label_tag :year, '対象期間:', style: 'font-weight: bold;' %>
diff --git a/app/views/layouts/application.html.erb b/app/views/layouts/application.html.erb
@@ -88,8 +88,11 @@
   <body>
     <%= render 'shared/header' %>
 
+    <%# 'inline_' プレフィックスがついたflashメッセージは、ここでは表示せず、各ビュー内でカスタム表示する %>
     <% flash.each do |message_type, message| %>
-      <div class="alert alert-<%= message_type %>"><%= message %></div>
+      <% unless message_type.to_s.start_with?('inline_') %>
+        <div class="alert alert-<%= message_type %>"><%= message %></div>
+      <% end %>
     <% end %>
 
     <%= yield %>
diff --git a/spec/requests/dojos_spec.rb b/spec/requests/dojos_spec.rb
@@ -55,20 +55,20 @@
       it "rejects years before 2012" do
         get dojos_path(year: 2011, format: :json)
         expect(response).to redirect_to(dojos_path(anchor: 'table'))
-        expect(flash[:alert]).to include("2012年から")
+        expect(flash[:inline_alert]).to include("2012年から")
       end
       
       it "rejects years after current year" do
         future_year = Date.current.year + 1
         get dojos_path(year: future_year, format: :json)
         expect(response).to redirect_to(dojos_path(anchor: 'table'))
-        expect(flash[:alert]).to include("指定された年は無効です")
+        expect(flash[:inline_alert]).to include("指定された年は無効です")
       end
       
       it "handles invalid year strings" do
         get dojos_path(year: "invalid", format: :json)
         expect(response).to redirect_to(dojos_path(anchor: 'table'))
-        expect(flash[:alert]).to include("無効")
+        expect(flash[:inline_alert]).to include("無効")
       end
     end
     
@@ -253,6 +253,8 @@
         get dojos_path(year: 2020)
         expect(response.body).to include('2020年末時点')
         expect(response.body).to include('アクティブな道場を表示中')
+        # inline_infoメッセージが表示されることを確認
+        expect(response.body).to include('alert-info')
       end
       
       it "includes CSV and JSON download links with year parameter" do
