{
"llm" : {
"feedback" : "# Exercise: fractal\n\n### Correctness\n- In `Boxes.drawRec`, the recursive step does nicht die geforderte Box-Fraktal-Struktur zeichnen: du rufst mehrfach **dieselbe** Rekursion mit identischen `(x, y)`-Koordinaten auf, statt die Fläche in Teilquadrate an verschiedenen Positionen aufzuteilen.\n- In `Boxes.drawRec`, die Größe `s - s/5` passt nicht zur üblichen rekursiven Unterteilung (die Aufgabe zielt darauf ab, aus einem Quadrat mehrere kleinere Quadrate mit klaren Offsets zu zeichnen); dadurch entsteht nicht das erwartete Muster.\n- `Triangles.drawRec` ist nicht implementiert; damit wird die zweite geforderte Form nicht gezeichnet.\n\n### Suggestion\n- Überlege dir bei den Boxes: Wenn `n > 0`, welche **vier Teilquadrate** (Positionen) entstehen aus einem Quadrat `(x, y, s)`? Skizziere sie (oben-links, oben-rechts, unten-links, unten-rechts) und berechne jeweils die neuen `x/y`-Startpunkte über passende Offsets mit einer verkleinerten Seitenlänge.\n- Statt einer `for`-Schleife: Frage dich, wie viele rekursive Aufrufe du pro Ebene wirklich brauchst (typisch: ein fester Satz an Aufrufen für die Teilbereiche), und warum identische Aufrufe übereinander nichts Neues hinzufügen.\n- Für Triangles: Baue eine klare Basis (`n == 0`): Zeichne das Dreieck mit drei Linien. Im Rekursionsfall: Berechne die Mittelpunkte der Dreiecksseiten und rufe dann rekursiv für die entstehenden Teildreiecke auf (mit `n-1`).\n\n3. Code Style:\n- In `Boxes.drawRec` ist der Kommentar `// TODO implement` nicht mehr zutreffend, wenn du schon Code drin hast – entweder entfernen oder durch eine passende Erklärung ersetzen.\n- Die `for (int i = 0; i < n; i++)`-Schleife wirkt hier irreführend: Rekursionstiefe (`n`) und “wie oft zeichne ich pro Ebene” sollten konzeptionell getrennt sein (auch wenn du es später korrekt lösen kannst, ist das aktuell schwer nachvollziehbar).\n\n\n# Exercise: knapsack\n\n### Correctness\n- In `KnapsackImpl` sind `pack(int i)`, `loadItem(int i)` und `unloadItem(int i)` nicht implementiert; damit kann `solve()` keine optimale Lösung berechnen und die Tests werden nicht bestehen.\n- In `KnapsackN` hat `main()` nicht die übliche Signatur `main(String[] args)`; je nach Test/Runner wird das Programm so nicht als Startpunkt erkannt.\n\n### Suggestion\n- Überlege dir für `pack(i)` die rekursive Struktur: Was passiert, wenn du bei einem Index `i` angekommen bist? Du musst alle sinnvollen Möglichkeiten für diesen Gegenstand durchprobieren (0-mal, 1-mal, …, bis `maxN`-mal) und dann mit `i+1` weitermachen. Achte darauf, wann du `maxValue` aktualisierst (typisch im Basisfall, wenn alle Gegenstände betrachtet sind).\n- `loadItem(i)`/`unloadItem(i)` sollten den Zustand (`currWeight`, `currValue`) konsistent verändern. Wichtig ist dabei, dass du nach dem Durchprobieren mehrerer Kopien wieder exakt auf den Zustand vor dem Gegenstand zurückkommst.\n- Falls dein Code über einen Runner gestartet werden muss: passe die `main`-Signatur so an, dass Java sie als Entry-Point akzeptiert.\n\n### Code Style\n- Die TODO-Methoden sind aktuell leer; entweder implementieren oder (wenn du anders lösen willst) die unbenutzten Methoden entfernen, damit die Klasse nicht “halb fertig” wirkt.\n- Benennung ist uneinheitlich zur Aufgabenstellung: Wenn du `loadItem/unloadItem` nutzt, bleib dabei konsistent (und verwende sie auch wirklich in `pack`), sonst verwirrt das beim Lesen.\n\n\n# Exercise: queens\n\n### Correctness\n- `QueensSolver.solve()` ist nicht implementiert und gibt immer `false` zurück, dadurch wird keine Lösung gesucht/gefunden.\n- `QueensSolver.count()` ist nicht implementiert und gibt immer `0` zurück, dadurch wird die Anzahl Lösungen nicht gezählt.\n\n### Suggestion\n- Für `solve()`: Überlege dir eine rekursive Backtracking-Funktion, die zeilenweise vorgeht: in jeder Zeile eine Spalte ausprobieren, mit `checkPlacement` testen, eine Dame setzen, rekursiv die nächste Zeile lösen, und bei Misserfolg die Dame wieder entfernen (Backtracking).\n- Für `count()`: Nutze ein sehr ähnliches rekursives Vorgehen wie bei `solve()`, aber statt beim ersten Fund abzubrechen, sammelst du die Anzahl vollständiger Platzierungen (Basisfall: alle Zeilen belegt ⇒ 1) und addierst die Ergebnisse aller gültigen Spaltenversuche; nach jedem Versuch unbedingt wieder zurücksetzen.\n\n### Code Style\n- In `Queens.getBoard()` ist `return this.board;` unnötig mit `this` (konsistent wäre einfach `return board;`), ist aber nur eine Kleinigkeit.\n- In `Queens.java` fehlt die übliche Java-`main`-Signatur (`static void main(String[] args)`); falls das in eurer Umgebung nicht speziell unterstützt wird, startet das Programm so nicht.\n\n\n# Exercise: sudoku\n\n### Correctness\n- In `SudokuSolverImpl.solved(SudokuModel model, int fieldNr)` ist der Backtracking-Algorithmus nicht implementiert und liefert immer `false`; damit wird nie eine Lösung gefunden und das Sudoku nicht gelöst.\n- In `SudokuSolverImpl.nofSolutions(SudokuModel model, int fieldNr, int max)` ist das Zählen der Lösungen nicht implementiert und liefert immer `0`; damit werden nie Lösungen gezählt.\n- In `Sudoku.java` ist die `main`-Methode nicht als `public static void main(String[] args)` deklariert; dadurch startet die Applikation in der Regel nicht über den Standard-Java-Entry-Point.\n\n### Suggestion\n- Für `solved(...)`: Überlege dir eine Rekursion über alle Felder (z.B. über `fieldNr`), mit einem Abbruchfall „alle Felder bearbeitet“. Wenn ein Feld schon belegt ist, gehst du direkt zum nächsten. Wenn es leer ist, probierst du Werte durch, prüfst nach jedem Setzen mit dem `checker` die Gültigkeit, und gehst nur dann rekursiv weiter. Wenn die Rekursion scheitert, musst du das Feld wieder zurücksetzen (Backtracking).\n- Für `nofSolutions(...)`: Nutze dieselbe Traversierung wie beim Lösen, aber statt „sobald eine Lösung gefunden wurde → true“ zählst du Lösungen hoch, wenn der Abbruchfall erreicht ist. Achte darauf, dass du bei der Rekursion den Parameter `max` sinnvoll reduzierst/berücksichtigst, damit du bei `MAX` frühzeitig stoppen kannst. Und auch hier: nach dem Durchprobieren eines Feldes wieder leeren.\n- Für `Sudoku.main`: Stell sicher, dass deine Startmethode exakt die Java-Signatur hat, damit die GUI überhaupt gestartet werden kann.\n\n### Code Style\n- In `SudokuSolverImpl` stehen noch `// TODO implement`-Platzhalter; sobald du implementierst, entferne diese oder ersetze sie durch sinnvolle Kommentare (z.B. „Ankerfall“, „Backtracking-Schritt“), damit die Logik nachvollziehbar bleibt.\n- Die Einrückung/Formatierung ist teils uneinheitlich (Tabs/Spaces, Methoden-Alignment); einmal automatisch formatieren (IDE) verbessert die Lesbarkeit deutlich.\n",
"status" : "SUCCESS"
},
"unitTest" : {
"tests" : [ {
"name" : "allZero()",
"status" : "PASSED",
"message" : null
}, {
"name" : "uselessStuff()",
"status" : "PASSED",
"message" : null
}, {
"name" : "random1()",
"status" : "FAILED",
"message" : "expected: <190> but was: <0>"
}, {
"name" : "random2()",
"status" : "FAILED",
"message" : "expected: <340> but was: <0>"
}, {
"name" : "random3()",
"status" : "FAILED",
"message" : "expected: <558> but was: <0>"
}, {
"name" : "boardIsValidAfterSolveN4()",
"status" : "FAILED",
"message" : "Board must contain exactly N queens ==> expected: <4> but was: <0>"
}, {
"name" : "boardIsValidAfterSolveN5()",
"status" : "FAILED",
"message" : "Board must contain exactly N queens ==> expected: <5> but was: <0>"
}, {
"name" : "boardIsValidAfterSolveN8()",
"status" : "FAILED",
"message" : "Board must contain exactly N queens ==> expected: <8> but was: <0>"
}, {
"name" : "boardSizeIsCorrectForN4()",
"status" : "PASSED",
"message" : null
}, {
"name" : "boardSizeIsCorrectForN8()",
"status" : "PASSED",
"message" : null
}, {
"name" : "countN1()",
"status" : "FAILED",
"message" : "N=1 has exactly 1 solution ==> expected: <1> but was: <0>"
}, {
"name" : "countN2()",
"status" : "PASSED",
"message" : null
}, {
"name" : "countN3()",
"status" : "PASSED",
"message" : null
}, {
"name" : "countN4()",
"status" : "FAILED",
"message" : "N=4 has exactly 2 solutions ==> expected: <2> but was: <0>"
}, {
"name" : "countN5()",
"status" : "FAILED",
"message" : "N=5 has exactly 10 solutions ==> expected: <10> but was: <0>"
}, {
"name" : "countN6()",
"status" : "FAILED",
"message" : "N=6 has exactly 4 solutions ==> expected: <4> but was: <0>"
}, {
"name" : "countN8()",
"status" : "FAILED",
"message" : "N=8 has exactly 92 solutions ==> expected: <92> but was: <0>"
}, {
"name" : "solveReturnsTrueForN1()",
"status" : "FAILED",
"message" : "N=1 has exactly one solution ==> expected: <true> but was: <false>"
}, {
"name" : "solveReturnsTrueForN4()",
"status" : "FAILED",
"message" : "N=4 has valid queen placements ==> expected: <true> but was: <false>"
}, {
"name" : "solveReturnsTrueForN8()",
"status" : "FAILED",
"message" : "N=8 has valid queen placements ==> expected: <true> but was: <false>"
}, {
"name" : "solveReturnsFalseForN2()",
"status" : "PASSED",
"message" : null
}, {
"name" : "solveReturnsFalseForN3()",
"status" : "PASSED",
"message" : null
} ]
}
}