{
"llm" : {
"feedback" : "Exercise: fractal\n\n1. Correctness\n- Du hast die beiden geforderten `draw`-Implementationen (genauer: die rekursiven Hilfsmethoden `drawRec`) in `Boxes` und `Triangles` noch nicht umgesetzt; damit werden keine Formen gezeichnet und die Aufgabe ist nicht erfüllt.\n\n2. Suggestion\n- Starte in beiden `drawRec`-Methoden mit einem klaren Rekursionsabbruch über `n` (z.B. wenn `n` zu klein ist, nur noch die aktuelle Form zeichnen oder gar nichts mehr).\n- `Boxes`: Zeichne zuerst das aktuelle Quadrat/Rechteck mit `g.drawRect(...)` anhand von `(x, y, s)` und rufe danach `drawRec` für die kleineren Teil-Quadrate auf, mit passend angepassten Koordinaten und verkleinerter Seitenlänge (und `n-1`).\n- `Triangles`: Zeichne zuerst das aktuelle Dreieck über drei `g.drawLine(...)`-Aufrufe (Kanten zwischen den drei Punkten). Für die Rekursion brauchst du dann neue Dreiecke, deren Eckpunkte sich aus den bestehenden Punkten ableiten (typisch über Mittelpunkte der Kanten), und rufst `drawRec` mit diesen neuen Punkten und `n-1` auf.\n\n3. Code Style\n- In `Shape` ist `public` bei Interface-Methoden redundant; du könntest es weglassen, um es idiomatischer zu machen.\n- Die TODO-Kommentare sind noch vorhanden; sobald du implementierst, entferne/aktualisiere sie, damit klar ist, was fertig ist.\n\n\nExercise: knapsack\n\n1. Correctness\n\n\n2. Suggestion\n\n\n3. Code Style\n- In `pack(...)` ist die Verschachtelung der `if`-Blöcke recht tief; du könntest mit frühen Returns (Guard Clauses) arbeiten, um die Lesbarkeit zu erhöhen.\n- Magische Grenzen/Checks wie `currWeight <= maxWeight` kommen mehrfach vor; eine kleine Hilfsmethode (z.B. `isValidWeight()`) könnte das verständlicher machen.\n\n\nExercise: queens\n\n1. Correctness\n- `solve()` ist nicht implementiert und liefert immer `false`, damit wird keine N‑Damen-Lösung per Backtracking gefunden und das Board bleibt leer.\n- `count()` ist nicht implementiert und liefert immer `0`, damit wird die Anzahl der möglichen Lösungen nicht gezählt.\n\n2. Suggestion\n- Für `solve()`: Überlege dir eine rekursive Backtracking-Funktion, die zeilenweise (oder spaltenweise) eine Dame setzt, mit `checkPlacement(row, col)` prüft, und bei Misserfolg den Zug wieder rückgängig macht (Queen entfernen) und die nächste Spalte probiert.\n- Für `count()`: Nutze ein ähnliches Backtracking wie bei `solve()`, aber statt beim ersten Fund abzubrechen, zählst du jede vollständige Platzierung (Basisfall: alle Reihen belegt) und sammelst die Anzahl über alle Rückwege hinweg.\n\n3. Code Style\n- Die `// TODO implement`-Marker sind noch vorhanden; sobald du implementierst, entferne sie oder ersetze sie durch sinnvolle Kommentare (z.B. Basisfall/Backtracking-Schritte), damit klar ist, wie der Algorithmus aufgebaut ist.\n\n\nExercise: sudoku\n\n## 1. Correctness\n- `solved(SudokuModel model)` ist nicht implementiert: Die rekursive Variante gibt immer `false` zurück und löst damit kein Sudoku.\n- `nofSolutions(SudokuModel model)` ist nicht implementiert: Die rekursive Variante gibt immer `0` zurück und zählt damit keine Lösungen.\n- Anforderung „falls keine Lösung gefunden, model wurde reset to its initial state“ wird aktuell nicht erfüllt, weil bei `solved`/`nofSolutions` gar keine Änderungen/Backtracking-Resets stattfinden.\n\n## 2. Suggestion\n- Für `solved`: Überlege dir, wie du `fieldNr` in `(i,j)` umrechnest und wie du zum nächsten Feld weitergehst; wichtig ist dabei der Abbruchfall, wenn du „hinter“ dem letzten Feld angekommen bist.\n- Für `solved`: Wenn ein Feld bereits belegt ist, solltest du es nicht verändern, sondern direkt mit dem nächsten Feld weitermachen.\n- Für `solved`: Wenn ein Feld leer ist, probiere Werte von `1..size()` aus, setze sie ins Model und verwende `checker.oneOK(model, i, j)` (oder passend) um zu entscheiden, ob du weiter recursen darfst; wenn der rekursive Versuch scheitert, musst du das Feld wieder leeren (Backtracking).\n- Für `nofSolutions`: Nimm die gleiche Traversierungslogik wie bei `solved`, aber statt beim ersten Fund zu stoppen, erhöhst du einen Zähler, sobald du eine vollständige gültige Belegung erreicht hast.\n- Für `nofSolutions`: Achte darauf, dass du das `max`/`MAX` Limit wirklich als Abbruchkriterium nutzt, damit du bei Sudokus mit sehr vielen Lösungen frühzeitig stoppen kannst.\n- Für `nofSolutions`: Auch hier nach jedem gesetzten Testwert bei Rückkehr unbedingt wieder `clear`en, damit das Model am Ende wieder im Ausgangszustand ist.\n\n## 3. Code Style\n- Die Kommentare `// TODO implement` sind okay während der Arbeit, sollten aber entfernt/ersetzt werden, sobald du fertig bist.\n- In `SudokuSolverImpl` würde ich bei den privaten Helfermethoden kurz dokumentieren, was `fieldNr` bedeutet (linearer Index), damit die Rekursion leichter nachvollziehbar bleibt.\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
} ]
}
}