Replace magic values with appsPerScreen and entriesPerScreen

2 files changed, 6 insertions, 5 deletions

diff --git a/src/displayapp/screens/settings/Settings.cpp b/src/displayapp/screens/settings/Settings.cpp
index 98d8c22..1c964c4 100644
--- a/src/displayapp/screens/settings/Settings.cpp
+++ b/src/displayapp/screens/settings/Settings.cpp
@@ -33,9 +33,9 @@ bool Settings::OnTouchEvent(Pinetime::Applications::TouchEvents event) {
 }
 
 std::unique_ptr<Screen> Settings::CreateScreen(unsigned int screenNum) const {
-  std::array<List::Applications, 4> screens;
-  for (int i = 0; i < 4; i++) {
-    screens[i] = entries[screenNum * 4 + i];
+  std::array<List::Applications, entriesPerScreen> screens;
+  for (int i = 0; i < entriesPerScreen; i++) {
+    screens[i] = entries[screenNum * entriesPerScreen + i];
   }
 
   return std::make_unique<Screens::List>(screenNum, nScreens, app, settingsController, screens);
diff --git a/src/displayapp/screens/settings/Settings.h b/src/displayapp/screens/settings/Settings.h
index 4ef507a..ad4bf08 100644
--- a/src/displayapp/screens/settings/Settings.h
+++ b/src/displayapp/screens/settings/Settings.h
@@ -24,7 +24,8 @@ namespace Pinetime {
 
         Controllers::Settings& settingsController;
 
-        static constexpr std::array<List::Applications, 16> entries {{
+        static constexpr int entriesPerScreen = 4;
+        static constexpr std::array<List::Applications, entriesPerScreen * 4> entries {{
           {Symbols::sun, "Display", Apps::SettingDisplay},
           {Symbols::eye, "Wake Up", Apps::SettingWakeUp},
           {Symbols::clock, "Time format", Apps::SettingTimeFormat},
@@ -45,7 +46,7 @@ namespace Pinetime {
           {Symbols::none, "None", Apps::None},
           {Symbols::none, "None", Apps::None},
         }};
-        static constexpr int nScreens = (entries.size() + 3) / 4;
+        static constexpr int nScreens = (entries.size() + entriesPerScreen - 1) / entriesPerScreen;
         ScreenList<nScreens> screens;
       };
     }