{
"llm" : {
"feedback" : "# Exercise: parking\n\n### Correctness\n- `ParkingCard` ist nicht `public`, obwohl es laut Aufgabenstellung als `public interface ParkingCard` definiert sein soll (sonst ist es ausserhalb des Packages nicht verwendbar und entspricht nicht der Vorgabe).\n- `FlatRateCard.cost()` gibt immer `150` zurück, unabhängig davon, ob überhaupt geparkt wurde; damit wird auch für unbenutzte Karten Umsatz gezählt (das kann dazu führen, dass der geforderte Gesamtumsatz nicht stimmt).\n- `FlatRateCard.reset()` setzt keinen Zustand zurück; falls du (jetzt oder später) doch Park-Zeit/Abrechnungsperiode modellierst, wird das mit `calculateRevenueAndReset()` nicht korrekt zusammenspielen.\n\n### Suggestion\n- Prüfe die Sichtbarkeit des Interfaces: Vergleiche deine `ParkingCard`-Deklaration mit der geforderten Signatur in der Aufgabenbeschreibung und überlege, welches Keyword dort zusätzlich steht und warum.\n- Überlege bei der `FlatRateCard`, was “fixe Monatskosten” im Kontext von `calculateRevenueAndReset()` bedeutet: Soll die Monatsgebühr jedes Mal anfallen, wenn `calculateRevenueAndReset()` aufgerufen wird, oder nur dann, wenn die Karte im aktuellen Abrechnungszeitraum aktiv/benutzt war? Passe die Logik so an, dass du auf den Ziel-Umsatz (156.75 CHF) kommst.\n- Falls du für die `FlatRateCard` entscheidest, dass die Gebühr nur unter bestimmten Bedingungen anfällt: Du brauchst dafür vermutlich einen internen Zustand (z.B. “wurde im Zeitraum benutzt?”), der in `park(...)` gesetzt und in `reset()` wieder gelöscht wird.\n\n### Code Style\n- Konstruktor von `GroupCard` ist package-private (`GroupCard(int people)`); setze ihn auf `public`, damit die Klasse konsistent von überall instanziiert werden kann (wie in deinem Example gezeigt/gedacht).\n- In `FlatRateCard` sind leere Methodenblöcke (`park`, `reset`) ohne Kommentar irritierend; wenn das absichtlich “no-op” ist, kurz begründen (Kommentar) oder einen minimalen Zustand führen, damit das Verhalten nachvollziehbar bleibt.\n- Einheitliche Formatierung: Doppelte Leerzeichen wie `implements ParkingCard` vermeiden.\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 `City`, `Lake`, `Mountain` und `ModeButton` verwendest du `new Rectangle(...)`, aber du importierst nirgends deine eigene `ch.fhnw.prog1.exercise.swissmap.Rectangle`-Klasse und es gibt auch kein `ch.trick17.gui.component.Rectangle`-Import in diesen Dateien. So wie es jetzt steht, wird das sehr wahrscheinlich nicht kompilieren („Rectangle cannot be resolved“).\n- `SwissMapApp`: Deine `main`-Methode hat die Signatur `void main()`. Wenn der Autograder/Startmechanismus eine normale Java-Entry-Point-Signatur erwartet, startet das Programm so evtl. nicht automatisch.\n\n### Suggestion\n- Prüfe, woher `Rectangle` kommen soll: Entweder musst du deine eigene `Rectangle`-Klasse explizit importieren/qualifizieren (Package beachten) oder die im Auftrag erwähnte vordefinierte `Rectangle` aus der GUI-Bibliothek verwenden (dann den richtigen Import nutzen und deine eigene Klasse wäre nicht nötig).\n- Schau in den Aufgaben-/Projektvorgaben nach, welche `main`-Signatur erwartet wird (typisch ist eine `public static void main(String[] args)`-Methode). Passe die Signatur so an, dass die App sicher gestartet werden kann.\n\n### Code Style\n- `mouseOver` ist in mehreren Klassen package-private (kein `private`). Mach es lieber `private`, damit der Zustand wirklich gekapselt ist.\n- In `ModeButton` ist der Konstruktor package-private (`ModeButton(...)` ohne `public`). Wenn die Klasse von ausserhalb des Packages erzeugt werden soll, könnte das unnötig einschränken.\n- Du hast in `City/Lake/Mountain` jeweils `Shape` importiert, aber nicht wirklich als Typ ausserhalb der Methodensignatur genutzt; das ist zwar ok, aber achte generell darauf, Imports schlank zu halten (v.a. wenn du am Ende doch die GUI-`Rectangle` verwendest).\n- In `ModeButton.onRightClick` leitest du auf `onLeftClick` weiter. Wenn rechtsklick wirklich „egal“ sein soll, wäre eine leere Implementierung klarer (weniger überraschendes Verhalten).\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"
}
}