{
"llm" : {
"feedback" : "# Exercise: textanalysis\n\n### Correctness\n- In `add`, neue Wörter werden mit `0` gespeichert; dadurch ist die gezählte Häufigkeit für jedes Wort um 1 zu klein (das erste Auftreten zählt nicht als 1).\n\n### Suggestion\n- Überlege, welchen Zählerstand ein Wort direkt nach dem *ersten* `add(word)` haben sollte, und setze den Initialwert entsprechend (bzw. nutze ein Muster wie “aktuellen Wert holen, falls nicht vorhanden mit einem Startwert arbeiten, dann erhöhen”).\n\n### Code Style\n- `wordList` sollte als `private` gekapselt werden, damit die interne Datenstruktur nicht von außen verändert werden kann.\n- Java-Konventionen: Variablennamen üblicherweise `camelCase` beginnen klein (z.B. `topWord` statt `TopWord`, `startValue` statt `startvalue`).\n\n\n# Exercise: lotto\n\n### Correctness\n- In `Lottery.buyTicket`: Du akzeptierst doppelte Zahlen, obwohl 6 **verschiedene** Zahlen gefordert sind (du prüfst `differentNumbers.size() == 6`, aber kombinierst das mit einer falschen Zählbedingung, sodass Duplikate durchrutschen können).\n- In `Lottery.buyTicket`: Die Bedingung `nextNumberCounter == 5` ist falsch; damit wird selbst bei 6 gültigen Zahlen kein Ticket erstellt (oder du landest am Ende bei `return null`), obwohl ein gültiges Ticket erwartet wird.\n- In `Lottery.buyTicket`: Wenn die Eingaben ungültig sind (z. B. Duplikate oder falsche Anzahl), soll eine `IllegalArgumentException` geworfen werden; aktuell gibst du in manchen Fällen `null` zurück.\n- `Lottery.draw` ist nicht implementiert; es werden keine 6 zufälligen, verschiedenen Gewinnzahlen gezogen und gespeichert.\n- In `Ticket`: Konstruktor speichert die Ticket-Zahlen nicht in einer Instanzvariable; `getNumbers`, `getCorrectNumbers`, `getPrize` sind nicht implementiert, damit kann das Ticket weder Zahlen noch korrekte Zahlen noch Gewinn liefern.\n\n### Suggestion\n- Für die “6 verschiedene Zahlen”-Prüfung: Überlege dir eine Logik, bei der du **direkt** nach dem Einfügen in ein `Set` erkennst, ob ein Duplicate dabei war (und dann sofort abbrichst), statt erst am Ende mit separatem Counter zu arbeiten.\n- Zur `nextNumberCounter == 5`-Stelle: Kontrolliere, was dein Counter eigentlich zählt und welcher Wert bei genau 6 gültigen Zahlen herauskommen muss; vergleiche das mit deiner if-Bedingung.\n- Statt am Ende `null` zurückzugeben: Überlege, in welchen Fällen du laut Aufgabenstellung **immer** eine Exception werfen musst, damit Tests nicht mit `null` weiterlaufen.\n- Für `draw`: Nimm eine Collection, die automatisch nur unterschiedliche Werte enthält, und fülle sie in einer Schleife mit Zufallszahlen, bis genau 6 drin sind; denke auch an den Zustand “draw schon passiert” → Exception.\n- Für `Ticket`: Lege eine Instanzvariable an, in der du die 6 Zahlen speicherst (so, dass sie später nicht “von außen” veränderbar sind). `getCorrectNumbers` kann dann die Schnittmenge aus Ticket-Zahlen und Gewinnzahlen bestimmen (aber erst nach dem Draw). `getPrize` sollte nur von der Anzahl korrekter Zahlen abhängen und den “*5, dann *20 pro weitere Zahl*”-Mechanismus abbilden.\n\n### Code Style\n- In `Lottery.buyTicket` ist die Logik unnötig kompliziert (Counter + Set + spätere Kombi-Bedingung). Eine klarere Struktur (frühe Checks, früh Exceptions werfen) macht den Code leichter korrekt zu halten.\n- `return null` am Ende von `buyTicket` ist riskant und erschwert Debugging/Tests; besser ist es, ungültige Fälle konsequent über Exceptions zu behandeln (wie es die Methode dokumentiert).\n- Benennung: `nextNumberCounter` sagt nicht klar, *was* gezählt wird (gültige Zahlen? eingelesene Zahlen?). Klarere Namen reduzieren Logikfehler wie den `== 5`-Bug.\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
} ]
}
}