{
"llm" : {
"feedback" : "# Exercise: fractal\n\n### Correctness\n- In `Boxes.drawRec(...)` wird bei `n == 0` gar nichts gezeichnet (dein Code zeichnet nur, wenn `n != 0`), damit fehlt der Basisfall und bei kleinen Rekursionstiefen bleibt das Bild leer bzw. unvollständig.\n- In `Boxes.drawRec(...)` zeichnest du pro Rekursionsschritt zwar 3 Teil-Quadrate, aber das vierte Teil-Quadrat (unten links) wird nie gezeichnet.\n- In `Boxes.drawRec(...)` zeichnest du das aktuelle große Rechteck bei jedem Schritt (`g.drawRect(x, y, s, s)`), dadurch enthält das Ergebnis zusätzliche Rechtecke, die so nicht zur rekursiven Endstruktur gehören.\n- In `Triangles.drawRec(...)` wird bei `n == 0` nichts gezeichnet (du zeichnest nur, wenn `n != 0`), dadurch fehlt auch hier der Basisfall.\n- In `Triangles.drawRec(...)` zeichnest du das “große” Dreieck in jedem Rekursionsschritt erneut, was zusätzliche Linien erzeugt, die nicht nur aus den kleinsten Dreiecken bestehen.\n\n### Suggestion\n- Überlege dir für beide Shapes: Was genau soll im **Basisfall** passieren (also wenn keine weitere Unterteilung mehr stattfinden soll)? Prüfe, ob genau dann gezeichnet wird.\n- Bei den Boxes: Skizziere einmal ein Quadrat, teile es in 4 gleich große Quadrate und markiere, welche Quadrate rekursiv weiter unterteilt werden sollen und welches “direkt” gezeichnet werden soll – dir fehlt aktuell eines dieser vier Teilquadrate.\n- Bei den Boxes: Entscheide dich, ob du in den Rekursionsschritten überhaupt das aktuelle Quadrat zeichnen willst, oder ob das Zeichnen nur im Basisfall passieren soll (sonst bekommst du zusätzliche Rahmen).\n- Bei den Triangles: Trenne “Dreieck zeichnen” und “Unterdreiecke berechnen”: Zeichne das Dreieck nur in dem Fall, in dem nicht weiter rekursiv unterteilt wird.\n- Bei den Triangles: Kontrolliere anhand der Midpoints (Seitenmitten), ob deine drei rekursiven Aufrufe wirklich die drei gewünschten Teil-Dreiecke abdecken (und nicht zusätzlich/anders überlagern), wenn du nur im Basisfall zeichnest.\n\n### Code Style\n- In `Triangles.drawRec(...)` ist der Kommentar `// TODO implement` am Ende irreführend, weil du schon implementiert hast – besser entfernen oder an die richtige Stelle setzen.\n- In `Boxes.drawRec(...)` und `Triangles.drawRec(...)` ist `if (n != 0)` weniger klar als eine explizite Basisfall-Prüfung (Lesbarkeit/Intent).\n- In `Triangles.drawRec(...)` berechnest du viele Mittelwerte mehrfach direkt in den Argumenten; speichere sie in gut benannten Variablen (z.B. Seitenmitten), das macht den Code deutlich verständlicher und weniger fehleranfällig.\n\n\n# Exercise: knapsack\n\n### Correctness\n- In `KnapsackImpl` sind `pack(int i)`, `loadItem(int i)` und `unloadItem(int i)` nicht implementiert, dadurch kann `solve()` keine optimale Lösung berechnen und die Tests können nicht bestehen.\n- In `KnapsackN` hat `main()` nicht die übliche Signatur `main(String[] args)`. Falls die Tests/Runner explizit diese Signatur erwarten, wird das Programm nicht korrekt gestartet.\n\n### Suggestion\n- Überlege dir für `pack(i)` eine Rekursion/Backtracking-Struktur mit Abbruchbedingung: Was passiert, wenn `i == weights.length` (also keine Items mehr übrig sind), und wann aktualisierst du `maxValue`?\n- Für “n gleiche Gegenstände” brauchst du pro Item-Index nicht nur die Entscheidung “nehmen/nicht nehmen”, sondern eine Schleife über die Anzahl Exemplare (z.B. 0 bis `maxN`). Achte darauf, dass du nach jedem “Nehmen” den Zustand wieder sauber zurücksetzt, bevor du andere Varianten ausprobierst.\n- Prüfe bei jeder Teil-Lösung, ob `currWeight` das `maxWeight` überschreitet, damit du unzulässige Zweige früh abbrechen kannst.\n- Wenn deine Umgebung `main(String[] args)` verlangt, passe die Signatur entsprechend an und vergleiche sie mit der Vorlage/den Tests.\n\n### Code Style\n- Die Methoden heißen bei dir `loadItem/unloadItem`, inhaltlich passen sie, aber wähle konsistente Namen (z.B. “pack/unpack” oder “load/unload”) und nutze sie dann auch konsequent in `pack`.\n- Aktuell ist viel TODO-Code stehen geblieben; entferne TODO-Kommentare, sobald du die Funktionalität implementiert hast, damit klar ist, was fertig ist.\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" : "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
} ]
}
}