{
"llm" : {
"feedback" : "# Exercise: textanalysis\n\n### Correctness\n- `topWord()` wirft bei leerem Counter eine `IllegalArgumentException`; die Aufgabe verlangt aber, dass `topWord()` das häufigste Wort zurückgibt (und macht keine Vorgabe, dass hier eine Exception geworfen werden soll). Außerdem wird diese Prüfung an der falschen Stelle durchgeführt (sie hängt nicht vom aktuellen Entry ab).\n- In `topWord()` ist die Leerfall-Prüfung innerhalb der Schleife platziert; wenn die Map leer ist, läuft die Schleife gar nicht und die Exception wird nie geworfen (es wird dann `\"\"` zurückgegeben), was zu inkonsistentem Verhalten führt.\n\n### Suggestion\n- Überlege dir, welches Verhalten für einen leeren `WordCounter` sinnvoll ist (z.B. leeren String zurückgeben) und implementiere dieses Verhalten konsistent, ohne eine Prüfung in der Schleife zu verstecken.\n- Wenn du eine Leerfall-Prüfung machen willst, dann prüfe die Datenstruktur bevor du über sie iterierst; ansonsten kann der Fall “leer” nie im Loop erkannt werden.\n\n### Code Style\n- Verwende für `maxCount` statt `Integer.MIN_VALUE` lieber einen Wert, der zur Logik passt (z.B. 0), wenn du ohnehin nur Zählwerte speicherst.\n- Die Exception in `topWord()` ist ohne Message; falls du überhaupt eine werfen willst, wäre eine aussagekräftige Fehlermeldung hilfreich.\n\n\n# Exercise: lotto\n\n### Correctness\n- `Lottery.buyTicket` ist nicht implementiert und gibt immer `null` zurück; damit werden keine Tickets erstellt, IDs nicht vergeben, und die Validierungsregeln (genau 6 Zahlen, Bereich 1..maxNumber, alle verschieden, sowie “nicht mehr kaufbar nach draw”) werden nicht erfüllt.\n- `Lottery.draw` ist nicht implementiert; dadurch werden nie 6 zufällige, verschiedene Gewinnzahlen gezogen/gespeichert und der Zustand “nach der Ziehung” wird nicht erreicht.\n- `Ticket` speichert die übergebenen Zahlen nicht; damit können `getNumbers`, `getCorrectNumbers` und `getPrize` nicht korrekt arbeiten.\n- `Ticket.getNumbers` ist nicht implementiert und liefert `null` statt der 6 Ticketzahlen (als Kopie).\n- `Ticket.getCorrectNumbers` ist nicht implementiert und liefert `null`; außerdem fehlt die geforderte Zustandsprüfung (vor draw muss eine `IllegalStateException` kommen).\n- `Ticket.getPrize` ist nicht implementiert und liefert immer `0`; außerdem fehlt die geforderte Zustandsprüfung (vor draw `IllegalStateException`) und die Gewinnberechnung gemäss Regeln (0→0, 1→5, jede weitere Zahl *20).\n\n### Suggestion\n- Implementiere `buyTicket` so, dass du zuerst den Zustand prüfst (ob bereits gezogen wurde) und danach die Eingabe validierst: Anzahl Elemente, Wertebereich, und “alle verschieden” (eine Set-Struktur hilft hier).\n- Den Ticket-Zähler/ID kannst du über `soldTickets` lösen: erst nach erfolgreicher Validierung erhöhen und diese Zahl als ID fürs neue Ticket verwenden.\n- Für `draw` brauchst du eine Datenstruktur, die automatisch Duplikate verhindert, und du füllst sie mit Zufallszahlen, bis genau 6 verschiedene drin sind; danach muss `hasDrawn()` wahr sein.\n- Lege im `Ticket` eine Instanzvariable für die Zahlen an (z.B. Collection oder Array), initialisiere sie im Konstruktor aus dem Parameter `numbers`, und gib in `getNumbers` immer eine Kopie zurück, nicht die interne Referenz.\n- Für `getCorrectNumbers`: hol dir die Gewinnzahlen über die Lottery-Referenz; wenn diese Methode vor der Ziehung aufgerufen wird, soll genau dadurch die geforderte `IllegalStateException` entstehen (oder du prüfst explizit vorher).\n- Für `getPrize`: nutze die Anzahl korrekter Zahlen als Basis und bilde daraus den Betrag (bei 1 Richtigen Startwert 5, dann für jede weitere richtige Zahl den Betrag mit 20 multiplizieren).\n\n### Code Style\n- In `Lottery` und `Ticket` sind noch mehrere `// TODO`-Stellen und Platzhalter-Returns (`null`, `0`), die das Verhalten verschleiern; sobald du implementierst, sollten diese entfernt werden.\n- Achte darauf, dass die Rückgabetypen/Dokumentation konsistent sind: In den Kommentaren steht teils “ArrayList”, die Signatur gibt aber `Integer[]` zurück; halte dich an die Signatur und formuliere Kommentare ggf. passend.\n",
"status" : "SUCCESS"
},
"unitTest" : {
"tests" : [ {
"name" : "buyTicketIds()",
"status" : "FAILED",
"message" : null
}, {
"name" : "buyTicketsIllegalNumbers()",
"status" : "PASSED",
"message" : null
}, {
"name" : "draw()",
"status" : "FAILED",
"message" : "expected: <true> but was: <false>"
}, {
"name" : "drawNoDuplicates()",
"status" : "FAILED",
"message" : null
}, {
"name" : "drawRandom()",
"status" : "FAILED",
"message" : null
}, {
"name" : "drawAlreadyDrawn()",
"status" : "FAILED",
"message" : "Expected java.lang.Exception to be thrown, but nothing was thrown."
}, {
"name" : "getWinningNumbersNotDrawnYet()",
"status" : "FAILED",
"message" : null
}, {
"name" : "buyTicketAlreadyDrawn()",
"status" : "FAILED",
"message" : null
} ]
}
}