{
"llm" : {
"feedback" : "# Exercise: fractal\n\n### Correctness\n- In `Triangles.drawRec` (Rekursionsfall) benutzt du für den unteren Mittelpunkt die y‑Koordinate `y1` fest (z.B. `(x1 + x2) / 2, y1`). Das ist nur korrekt, wenn die Basis wirklich exakt horizontal ist; die Methode soll aber allgemein mit beliebigen Dreieckskoordinaten funktionieren (wie die Signatur nahelegt).\n- In `Triangles.drawRec` (Basisfall) zeichnest du die dritte Kante als `g.drawLine(x0, y0, x2, y2)` statt `g.drawLine(x2, y2, x0, y0)`. Das ist zwar geometrisch dieselbe Linie, aber wenn die Aufgabe erwartet, dass “alle drei Seiten” konsistent aus den drei Punktpaaren gezeichnet werden, kann das als Abweichung gewertet werden.\n\n### Suggestion\n- Berechne für die Rekursion bei den Dreiecken die drei Mittelpunkte jeweils als Durchschnitt der beiden Endpunkte der entsprechenden Kante: für **jede** der drei Kanten sowohl x **und** y mitteln (nicht y von einem Eckpunkt “übernehmen”). Dann kannst du daraus die 3 Teildreiecke zusammensetzen.\n- Wenn du ganz sicher gehen willst beim Basisfall: zeichne die drei Kanten immer explizit als (0→1), (1→2), (2→0), damit klar ist, dass wirklich das Dreieck aus genau diesen drei Punkten gezeichnet wird.\n\n### Code Style\n- In `Triangles.drawRec` wiederholst du sehr viele Ausdrücke wie `(x1 + x0) / 2` mehrfach; speichere die Mittelpunkte in gut benannten Variablen (lesbarer und weniger fehleranfällig).\n- In `Boxes.drawRec` sind die Kommentare (rechts oben/rechts unten/…) ok, aber die Reihenfolge der Aufrufe wirkt gemischt; eine konsistente Reihenfolge (z.B. im Uhrzeigersinn) macht das Nachvollziehen leichter.\n\n\n# Exercise: knapsack\n\n### Correctness\n- In `KnapsackN`, die `main`-Methode hat nicht die übliche Signatur `main(String[] args)`. Falls die Tests/Runner genau diese Signatur erwarten, wird das Programm nicht gestartet.\n\n### Suggestion\n- Schau dir an, wie die Applikation in den Übungen/Tests gestartet wird: Wird explizit `public static void main(String[] args)` erwartet? Falls ja, passe nur die Signatur an (Inhalt kann gleich bleiben).\n\n### Code Style\n- Der `Logger log` sowie die `slf4j`-Imports werden nicht verwendet; das erzeugt unnötige Abhängigkeiten/Warnungen. Entfernen oder tatsächlich nutzen.\n- In `pack` könntest du das “Zurücksetzen” nach der Schleife kompakter ausdrücken (aktuell zweimalige Schleife zum Entladen). Das ist nicht falsch, aber schwerer zu lesen als nötig.\n\n\n# Exercise: queens\n\n### Correctness\n- `QueensSolver.solve()` ist nicht implementiert und liefert immer `false`, damit wird nie eine Lösung auf dem Board platziert.\n- `QueensSolver.count()` ist nicht implementiert und liefert immer `0`, damit wird die Anzahl Lösungen nie korrekt gezählt.\n- In `Queens.java` ist `main()` nicht `static` und hat keine `String[] args`-Parameter; so wird das Programm in Java typischerweise nicht als Einstiegspunkt erkannt/ausgeführt.\n\n### Suggestion\n- Für `solve()`: Überlege dir eine rekursive Backtracking-Funktion, die zeilenweise arbeitet: in jeder Zeile alle Spalten ausprobieren, nur setzen wenn `checkPlacement(...)` true ist, und bei Misserfolg die Dame wieder entfernen (Backtracking).\n- Für `count()`: Ähnlich wie bei `solve()`, aber statt beim ersten Fund abzubrechen, alle möglichen Platzierungen weiterverfolgen und die Anzahl der vollständigen Platzierungen aufsummieren.\n- Für `main()`: Prüfe die Java-Signatur für den Programmeinstiegspunkt und passe die Methodendeklaration entsprechend an, damit das Starten über die JVM/IDE klappt.\n\n### Code Style\n- `getBoard()` gibt einmal `this.board` zurück; konsistent könntest du überall entweder mit oder ohne `this` arbeiten (kein Muss, aber einheitlicher).\n- In `QueensSolver` sind noch `// TODO implement`-Marker enthalten; sobald du implementierst, entferne sie bzw. ersetze sie durch passende Kommentare.\n\n\n# Exercise: sudoku\n\n### Correctness\n- `SudokuSolverImpl.solved(...)` ist nicht implementiert und liefert immer `false`, dadurch kann kein Sudoku gelöst werden.\n- `SudokuSolverImpl.nofSolutions(...)` ist nicht implementiert und liefert immer `0`, dadurch werden Lösungen nie korrekt gezählt.\n- `Sudoku.main()` hat nicht die korrekte Java-Signatur für den Programmeinstieg (muss `public static void main(String[] args)` sein), sonst startet die Applikation typischerweise nicht.\n\n### Suggestion\n- Für `solved`: Überlege dir eine rekursive Backtracking-Struktur mit einer Abbruchbedingung “alle Felder abgearbeitet”. Für jedes Feld: wenn schon belegt → nächstes Feld; sonst Werte 1..size ausprobieren, nach jedem Setzen mit `checker.oneOK(model, i, j)` prüfen und bei Fehlschlag wieder zurücksetzen (`clear`).\n- Für `nofSolutions`: Gleiche Traversierung wie beim Lösen, aber statt “erste Lösung reicht” musst du die Anzahl aufsummieren. Achte darauf, dass du bei vielen Lösungen frühzeitig stoppst (Parameter `max` / `MAX`) und beim Zurückkehren das Feld wieder leerst.\n- Für `main`: Vergleiche die erwartete Signatur mit einer normalen Java-Startmethode und passe Sichtbarkeit (`public`) und `static` sowie das `String[] args`-Argument an.\n\n### Code Style\n- Die `// TODO implement`-Stellen sind noch drin; sobald du implementierst, entferne die TODOs bzw. ersetze sie durch sinnvolle Kommentare (z.B. “Abbruchbedingung”, “Versuche Werte 1..9”, “Backtracking”).\n- In `Sudoku.java` fehlt der `public static`-Modifier bei `main`; selbst wenn es “nur” ein kleiner Unterschied ist, ist das in Java konventionell und entscheidend für die Ausführbarkeit.\n",
"status" : "SUCCESS"
},
"unitTest" : {
"tests" : [ {
"name" : "allZero()",
"status" : "FAILED",
"message" : "Error occurred in server thread; nested exception is: \n\tjava.lang.NoClassDefFoundError: org/slf4j/LoggerFactory"
}, {
"name" : "uselessStuff()",
"status" : "FAILED",
"message" : "Error occurred in server thread; nested exception is: \n\tjava.lang.NoClassDefFoundError: org/slf4j/LoggerFactory"
}, {
"name" : "random1()",
"status" : "FAILED",
"message" : "Error occurred in server thread; nested exception is: \n\tjava.lang.NoClassDefFoundError: org/slf4j/LoggerFactory"
}, {
"name" : "random2()",
"status" : "FAILED",
"message" : "Error occurred in server thread; nested exception is: \n\tjava.lang.NoClassDefFoundError: org/slf4j/LoggerFactory"
}, {
"name" : "random3()",
"status" : "FAILED",
"message" : "Error occurred in server thread; nested exception is: \n\tjava.lang.NoClassDefFoundError: org/slf4j/LoggerFactory"
}, {
"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
} ]
}
}