{
"llm" : {
"feedback" : "# Exercise: textanalysis\n\n### Correctness\n- In `add`, when a word is seen for the first time you store `0` as its count; that makes every count one too small and can lead to `topWord()` returning `\"\"` when all words occur only once (because no count is ever `> 0`).\n\n### Suggestion\n- Think about what the stored number in the map is supposed to represent (e.g., “how many times have I seen this word so far?”). Then check what value you should assign on the very first occurrence so that after one `add(word)` the count reflects one occurrence.\n\n### Code Style\n- Make `wordList` `private` to encapsulate the internal data structure.\n- Use Java naming conventions: local variables should start with lowercase (e.g., `topWord` instead of `TopWord`), and consider a clearer name than `startvalue` (it’s the current maximum count).\n\n\n# Exercise: lotto\n\n### Correctness\n- `buyTicket` muss bei ungültigen Zahlen bzw. falscher Anzahl Zahlen eine `IllegalArgumentException` werfen; dein Code gibt stattdessen `null` zurück.\n- `buyTicket` muss nach einer Ziehung das Kaufen verhindern und eine `IllegalStateException` werfen; in deiner Implementierung fehlt die Zustandsprüfung (`hasDrawn()` / `winningNumbers != null`).\n- In der Zahlenprüfung in `buyTicket` ist der Bereichstest nicht korrekt: du prüfst `< maxNumber` statt `<= maxNumber` (die Höchstzahl muss erlaubt sein).\n- Die Schleifenlogik in `buyTicket` validiert nicht zuverlässig alle 6 Zahlen (Off-by-one/Abbruchbedingung: `nextNumberCounter == 5` bedeutet nicht “6 Zahlen gültig”).\n- `buyTicket` prüft nicht, ob alle 6 Zahlen verschieden sind (Duplikate müssten als ungültig gelten).\n- `draw()` ist nicht implementiert: es fehlen das Ziehen von 6 zufälligen *verschiedenen* Gewinnzahlen, das Speichern dieser Zahlen, und die `IllegalStateException`, falls schon gezogen wurde.\n- In `Ticket` fehlen: Speichern der Ticketzahlen, `getNumbers()`, `getCorrectNumbers()` (inkl. `IllegalStateException` vor Ziehung), und `getPrize()` (inkl. korrekter Gewinnlogik und `IllegalStateException` vor Ziehung).\n\n### Suggestion\n- Überlege dir bei `buyTicket`: Welche Fälle müssen *immer* mit `throw new IllegalArgumentException(...)` enden, statt “irgendwie kein Ticket erstellen”? (Anzahl != 6, Zahl ausserhalb 1..maxNumber, Duplikate).\n- Baue als ersten Schritt in `buyTicket` eine klare Zustandsprüfung ein: “darf ich überhaupt noch Lose verkaufen, wenn schon gezogen wurde?”\n- Für die Prüfung “6 verschiedene Zahlen” ist eine Collection hilfreich, die automatisch Duplikate entfernt; vergleiche danach die Größe mit 6.\n- Achte bei der Bereichsprüfung auf die Randwerte: sowohl 1 als auch `maxNumber` müssen gültig sein.\n- Für `draw()`: nutze eine Collection, in die du wiederholt Zufallszahlen einfügst, bis die Collection genau 6 Elemente enthält; prüfe vorher, ob schon gezogen wurde.\n- Für `Ticket`: speichere die Zahlen in einer passenden Collection als Instanzvariable und gib in `getNumbers()` eine Kopie zurück.\n- Für `getCorrectNumbers()`: besorge dir die Gewinnzahlen über die `lottery`-Referenz (damit kommt automatisch die `IllegalStateException` vor der Ziehung) und bilde daraus die Schnittmenge mit den Ticketzahlen, ohne die internen Datenstrukturen dauerhaft zu verändern.\n- Für `getPrize()`: leite den Gewinn nur aus der Anzahl korrekter Zahlen ab (0 → 0; 1 → Basis; jede weitere Zahl → mal Multiplikator), statt einzelne Fälle hart zu codieren.\n\n### Code Style\n- `buyTicket` enthält unnötig komplizierte Zähl-/While-Logik; das macht Off-by-one-Fehler wahrscheinlich und ist schwer lesbar im Vergleich zu einer direkten Iteration über die 6 Zahlen.\n- `buyTicket` hat noch ein `// TODO` obwohl Code darunter steht; entferne oder ersetze TODOs, wenn du den Teil implementiert hast.\n- `return null` bei Fehlern ist hier unpassend und führt später zu schwer auffindbaren `NullPointerException`s; Exceptions machen das Verhalten klarer (und sind in den Anforderungen explizit erwähnt).\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
} ]
}
}