{
"llm" : {
"feedback" : "# Exercise: parking\n\n### Correctness\n- `GroupCard` verwendet für den Tarif ein `static` Feld (`HOURLY_RATE`): dadurch teilen sich **alle** GroupCard-Objekte denselben Tarif, und der Konstruktor eines später erstellten Objekts kann den Tarif von bereits registrierten GroupCards verändern (falsche Umsätze, sobald mehrere GroupCards mit unterschiedlichen Personenzahlen existieren).\n- `FlatRateCard.cost()` gibt immer `150` zurück, auch **nachdem** `calculateRevenueAndReset()` bereits einmal gelaufen ist: bei wiederholtem Aufruf der Umsatzberechnung würdest du die 150 CHF pro Abo jedes Mal erneut zählen, obwohl durch `reset()` eigentlich ein “neuer Abrechnungszeitraum” starten soll.\n\n### Suggestion\n- Überlege bei `GroupCard`, welche Daten wirklich **pro Karte** unterschiedlich sein müssen. Alles, was von der Personenzahl abhängt, sollte nicht global für die ganze Klasse gelten, sondern an einem einzelnen Objekt hängen.\n- Überlege bei `FlatRateCard`, was `reset()` semantisch bedeuten soll: Wenn das System nach der Umsatzberechnung “zurückgesetzt” wird, wie verhinderst du, dass ein Flat-Rate-Abo beim nächsten `calculateRevenueAndReset()` ohne neue “Monatsperiode” nochmals 150 CHF beiträgt? (Du könntest z.B. einen internen Zustand speichern, ob die Monatsgebühr für die aktuelle Periode bereits verrechnet wurde.)\n\n### Code Style\n- Verwende in Java übliche Namenskonventionen: Konstruktorparameter wie `Personen` besser klein schreiben (z.B. `personen`), damit es nicht wie ein Typ aussieht.\n- In `GroupCard` ist `HOURLY_RATE` als `static` und veränderlich etwas irreführend benannt (Konstanten sind normalerweise `static final` und ändern nie). Hier passt eher ein normales Instanzfeld (dann auch kein “Konstanten-Name” nötig).\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 in Java nicht als Einstiegspunkt erkannt (üblich ist eine `public static void main(String[] args)`-Signatur), wodurch die App evtl. gar nicht startet.\n- Aufgabe d) fehlt: Es gibt keine Klasse `ModeButton`, die `Drawable`, `Hoverable` und `Clickable` implementiert, und es wird auch kein Button zum `Gui` hinzugefügt, der den Satellitenmodus umschaltet.\n- Für Aufgabe d) fehlt damit auch die geforderte Klick-Logik (`onLeftClick` zum Umschalten via `SwissMap#setSatelliteMode(...)`, `onRightClick` zumindest leer).\n\n### Suggestion\n- Prüfe die erwartete Signatur der Java-Startmethode in dieser Übungsumgebung und passe `main` so an, dass sie wirklich als Programmeinstieg verwendet wird (Achte dabei auf `public/static` und den Parameter).\n- Leite aus den Interfaces ab, welche Methoden `Clickable` zusätzlich verlangt, und überlege dann, welche Referenz dein `ModeButton` speichern muss, um beim Klick den Zustand im `SwissMap`-Objekt ändern zu können.\n- Denk beim Umschalten daran: Du brauchst nicht neu zu zeichnen im Click-Handler, sondern nur den Modus-Zustand ändern; das Neuzeichnen passiert danach automatisch über die Render-Schleife.\n\n### Code Style\n- In `Circle.contains(double v, double v1)` sind die Parameternamen nichtssagend; sprechende Namen (z. B. `x`, `y` oder `mouseX`, `mouseY`) machen die Methode viel lesbarer.\n- In `City` sind `r` und `offset` als Felder konstant, aber in `draw` kopierst du sie erst in lokale Variablen und skalierst diese; das ist ok, aber benenne die lokalen Variablen dann klarer (z. B. `radius`, `border`) und vermeide magische Ausdrücke wie `rad*1/4`.\n- Du verwendest in `Lake` und `Mountain` teils doppelte Skalierungslogik (einmal inline, einmal via Methode). Das könntest du konsistenter kapseln (pro Klasse eine Methode) um Redundanz zu reduzieren.\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"
}
}