{
"llm" : {
"feedback" : "# Exercise: fractal\n\n### Correctness\n- In `Boxes`, die rekursive Methode `drawRec(...)` ist nicht implementiert, dadurch wird beim Zeichnen nichts angezeigt.\n- In `Triangles`, die rekursive Methode `drawRec(...)` ist nicht implementiert, dadurch wird beim Zeichnen nichts angezeigt.\n\n### Suggestion\n- Für `Boxes.drawRec(...)`: Überlege dir einen klaren Abbruchfall für die Rekursion (z.B. wenn `n` einen bestimmten Wert erreicht) und zeichne dann genau ein Rechteck. In allen anderen Fällen solltest du die Fläche in kleinere Teilquadrate aufteilen und die Methode mit angepassten Koordinaten/Grössen erneut aufrufen.\n- Für `Triangles.drawRec(...)`: Definiere ebenfalls einen Abbruchfall, in dem du die drei Kanten mit `g.drawLine(...)` zeichnest. Für den Rekursionsschritt hilft es, die Mittelpunkte der drei Dreiecksseiten zu berechnen und daraus die kleineren Teil-Dreiecke zusammenzusetzen, die du rekursiv weiterzeichnest.\n\n### Code Style\n- In beiden Klassen (`Boxes`, `Triangles`) stehen noch `// TODO implement` Kommentare; sobald du fertig bist, entferne sie oder ersetze sie durch kurze, beschreibende Kommentare zur Rekursionsidee.\n\n\n# Exercise: knapsack\n\n### Correctness\n- In `pack`, beim Zurücksetzen nach dem `while` entlädst du mit `for (int x = amount; x >= 0; x--)` **amount+1-mal**. Du lädst aber nur `amount`-mal ⇒ am Ende wird ein Item zu viel entfernt und `currWeight/currValue` können falsch werden (sogar negativ).\n- Die `while (currWeight < maxWeight && amount < maxN)`-Bedingung verhindert, dass du ein Item lädst, wenn `currWeight == maxWeight` (okay), aber sie verhindert nicht, dass du ein Item lädst, das **maxWeight überschreitet** (z.B. `currWeight=18`, `weights[i]=3`). Damit explorierst du Zustände, die über dem Limit liegen; das ist zwar nicht automatisch falsch, aber du musst sicherstellen, dass das Backtracking dann korrekt wieder auf den vorherigen Zustand zurückkommt (was wegen des Off-by-one beim Entladen aktuell nicht garantiert ist).\n\n### Suggestion\n- Schau dir genau an, wie viele Items du in deiner Schleife wirklich lädst (`amount`) und wie viele du danach wieder entlädst. Überprüfe die Laufgrenzen der `for`-Schleife: Wenn du `amount`-mal geladen hast, solltest du auch nur `amount`-mal entladen.\n- Überlege, ob du vor `loadItem(i)` prüfen willst, ob `currWeight + weights[i]` noch innerhalb von `maxWeight` bleibt. Alternativ kannst du auch weiterhin „überladen“ erlauben, aber dann muss das Backtracking (Entladen) in jedem Fall exakt stimmen.\n\n### Code Style\n- Die Kommentare `// TODO implement` in `loadItem`/`unloadItem` sind nicht mehr passend, weil du die Methoden bereits implementiert hast.\n- In `pack` ist die Logik mit `amount` + `while` + anschließendem `for` etwas schwer zu lesen; eine klarere Struktur (z.B. konsistente Zählweise/Loop-Grenzen) würde die Gefahr von Off-by-one-Fehlern reduzieren.\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: <150>"
}, {
"name" : "random2()",
"status" : "FAILED",
"message" : "expected: <340> but was: <246>"
}, {
"name" : "random3()",
"status" : "FAILED",
"message" : "expected: <558> but was: <544>"
}, {
"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
} ]
}
}