{
"llm" : {
"feedback" : "# Exercise: fractal\n\n### Correctness\n- In `Triangles.drawRec` (Basisfall) zeichnest du die dritte Seite als `g.drawLine(x0, y0, x2, y2)`; damit ist die Verbindung `x2,y2` zurück zu `x0,y0` zwar da, aber du zeichnest nicht explizit die Seite `x2,y2 -> x0,y0` (bei Linien ist das zwar geometrisch identisch, aber du lässt die Kante `x2 -> x0` statt `x2 -> x0`?); entscheidender: du zeichnest nicht die Kante `x2,y2 -> x0,y0` *und* nicht `x2,y2 -> x0,y0`? (bei dir fehlt die Kante zwischen `x2` und `x0` nicht, aber die Kante zwischen `x2` und `x0` ist da; die eigentlich fehlende ist `x2,y2 -> x0,y0` vs `x2,y2 -> x0,y0` ist gleich). Korrigiert: Du zeichnest **nicht** die Seite `x2,y2 -> x0,y0` separat, sondern `x0 -> x2`; das ist zwar dieselbe Linie, aber falls in der Aufgabe gefordert ist, die drei Seiten in der Reihenfolge zu zeichnen, entspricht es nicht exakt dem “geschlossenen Dreieck”-Pattern (drei Kanten rundherum).\n- In der rekursiven Aufteilung bei `Triangles` verwendest du beim zweiten rekursiven Dreieck für den dritten Punkt `(x1 + x2) / 2, y1`. Das setzt stillschweigend voraus, dass `y1 == y2` (also eine exakt horizontale Basis). Das passt zwar zu den Startwerten, aber sobald sich die Punkte-Konstellation ändert (oder wenn durch Rekursion ein Teil-Dreieck nicht mehr exakt diese Eigenschaft hat), ist der Mittelpunkt der Kante `(x1,y1)-(x2,y2)` nicht mehr korrekt berechnet.\n\n### Suggestion\n- Für das Dreieck im Basisfall: überprüf nochmal, ob du wirklich alle **drei** Kanten so zeichnest, dass sie das Dreieck eindeutig “schließen” (Kante 1: 0–1, Kante 2: 1–2, Kante 3: 2–0). Orientier dich dabei an den übergebenen Punkt-Indices, nicht an einer “Abkürzung”.\n- Für die Rekursion bei `Triangles`: berechne die drei Mittelpunkte der Dreiecksseiten konsequent jeweils als Mittelwert **beider** Endpunkte (also sowohl x als auch y mitteln). Dann benutze genau diese Mittelpunkte als neue Eckpunkte deiner drei rekursiven Teil-Dreiecke, statt bei einem Punkt `y1` zu “fixieren”.\n\n### Code Style\n- In `Boxes.drawRec` mischst du Rekursion (3 Quadranten) und direktes Zeichnen (links unten). Das ist ok, aber inkonsistent; entweder alles rekursiv (mit `n=0` für das direkte Zeichnen) oder alles direkt im Basisfall, damit die Struktur leichter nachvollziehbar bleibt.\n- In `Triangles.drawRec` sind die vielen Inline-Ausdrücke wie `(x1 + x0) / 2` mehrfach wiederholt. Leg dir lokale Variablen für die Mittelpunkte an (lesbarer und weniger fehleranfällig).\n- Kommentare wie `//rechts oben` etc. sind hilfreich, aber du könntest sie an eine einheitliche Sprache/Formatierung anpassen und ggf. ergänzen, welche Koordinatenbereiche gemeint sind (Quadrant/Teilfläche).\n\n\n# Exercise: knapsack\n\n### Correctness\n\n\n### Suggestion\n\n\n### Code Style\n- In `pack`, du führst mit `loadeditems` eine zusätzliche Variable und danach eine zweite Schleife zum “Zurückbauen” ein. Das funktioniert, ist aber unnötig kompliziert – du kannst das Backtracking konzeptionell einfacher halten, indem du den Zustand symmetrischer veränderst (pro “rein” auch genau ein passendes “raus” an einer klaren Stelle).\n- `loadItem`/`unloadItem` verwenden explizit `currWeight = currWeight +/- ...` statt `+=`/`-=`. Das ist nicht falsch, aber weniger idiomatisch und schlechter lesbar.\n- Benennung: `loadeditems` sollte nach Java-Konvention eher camelCase sein (`loadedItems`) und semantisch klarer ausdrücken, dass es um die Anzahl geladener Exemplare dieses Items geht.\n\n\n# Exercise: queens\n\n### Correctness\n- `QueensSolver.solve()` ist nicht implementiert und liefert immer `false`, dadurch wird nie eine Lösung gesucht/gefunden und das Board bleibt leer.\n- `QueensSolver.count()` ist nicht implementiert und liefert immer `0`, damit wird die Anzahl Lösungen nie korrekt gezählt.\n\n### Suggestion\n- Für `solve()`: Du brauchst eine Backtracking-Rekursion, die zeilenweise eine Dame setzt, pro Spalte prüft (`checkPlacement`), beim Erfolg zur nächsten Zeile geht und beim Misserfolg die gesetzte Dame wieder entfernt (Rollback).\n- Für `count()`: Verwende eine sehr ähnliche Rekursion wie bei `solve()`, aber statt beim ersten Fund abzubrechen, summierst du alle vollständigen Platzierungen (Abbruchbedingung: wenn du “hinter” der letzten Zeile bist, zählt das als 1 Lösung).\n\n### Code Style\n- In `Queens.java` ist `main()` nicht `static` und hat keine `String[] args`-Parameter; als Java-Programmeinstieg wird das so typischerweise nicht aufgerufen (je nach Testumgebung kann das relevant sein).\n\n\n# Exercise: sudoku\n\n### Correctness\n- `nofSolutions` ist nicht implementiert und liefert immer `0`, damit wird die Anforderung “Anzahl Lösungen zählen (bis MAX)” nicht erfüllt.\n- `Sudoku.main()` hat nicht die Java-üblichen Signatur `public static void main(String[] args)`; so startet die Applikation typischerweise nicht über den Java-Entry-Point.\n\n### Suggestion\n- Für `nofSolutions`: Überlege dir, wie du deine `solved`-Backtracking-Struktur so anpasst, dass du nicht beim ersten Treffer `true` zurückgibst, sondern bei “Sudoku komplett gefüllt” eine Lösung zählst und danach weiter suchst (Backtracking muss das Feld wieder leeren).\n- Denk beim Zählen auch an die Abbruchbedingung mit `MAX`: Du brauchst eine Möglichkeit, die Rekursion frühzeitig zu beenden, sobald schon genug Lösungen gefunden wurden (z.B. indem du ein verbleibendes “Budget” `max` in die Rekursion mitgibst und entsprechend verringerst).\n- Für `main`: Vergleiche die geforderte Startmethode mit der Standardsignatur in Java und passe Zugriff (`public`) und `static` sowie den Parameter an.\n\n### Code Style\n- In `solved(...)` räumst du das Feld innerhalb der Schleife in mehreren Zweigen auf (`clear` im `else` und nochmal beim “nicht gefunden” Fall). Das funktioniert zwar, aber ist unnötig verstreut und macht den Kontrollfluss schwerer nachzuvollziehen; versuch das Zurücksetzen pro Versuch/Iteration klarer zu bündeln.\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" : "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
} ]
}
}