{
"llm" : {
"feedback" : "# Exercise: fractal\n\n### Correctness\n- In `Boxes.drawRec`: du rufst als ersten rekursiven Schritt `drawRec(g, x, y, s, n-1)` mit **derselben** Quadratgröße `s` und denselben Koordinaten auf; dadurch wird das große Quadrat bei jedem Level immer wieder komplett neu “aufgespannt” und die Rekursion zeichnet nicht die geforderte Aufteilung in Teilquadrate.\n- In `Boxes.drawRec`: du zeichnest nur drei der vier Quadranten in der rekursiven Aufteilung (`top-left`, `top-right`, `bottom-right`), der `bottom-left`-Quadrant fehlt.\n- In `Boxes.drawRec`: durch den ersten rekursiven Aufruf mit unverändertem `s` entsteht sehr viel redundantes Zeichnen (und das Muster entspricht nicht dem Ziel-Fraktal mit “fehlendem”/anders behandelten Quadranten).\n\n### Suggestion\n- Überlege dir für `Boxes`, was sich bei jedem Rekursionsschritt **ändern muss**: Die Teilquadrate sollten eine kleinere Seitenlänge haben (typisch `s/2`) und ihre `x/y`-Positionen sollten so verschoben werden, dass sie die vier Quadranten abdecken.\n- Prüfe systematisch die vier Zielpositionen der Quadranten: `(x, y)`, `(x + s/2, y)`, `(x, y + s/2)`, `(x + s/2, y + s/2)` – und entscheide dann, welcher Quadrant “normal” rekursiv weitergeht und welcher ggf. direkt im Basisfall gezeichnet wird (je nach gewünschtem Muster).\n- Wenn du sicherstellen willst, dass das Muster stimmt, zeichne für ein kleines `n` (z.B. `n=1` oder `n=2`) einmal auf Papier, welche Quadrate erscheinen sollen, und gleiche das dann mit deinen rekursiven Aufrufen ab.\n\n### Code Style\n- In `Boxes.drawRec` ist das `return` nach dem `if(n == 0)` nicht nötig, weil danach sowieso nichts mehr kommen müsste, wenn du ein `else` verwendest (funktional ok, aber etwas unnötig).\n- In `Boxes.drawRec` sind die rekursiven Aufrufe aktuell schwer zu “lesen”, weil ein Aufruf (der mit unverändertem `s`) aus dem Muster herausfällt; benenne/ordne die Aufrufe lieber klar nach Quadranten (z.B. Kommentar “top-left”, “top-right”, …), dann fällt sofort auf, ob einer fehlt.\n\n\n# Exercise: knapsack\n\n### Correctness\n- Deine Abbruchbedingung in `pack(int i)` nutzt `if (i == maxN)`, aber `i` ist der Index des Gegenstands (läuft über `weights.length`) und nicht die Anzahl erlaubter Kopien; dadurch wird die Rekursion zu früh/zu spät beendet und es werden nicht alle Gegenstände betrachtet.\n- Mit `pack(i)` nach `loadItem(i)` erlaubst du, denselben Gegenstand beliebig oft zu nehmen, solange das Gewicht passt; die Vorgabe ist aber „jeden Gegenstand maximal `maxN`-mal“, nicht unendlich oft bis zum Gewichtslimit.\n- Durch das falsche Stoppkriterium kann es außerdem passieren, dass du `weights[i]`/`values[i]` mit einem `i` verwendest, das gar kein gültiger Index mehr ist (je nach Verlauf der Rekursion), was die Tests zum Absturz bringen kann.\n\n### Suggestion\n- Überlege dir genau, welche Bedeutung `i` in der Rekursion haben soll: typischerweise ist das „welcher Gegenstand als nächstes dran ist“. Dann sollte die Abbruchbedingung an die Anzahl Gegenstände gekoppelt sein (also an die Länge der Arrays), nicht an `maxN`.\n- Wenn `maxN` die maximale Anzahl Kopien pro Gegenstand ist, brauchst du irgendwo eine Zählung/Schleife/Logik, die pro Gegenstand nur `0..maxN` Auswahlmöglichkeiten zulässt, bevor du zum nächsten Gegenstand weitergehst.\n- Den Zustand beim Zurückgehen aus der Rekursion musst du so zurücksetzen, dass genau die Anzahl hinzugefügter Kopien wieder entfernt wird (nicht „irgendwie einmal“), sonst bleiben Werte/ Gewichte aus vorherigen Pfaden im Zustand hängen.\n\n### Code Style\n- In `pack` ist der Kommentar `// remove item if it was to much` irreführend: `unloadItem(i)` ist hier Teil des normalen Backtrackings und nicht „nur wenn es zu viel war“.\n- `KnapsackN.main()` hat keine `String[] args`-Signatur; falls eure Test-/Run-Umgebung eine echte Java-`main` erwartet, kann das verwirrend sein (auch wenn es nicht Teil der eigentlichen Knapsack-Logik ist).\n\n\n# Exercise: sudoku\n\n### Correctness\n- In `Sudoku.java` ist `main()` nicht als `public static void main(String[] args)` deklariert; so wird das Programm typischerweise nicht starten (Entry-Point fehlt).\n- `solved(SudokuModel model)` muss laut Interface/Kommentar bei “keine Lösung” das Modell auf den Initialzustand zurücksetzen; deine Implementierung lässt evtl. gesetzte Werte aus erfolglosen Versuchen im Modell stehen.\n- `nofSolutions(SudokuModel model)` muss das Modell nach dem Zählen ebenfalls auf den Initialzustand zurücksetzen; aktuell bleibt das Modell nach dem Aufruf nicht garantiert im Ausgangszustand.\n- Deine `isValid(...)`-Prüfung ignoriert den bereitgestellten `SudokuChecker` komplett; falls die Umgebung erwartet, dass die Validierung über `checker.oneOK(...)` läuft (z.B. wegen spezieller Regeln/Generalisierung), kann das von den Anforderungen abweichen.\n\n### Suggestion\n- Schau dir an, was “Initialzustand” konkret bedeutet: Du brauchst eine Strategie, wie du nach einem Fehlschlag (bei `solved`) bzw. nach dem vollständigen Durchzählen (bei `nofSolutions`) alle Veränderungen gegenüber dem Start wieder rückgängig machst.\n- Überlege bei `solved`: Wenn du im Rekursionsbaum zurückläufst, werden nicht nur die aktuell getesteten Felder wieder geleert, sondern am Ende (wenn keine Lösung existiert) auch alle Felder, die während des gesamten Versuchs gesetzt wurden.\n- Für `nofSolutions`: Achte darauf, dass du vor dem Return aus der Methode (auch bei Early-Exit durch `MAX`) den aktuellen Pfad sauber “aufrollst”, damit am Ende keine Testwerte im Grid verbleiben.\n- Wenn du den `SudokuChecker` verwenden sollst: Du könntest deine Validierung so ausrichten, dass sie die vorhandenen Checker-Methoden nutzt (zumindest an der Stelle, wo du gerade einen Wert gesetzt hast), statt die Regeln doppelt zu implementieren.\n\n### Code Style\n- Du hast mit `isValid(...)` Logik dupliziert, die bereits über `SudokuChecker` vorhanden ist; das erhöht Wartungsaufwand und Risiko von Inkonsistenzen (eine Variante ändern, andere vergessen).\n- In `Sudoku.java` fehlt `public static` bei `main`; auch wenn das “nur” eine Signatur ist, ist das ein sehr auffälliger Konventions-/Strukturbruch für Java-Startklassen.\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" : "FAILED",
"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
} ]
}
}