{
"llm" : {
"feedback" : "# Exercise: fractal\n\n### Correctness\n- In `Boxes.drawRec(...)` zeichnest du bei jedem Rekursionsschritt sofort ein Rechteck (`g.drawRect(...)`), statt nur im Abbruchfall; dadurch entstehen zusätzliche Rechtecke und nicht die geforderte rekursive Struktur (insbesondere wird auch bei `n > 0` das große Quadrat immer wieder gezeichnet).\n- In `Boxes.drawRec(...)` fehlen Rekursionsaufrufe für alle vier Teilquadrate (du zeichnest nur drei Quadranten rekursiv; unten links wird nie gezeichnet).\n- In `Triangles.drawRec(...)` zeichnest du bei jedem Rekursionsschritt sofort das aktuelle Dreieck, statt nur im Basisfall; damit erhältst du überlagerte Dreiecke und nicht nur die kleinsten Dreiecke der Rekursionstiefe.\n- In `Triangles.drawRec(...)` ist dein Abbruchkriterium `n < 0`; bei `n == 0` wird trotzdem weiter rekursiv aufgerufen (mit `n - 1`), was zwar irgendwann stoppt, aber eine Rekursionstiefe mehr erzeugt als vorgesehen und die Struktur verändert.\n\n### Suggestion\n- Überlege dir für beide Formen ein klares Basisfall-Kriterium (z.B. „wenn die Tiefe erreicht ist, zeichnen“) und zeichne die primitive Form nur dort; im Rekursionsfall sollte dann nur noch in Teilprobleme zerlegt werden.\n- Bei den Boxen: prüfe, welche vier Teilquadrate (Positionen) aus einem Quadrat der Seitenlänge `s` entstehen, wenn du `s` halbierst, und stelle sicher, dass alle vier berücksichtigt werden (auch unten links).\n- Bei den Dreiecken: wenn du Mittelpunkte der Kanten berechnest, nutze diese drei Punkte, um die drei „Eck“-Dreiecke zu definieren; achte darauf, dass du im Rekursionsfall nicht zusätzlich das große Dreieck zeichnest, wenn eigentlich nur die kleinsten gezeichnet werden sollen.\n- Ändere den Abbruch so, dass bei genau der gewünschten Tiefe Schluss ist (also nicht erst „unter 0“), damit die Anzahl Rekursionsschritte exakt passt.\n\n### Code Style\n- Vermeide „magische“ Abbruchbedingungen wie `n < 0`, wenn die Aufgabe eine diskrete Tiefe nahelegt; ein klarer Basisfall (`n == 0`) macht die Logik leichter verständlich.\n- In `Boxes.drawRec(...)` überschreibst du den Parameter `s` direkt (`s = s / 2;`); lesbarer ist es oft, eine neue Variable wie `half` zu verwenden, damit klar bleibt, wofür welcher Wert steht.\n\n\n# Exercise: knapsack\n\n### Correctness\n- In `pack(int i)` entlädst du am Ende immer `maxN`-mal (`for (int j = 0; j < maxN; j++) unloadItem(i);`), auch wenn du das Item in dieser Rekursions-Ebene evtl. weniger als `maxN`-mal geladen hast (z.B. wenn du wegen `currWeight > maxWeight` früher zurückkehrst oder wenn die Schleife nicht alle `loadItem`-Schritte ausführt). Dadurch kann `currWeight/currValue` zu klein werden (sogar negativ) und die Zustände der Rekursion stimmen nicht mehr.\n- Der Early-Return `if (currWeight > maxWeight) return;` passiert, ohne den aktuellen Zustand dieser Ebene wieder “aufzuräumen”. Wenn in dieser Ebene davor schon `loadItem(i)` aufgerufen wurde, bleibt das Laden “hängen” (weil dein pauschales Entladen erst nach der Schleife käme, aber wegen `return` nie erreicht wird).\n- In `KnapsackN` hat `main()` nicht die erwartete Java-Signatur `main(String[] args)`. Falls die Tests/Runner direkt `public static void main(String[] args)` erwarten, startet das Programm nicht wie vorgesehen.\n\n### Suggestion\n- Achte darauf, dass jede Rekursions-Ebene genau so viele `unloadItem(i)` macht, wie sie in dieser Ebene vorher `loadItem(i)` gemacht hat—auch dann, wenn du wegen Übergewicht früh abbrichst. Eine einfache Strategie ist: “laden → rekursiver Aufruf → sofort wieder entladen” (symmetrisch pro Versuch) oder du zählst mit, wie oft du wirklich geladen hast und entlädst genau diese Anzahl.\n- Vermeide Rücksprünge (`return`) an Stellen, wo noch “Cleanup” (Entladen) nötig wäre. Du kannst stattdessen Übergewicht als Bedingung nutzen, um bestimmte Rekursionszweige gar nicht weiter zu verfolgen, ohne die Methode abrupt zu verlassen.\n- Passe die `main`-Methode an die Standard-Signatur an, damit die Ausführung in typischen Java-Setups/Testumgebungen zuverlässig klappt.\n\n### Code Style\n- Entferne die `// TODO implement`-Kommentare dort, wo du bereits implementiert hast, damit klar ist, was noch fehlt.\n- Die beiden Schleifen am Ende von `pack` (eine zum Laden, eine zum Entladen) sind fehleranfällig; lesbarer und weniger riskant wird es, wenn Laden/Entladen strukturell näher beieinander stehen (z.B. direkt um den rekursiven Aufruf herum).\n\n\n# Exercise: sudoku\n\n### Correctness\n- In `nofSolutions` wird das Parameter `max` beim Rekursionsaufruf nicht reduziert/berücksichtigt; dadurch kann es passieren, dass du insgesamt mehr als `MAX` Lösungen “zählst”, obwohl laut Aufgabenstellung bei `MAX` abgebrochen werden soll.\n- In `nofSolutions` wird das Model am Ende nicht garantiert auf den Ausgangszustand zurückgesetzt, wenn du wegen `zähler >= max` frühzeitig zurückkehrst (du clearest nur das aktuelle Feld, aber alle tieferen Rekursions-Änderungen bleiben bestehen).\n\n### Suggestion\n- Überlege dir, wie viele Lösungen du “noch suchen darfst”, wenn du bereits `zähler` Lösungen gefunden hast, und gib genau diese Restgrenze an den nächsten Rekursionsaufruf weiter.\n- Prüfe die Stellen mit `return` in `nofSolutions`: An welchen Rückgabepfaden muss zwingend ein vollständiges Backtracking passieren, damit nach dem Zählen wieder exakt der ursprüngliche Zustand im Model steht?\n\n### Code Style\n- Verwende besser nur ASCII-Bezeichner (z.B. `zaehler` statt `zähler`), damit es in allen Tools/Encodings problemlos kompiliert und lesbar bleibt.\n- Viele Kommentare wiederholen nur, was der Code ohnehin ausdrückt; versuche eher die “Warum”-Frage zu kommentieren (z.B. warum du `max` weiterreichst), nicht das Offensichtliche.\n- In `Sudoku.java` ist `main()` nicht `public static void main(String[] args)`; falls das so bei dir eingereicht wird, startet die Applikation typischerweise nicht direkt über die JVM-Standard-Einstiegsmethode.\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" : "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" : "Connection refused to host: 172.25.0.17; nested exception is: \n\tjava.net.ConnectException: Connection refused"
}, {
"name" : "boardSizeIsCorrectForN4()",
"status" : "PASSED",
"message" : null
}, {
"name" : "boardSizeIsCorrectForN8()",
"status" : "FAILED",
"message" : "Connection refused to host: 172.25.0.18; nested exception is: \n\tjava.net.ConnectException: Connection refused"
}, {
"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" : "FAILED",
"message" : "Connection refused to host: 172.25.0.19; nested exception is: \n\tjava.net.ConnectException: Connection refused"
} ]
}
}