From 655c5b70ed9a7b4e12cad112d1acad97572bdb17 Mon Sep 17 00:00:00 2001
From: Darinka Loncar <149894872+darinkaloncar@users.noreply.github.com>
Date: Mon, 27 May 2024 09:41:10 -0700
Subject: [PATCH 1/9] [Add] method for forming grid

---
 LangLang/BusinessLogic/UseCases/PdfService.cs | 15 ++++++++++++++-
 1 file changed, 14 insertions(+), 1 deletion(-)

diff --git a/LangLang/BusinessLogic/UseCases/PdfService.cs b/LangLang/BusinessLogic/UseCases/PdfService.cs
index 4e602a8..4efadd4 100644
--- a/LangLang/BusinessLogic/UseCases/PdfService.cs
+++ b/LangLang/BusinessLogic/UseCases/PdfService.cs
@@ -49,6 +49,19 @@ public PdfGrid DataToGrid(Dictionary<string, double> data)
             }
             return grid;
         }
-
+        public PdfGrid DataToGrid(Dictionary<string, List<double>> data)
+        {
+            PdfGrid grid = new PdfGrid();
+            grid.Columns.Add(4);
+            foreach (var item in data)
+            {
+                PdfGridRow row = grid.Rows.Add();
+                row.Cells[0].Value = item.Key;
+                row.Cells[1].Value = item.Value[0].ToString();
+                row.Cells[2].Value = item.Value[1].ToString();
+                row.Cells[3].Value = item.Value[2].ToString();
+            }
+            return grid;
+        }
     }
 }

From 0b365e755af725abd782e0186cf311f6b855592c Mon Sep 17 00:00:00 2001
From: Darinka Loncar <149894872+darinkaloncar@users.noreply.github.com>
Date: Mon, 27 May 2024 09:41:48 -0700
Subject: [PATCH 2/9] [Update] method for generating a report of course grades

---
 LangLang/BusinessLogic/UseCases/ReportService.cs | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/LangLang/BusinessLogic/UseCases/ReportService.cs b/LangLang/BusinessLogic/UseCases/ReportService.cs
index dad98fc..3cfc21b 100644
--- a/LangLang/BusinessLogic/UseCases/ReportService.cs
+++ b/LangLang/BusinessLogic/UseCases/ReportService.cs
@@ -89,9 +89,9 @@ public double CalculatePassingPercentage(Course course)
             return (passed / attended)*100;
         }
 
-        public Dictionary<Course, List<double>> GetAverageGradesOfCourses()
+        public Dictionary<string, List<double>> GetAverageGradesOfCourses()
         {
-            Dictionary<Course, List<double>> averages = new();
+            Dictionary<string, List<double>> averages = new();
             CourseService courseService = new();
             List<Course> courses = courseService.GetCoursesHeldInLastYear();
             GradeService gradeService = new();
@@ -102,7 +102,7 @@ public Dictionary<Course, List<double>> GetAverageGradesOfCourses()
                 avg.Add(gradeService.GetAverageKnowledgeGrade(course));
                 avg.Add(gradeService.GetAverageActivityGrade(course));
                 avg.Add(tutorRatingService.GetAverageTutorRating(course));
-                averages.Add(course, avg);
+                averages.Add(course.ToPdfString(), avg);
             }
             return averages;
         }

From 86812383f588d2c5a83e4943726194cd7c0019d2 Mon Sep 17 00:00:00 2001
From: Darinka Loncar <149894872+darinkaloncar@users.noreply.github.com>
Date: Mon, 27 May 2024 09:42:57 -0700
Subject: [PATCH 3/9] [Add] method for sending course grades report to director

---
 LangLang/BusinessLogic/UseCases/SenderService.cs     | 12 ++++++++++++
 .../DirectorViewModels/ReportsViewModel.cs           |  7 +++++++
 2 files changed, 19 insertions(+)

diff --git a/LangLang/BusinessLogic/UseCases/SenderService.cs b/LangLang/BusinessLogic/UseCases/SenderService.cs
index cab8127..6d1dc4d 100644
--- a/LangLang/BusinessLogic/UseCases/SenderService.cs
+++ b/LangLang/BusinessLogic/UseCases/SenderService.cs
@@ -37,6 +37,18 @@ public void SendResults(ExamSlot exam)
                 EmailService.SendEmail(student.Profile.Email, subject, body);
             }
         }
+        public void SendAverageCourseGrades(Director director)
+        {
+            var reportService = new ReportService();
+            var pdfService = new PdfService();
+
+            var reportName = "Average grades and tutor rating per course";
+            var headers = new string[] { "Course", "Average knowledge grade", "Average activity grade", "Average tutor rating" };
+
+            var document = PdfService.GeneratePdf<Dictionary<string, List<double>>>(reportService.GetAverageGradesOfCourses(), headers, reportName, data => pdfService.DataToGrid(data));
+            EmailService.SendEmail(director.Profile.Email, reportName, "", document);
+
+        }
 
         public void SendAveragePoints(Director director)
         {
diff --git a/LangLang/WPF/ViewModels/DirectorViewModels/ReportsViewModel.cs b/LangLang/WPF/ViewModels/DirectorViewModels/ReportsViewModel.cs
index c951099..74bf378 100644
--- a/LangLang/WPF/ViewModels/DirectorViewModels/ReportsViewModel.cs
+++ b/LangLang/WPF/ViewModels/DirectorViewModels/ReportsViewModel.cs
@@ -25,6 +25,13 @@ public void SentAveragePenaltyPoints()
             ShowSuccess();
         }
 
+        public void SentAverageCourseGrades()
+        {
+            var senderService = new SenderService();
+            senderService.SendAverageCourseGrades(_director);
+            ShowSuccess();
+        }
+
         private void ShowSuccess()
         {
             MessageBox.Show("Successfully completed!", "Notification", MessageBoxButton.OK, MessageBoxImage.Information);

From 4cc27292d328fd2f3d077d6488278290ce3f522f Mon Sep 17 00:00:00 2001
From: Darinka Loncar <149894872+darinkaloncar@users.noreply.github.com>
Date: Mon, 27 May 2024 09:43:36 -0700
Subject: [PATCH 4/9] [Update] button for sending the report and implement
 click action listener

---
 LangLang/WPF/Views/DirectorView/Tabs/Reports.xaml    | 2 +-
 LangLang/WPF/Views/DirectorView/Tabs/Reports.xaml.cs | 5 +++++
 2 files changed, 6 insertions(+), 1 deletion(-)

diff --git a/LangLang/WPF/Views/DirectorView/Tabs/Reports.xaml b/LangLang/WPF/Views/DirectorView/Tabs/Reports.xaml
index 29601f1..2c25ff3 100644
--- a/LangLang/WPF/Views/DirectorView/Tabs/Reports.xaml
+++ b/LangLang/WPF/Views/DirectorView/Tabs/Reports.xaml
@@ -26,7 +26,7 @@
             <Button Grid.Row="3" Content="Number of average penalty points per language" BorderBrush="White" Background="#FFFFDBDB" Foreground="#FF817C7C" FontFamily="Segoe UI Black" Click="AveragePenaltyPoints_Click" />
             <Button Grid.Row="4" Content="Number of created courses for each of the existing languages in the system in the last year" BorderBrush="White" Background="#FFFFDBDB" Foreground="#FF817C7C" FontFamily="Segoe UI Black" />
             <Button Grid.Row="5" Content="Number of created  exams for each of the existing languages in the system in the last year" BorderBrush="White" Background="#FFFFDBDB" Foreground="#FF817C7C" FontFamily="Segoe UI Black" />
-            <Button Grid.Row="6" Content="Report b" BorderBrush="White" Background="#FFFFDBDB" Foreground="#FF817C7C" FontFamily="Segoe UI Black" />
+            <Button Grid.Row="6" Content="The average grades of students as well as the average rating of tutor per course in the last year" BorderBrush="White" Background="#FFFFDBDB" Foreground="#FF817C7C" FontFamily="Segoe UI Black" Click="AverageCourseGrades_Click" />
             <Button Grid.Row="7" Content="Report c" BorderBrush="White" Background="#FFFFDBDB" Foreground="#FF817C7C" FontFamily="Segoe UI Black" />
         </Grid>
     </Grid>
diff --git a/LangLang/WPF/Views/DirectorView/Tabs/Reports.xaml.cs b/LangLang/WPF/Views/DirectorView/Tabs/Reports.xaml.cs
index 3d446af..ec932c0 100644
--- a/LangLang/WPF/Views/DirectorView/Tabs/Reports.xaml.cs
+++ b/LangLang/WPF/Views/DirectorView/Tabs/Reports.xaml.cs
@@ -26,5 +26,10 @@ private void AveragePenaltyPoints_Click(object sender, System.Windows.RoutedEven
         {
             _viewModel.SentAveragePenaltyPoints();
         }
+
+        private void AverageCourseGrades_Click(object sender, System.Windows.RoutedEventArgs e)
+        {
+            _viewModel.SentAverageCourseGrades();
+        }
     }
 }

From bf2ddf2571cbd0c783a7a7f7c43af5beec7e0684 Mon Sep 17 00:00:00 2001
From: Darinka Loncar <149894872+darinkaloncar@users.noreply.github.com>
Date: Mon, 27 May 2024 09:44:08 -0700
Subject: [PATCH 5/9] [Update] Course class

Add new method and update an old one.
---
 LangLang/Domain/Models/Course.cs | 7 ++++++-
 1 file changed, 6 insertions(+), 1 deletion(-)

diff --git a/LangLang/Domain/Models/Course.cs b/LangLang/Domain/Models/Course.cs
index 905ba4c..a1b5233 100644
--- a/LangLang/Domain/Models/Course.cs
+++ b/LangLang/Domain/Models/Course.cs
@@ -107,7 +107,7 @@ public bool IsHeldInLastYear()
         {
             var endDate = TimeSlots[^1].GetEnd();
             DateTime oneYearAgo = DateTime.Now.AddYears(-1);
-            return endDate > oneYearAgo;
+            return endDate > oneYearAgo && endDate <= DateTime.Now;
         }
 
         public override string ToString()
@@ -125,5 +125,10 @@ public override string ToString()
             }
             return string.Join("|", new object[] { Id, TutorId, Language, Level.ToString(), NumberOfWeeks, sbDays.ToString(), Online, NumberOfStudents, MaxStudents, StartDateTime.ToString(), CreatedByDirector, Modifiable, GratitudeEmailSent });
         }
+
+        public string ToPdfString()
+        {
+            return Id + " " + " " + Language + " " + Level;
+        }
     }
 }

From d2cc8be989a15c0f213f5c15b3fdddd5fcc4edcb Mon Sep 17 00:00:00 2001
From: DusicaPesic <130140285+DusicaPesic@users.noreply.github.com>
Date: Mon, 27 May 2024 20:36:52 +0200
Subject: [PATCH 6/9] [Add] functions for Report C

---
 LangLang/BusinessLogic/UseCases/PdfService.cs | 26 +++++++++++++++++++
 .../BusinessLogic/UseCases/ReportService.cs   | 23 ++++++++++++++--
 .../BusinessLogic/UseCases/SenderService.cs   | 24 +++++++++++++++++
 .../DirectorViewModels/ReportsViewModel.cs    | 13 +++++++++-
 4 files changed, 83 insertions(+), 3 deletions(-)

diff --git a/LangLang/BusinessLogic/UseCases/PdfService.cs b/LangLang/BusinessLogic/UseCases/PdfService.cs
index 4efadd4..63ca3bf 100644
--- a/LangLang/BusinessLogic/UseCases/PdfService.cs
+++ b/LangLang/BusinessLogic/UseCases/PdfService.cs
@@ -63,5 +63,31 @@ public PdfGrid DataToGrid(Dictionary<string, List<double>> data)
             }
             return grid;
         }
+        public PdfGrid DataToGrid(float[] data)
+        {
+            PdfGrid grid = new PdfGrid();
+            grid.Columns.Add(4);
+            PdfGridRow row = grid.Rows.Add();
+            row.Cells[0].Value = data[0].ToString();
+            row.Cells[1].Value = data[1].ToString();
+            row.Cells[2].Value = data[2].ToString();
+            row.Cells[3].Value = data[3].ToString();
+            return grid;
+        }
+        public PdfGrid DataToGrid(Dictionary<string, float[]> data)
+        {
+            PdfGrid grid = new PdfGrid();
+            grid.Columns.Add(4);
+            foreach (var item in data)
+            {
+                PdfGridRow row = grid.Rows.Add();
+                row.Cells[0].Value = item.Key;
+                row.Cells[1].Value = item.Value[0].ToString();
+                row.Cells[2].Value = item.Value[1].ToString();
+                row.Cells[3].Value = item.Value[2].ToString();
+
+            }
+            return grid;
+        }
     }
 }
diff --git a/LangLang/BusinessLogic/UseCases/ReportService.cs b/LangLang/BusinessLogic/UseCases/ReportService.cs
index 3cfc21b..79c409a 100644
--- a/LangLang/BusinessLogic/UseCases/ReportService.cs
+++ b/LangLang/BusinessLogic/UseCases/ReportService.cs
@@ -66,10 +66,14 @@ public float[] GetAverageResults()
                 averages[2] += result.WritingPoints;
                 averages[3] += result.ListeningPoints;
             }
+
+            if (results.Count == 0) return averages;
+
             for (int i = 0; i < averages.Length; i++)
             {
                 averages[i] /= results.Count;
             }
+
             return averages;
         }
         public int NumStudentsAttended(Course course)
@@ -82,13 +86,28 @@ public int NumStudentsPassed(Course course)
             CourseService coursesService = new();
             return coursesService.NumStudentsPassed(course);
         }
-        public double CalculatePassingPercentage(Course course)
+        public float CalculatePassingPercentage(Course course)
         {
             int attended = NumStudentsAttended(course);
+            if (attended == 0) return 0;
             int passed = NumStudentsPassed(course);
             return (passed / attended)*100;
         }
-
+        public Dictionary<string,float[]> GetCoursesAccomplishment()
+        {
+            Dictionary<string, float[]> accomplishments = new();
+            CourseService courseService = new();
+            List<Course> courses = courseService.GetCoursesHeldInLastYear();
+            foreach (Course course in courses)
+            {
+                float[] res = new float[3];
+                res[0] = NumStudentsAttended(course);
+                res[1] = NumStudentsPassed(course);
+                res[2] = CalculatePassingPercentage(course);
+                accomplishments.Add(course.ToPdfString(), res);
+            }
+            return accomplishments;
+        }
         public Dictionary<string, List<double>> GetAverageGradesOfCourses()
         {
             Dictionary<string, List<double>> averages = new();
diff --git a/LangLang/BusinessLogic/UseCases/SenderService.cs b/LangLang/BusinessLogic/UseCases/SenderService.cs
index 6d1dc4d..2b7a6e0 100644
--- a/LangLang/BusinessLogic/UseCases/SenderService.cs
+++ b/LangLang/BusinessLogic/UseCases/SenderService.cs
@@ -48,6 +48,30 @@ public void SendAverageCourseGrades(Director director)
             var document = PdfService.GeneratePdf<Dictionary<string, List<double>>>(reportService.GetAverageGradesOfCourses(), headers, reportName, data => pdfService.DataToGrid(data));
             EmailService.SendEmail(director.Profile.Email, reportName, "", document);
 
+        }
+        public void SendAverageResultsPerSkill(Director director)
+        {
+            var reportService = new ReportService();
+            var pdfService = new PdfService();
+
+            var reportName = "Average exams results per skill in last year";
+            var headers = new string[] { "Reading average score", "Writing average score", "Listening average score", "Speaking average score" };
+            
+            var document = PdfService.GeneratePdf<float[]>(reportService.GetAverageResults(), headers, reportName, data => pdfService.DataToGrid(data));
+            EmailService.SendEmail(director.Profile.Email, reportName, "", document);
+
+        }
+        public void SendCoursesAccomplishments(Director director)
+        {
+            var reportService = new ReportService();
+            var pdfService = new PdfService();
+
+            var reportName = "Courses enrollments and pass rates analysis";
+            var headers = new string[] { "Course", "Total Attendees", "Students Passed", "Pass Rate (%)" };
+
+            var document = PdfService.GeneratePdf<Dictionary<string, float[]>>(reportService.GetCoursesAccomplishment(), headers, reportName, data => pdfService.DataToGrid(data));
+            EmailService.SendEmail(director.Profile.Email, reportName, "", document);
+
         }
 
         public void SendAveragePoints(Director director)
diff --git a/LangLang/WPF/ViewModels/DirectorViewModels/ReportsViewModel.cs b/LangLang/WPF/ViewModels/DirectorViewModels/ReportsViewModel.cs
index 74bf378..5bc0dbc 100644
--- a/LangLang/WPF/ViewModels/DirectorViewModels/ReportsViewModel.cs
+++ b/LangLang/WPF/ViewModels/DirectorViewModels/ReportsViewModel.cs
@@ -31,7 +31,18 @@ public void SentAverageCourseGrades()
             senderService.SendAverageCourseGrades(_director);
             ShowSuccess();
         }
-
+        public void SentAverageResultsPerSkill()
+        {
+            var senderService = new SenderService();
+            senderService.SendAverageResultsPerSkill(_director);
+            ShowSuccess();
+        }
+        public void SentCoursesAccomplishments()
+        {
+            var senderService = new SenderService();
+            senderService.SendCoursesAccomplishments(_director);
+            ShowSuccess();
+        }
         private void ShowSuccess()
         {
             MessageBox.Show("Successfully completed!", "Notification", MessageBoxButton.OK, MessageBoxImage.Information);

From e1f49f07c8ed3e01c7a353297a38b64a03d29e65 Mon Sep 17 00:00:00 2001
From: DusicaPesic <130140285+DusicaPesic@users.noreply.github.com>
Date: Mon, 27 May 2024 20:37:08 +0200
Subject: [PATCH 7/9] [Add] GUI for Report C

---
 LangLang/WPF/Views/DirectorView/Tabs/Reports.xaml    | 3 ++-
 LangLang/WPF/Views/DirectorView/Tabs/Reports.xaml.cs | 8 ++++++++
 2 files changed, 10 insertions(+), 1 deletion(-)

diff --git a/LangLang/WPF/Views/DirectorView/Tabs/Reports.xaml b/LangLang/WPF/Views/DirectorView/Tabs/Reports.xaml
index 2c25ff3..9d7479f 100644
--- a/LangLang/WPF/Views/DirectorView/Tabs/Reports.xaml
+++ b/LangLang/WPF/Views/DirectorView/Tabs/Reports.xaml
@@ -27,7 +27,8 @@
             <Button Grid.Row="4" Content="Number of created courses for each of the existing languages in the system in the last year" BorderBrush="White" Background="#FFFFDBDB" Foreground="#FF817C7C" FontFamily="Segoe UI Black" />
             <Button Grid.Row="5" Content="Number of created  exams for each of the existing languages in the system in the last year" BorderBrush="White" Background="#FFFFDBDB" Foreground="#FF817C7C" FontFamily="Segoe UI Black" />
             <Button Grid.Row="6" Content="The average grades of students as well as the average rating of tutor per course in the last year" BorderBrush="White" Background="#FFFFDBDB" Foreground="#FF817C7C" FontFamily="Segoe UI Black" Click="AverageCourseGrades_Click" />
-            <Button Grid.Row="7" Content="Report c" BorderBrush="White" Background="#FFFFDBDB" Foreground="#FF817C7C" FontFamily="Segoe UI Black" />
+            <Button Grid.Row="7" Content="The average exams score for each skill in the last year" BorderBrush="White" Background="#FFFFDBDB" Foreground="#FF817C7C" FontFamily="Segoe UI Black" Click="AverageResultsPerSkill_Click"/>
+            <Button Content="Enrollments and pass rate of courses held in the last year" BorderBrush="White" Background="#FFFFDBDB" Foreground="#FF817C7C" FontFamily="Segoe UI Black" Click="CoursesAccomplishments_Click" Margin="0,50,0,-50" Grid.Row="7" />
         </Grid>
     </Grid>
 </UserControl>
diff --git a/LangLang/WPF/Views/DirectorView/Tabs/Reports.xaml.cs b/LangLang/WPF/Views/DirectorView/Tabs/Reports.xaml.cs
index ec932c0..29f0f1d 100644
--- a/LangLang/WPF/Views/DirectorView/Tabs/Reports.xaml.cs
+++ b/LangLang/WPF/Views/DirectorView/Tabs/Reports.xaml.cs
@@ -31,5 +31,13 @@ private void AverageCourseGrades_Click(object sender, System.Windows.RoutedEvent
         {
             _viewModel.SentAverageCourseGrades();
         }
+        private void AverageResultsPerSkill_Click(object sender, System.Windows.RoutedEventArgs e)
+        {
+            _viewModel.SentAverageResultsPerSkill();
+        }
+        private void CoursesAccomplishments_Click(object sender, System.Windows.RoutedEventArgs e)
+        {
+            _viewModel.SentCoursesAccomplishments();
+        }
     }
 }

From 2ef0498957cdfb212086da3219eea655196866e2 Mon Sep 17 00:00:00 2001
From: DusicaPesic <130140285+DusicaPesic@users.noreply.github.com>
Date: Mon, 27 May 2024 22:09:37 +0200
Subject: [PATCH 8/9] [Update] MostSuitableTutor function to consider how busy
 tutors are when recommending

---
 .../BusinessLogic/UseCases/CourseService.cs   | 14 ++++-
 .../BusinessLogic/UseCases/SmartSystem.cs     | 62 ++++++++++++++++---
 2 files changed, 67 insertions(+), 9 deletions(-)

diff --git a/LangLang/BusinessLogic/UseCases/CourseService.cs b/LangLang/BusinessLogic/UseCases/CourseService.cs
index 6dbe1dc..99e7153 100644
--- a/LangLang/BusinessLogic/UseCases/CourseService.cs
+++ b/LangLang/BusinessLogic/UseCases/CourseService.cs
@@ -190,7 +190,19 @@ public bool IsActive(Course course)
             if (course.StartDateTime <= DateTime.Now && GetEnd(course) >= DateTime.Now) return true;
             return false;
         }
-
+        public int NumActiveCourses(Tutor tutor)
+        {
+            int active = 0;
+            List<Course> coursesByTutor = GetByTutor(tutor);
+            foreach (Course course in coursesByTutor)
+            {
+                if (course.IsActive())
+                {
+                    active++;
+                }
+            }
+            return active;
+        }
         public List<Course> GetByTutor(Tutor tutor)
         {
             List<Course> coursesByTutor = new List<Course>();
diff --git a/LangLang/BusinessLogic/UseCases/SmartSystem.cs b/LangLang/BusinessLogic/UseCases/SmartSystem.cs
index ccc09dd..5306db8 100644
--- a/LangLang/BusinessLogic/UseCases/SmartSystem.cs
+++ b/LangLang/BusinessLogic/UseCases/SmartSystem.cs
@@ -2,6 +2,7 @@
 using LangLang.Configuration;
 using LangLang.Domain.Models;
 using System.Collections.Generic;
+using System.Diagnostics;
 using System.Linq;
 
 namespace LangLang.BusinessLogic.UseCases
@@ -65,21 +66,66 @@ public static int MostSuitableTutor(Course course)
         {
             TutorService tutorService = new();
             List<Tutor> tutors = tutorService.GetBySkill(course.Language, course.Level);
+            //there is no free tutors for given course
             if (tutors.Count == 0) return -1;
-            TutorRatingService tutorRatingService = new();
-            Dictionary<Tutor, double> tutorsAndRatings = new();
+
+            //consider only tutors that are free in time of course
+            List<Tutor> availableTutors = new();
+            CourseService coursesService = new();
             foreach (Tutor tutor in tutors)
             {
-                tutorsAndRatings[tutor] = tutorRatingService.GetAverageRating(tutor);
+                course.TutorId = tutor.Id;
+                if (coursesService.CanCreateOrUpdate(course)) availableTutors.Add(tutor);
             }
-            Dictionary<Tutor, double> sorted = tutorsAndRatings.OrderBy(pair => pair.Value).ToDictionary(pair => pair.Key, pair => pair.Value);
+            //find least busy tutors
+            List<Tutor> leastBusyTutors = WithLeastActiveCourses(availableTutors);
+            //check who is rated the best
+            return GetBestRankedTutor(leastBusyTutors);
+        }
+        private static int MinimumActiveCourses(List<Tutor> tutors)
+        {
             CourseService coursesService = new();
-            foreach (Tutor tutor in sorted.Keys)
+            int minActive = int.MaxValue;
+            int active = 0;
+            foreach (var tutor in tutors)
             {
-                course.TutorId = tutor.Id;
-                if (coursesService.CanCreateOrUpdate(course)) return tutor.Id;
+                active = coursesService.NumActiveCourses(tutor);
+                if (active < minActive)
+                {
+                    minActive = active;
+                }
             }
-            return -1;
+
+            return minActive;
+        }
+        private static List<Tutor> WithLeastActiveCourses(List<Tutor> tutors)
+        {
+            int minActive = MinimumActiveCourses(tutors);
+            List<Tutor> leastBusyTutors = new List<Tutor>();
+            CourseService coursesService = new();
+
+            foreach (var tutor in tutors)
+            {
+                int active = coursesService.NumActiveCourses(tutor);
+                if (active == minActive)
+                {
+                    leastBusyTutors.Add(tutor);
+                }
+            }
+
+            return leastBusyTutors;
+        }
+        private static int GetBestRankedTutor(List<Tutor> tutors)
+        {
+            TutorRatingService tutorRatingService = new();
+            Dictionary<Tutor, double> ratings = new();
+            foreach (Tutor tutor in tutors)
+            {
+                ratings[tutor] = tutorRatingService.GetAverageRating(tutor);
+            }
+            Tutor bestRanked = ratings.Aggregate((x, y) => x.Value < y.Value ? x : y).Key;
+            return bestRanked.Id;
         }
+        
     }
 }

From 86928dc8e63c041b50d53826eae9eb0e8eb64909 Mon Sep 17 00:00:00 2001
From: Darinka Loncar <149894872+darinkaloncar@users.noreply.github.com>
Date: Mon, 27 May 2024 13:32:03 -0700
Subject: [PATCH 9/9] [Update] service and model class for Course

---
 .../BusinessLogic/UseCases/CourseService.cs   | 27 ++-----------------
 .../UseCases/TutorRatingService.cs            |  2 +-
 LangLang/Domain/Models/Course.cs              | 17 ++++++++++--
 .../CourseViewModels/CoursesTutorViewModel.cs |  2 +-
 .../WPF/Views/TutorView/Tabs/Courses.xaml.cs  | 13 ++++-----
 5 files changed, 24 insertions(+), 37 deletions(-)

diff --git a/LangLang/BusinessLogic/UseCases/CourseService.cs b/LangLang/BusinessLogic/UseCases/CourseService.cs
index 99e7153..b0ee04e 100644
--- a/LangLang/BusinessLogic/UseCases/CourseService.cs
+++ b/LangLang/BusinessLogic/UseCases/CourseService.cs
@@ -47,7 +47,7 @@ public void Delete(int id)
         }
         public void DeleteByTutor(Tutor tutor)
         {
-            foreach (Course course in GetByTutor(tutor))
+            foreach (Course course in GetByTutor(tutor.Id))
             {
                 if (course.StartDateTime > DateTime.Now)
                 {
@@ -180,20 +180,11 @@ public List<Course> GetAll()
         {
             return _courses.GetAll();
         }
-        public DateTime GetEnd(Course course)
-        {
-            return course.TimeSlots[course.TimeSlots.Count - 1].GetEnd();
-        }
 
-        public bool IsActive(Course course)
-        {
-            if (course.StartDateTime <= DateTime.Now && GetEnd(course) >= DateTime.Now) return true;
-            return false;
-        }
         public int NumActiveCourses(Tutor tutor)
         {
             int active = 0;
-            List<Course> coursesByTutor = GetByTutor(tutor);
+            List<Course> coursesByTutor = GetByTutor(tutor.Id);
             foreach (Course course in coursesByTutor)
             {
                 if (course.IsActive())
@@ -203,20 +194,6 @@ public int NumActiveCourses(Tutor tutor)
             }
             return active;
         }
-        public List<Course> GetByTutor(Tutor tutor)
-        {
-            List<Course> coursesByTutor = new List<Course>();
-
-            foreach (Course course in GetAll())
-            {
-                if (course.TutorId == tutor.Id)
-                {
-                    coursesByTutor.Add(course);
-                }
-            }
-
-            return coursesByTutor;
-        }
 
         public List<Course> GetByTutor(int tutorId)
         {
diff --git a/LangLang/BusinessLogic/UseCases/TutorRatingService.cs b/LangLang/BusinessLogic/UseCases/TutorRatingService.cs
index bd8d21c..bc6b488 100644
--- a/LangLang/BusinessLogic/UseCases/TutorRatingService.cs
+++ b/LangLang/BusinessLogic/UseCases/TutorRatingService.cs
@@ -72,7 +72,7 @@ public double GetAverageRating(Tutor tutor)
         {
             CourseService courseService = new();
             List<double> ratings = new();
-            foreach (Course course in courseService.GetByTutor(tutor))
+            foreach (Course course in courseService.GetByTutor(tutor.Id))
             {
                 ratings.Add(GetAverageTutorRating(course));
             }
diff --git a/LangLang/Domain/Models/Course.cs b/LangLang/Domain/Models/Course.cs
index a1b5233..9d5cd06 100644
--- a/LangLang/Domain/Models/Course.cs
+++ b/LangLang/Domain/Models/Course.cs
@@ -105,9 +105,8 @@ public int DaysUntilStart()
 
         public bool IsHeldInLastYear()
         {
-            var endDate = TimeSlots[^1].GetEnd();
             DateTime oneYearAgo = DateTime.Now.AddYears(-1);
-            return endDate > oneYearAgo && endDate <= DateTime.Now;
+            return GetEnd() > oneYearAgo && GetEnd() <= DateTime.Now;
         }
 
         public override string ToString()
@@ -130,5 +129,19 @@ public string ToPdfString()
         {
             return Id + " " + " " + Language + " " + Level;
         }
+
+        public DateTime GetEnd()
+        {
+            return TimeSlots[TimeSlots.Count - 1].GetEnd();
+        }
+        public bool IsActive()
+        {
+            if (StartDateTime <= DateTime.Now && GetEnd() >= DateTime.Now) return true;
+            return false;
+        }
+        public bool CanChange()
+        {
+            return StartDateTime >= DateTime.Now.AddDays(Constants.COURSE_MODIFY_PERIOD); ;
+        }
     }
 }
diff --git a/LangLang/WPF/ViewModels/CourseViewModels/CoursesTutorViewModel.cs b/LangLang/WPF/ViewModels/CourseViewModels/CoursesTutorViewModel.cs
index 247eac8..bf0c2d6 100644
--- a/LangLang/WPF/ViewModels/CourseViewModels/CoursesTutorViewModel.cs
+++ b/LangLang/WPF/ViewModels/CourseViewModels/CoursesTutorViewModel.cs
@@ -28,7 +28,7 @@ public void Update()
         {
             Courses.Clear();
             CourseService courseService = new();
-            foreach (Course course in courseService.GetByTutor(LoggedIn))
+            foreach (Course course in courseService.GetByTutor(LoggedIn.Id))
             {
                 Courses.Add(new CourseViewModel(course));
             }
diff --git a/LangLang/WPF/Views/TutorView/Tabs/Courses.xaml.cs b/LangLang/WPF/Views/TutorView/Tabs/Courses.xaml.cs
index 784abe7..c03d0cd 100644
--- a/LangLang/WPF/Views/TutorView/Tabs/Courses.xaml.cs
+++ b/LangLang/WPF/Views/TutorView/Tabs/Courses.xaml.cs
@@ -67,8 +67,7 @@ private void CourseSearchWindowBtn_Click(object sender, RoutedEventArgs e)
 
         private void CourseUpdateWindowBtn_Click(object sender, RoutedEventArgs e)
         {
-            CourseService courseService = new();
-            if (courseService.CanChange(CoursesViewModel.SelectedCourse.Id))
+            if (CoursesViewModel.SelectedCourse.ToCourse().CanChange())
             {
                 CourseUpdateWindow updateWindow = new(this, CoursesViewModel.SelectedCourse.ToCourse());
                 updateWindow.Show();
@@ -81,9 +80,9 @@ private void CourseUpdateWindowBtn_Click(object sender, RoutedEventArgs e)
 
         private void CourseDeleteBtn_Click(object sender, RoutedEventArgs e)
         {
-            CourseService courseService = new();
-            if (courseService.CanChange(CoursesViewModel.SelectedCourse.Id))
+            if (CoursesViewModel.SelectedCourse.ToCourse().CanChange())
             {
+                CourseService courseService = new();
                 courseService.Delete(CoursesViewModel.SelectedCourse.Id);
                 CoursesViewModel.Update();
                 MessageBox.Show("The course has been successfully deleted.", "Information", MessageBoxButton.OK, MessageBoxImage.Information);
@@ -96,8 +95,7 @@ private void CourseDeleteBtn_Click(object sender, RoutedEventArgs e)
 
         private void EnterGradeBtn_Click(object sender, RoutedEventArgs e)
         {
-            CourseService courseService = new();
-            if (courseService.GetEnd(CoursesViewModel.SelectedCourse.ToCourse()) < DateTime.Now)
+            if (CoursesViewModel.SelectedCourse.ToCourse().GetEnd() < DateTime.Now)
             {
                 EnterGradesWindow gradesWindow = new(CoursesViewModel.SelectedCourse);
                 gradesWindow.Show();
@@ -109,8 +107,7 @@ private void EnterGradeBtn_Click(object sender, RoutedEventArgs e)
         }
         private void DurationOfCourseBtn_Click(object sender, RoutedEventArgs e)
         {
-            CourseService courseService = new();
-            if (courseService.IsActive(CoursesViewModel.SelectedCourse.ToCourse()))
+            if (CoursesViewModel.SelectedCourse.ToCourse().IsActive())
             {
                 DurationOfCourseWindow courseWindow = new(CoursesViewModel.SelectedCourse);
                 courseWindow.Show();