From c4d4f8fd98c982a433b5e25b6bbdcc503a7e3ad7 Mon Sep 17 00:00:00 2001
From: Olga Khlopkova <o.khlopkova@corp.mail.ru>
Date: Wed, 2 Sep 2020 17:55:16 +0300
Subject: [PATCH] [transit] Replace magic numbers with enum in Schedule.

---
 transit/transit_schedule.cpp | 14 +++++++-------
 transit/transit_schedule.hpp | 11 +++++++++++
 2 files changed, 18 insertions(+), 7 deletions(-)

diff --git a/transit/transit_schedule.cpp b/transit/transit_schedule.cpp
index 0a48a445cd..7ca110c491 100644
--- a/transit/transit_schedule.cpp
+++ b/transit/transit_schedule.cpp
@@ -147,13 +147,13 @@ std::tuple<Date, Date, WeekSchedule> DatesInterval::Extract() const
   date2.m_month = (m_data >> 12) & kMask4bits;
   date2.m_day = (m_data >> 7) & kMask5bits;
 
-  week[0] = m_data & 0x40;
-  week[1] = m_data & 0x20;
-  week[2] = m_data & 0x10;
-  week[3] = m_data & 0x8;
-  week[4] = m_data & 0x4;
-  week[5] = m_data & 0x2;
-  week[6] = m_data & 0x1;
+  week[WeekDays::Sunday] = m_data & 0x40;
+  week[WeekDays::Monday] = m_data & 0x20;
+  week[WeekDays::Tuesday] = m_data & 0x10;
+  week[WeekDays::Wednesday] = m_data & 0x8;
+  week[WeekDays::Thursday] = m_data & 0x4;
+  week[WeekDays::Friday] = m_data & 0x2;
+  week[WeekDays::Saturday] = m_data & 0x1;
 
   return {date1, date2, week};
 }
diff --git a/transit/transit_schedule.hpp b/transit/transit_schedule.hpp
index 816d95f8b4..570d8fe4cf 100644
--- a/transit/transit_schedule.hpp
+++ b/transit/transit_schedule.hpp
@@ -87,6 +87,17 @@ struct Time
 };
 
 using WeekSchedule = std::array<bool, 7>;
+
+enum WeekDays
+{
+  Sunday = 0,
+  Monday,
+  Tuesday,
+  Wednesday,
+  Thursday,
+  Friday,
+  Saturday
+};
 // Service dates specified using a weekly schedule with start and end dates. Dates range is
 // specified by the start_date and end_date fields in the GTFS calendar.txt.
 // Dates interval and open/closed states for week days are stored |m_data|.