{
"llm" : {
"feedback" : "# Exercise: parking\n\n### Correctness\n- `FlatRateCard.cost()` liefert immer `150`, auch nachdem das System `reset()` aufruft; damit wird der Umsatz bei jedem `calculateRevenueAndReset()` erneut um 150 erhöht, obwohl „resetten“ typischerweise den aktuell abgerechneten Betrag/Zeitraum zurücksetzen soll.\n- In `GroupCard` ist `HOURLY_RATE` als `static` definiert und wird im Konstruktor abhängig von der Personenanzahl gesetzt; dadurch teilen sich alle `GroupCard`-Objekte denselben Tarif, was falsch wird, sobald mehrere Gruppenkarten mit unterschiedlicher Personenzahl registriert sind.\n\n### Suggestion\n- Überlege bei `FlatRateCard`, was „Monatskosten“ im Kontext des bestehenden Systems bedeutet: Soll die Pauschale pro Abrechnungsperiode nur einmal anfallen und danach „verbraucht“ sein? Dann muss sich das Objekt merken, ob (seit dem letzten Reset) schon abgerechnet wurde.\n- Mach den Tarif in `GroupCard` zu einem Wert, der pro Objekt gespeichert wird (und nicht global für alle Instanzen). Prüfe dann gedanklich den Fall: eine `GroupCard(5)` und eine `GroupCard(15)` gleichzeitig im System – beide müssen unabhängig korrekt rechnen.\n\n### Code Style\n- In `GroupCard` sollte `HOURLY_RATE` ein `final` Instanzfeld sein (nachdem du ihn im Konstruktor bestimmt hast), und der Parametername `Personen` besser nach Java-Konvention klein (`personen`) und aussagekräftig.\n- `FlatRateCard.park()`/`reset()` sind leer – das ist zwar kompiliert, aber wirkt „unfertig“; kommentiere zumindest kurz, warum sie leer sind bzw. welche Bedeutung Parkzeit bei der Flatrate hat.\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- Teil c) ist nicht vollständig umgesetzt: `City` und `Lake` implementieren noch nicht `Hoverable`, dadurch kann man bei diesen Objekten die Beschreibung nicht per Mouse-Hover anzeigen lassen.\n- Teil d) fehlt komplett: Es gibt keine Klasse `ModeButton`, und damit auch keine Möglichkeit, per Klick den Satelliten-Modus umzuschalten.\n- In `SwissMapApp` ist die `main`-Methode als `void main()` deklariert; als Einstiegspunkt wird normalerweise `public static void main(String[] args)` erwartet (sonst startet das Programm je nach Umgebung nicht).\n\n### Suggestion\n- Für c): Orientiere dich an deiner `Mountain`-Implementierung: Du brauchst bei `City` und `Lake` ebenfalls einen Hover-Zustand (z.B. `hovered`-Boolean), `onMouseEnter/onMouseExit` zum Umschalten und eine passende `getInteractiveArea(...)` (z.B. `Rectangle`) an der Position des Objekts.\n- Für d): Überlege dir, welche Informationen der Button braucht, um den Modus ändern zu können (Tipp aus Aufgabe: Referenz auf das `SwissMap`-Objekt im Konstruktor speichern) und welche Interfaces er implementieren muss (`Drawable`, `Hoverable`, `Clickable`). Beim Klick dann den aktuellen Modus umschalten statt nur “setzen”.\n- Für `main`: Prüfe, wie deine IDE/der Autograder Programme startet. Falls er eine echte Java-`main` verlangt, musst du Methodensignatur und `static/public` anpassen.\n\n### Code Style\n- In `City.draw(...)` rufst du `getCoord().toGuiX(gui)` und `toGuiY(gui)` mehrfach auf; speichere die Werte in lokale Variablen (`x`, `y`), das macht den Code lesbarer und vermeidet doppelte Rechnungen.\n- In `Lake.draw(...)` ist die Skalierung etwas “magisch” und die `-10` Offsets sind hardcoded; benenne solche Konstanten (Icon-Breite/Höhe bzw. Halbe) ähnlich wie du es bei `Mountain` mit `IMG_W/IMG_H` gemacht hast, dann ist klarer, warum diese Werte existieren.\n- `Mountain`: `hovered` könntest du `private` ohne Initialisierung lassen (Standard ist `false`), und für den Text wäre es lesbarer, wenn du das Format konsistent über `toString()` oder eine eigene Methode erzeugst statt es in `draw` zusammenzubauen.\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"
}
}