{
"llm" : {
"feedback" : "# Exercise: fractal\n\n### Correctness\n- In `Boxes`, die rekursive Zeichenmethode `drawRec(...)` ist nicht implementiert; damit wird keine Box-Fraktalform gezeichnet.\n- In `Triangles`, die rekursive Zeichenmethode `drawRec(...)` ist nicht implementiert; damit wird keine Dreiecks-Fraktalform gezeichnet.\n\n### Suggestion\n- Für `Boxes.drawRec(...)`: Überlege dir einen klaren **Abbruchfall** (z.B. bei `n == 0`), in dem du **genau ein Quadrat** mit `g.drawRect(...)` zeichnest. Im rekursiven Fall teilst du die Seitenlänge sinnvoll (typisch halbieren) und platzierst mehrere kleinere Quadrate an den passenden Positionen, wobei `n` jeweils kleiner wird.\n- Für `Triangles.drawRec(...)`: Baue ebenfalls einen **Abbruchfall** (z.B. `n == 0`), in dem du das Dreieck mit **drei Linien** zeichnest (`g.drawLine(...)`). Im rekursiven Fall berechnest du die **Mittelpunkte der Dreiecksseiten** und rufst `drawRec` dann für die entstehenden kleineren Teildreiecke mit `n - 1` auf.\n\n### Code Style\n- Die `// TODO implement`-Kommentare sind noch vorhanden; sobald du fertig bist, solltest du sie entfernen oder durch kurze, konkrete Kommentare ersetzen (z.B. “base case”, “recursive step”).\n- In `Triangles.drawRec(...)` steht noch ein Hinweis-Kommentar aus der Vorlage; nach der Implementierung kann der entweder entfernt oder zu einer echten Erklärung deiner Schritte umformuliert werden.\n\n\n# Exercise: knapsack\n\n### Correctness\n- In `pack(int i)` rufst du `pack(i + 1)` im `while` **bevor** du das aktuelle Item lädst; dadurch wird der Fall „0 Stück von Item i“ mehrfach geprüft, während der Fall „genau maxN Stück von Item i“ nie geprüft wird (weil nach dem letzten `loadItem` kein `pack(i + 1)` mehr folgt).\n- Deine Rücksetz-Logik nach dem `while` ist inkonsistent: Im zweiten `for`-Block rufst du bei jedem Entladen nochmals `pack(i + 1)` auf. Das erzeugt Zustände, die nicht der üblichen „für jedes n: erst Zustand herstellen, dann rekursiv weiter“ Struktur entsprechen und kann Kombinationen doppelt zählen bzw. wichtige Kombinationen in der falschen Reihenfolge evaluieren.\n- `KnapsackN.main()` hat nicht die erwartete Java-Signatur `main(String[] args)`, wodurch das Programm so ggf. nicht direkt gestartet werden kann (auch wenn Tests evtl. nur `KnapsackImpl` prüfen).\n\n### Suggestion\n- Überlege dir eine klare Schleifenstruktur pro Item `i`: Für **jede** mögliche Anzahl `n` von 0 bis `maxN` solltest du genau einmal „aktuellen Zustand“ → `pack(i+1)` ausführen. Prüfe dabei speziell, ob du nach dem letzten Hinzufügen noch einen Rekursionsaufruf machst.\n- Achte darauf, dass du nach dem Durchprobieren aller `n` den Zustand (`currWeight/currValue`) wieder exakt auf den Stand vor Item `i` zurücksetzt, ohne dabei zusätzliche Rekursionsaufrufe in der „Aufräumphase“ zu machen.\n- Passe die `main`-Methode an die Standard-Signatur an, damit sie zuverlässig ausführbar ist.\n\n### Code Style\n- Die Kommentare `// TODO implement` in `loadItem`/`unloadItem` sind nicht mehr aktuell und sollten entfernt werden.\n- `amount += 1;` kann idiomatischer als `amount++;` geschrieben werden (kleine Lesbarkeitsverbesserung).\n\n\n# Exercise: queens\n\nLlm Evaluation ist für diese Aufgabe deaktiviert. Entferne die .llmignore Datei vom Package der Aufgabe.\n\n\n# Exercise: sudoku\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" : "PASSED",
"message" : null
}, {
"name" : "random2()",
"status" : "PASSED",
"message" : null
}, {
"name" : "random3()",
"status" : "PASSED",
"message" : null
}, {
"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
} ]
}
}