From c35e840694bdcab16270630752f5ba02fc4b813a Mon Sep 17 00:00:00 2001
From: Alex Ling <hkalexling@gmail.com>
Date: Tue, 19 May 2020 12:16:32 +0000
Subject: [PATCH] Refactor the `/` route

---
 src/routes/main.cr | 11 ++++++-----
 1 file changed, 6 insertions(+), 5 deletions(-)

diff --git a/src/routes/main.cr b/src/routes/main.cr
index c4c6cb8..523388c 100644
--- a/src/routes/main.cr
+++ b/src/routes/main.cr
@@ -68,11 +68,12 @@ class MainRouter < Router
         titles = @context.library.titles
         username = get_username env
 
-        on_deck_entries = [] of Entry
-        titles.each do |title|
-          on_deck_entry = title.get_on_deck_entry username
-          on_deck_entries << on_deck_entry if on_deck_entry # ingnore titles without latest on deck entry
-        end
+        # map: get the on-deck entry or nil for each Title
+        # select: select only entries (and ignore Nil's) from the array
+        #   produced by map
+        on_deck_entries = titles.map { |t|
+          t.get_on_deck_entry username
+        }.select Entry
 
         layout "home"
       rescue e