[fix:] course amount per semester synchronized with NumCoursesPerSemester, fixing #1

MainWindow.axaml.cs

@@ -11,7 +11,7 @@ namespace spplus;
 public partial class MainWindow : Window
 {
     public static MainWindow Instance { get; set; }
-    public static string ApplicationVersion = "v1.1.17";
+    public static string ApplicationVersion = "v1.2.24";
     public MainWindow()
     {
         InitializeComponent();

README.md

@@ -1,13 +1,15 @@
 # SP+
 
-Interaktiver Sportkursplaner für Oberstufen auf Basis einer Sportkurswahl durch SuS.
+Plattformunabhängiger (Windows, Linux, Mac), interaktiver Sportkursplaner für Oberstufen auf Basis einer Sportkurswahl durch SuS.
 
 ## Features
 * \+ Import von CSV-Dateien mit Kurswahl
 * \+ Wahlansicht 
 * \+ Statistiken
 * \+ Pflege von Sportkursen (inkl. Kürzel/ alternativen Bezeichnungen)
+* \+ CSV-Export
 * ~ Fehleransicht für nicht-existente, aber gewählte Kurse
+* ~ PDF-Export
 
 \+ Vorhanden, ~ Pending
 

crafter.cs

@@ -193,6 +193,94 @@ public class CourseCrafter
 
         } while (changed && iteration < maxIterations);
         
+        // // --- Kurse nachträglich aufteilen, um NumCoursesPerSemester exakt zu erreichen ---
+        // foreach (var course in GeneratedCourses)
+        // {
+        //     int sem_count_total = GeneratedCourses.Count(tuple => tuple.Semester == course.Semester);
+        //     if (sem_count_total >= Settings.Instance.NumCoursesPerSemester) break;
+        //     
+        //     int sem_count = GeneratedCourses.Count(tuple => tuple.Semester == course.Semester && tuple.Instance.Sport.Name == course.Instance.Sport.Name);
+        //     
+        //
+        //     if (sem_count < course.Instance.Sport.Semester[course.Semester - 1] && course.Instance.Students.Count >= course.Instance.Sport.MinStudents *2)
+        //     {
+        //         // hier aufteilen
+        //         Console.WriteLine("Könnte aufgeteilt werden.");
+        //     }
+        // }
+        //
+        // --- Kurse nachträglich aufteilen, um NumCoursesPerSemester exakt zu erreichen ---
+        for (int semester = 1; semester <= 4; semester++)
+        {
+            int cancel = 0;
+            while (GeneratedCourses.Count(c => c.Semester == semester) < Settings.Instance.NumCoursesPerSemester)
+            {
+                cancel++;
+                if (cancel >= 5) break;
+                // Kandidaten suchen: splittbare Kurse, deren Sport noch Kapazität hat
+                var candidate = GeneratedCourses
+                    .Where(c => c.Semester == semester)
+                    .Where(c => c.Instance.Students.Count >= c.Instance.Sport.MinStudents * 2)
+                    .Where(c =>
+                    {
+                        int sportCount = GeneratedCourses.Count(g =>
+                            g.Semester == semester &&
+                            g.Instance.Sport.Name == c.Instance.Sport.Name);
+
+                        int allowed = c.Instance.Sport.Semester[semester - 1];
+                        return sportCount < allowed;
+                    })
+                    .OrderByDescending(c => c.Instance.Students.Count)
+                    .FirstOrDefault();
+
+                try
+                {
+
+                    var students = candidate.Instance.Students;
+
+                    var newCourse = new CourseInstance
+                    {
+                        Sport = candidate.Instance.Sport,
+                        Students = new List<string>()
+                    };
+
+                    List<string> moved = new();
+
+                    for (int i = students.Count - 1; i >= 0; i--)
+                    {
+                        string stud = students[i];
+
+                        if (newCourse.Students.Count >= candidate.Instance.Sport.MaxStudents)
+                            break;
+
+                        // Sicherstellen, dass beide Kurse >= MinStudents bleiben
+                        if (students.Count - moved.Count <= candidate.Instance.Sport.MinStudents)
+                            break;
+
+                        newCourse.Students.Add(stud);
+                        moved.Add(stud);
+                    }
+
+                    // Validierung
+                    if (newCourse.Students.Count < candidate.Instance.Sport.MinStudents)
+                        break;
+
+                    // Move durchführen
+                    foreach (var s in moved)
+                    {
+                        candidate.Instance.Students.Remove(s);
+                    }
+
+                    GeneratedCourses.Add((semester, newCourse));
+                }
+                catch
+                {
+                    
+                }
+                
+            }
+        }
+        
 
         bool isStudentFree(int semester, string studentID)
         {