{
"llm" : {
"feedback" : "# Exercise: textanalysis\n\n### Correctness\n- `topWord()` gibt bei leerer Datenstruktur (`add` nie aufgerufen) `null` zurück; die Aufgaben-Vorlage erwartet einen `String` (im Muster ist `\"\"` als Rückgabewert vorgegeben), und `TextAnalyzer` druckt das Ergebnis direkt aus.\n\n### Suggestion\n- Überlege dir, was `topWord()` zurückgeben soll, wenn noch kein Wort hinzugefügt wurde (z.B. ein leerer String oder ein anderer definierter Default), und stelle sicher, dass dieser Fall explizit behandelt wird, bevor du iterierst.\n\n### Code Style\n- Mache `counts` besser `private` (Kapselung) und gib ihm einen sprechenden Namen wie `wordCounts`.\n- In `add()` rufst du `counts.get(word)` mehrfach auf; speichere den Wert einmal in einer lokalen Variable oder nutze eine Map-Operation, die “default wenn nicht vorhanden” ausdrückt.\n- In `topWord()` könntest du statt eines expliziten `Iterator` auch eine enhanced for-loop über `counts.entrySet()` verwenden; das macht den Zugriff auf Wort und Count direkter und vermeidet wiederholte `get()`-Aufrufe.\n\n\n# Exercise: lotto\n\n### Correctness\n- `Lottery.buyTicket` gibt immer `null` zurück, statt ein neues `Ticket` zu erstellen und zurückzugeben.\n- In `Lottery.buyTicket` wird `soldTickets` nicht erhöht und dadurch werden die Ticket-IDs nicht wie gefordert (1, 2, 3, …) vergeben.\n- `Lottery.draw` ist nicht implementiert; es werden keine 6 zufälligen, verschiedenen Gewinnzahlen gezogen und gespeichert.\n- Die Zustandsregeln sind nicht vollständig umgesetzt: Nach der Ziehung darf man keine Tickets mehr kaufen und `draw()` darf nur einmal aufgerufen werden; das ist bei `draw()` aktuell gar nicht geprüft (weil nicht implementiert).\n- In `Ticket` fehlen Konstruktor-Logik zum Speichern der Zahlen sowie die Implementationen von `getNumbers`, `getCorrectNumbers` und `getPrize`.\n\n### Suggestion\n- In `buyTicket`: Überlege dir, was nach den Validierungen passieren muss, damit wirklich ein Ticket entsteht. Du brauchst dafür eine neue ID (basierend auf `soldTickets`) und musst dann ein `Ticket`-Objekt zurückgeben.\n- Achte darauf, dass `soldTickets` genau dann hochgezählt wird, wenn ein Ticket erfolgreich verkauft wurde (nicht schon bei ungültigen Eingaben).\n- Für `draw()`: Du brauchst eine Zufallsquelle und eine Collection, die automatisch Duplikate verhindert. Ziehe so lange Zahlen, bis es genau 6 verschiedene sind, und speichere sie in `winningNumbers`.\n- Für die Zustandsprüfungen: Nutze `winningNumbers == null`/`!= null` konsequent, um zu entscheiden, ob etwas erlaubt ist (z.B. `buyTicket` nur vor der Ziehung, `draw` nur vor der Ziehung).\n- In `Ticket`: Lege eine Instanzvariable für die Ticket-Zahlen an (Collection oder Array) und gib in `getNumbers()` eine Kopie davon zurück.\n- Für `getCorrectNumbers()`: Bestimme die Schnittmenge zwischen Ticketzahlen und Gewinnzahlen der zugehörigen `Lottery` (und denk daran: vor der Ziehung soll eine `IllegalStateException` passieren – das ergibt sich oft automatisch, wenn du über die Lottery die Gewinnzahlen holst).\n- Für `getPrize()`: Rechne nur über die Anzahl korrekter Zahlen; Start bei 0 → 0 CHF, bei 1 → 5 CHF, und dann pro weiterer richtiger Zahl jeweils mal 20.\n\n### Code Style\n- In `buyTicket` hast du sehr viele Kommentarzeilen, die den Code “übersetzen” (z.B. was `for` macht). Besser wäre: kurze, zielgerichtete Kommentare nur dort, wo die Absicht nicht offensichtlich ist.\n- Die doppelte (englische + deutsche) Javadoc-Beschreibung bei `buyTicket` ist redundant; halte dich für Lesbarkeit an eine Version.\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
} ]
}
}