{
"llm" : {
"feedback" : "# Exercise: fractal\n\n### Correctness\n- In `Boxes`, die rekursive Zeichenmethode `drawRec(...)` ist nicht implementiert; dadurch wird beim Zeichnen der Boxen nichts gezeichnet.\n- In `Triangles`, die rekursive Zeichenmethode `drawRec(...)` ist nicht implementiert; dadurch wird beim Zeichnen der Dreiecke nichts gezeichnet.\n\n### Suggestion\n- Überlege dir für `Boxes.drawRec(...)` zuerst einen klaren **Abbruchfall** (z.B. wenn `n` eine bestimmte Tiefe erreicht) und zeichne dann genau **ein Rechteck** mit `g.drawRect(...)`. Im rekursiven Fall solltest du die Fläche in kleinere Quadrate aufteilen und die Methode mit passenden neuen `(x, y, s, n-1)`-Werten aufrufen.\n- Für `Triangles.drawRec(...)` brauchst du ebenfalls einen **Abbruchfall**, in dem du das Dreieck wirklich zeichnest (mit drei `g.drawLine(...)`). Für den rekursiven Schritt: berechne die **Mittelpunkte der Dreiecksseiten** (jeweils Mittelwert der Koordinaten) und rufe `drawRec(...)` dann für die entstehenden kleineren Dreiecke mit `n-1` auf.\n\n### Code Style\n- Die `// TODO implement` Kommentare sind noch drin; sobald du fertig bist, kannst du sie entfernen, damit klar ist, dass die Aufgabe gelöst ist.\n\n\n# Exercise: knapsack\n\nLlm Evaluation ist für diese Aufgabe deaktiviert. Entferne die .llmignore Datei vom Package der Aufgabe.\n\n\n# Exercise: sudoku\n\n### Correctness\n- `Sudoku.main` hat nicht die korrekte Signatur (muss `public static void main(String[] args)` sein), sonst startet die Applikation nicht über den üblichen Java-Einstiegspunkt.\n- `solved(...)` erfüllt das Backtracking-Verhalten nicht korrekt: Du rufst rekursiv `solved(model, fieldNr + 1)` auf, ignorierst aber dessen Rückgabewert; dadurch brichst du nicht ab, wenn eine Lösung gefunden wurde, und gibst das Ergebnis nicht sauber nach oben zurück.\n- In `solved(...)` fehlt das eigentliche Backtracking-Aufräumen: Wenn ein getesteter Wert nicht zur Lösung führt, muss das Feld wieder geleert werden, sonst “verschmutzt” der Versuch das Modell.\n- `solved(...)` garantiert bei `false` nicht, dass das Model auf den Initialzustand zurückgesetzt wird (Anforderung: bei “keine Lösung” muss das Sudoku wieder wie am Anfang aussehen).\n- Die Abbruchbedingung in `solved(...)` prüft `checker.allOK(model)` am Ende/bei voller Belegung; damit suchst du nicht korrekt entlang “partiell gültiger” Zustände, sondern riskierst, dass du am Ende mit einem teilweise gefüllten, aber “lokal ok” Zustand rauskommst bzw. unnötig weiterrechnest.\n- `nofSolutions(...)` ist nicht implementiert und liefert immer `0`, damit ist Teil 4 der Aufgabe nicht erfüllt.\n\n### Suggestion\n- Schau dir an, wie du in der Rekursion ein “found”-Resultat von tieferen Aufrufen nach oben propagieren kannst, sodass du bei der ersten gefundenen Lösung sofort stoppen kannst.\n- Überlege dir für den Backtracking-Schritt: Was muss mit dem aktuell gesetzten Feld passieren, wenn kein Wert (1..size) zu einer Lösung führt? (Stichwort: Zustand vor dem Versuch wiederherstellen.)\n- Um das “Model bei false zurücksetzen” zu erreichen: Du brauchst eine Strategie, wie du nur die von dir gesetzten Felder wieder entfernst (vordefinierte Felder dürfen nicht verändert werden).\n- Für die Verankerung (Base Case): Trenne klar “alle Felder abgearbeitet” von “Zustand ist gültig”. Meist reicht “alle Felder abgearbeitet ⇒ Lösung”, wenn du bei jedem Setzen bereits mit `oneOK` (oder ähnlich) prüfst.\n- Für `nofSolutions`: Denk an denselben Suchbaum wie bei `solved`, aber statt beim ersten Fund zu stoppen zählst du jeden vollständigen gültigen Endzustand als 1 und summierst; zusätzlich musst du abbrechen, sobald `MAX` erreicht ist.\n\n3. Code Style:\n- In `solved(...)` sind `9`, `10` hart codiert; nutze stattdessen konsequent `model.size()` (auch für Zeile/Spalte), sonst ist der Code unnötig unflexibel.\n- Vermeide `this.checker...` überall, wenn kein Namenskonflikt besteht; das macht den Code etwas schwerer lesbar.\n- Die Logik am Ende von `solved(...)` (`return checker.allOK(...) && fieldNr >= ...`) dupliziert die Base-Case-Idee und macht den Kontrollfluss schwer nachvollziehbar; besser ist ein klarer Rückgabepfad pro Fall (Base Case / vorgegebenes Feld / ausprobieren).\n- In `solved(...)` könntest du `row/column` als `final` deklarieren, da sie sich nicht ändern; erhöht Lesbarkeit.\n\n\n# Exercise: queens\n\nLlm Evaluation ist für diese Aufgabe deaktiviert. Entferne die .llmignore Datei vom Package der Aufgabe.\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
} ]
}
}