refactor: 非アクティブ判定の閾値をDojoモデルに定数化

- INACTIVE_THRESHOLD = 1.year をDojoモデルに定義
- メンテナンス性向上（閾値変更が容易に）
- ビューでは Time.current.prev_year の代わりに定数を使用
- Claudeからのフィードバックを適用

Author: yasulab

app/controllers/dojos_controller.rb

@@ -109,6 +109,9 @@ def show
   # GET /dojos/activity
   # 道場の活動状況を表示（旧 /events/latest から移行）
   def activity
+    # ビューで使用するための閾値をインスタンス変数に設定（モデルから取得）
+    @inactive_threshold = Dojo::INACTIVE_THRESHOLD
+    
     @latest_event_by_dojos = []
     Dojo.active.each do |dojo|
       link_in_note = dojo.note.match(URI.regexp)

app/models/dojo.rb

@@ -5,6 +5,9 @@ class Dojo < ApplicationRecord
   NUM_OF_TOTAL_EVENTS  = "10,000"
   NUM_OF_TOTAL_NINJAS  = "62,000"
   DOJO_INFO_YAML_PATH = Rails.root.join('db', 'dojos.yml')
+  
+  # 1年以上イベント開催していない道場を判定する閾値
+  INACTIVE_THRESHOLD = 1.year
 
   belongs_to :prefecture
   has_many   :dojo_event_services, dependent: :destroy

app/views/dojos/activity.html.erb

@@ -74,13 +74,13 @@
             <small>
               <% if dojo[:latest_event_at] %>
                 <!-- イベント履歴がある場合 -->
-                <% expired = dojo[:latest_event_at] <= Time.current.prev_year && !dojo[:note].include?('Active') %>
+                <% expired = dojo[:latest_event_at] <= Time.current - @inactive_threshold && !dojo[:note].include?('Active') %>
                 <span class="<%= 'expired' if expired %>">
                   <%= link_to dojo[:latest_event_at].strftime("%Y-%m-%d"), dojo[:latest_event_url] %>
                 </span>
               <% elsif dojo[:note_date] %>
                 <!-- note内に日付がある場合 -->
-                <% expired = dojo[:note_date] <= Time.current.prev_year && !dojo[:note].include?('Active') %>
+                <% expired = dojo[:note_date] <= Time.current - @inactive_threshold && !dojo[:note].include?('Active') %>
                 <span class="<%= 'expired' if expired %>">
                   <% if dojo[:note_link] %>
                     <%= link_to dojo[:note_date].strftime("%Y-%m-%d"), dojo[:note_link] %>