From c403c479d714934e204448ea16e146947cb9de4b Mon Sep 17 00:00:00 2001 From: Giuseppe Tornello Date: Thu, 30 Oct 2025 17:50:17 +0100 Subject: [PATCH 01/11] refactor: replacing if statement with switch --- exercises/ex1.cpp | 58 +++++++++++++++++++++-------------------------- 1 file changed, 26 insertions(+), 32 deletions(-) diff --git a/exercises/ex1.cpp b/exercises/ex1.cpp index 22069589..a20efb90 100644 --- a/exercises/ex1.cpp +++ b/exercises/ex1.cpp @@ -11,38 +11,32 @@ int main() cout << "Enter week number(1-7): " << endl; cin >> week; - if (week == 1) - { - cout << "Monday" << endl; - } - else if (week == 2) - { - cout << "Tuesday" << endl; - } - else if (week == 3) - { - cout << "Wednesday" << endl; - } - else if (week == 4) - { - cout << "Thursday" << endl; - } - else if (week == 5) - { - cout << "Friday" << endl; - } - else if (week == 6) - { - cout << "Saturday" << endl; - } - else if (week == 7) - { - cout << "Sunday" << endl; - } - else - { - cout << "Invalid input! Please enter week number between 1-7." << endl; + switch(week){ + case 1: + cout << "Monday" << endl; + break; + case 2: + cout << "Tuesday" << endl; + break; + case 3: + cout << "Wednesday" << endl; + break; + case 4: + cout << "Thursday" << endl; + break; + case 5: + cout << "Friday" << endl; + break; + case 6: + cout << "Saturday" << endl; + break; + case 7: + cout << "Sunday" << endl; + break; + default: + cout << "Invalid input! Please enter week number between 1-7." << endl; + break; } return 0; -} \ No newline at end of file +} From 648684e00cfe1392ee2bd02225e95e093898c138 Mon Sep 17 00:00:00 2001 From: Giuseppe Tornello Date: Mon, 3 Nov 2025 11:05:57 +0100 Subject: [PATCH 02/11] Fix:indentation --- exercises/ex1.cpp | 53 +++++++++++++++++++++++------------------------ 1 file changed, 26 insertions(+), 27 deletions(-) diff --git a/exercises/ex1.cpp b/exercises/ex1.cpp index a20efb90..b43228e9 100644 --- a/exercises/ex1.cpp +++ b/exercises/ex1.cpp @@ -3,39 +3,38 @@ #include using namespace std; -int main() -{ +int main() { int week; /* Input week number from user */ cout << "Enter week number(1-7): " << endl; cin >> week; - switch(week){ - case 1: - cout << "Monday" << endl; - break; - case 2: - cout << "Tuesday" << endl; - break; - case 3: - cout << "Wednesday" << endl; - break; - case 4: - cout << "Thursday" << endl; - break; - case 5: - cout << "Friday" << endl; - break; - case 6: - cout << "Saturday" << endl; - break; - case 7: - cout << "Sunday" << endl; - break; - default: - cout << "Invalid input! Please enter week number between 1-7." << endl; - break; + switch (week) { + case 1: + cout << "Monday" << endl; + break; + case 2: + cout << "Tuesday" << endl; + break; + case 3: + cout << "Wednesday" << endl; + break; + case 4: + cout << "Thursday" << endl; + break; + case 5: + cout << "Friday" << endl; + break; + case 6: + cout << "Saturday" << endl; + break; + case 7: + cout << "Sunday" << endl; + break; + default: + cout << "Invalid input! Please enter week number between 1-7." << endl; + break; } return 0; From eac06289091994fc2878e0da8627faefbc621c66 Mon Sep 17 00:00:00 2001 From: Giuseppe Tornello Date: Mon, 3 Nov 2025 11:33:02 +0100 Subject: [PATCH 03/11] Revert "Fix:indentation" This reverts commit 648684e00cfe1392ee2bd02225e95e093898c138. --- exercises/ex1.cpp | 53 ++++++++++++++++++++++++----------------------- 1 file changed, 27 insertions(+), 26 deletions(-) diff --git a/exercises/ex1.cpp b/exercises/ex1.cpp index b43228e9..a20efb90 100644 --- a/exercises/ex1.cpp +++ b/exercises/ex1.cpp @@ -3,38 +3,39 @@ #include using namespace std; -int main() { +int main() +{ int week; /* Input week number from user */ cout << "Enter week number(1-7): " << endl; cin >> week; - switch (week) { - case 1: - cout << "Monday" << endl; - break; - case 2: - cout << "Tuesday" << endl; - break; - case 3: - cout << "Wednesday" << endl; - break; - case 4: - cout << "Thursday" << endl; - break; - case 5: - cout << "Friday" << endl; - break; - case 6: - cout << "Saturday" << endl; - break; - case 7: - cout << "Sunday" << endl; - break; - default: - cout << "Invalid input! Please enter week number between 1-7." << endl; - break; + switch(week){ + case 1: + cout << "Monday" << endl; + break; + case 2: + cout << "Tuesday" << endl; + break; + case 3: + cout << "Wednesday" << endl; + break; + case 4: + cout << "Thursday" << endl; + break; + case 5: + cout << "Friday" << endl; + break; + case 6: + cout << "Saturday" << endl; + break; + case 7: + cout << "Sunday" << endl; + break; + default: + cout << "Invalid input! Please enter week number between 1-7." << endl; + break; } return 0; From 2f814023a9b07c5c8f057261b3714df6f5907a7e Mon Sep 17 00:00:00 2001 From: Giuseppe Tornello Date: Mon, 3 Nov 2025 11:35:10 +0100 Subject: [PATCH 04/11] Revert "refactor: replacing if statement with switch" This reverts commit c403c479d714934e204448ea16e146947cb9de4b. --- exercises/ex1.cpp | 58 ++++++++++++++++++++++++++--------------------- 1 file changed, 32 insertions(+), 26 deletions(-) diff --git a/exercises/ex1.cpp b/exercises/ex1.cpp index a20efb90..22069589 100644 --- a/exercises/ex1.cpp +++ b/exercises/ex1.cpp @@ -11,32 +11,38 @@ int main() cout << "Enter week number(1-7): " << endl; cin >> week; - switch(week){ - case 1: - cout << "Monday" << endl; - break; - case 2: - cout << "Tuesday" << endl; - break; - case 3: - cout << "Wednesday" << endl; - break; - case 4: - cout << "Thursday" << endl; - break; - case 5: - cout << "Friday" << endl; - break; - case 6: - cout << "Saturday" << endl; - break; - case 7: - cout << "Sunday" << endl; - break; - default: - cout << "Invalid input! Please enter week number between 1-7." << endl; - break; + if (week == 1) + { + cout << "Monday" << endl; + } + else if (week == 2) + { + cout << "Tuesday" << endl; + } + else if (week == 3) + { + cout << "Wednesday" << endl; + } + else if (week == 4) + { + cout << "Thursday" << endl; + } + else if (week == 5) + { + cout << "Friday" << endl; + } + else if (week == 6) + { + cout << "Saturday" << endl; + } + else if (week == 7) + { + cout << "Sunday" << endl; + } + else + { + cout << "Invalid input! Please enter week number between 1-7." << endl; } return 0; -} +} \ No newline at end of file From a2b20f03119b9f1d65ce251dc4374e991dbb85eb Mon Sep 17 00:00:00 2001 From: Giuseppe Tornello Date: Mon, 3 Nov 2025 16:12:42 +0100 Subject: [PATCH 05/11] refactor: replacing deprecated C I/O functions --- exercises/ex4.cpp | 7 ++++--- 1 file changed, 4 insertions(+), 3 deletions(-) diff --git a/exercises/ex4.cpp b/exercises/ex4.cpp index 9d93d8ac..87af6aae 100644 --- a/exercises/ex4.cpp +++ b/exercises/ex4.cpp @@ -1,14 +1,15 @@ /* Surprise me. */ -#include +#include +using namespace std; int main() { int month; /* Input month number from user */ - printf("Enter month number(1-12): "); - scanf("%d", &month); + cout << "Enter month number(1-12): " << endl; + cin >> month; switch (month) { From e312c6bdf87eefb964538cbcc907d69e939ad105 Mon Sep 17 00:00:00 2001 From: Giuseppe Tornello Date: Mon, 3 Nov 2025 16:39:45 +0100 Subject: [PATCH 06/11] refactor: improve switch case handling --- exercises/ex4.cpp | 31 +++++++------------------------ 1 file changed, 7 insertions(+), 24 deletions(-) diff --git a/exercises/ex4.cpp b/exercises/ex4.cpp index 87af6aae..1dfadfde 100644 --- a/exercises/ex4.cpp +++ b/exercises/ex4.cpp @@ -14,41 +14,24 @@ int main() switch (month) { case 1: + case 3: + case 5: + case 7: + case 8: + case 10: + case 12: printf("31 days"); break; case 2: printf("28/29 days"); break; - case 3: - printf("31 days"); - break; case 4: - printf("30 days"); - break; - case 5: - printf("31 days"); - break; case 6: - printf("30 days"); - break; - case 7: - printf("31 days"); - break; - case 8: - printf("31 days"); - break; case 9: - printf("30 days"); - break; - case 10: - printf("31 days"); - break; case 11: printf("30 days"); break; - case 12: - printf("31 days"); - break; + default: printf("Invalid input! Please enter month number between 1-12"); } From 0f521df55aadcb333ac5d972a21c5fbb61dad99a Mon Sep 17 00:00:00 2001 From: Giuseppe Tornello Date: Mon, 3 Nov 2025 16:48:04 +0100 Subject: [PATCH 07/11] refactor: Improved switch readability --- exercises/ex4.cpp | 39 +++++++++++++++++++++++++++------------ 1 file changed, 27 insertions(+), 12 deletions(-) diff --git a/exercises/ex4.cpp b/exercises/ex4.cpp index 1dfadfde..5d363ad3 100644 --- a/exercises/ex4.cpp +++ b/exercises/ex4.cpp @@ -3,6 +3,21 @@ #include using namespace std; +enum months{ + January=1, + February, + March, + April, + May, + June, + July, + August, + September, + October, + November, + December +}; + int main() { int month; @@ -13,22 +28,22 @@ int main() switch (month) { - case 1: - case 3: - case 5: - case 7: - case 8: - case 10: - case 12: + case January: + case March: + case May: + case July: + case August: + case October: + case December: printf("31 days"); break; - case 2: + case February: printf("28/29 days"); break; - case 4: - case 6: - case 9: - case 11: + case April: + case June: + case September: + case November: printf("30 days"); break; From 0788ce5d8554d9f39a9e548383f69930562a27c3 Mon Sep 17 00:00:00 2001 From: Giuseppe Tornello Date: Mon, 3 Nov 2025 16:53:53 +0100 Subject: [PATCH 08/11] refactor: replaced deprecated C I/O functions pt2 --- exercises/ex4.cpp | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/exercises/ex4.cpp b/exercises/ex4.cpp index 5d363ad3..8a530438 100644 --- a/exercises/ex4.cpp +++ b/exercises/ex4.cpp @@ -35,16 +35,16 @@ int main() case August: case October: case December: - printf("31 days"); + cout << "31 days" << endl; break; case February: - printf("28/29 days"); + cout << "28/29 days" << endl; break; case April: case June: case September: case November: - printf("30 days"); + cout << "30 days" << endl; break; default: From 75fb69e3f401b06f8cf644f3cd72809cc5f4d51e Mon Sep 17 00:00:00 2001 From: Giuseppe Tornello Date: Mon, 3 Nov 2025 16:54:23 +0100 Subject: [PATCH 09/11] refactor: replaced deprecated C I/O functions pt3 --- exercises/ex4.cpp | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/exercises/ex4.cpp b/exercises/ex4.cpp index 8a530438..a247d2e5 100644 --- a/exercises/ex4.cpp +++ b/exercises/ex4.cpp @@ -48,7 +48,7 @@ int main() break; default: - printf("Invalid input! Please enter month number between 1-12"); + cout << "Invalid input! Please enter month number between 1-12" << endl; } return 0; From 2a9690058b5c054e1990f2f06debb21cc94ddda5 Mon Sep 17 00:00:00 2001 From: Giuseppe Tornello Date: Mon, 3 Nov 2025 17:29:04 +0100 Subject: [PATCH 10/11] refactor: moved switch to a different function --- exercises/ex4.cpp | 53 ++++++++++++++++++++++++++--------------------- 1 file changed, 29 insertions(+), 24 deletions(-) diff --git a/exercises/ex4.cpp b/exercises/ex4.cpp index a247d2e5..79aeaa05 100644 --- a/exercises/ex4.cpp +++ b/exercises/ex4.cpp @@ -3,6 +3,8 @@ #include using namespace std; +void printMonth(int& month); + enum months{ January=1, February, @@ -18,6 +20,32 @@ enum months{ December }; +void printMonth(int& month){ + switch (month){ + case January: + case March: + case May: + case July: + case August: + case October: + case December: + cout << "31 days" << endl; + break; + case February: + cout << "28/29 days" << endl; + break; + case April: + case June: + case September: + case November: + cout << "30 days" << endl; + break; + default: + cout << "Invalid input! Please enter month number between 1-12" << endl; + break; + } +} + int main() { int month; @@ -26,30 +54,7 @@ int main() cout << "Enter month number(1-12): " << endl; cin >> month; - switch (month) - { - case January: - case March: - case May: - case July: - case August: - case October: - case December: - cout << "31 days" << endl; - break; - case February: - cout << "28/29 days" << endl; - break; - case April: - case June: - case September: - case November: - cout << "30 days" << endl; - break; - - default: - cout << "Invalid input! Please enter month number between 1-12" << endl; - } + printMonth(month); return 0; } \ No newline at end of file From a8b6113bd70c6060723726b317d4f9927193a95f Mon Sep 17 00:00:00 2001 From: Giuseppe Tornello Date: Mon, 3 Nov 2025 17:52:10 +0100 Subject: [PATCH 11/11] chore(printMonth): set const parameter in funcion --- exercises/ex4.cpp | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/exercises/ex4.cpp b/exercises/ex4.cpp index 79aeaa05..b0ece680 100644 --- a/exercises/ex4.cpp +++ b/exercises/ex4.cpp @@ -20,7 +20,7 @@ enum months{ December }; -void printMonth(int& month){ +void printMonth(const int& month){ switch (month){ case January: case March: