{
"llm" : {
"feedback" : "# Exercise: parking\n\n### Correctness\n- In `FlatRateCard` wird in `cost()` immer 150 CHF zurückgegeben, unabhängig davon, ob überhaupt geparkt wurde; falls „Monatskosten“ nur dann zählen sollen, wenn die Karte im Abrechnungszeitraum verwendet wurde, wäre das so nicht erfüllt.\n- `FlatRateCard` zählt zwar `totalTime`, verwendet diese Information aber nirgends für die Kostenberechnung; je nach Aufgabenverständnis (Abo deckt Parken ab, aber Monatsgebühr nur einmal pro Abrechnung) kann das zu falschem Gesamtumsatz führen.\n\n### Suggestion\n- Überlege dir beim Abo genau, wann die 150 CHF in den Umsatz einfliessen sollen: pro `calculateRevenueAndReset()`-Abrechnungsperiode immer, oder nur wenn in dieser Periode mindestens einmal geparkt wurde? Prüfe das mit dem erwarteten Gesamtumsatz von **156.75 CHF** als Plausibilitätscheck.\n- Falls die Monatsgebühr nur einmal pro Periode zählen soll, kann dir ein zusätzlicher Zustand helfen (z.B. „wurde benutzt?“), der bei `park(...)` gesetzt und bei `reset()` wieder gelöscht wird.\n\n### Code Style\n- In `FlatRateCard` ist `totalTime` aktuell faktisch ungenutzt (du speicherst es, aber es beeinflusst nichts). Entweder verwenden oder entfernen, damit der Zweck der Variable klar ist.\n- `HOURLY_RATE` in `GroupCard` ist als Konstante benannt, ist aber instanzabhängig (wird im Konstruktor gesetzt). Besser wäre ein neutralerer Name (z.B. `hourlyRate`), um Missverständnisse zu vermeiden.\n- `IllegalArgumentException()` in `GroupCard` ohne Fehlermeldung macht Debugging unnötig schwer; eine kurze Message (z.B. mit dem übergebenen `personCount`) wäre hilfreicher.\n\n\n# Exercise: labyrinth\n\nLlm Evaluation ist für diese Aufgabe deaktiviert. Entferne die .llmignore Datei vom Package der Aufgabe.\n\n\n# Exercise: swissmap\n\n### Correctness\n- In `SwissMapApp` ist die `main`-Methode als `void main()` deklariert; so wird sie von Java nicht als Programmeinstieg erkannt (es fehlt die übliche Signatur).\n- In `Coordinate.toGuiX/toGuiY` greifst du auf `SwissMap.TOP_LEFT.east` bzw. `.north` zu, aber `east` und `north` sind in `Coordinate` `private` → das kompiliert so nicht.\n- In `ModeButton.java` liegt `enum Mode` als eigener Top-Level-Typ in derselben Datei; falls `ModeButton` `public` ist, führt das je nach Vorgaben/Build-Regeln schnell zu Struktur-/Dateinamensproblemen (oft wird erwartet: eine öffentliche Klasse pro Datei bzw. Enum separat/als nested type).\n- Bei `gui.addComponents(cities/lakes/mountains)`: Falls die `Gui`-Bibliothek diese Methode nicht anbietet (in vielen Aufgaben gibt es nur `addComponent`), kompiliert das nicht und die Objekte werden nicht registriert.\n\n### Suggestion\n- Schau dir die erwartete Java-Startmethode an: Welche Keywords (z. B. `public/static`) und welcher Parameter-Typ werden für `main` verlangt, damit das Programm startet?\n- In `Coordinate`: Du hast bereits Getter (`getEast`, `getNorth`). Überlege, wie du in `toGuiX/toGuiY` an die Werte von `TOP_LEFT` kommst, ohne auf private Felder direkt zuzugreifen.\n- Beim `Mode`-Enum: Prüfe, ob du es lieber als `private enum` **innerhalb** von `ModeButton` oder als eigene Datei ablegen solltest, damit die Projektstruktur sicher zu den Java-Regeln passt.\n- Falls `addComponents` nicht existiert: Überlege, wie du die Elemente aus den Arrays trotzdem einzeln als Komponenten hinzufügen kannst (z. B. mit einer Schleife).\n\n### Code Style\n- Viele Werte sind “Magic Numbers” (`0.3`, `20`, `10`, Button-Position/Grössen). Benenne sie als `static final` Konstanten, dann ist leichter klar, wofür sie stehen und du kannst sie zentral anpassen.\n- `ModeButton` hat Felder wie `WIDTH`/`HEIGHT` als Instanzfelder in Grossbuchstaben; üblich wären hier Konstanten (`static final`) oder camelCase für normale Felder.\n- `setHovered(...)` wird nur intern verwendet und ist ok, aber du könntest direkt in `onMouseEnter/onMouseExit` setzen; aktuell ist es etwas unnötig indirekt.\n- In `Mountain.draw` verwendest du `drawImage(...)`, bei `Lake/City` `drawImageCentered(...)`. Einheitlich bleiben (und bewusst entscheiden), damit Positionierung/Hitbox leichter zusammenpassen.\n\n\n# Exercise: visualizer\n\nLlm Evaluation ist für diese Aufgabe deaktiviert. Entferne die .llmignore Datei vom Package der Aufgabe.\n",
"status" : "SUCCESS"
}
}