{
"llm" : {
"feedback" : "# Exercise: address\n\n### Correctness\n\n\n### Suggestion\n\n\n### Code Style\n- In den `throw new IllegalArgumentException()`-Stellen wäre eine aussagekräftige Fehlermeldung hilfreich (z.B. welcher Parameter ungültig war), damit man beim Debuggen/Benutzen der Klasse schneller versteht, was schiefgelaufen ist.\n- In `format()` kannst du bei den Platzhaltern konsistent bleiben: Für `int` ist `%d` üblicher als `%s` (auch wenn `%s` hier durch Autoboxing/Conversion funktioniert).\n\n\n# Exercise: timespan\n\n### Correctness\n- Deine Precondition in `add` erlaubt z.B. `hours = 1, minutes = -120` (Summe = -60) zwar korrekt abzulehnen, aber auch Fälle wie `hours = 2, minutes = -30` (Summe = 90) würden **nicht** abgelehnt, obwohl du damit Minuten “negativ” übergeben würdest; die Aufgabe verlangt sinnvolle Preconditions (Client-Sicht), d.h. negative Eingaben sollen nicht einfach durch positive Stunden “kompensiert” werden.\n- `TimeSpan(int hours, int minutes)` setzt keine eigene Precondition für “Zeitspanne nie negativ”, sondern verlässt sich nur auf `add`; dadurch gilt derselbe Effekt wie oben: bestimmte negative Eingaben könnten durch die Summenprüfung “durchrutschen”, obwohl sie gegen die intendierte Konstruktor-Precondition verstossen.\n- Die Invariante “Zeitspannen können nur verlängert werden, nicht verkürzt” ist bei dir nur indirekt über die Summenprüfung abgedeckt; wenn du die Eingaben einzeln nicht auf Negativität prüfst, kann Client-Code weiterhin mit negativen Minuten/Stunden arbeiten und so fachlich “verkürzende” Aufrufe formulieren (auch wenn die Summe evtl. positiv bleibt).\n\n### Suggestion\n- Überlege bei den Preconditions: Was bedeutet “nicht negativ” aus Client-Sicht für die Parameter `hours` und `minutes` jeweils? Formuliere die Checks so, dass negative Werte nicht durch andere Parameter “ausgeglichen” werden können.\n- Prüfe im Konstruktor und in `add` dieselben fachlichen Bedingungen (oder leite sie sauber voneinander ab), damit die Invariante `totalMinutes ≥ 0` wirklich immer gilt – egal ob Objekt neu erstellt oder verändert wird.\n- Wenn du “nur verlängern” wirklich klar ausdrücken willst: orientiere die Bedingung daran, ob die Änderung die Zeitspanne verkleinern könnte, und verhindere genau solche Aufrufe per Exception.\n\n### Code Style\n- `throw new IllegalArgumentException();` ohne Nachricht ist wenig hilfreich beim Debuggen; gib eine aussagekräftige Fehlermeldung an (z.B. welche Bedingung verletzt wurde).\n- Der Ausdruck `60 * hours + minutes` kommt mehrfach vor; eine kleine Hilfsmethode oder eine lokale Variable würde die Lesbarkeit verbessern und verhindert, dass sich Logik bei Änderungen verstreut.\n\n\n# Exercise: asteroids\n\nLlm Evaluation ist für diese Aufgabe deaktiviert. Entferne die .llmignore Datei vom Package der Aufgabe.\n\n\n# Exercise: smarthome\n\n### Correctness\n- In `randomize()` erzeugst du die zufällige Helligkeit mit `random.nextDouble(0.5, 1.1)`: damit können Werte > 1.0 entstehen, gefordert ist aber ein Wert zwischen **0.5 und 1.0**.\n- `randomize()` verwendet `Random#nextDouble(min, max)`. Diese Überladung gibt es erst in neueren Java-Versionen; je nach Übungsumgebung kann das zu einem **Compile-Error** führen.\n\n### Suggestion\n- Achte bei der Zufallshelligkeit darauf, dass der obere Rand wirklich bei 1.0 liegt (und nicht darüber). Überlege dir, wie du aus einer Zufallszahl in \\[0.0, 1.0) durch Skalierung/Verschiebung genau den Bereich \\[0.5, 1.0) machst.\n- Falls deine Java-Version `nextDouble(min, max)` nicht kennt: nutze nur die Variante ohne Parameter und rechne den Zielbereich selbst aus.\n\n### Code Style\n- `private Random random = new Random();` kann (und sollte) `final` sein, da du es nicht neu zuweist: das macht die Absicht klarer.\n- In `findHallway()` setzt du zuerst `hallway = rooms[0]` und suchst dann weiter. Das funktioniert zwar bei der Annahme “Hallway existiert”, aber lesbarer wäre eine Struktur, die klarer ausdrückt, dass du genau den Raum mit dem Namen suchst (ohne “Default”-Zuweisung, die inhaltlich nichts mit “Hallway” zu tun hat).\n- In `nightMode()` und auch sonst hast du mehrfach sehr ähnliche Schleifen (alle Lampen ausschalten; eine Lampe im Raum auswählen und einschalten). Eine kleine Hilfsmethode würde hier Duplikation reduzieren und die Blockstruktur noch klarer machen.\n\n\n# Exercise: bosses\n\nLlm Evaluation ist für diese Aufgabe deaktiviert. Entferne die .llmignore Datei vom Package der Aufgabe.\n",
"status" : "SUCCESS"
}
}