Refactor: デフォルト値を活用して if 文の二段ネストを一段にした

Author: yasulab

app/controllers/events_controller.rb

@@ -22,28 +22,29 @@ def latest
     # ]
     @latest_event_by_dojos = []
     Dojo.active.each do |dojo|
-      latest_event = dojo.event_histories.newest.first
-
       link_in_note = dojo.note.match(URI.regexp)
       date_in_note = dojo.note.match(/(\d{4}-\d{1,2}-\d{1,2})/) # YYYY-MM-DD
       last_session_link = link_in_note.nil? ? dojo_path(dojo.id) : link_in_note.to_s
       last_session_date = date_in_note.nil? ? dojo.created_at    : Time.zone.parse(date_in_note.to_s)
+
+      latest_event    = dojo.event_histories.newest.first
+      latest_event_at = latest_event.nil? ? Time.zone.parse('2000-01-23') : latest_event.evented_at
       @latest_event_by_dojos << {
         id:   dojo.id,
         name: dojo.name,
         note: dojo.note,
         url:  dojo.url,
         has_event_histories: latest_event.nil?,
 
-        # 過去のイベント開催データが無ければ、note 内にある日付または掲載日を表示
-        event_at:  latest_event.nil? ?
+        # 過去のイベント開催日と note 内の日付を比較し、新しい方の日付を表示
+        event_at: (latest_event_at < last_session_date) ?
           last_session_date.strftime("%Y-%m-%d") :
-          (latest_event.evented_at < last_session_date) ? last_session_date.strftime("%Y-%m-%d") : latest_event.evented_at.strftime("%Y-%m-%d"),
+          latest_event.evented_at.strftime("%Y-%m-%d"),
 
-        # 過去のイベント開催データが無ければ、note 内にあるリンクまたは個別統計ページを表示
-        event_url: latest_event.nil? ?
+        # 過去のイベント開催日と note 内の日付を比較し、新しい方のリンクを表示
+        event_url: (latest_event_at < last_session_date) ?
           last_session_link :
-          (latest_event.evented_at < last_session_date) ? last_session_link : latest_event.event_url
+          latest_event.event_url
       }
     end
 

app/views/events/latest.html.erb

@@ -83,7 +83,7 @@
 		</span>
               <% else %>
 		<!-- 別サービスでイベント管理している Dojo で、"Active" マークが無い場合はハイライトする -->
-		<!-- 別サービスでイベント管理している Dojo の場合、note にある日付とリンクで分岐処理する -->
+		<!-- 別サービスでイベント管理している Dojo なら、note にある日付とリンクがあれば表示する -->
                 <span class=<%= 'expired' if dojo[:event_at] <= Time.current.prev_year && !dojo[:note].include?('Active') %>>
 		  <%= link_to dojo[:event_at], dojo[:event_url] %>
 		</span>
@@ -92,7 +92,7 @@
           </td>
           <td class="url-cell">
             <small>
-              <span title="<%= dojo[:note] %>"><%= raw Rinku.auto_link(dojo[:note]) %></span>
+              <span title="<%= dojo[:note] %>"><%= raw Addressable::URI.unescape(Rinku.auto_link(dojo[:note])) %></span>
             </small>
           </td>
         </tr>