{
"llm" : {
"feedback" : "# Exercise: fractal\n\n### Correctness\n- In `Boxes` ist `drawRec(...)` nicht implementiert; damit wird beim Zeichnen keine Box/kein Fraktal ausgegeben, obwohl die Aufgabe verlangt, die `draw`-Methoden in `Boxes` zu implementieren.\n- In `Triangles` ist `drawRec(...)` nicht implementiert; damit wird kein Dreiecks-Fraktal gezeichnet, obwohl die Aufgabe verlangt, die `draw`-Methoden in `Triangles` zu implementieren.\n\n### Suggestion\n- Bei `Boxes.drawRec(...)`: Überlege dir einen klaren Rekursions-Abbruch (z.B. wenn `n` ein bestimmtes Level erreicht). Im Abbruchfall zeichnest du genau **ein** Quadrat mit `g.drawRect(...)`; sonst teilst du das aktuelle Quadrat in kleinere Quadrate und rufst `drawRec` für diese Teilbereiche mit reduziertem `n` auf.\n- Bei `Triangles.drawRec(...)`: Definiere ebenfalls einen Basisfall, in dem du das Dreieck durch **drei Linien** zeichnest. Im Rekursionsfall berechnest du aus den drei Eckpunkten die **Mittelpunkte der Kanten** und rufst die Methode für die entstehenden kleineren Dreiecke mit `n-1` auf.\n\n### Code Style\n- In `Boxes.drawRec(...)` steht nur noch ein `// TODO implement` ohne Hinweise; ein kurzer Kommentar, welche Parameter was bedeuten (x/y/s/n), würde die Lesbarkeit verbessern.\n- In `Triangles.drawRec(...)` ist der Kommentar zwar da, aber die Methode bleibt komplett leer; entferne TODOs erst, wenn implementiert, oder ergänze sie um konkrete Zwischenschritte, die du als nächstes umsetzen willst.\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 erwartete Java-Entry-Signatur (`public static void main(String[] args)`), dadurch startet die Applikation typischerweise nicht.\n- `solved(...)` erfüllt die Spezifikation nicht: Du gibst am Ende immer `false` zurück, auch wenn eine Lösung gefunden wurde.\n- `solved(...)` setzt das Sudoku bei “keine Lösung” nicht zuverlässig auf den Anfangszustand zurück (Anforderung: bei `false` muss das Model wieder initial sein).\n- In `solved(...)` werden rekursive Rückgabewerte ignoriert: Du rufst `solved(model, fieldNr + 1)` auf, aber nutzt das Ergebnis nicht zum Abbrechen/Zurückpropagieren.\n- In `solved(...)` fehlt korrektes Backtracking: Wenn ein gesetzter Versuchswert nicht zum Erfolg führt, wird das Feld nicht wieder geleert, bevor der nächste Versuch kommt.\n- `solved(...)` verwendet hartcodiert `9`/`10` (z.B. `row = fieldNr / 9`, Schleife `x < 10`), statt mit `model.size()` zu arbeiten (die Schnittstelle ist allgemein gehalten).\n- Der `else if`-Zweig verändert auch “vordefinierte” Felder (nicht-0) durch Hochzählen des Werts; das verletzt die Idee, dass vorgegebene Werte fix bleiben.\n- `nofSolutions(...)` ist nicht implementiert und gibt immer `0` zurück, damit wird die Anforderung “Anzahl Lösungen zählen (bis MAX)” nicht erfüllt.\n\n### Suggestion\n- Schau dir die `main`-Methode an: Welche Signatur braucht Java, damit die JVM sie als Startpunkt erkennt?\n- Überlege bei `solved`: Wann genau soll `true` zurückgegeben werden, und wie kommt dieses `true` wieder aus der tiefen Rekursion nach oben (Stichwort: Rückgabewert sofort weiterreichen/Schleifen abbrechen)?\n- Plane Backtracking explizit ein: Wenn ein Versuch (Wert setzen) nicht zum Erfolg führt, welcher Schritt macht den Zustand wieder rückgängig, bevor du den nächsten Wert ausprobierst?\n- Prüfe die Basisbedingung: Wenn `fieldNr` am Ende angekommen ist, reicht “allOK” oder musst du auch sicherstellen, dass wirklich alle Felder befüllt sind? (Und: Was soll in diesem Fall zurückgegeben werden?)\n- Verwende überall `model.size()` (und ggf. `model.size()*model.size()`), damit dein Solver nicht implizit auf 9x9 fest verdrahtet ist.\n- Behandle vorgegebene Felder getrennt: Wenn `model.get(i,j) != 0`, dann sollte dieses Feld nicht verändert werden, sondern du gehst nur zum nächsten Feld weiter.\n- Für `nofSolutions`: Denk an eine sehr ähnliche Rekursion wie beim Lösen, nur dass du statt “erste Lösung finden” alle gültigen Fortsetzungen aufsummierst und bei `MAX` frühzeitig stoppst; wichtig ist dabei auch hier das Zurücksetzen (clear) beim Zurückgehen.\n\n### Code Style\n- In `solved(...)` sind Magic Numbers (`9`, `10`) verwendet; auch wenn es beim Standard-Sudoku passt, macht es den Code unnötig unflexibel und schlechter lesbar.\n- Der `else if`-Block mit `while` und Hochzählen macht den Kontrollfluss schwer nachvollziehbar; eine klarere Trennung der Fälle (vorgegeben vs. leer) reduziert Komplexität.\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
} ]
}
}