{
"llm" : {
"feedback" : "Exercise: fractal\n\n1. Correctness\n- Du hast die beiden geforderten `draw`-Implementationen (rekursive Logik) in `Boxes.drawRec(...)` und `Triangles.drawRec(...)` nicht umgesetzt; beide Methoden sind noch TODO und zeichnen daher keine Fraktal-Formen.\n\n2. Suggestion\n- Starte in beiden `drawRec`-Methoden mit einem klaren Abbruchfall für `n` (z.B. wenn `n == 0` oder kleiner), damit die Rekursion endet.\n- Zeichne in `Boxes.drawRec(...)` zuerst das aktuelle Rechteck mit `g.drawRect(...)` und rufe danach `drawRec` für kleinere Teil-Rechtecke mit angepassten Koordinaten/Seitenlängen auf (typisch: Aufteilen der Fläche in mehrere kleinere Quadrate/Boxen).\n- Zeichne in `Triangles.drawRec(...)` zuerst das aktuelle Dreieck, indem du die drei Kanten mit `g.drawLine(...)` verbindest, und berechne dann für die rekursiven Aufrufe die relevanten Teil-Dreiecke (häufig über Mittelpunkte der Seiten oder ähnliche Teilung).\n\n3. Code Style\n- In beiden Klassen stehen noch `// TODO implement`-Kommentare ohne Umsetzung; sobald du implementierst, entferne/aktualisiere diese Kommentare, damit der Codezustand klar ist.\n\n\nExercise: knapsack\n\n1. Correctness\n\n\n2. Suggestion\n\n\n3. Code Style\n- In `pack`, the nested `if` structure is a bit hard to scan; consider using early returns (guard clauses) to reduce indentation and make the recursion cases clearer.\n\n\nExercise: queens\n\n1. Correctness\n- `solve()` ist nicht implementiert und gibt immer `false` zurück; damit wird keine Backtracking-Lösung gefunden und auch kein Brett befüllt.\n- `count()` ist nicht implementiert und gibt immer `0` zurück; damit wird die Anzahl der möglichen Lösungen nicht gezählt.\n\n2. Suggestion\n- Für `solve()`: Überlege dir eine rekursive Hilfsmethode, die zeilenweise (oder spaltenweise) vorgeht: in der aktuellen Zeile jede Spalte ausprobieren, bei gültiger Platzierung (`checkPlacement`) eine Dame setzen, rekursiv zur nächsten Zeile gehen und bei Fehlschlag wieder entfernen (Backtracking). Sobald alle Zeilen belegt sind, kannst du „gefunden“ zurückgeben.\n- Für `count()`: Nutze eine sehr ähnliche Rekursion wie bei `solve()`, aber statt beim ersten Fund abzubrechen, zählst du jede vollständige Platzierung als 1 und summierst die Ergebnisse der rekursiven Aufrufe. Denk daran, dass du beim Zurückgehen ebenfalls wieder „aufräumen“ musst (Dame entfernen), sonst beeinflussen alte Platzierungen die nächsten Versuche.\n\n3. Code Style\n- In `BoardPrinter.printLine`: `if (board[row][col] == true)` ist unnötig umständlich; der direkte Boolean-Ausdruck ist lesbarer.\n- In `QueensSolver`: Die `// TODO implement`-Kommentare sind ok während der Arbeit, sollten aber nach Fertigstellung entfernt werden.\n\n\nExercise: sudoku\n\n1. Correctness\n- `solved(SudokuModel model)` ist nicht implementiert: die rekursive Variante gibt immer `false` zurück, damit wird nie eine Lösung gefunden und das Modell nie gefüllt.\n- `nofSolutions(SudokuModel model)` ist nicht implementiert: die rekursive Variante gibt immer `0` zurück, damit wird nie eine Lösung gezählt.\n- Anforderung aus dem Interface-Vertrag: Bei `solved(...) == false` muss das Modell auf den Initialzustand zurückgesetzt werden; aktuell passiert gar keine Veränderung/kein Reset, weil die Methode nicht arbeitet.\n- Anforderung aus dem Interface-Vertrag: `nofSolutions(...)` muss das Modell nach dem Zählen auf den Initialzustand zurücksetzen; aktuell findet kein Zählen statt und damit auch kein korrektes Zurücksetzen.\n\n2. Suggestion\n- Für `solved(model, fieldNr)`: Überlege dir eine Abbruchbedingung, die “alle Felder verarbeitet” bedeutet (z.B. `fieldNr` läuft von 0 bis `size*size`) und dann `true` liefert.\n- Rechne aus `fieldNr` die Koordinaten `(i, j)` aus (mit Division/Modulo), damit du systematisch Feld für Feld durchgehst.\n- Wenn ein Feld schon belegt ist (`get(i,j) != 0`), sollte der Solver direkt mit dem nächsten Feld weitermachen, statt dort Werte zu probieren.\n- Wenn ein Feld leer ist, iteriere Kandidaten `1..size()`, setze testweise, prüfe Gültigkeit mit `checker.oneOK(model, i, j)`, und gehe dann rekursiv weiter; falls es nicht klappt, Feld wieder leeren (Backtracking).\n- Für das “Reset bei Fehlschlag” (bei `solved` und nach `nofSolutions`): Du brauchst eine Möglichkeit, den Anfangszustand wiederherzustellen (z.B. indem du dir die ursprünglichen festen Werte merkst oder indem du beim Backtracking nur Felder veränderst, die vorher leer waren und diese am Ende wieder leer sind).\n- Für `nofSolutions(model, fieldNr, max)`: Es ist sehr ähnlich zu `solved`, nur dass du nicht beim ersten Treffer stoppst, sondern beim Erreichen einer vollständigen Belegung `1` zurückgibst und die Rückgaben aufsummierst.\n- Nutze den `max`-Parameter wirklich als Cutoff: Wenn du schon `max` Lösungen erreicht hast, solltest du nicht weiter in die Tiefe suchen, sondern frühzeitig abbrechen/zurückkehren.\n\n3. Code Style\n- In `SudokuSolverImpl` stehen noch `// TODO implement`-Kommentare; sobald du implementierst, entferne sie oder ersetze sie durch kurze, beschreibende Kommentare (z.B. “backtracking over fields”).\n- Die Einrückung ist teils gemischt (Tabs/Spaces wirken uneinheitlich rund um `@Override`-Methoden). Einheitliche Formatierung erleichtert das Lesen, besonders bei Rekursion.\n",
"status" : "SUCCESS"
},
"unitTest" : {
"tests" : [ {
"name" : "allZero()",
"status" : "PASSED",
"message" : null
}, {
"name" : "uselessStuff()",
"status" : "PASSED",
"message" : null
}, {
"name" : "random1()",
"status" : "PASSED",
"message" : null
}, {
"name" : "random2()",
"status" : "PASSED",
"message" : null
}, {
"name" : "random3()",
"status" : "PASSED",
"message" : null
} ]
}
}