{
"llm" : {
"feedback" : "# Exercise: parking\n\n### Correctness\n- Der geforderte Gesamtumsatz von **156.75 CHF** wird mit deiner aktuellen `FlatRateCard` nicht erreicht, weil `cost()` immer **150** zurückgibt – unabhängig davon, ob überhaupt „ein Monat“ abgerechnet werden soll bzw. wie die Beispiel-Ausführung den Betrag zusammensetzt.\n\n### Suggestion\n- Rechne nach, welche drei Teilbeträge im Beispiel zusammenkommen müssen, damit **156.75** entsteht (Individual + Group + FlatRate). Vergleiche das mit dem, was deine `FlatRateCard.cost()` aktuell beiträgt, und überlege, wann/unter welchen Bedingungen die **150 CHF** überhaupt in die Abrechnung einfliessen sollen (und ob sie ggf. nicht immer anfallen darf).\n\n### Code Style\n- In `ParkingSystem.java` ist `import javax.smartcardio.Card;` unbenutzt und sollte entfernt werden.\n- In `FlatRateCard` ist `totalTime` faktisch ungenutzt (weil `cost()` nicht davon abhängt). Entweder konsequent weglassen oder so gestalten, dass der Zustand einen Zweck erfüllt.\n- In `GroupCard` sollte `HOURLY_RATE` eher eine `private final` Instanzvariable sein (und nach Java-Konvention nicht in ALLCAPS, da es keine Konstante ist).\n\n\n# Exercise: labyrinth\n\n### Correctness\n- In `BacktrackingAlgorithm.navigate` setzt du `direction = 2` mit dem Kommentar „Anpassen je nach Level“ – das widerspricht der Anforderung eines *allgemeinen* Algorithmus, der für alle Labyrinthe funktioniert, ohne pro Level manuell angepasst zu werden.\n- In `LabyrinthApp.main` startest du die Level-Schleife bei `level = 7`; die App soll aber grundsätzlich alle Levels durchlaufen (und für Teilaufgaben ggf. nur durch Ersetzen des Algorithmus die ersten Levels schaffen).\n- Dein Backtracking-Ansatz merkt sich keine besuchten Felder / Kreuzungen; dadurch kann der Algorithmus in Labyrinthen mit Zyklen (Rundwegen) in Endlosschleifen geraten bzw. extrem lange laufen, statt zuverlässig zu terminieren (für „funktioniert für alle diese Labyrinthe“ ist das ein Problem).\n\n### Suggestion\n- Für die Initialrichtung: Nutze statt einer eigenen `direction`-Variable lieber den Zustand, den die `Figure` bereits hat (oder leite alles nur über `turnLeft/turnRight/moveForward` + `path…` ab), sodass kein „je nach Level“ nötig ist.\n- Für „alle Levels spielen“: Schau dir die for-Schleife in `LabyrinthApp` an und überlege, ob dein Startwert wirklich noch dem entspricht, was die App/Übung erwartet.\n- Für robustes Backtracking: Überlege dir, wie du vermeiden kannst, dass du immer wieder in denselben Schleifen landest (z.B. indem du an Kreuzungen/Feldern Informationen speicherst, welche Abzweigungen du schon ausprobiert hast, oder indem du „besuchte Zustände“ aus Position *und* Blickrichtung trackst).\n\n3. Code Style:\n- In `BacktrackingAlgorithm` ist die Variable `direction` redundant, weil die `Figure` ihre Richtung bereits intern führt (`dir()`); zwei Quellen für denselben Zustand machen den Code schwerer wartbar.\n- Die Kommentare zur Richtungs-Codierung (`0=Nord, 1=Ost, ...`) passen nicht zur im gegebenen Code verwendeten Interpretation von `dir` (dort ist `0` nach Osten, dann gegen den Uhrzeigersinn). Das ist verwirrend, auch wenn du `direction` aktuell kaum nutzt.\n- Du hast im Attempt viele vorgegebene Dateien (Figure/Labyrinth/LabyrinthGame/…) 1:1 mitkopiert; das ist für die Abgabe unnötig und erschwert das Review – normalerweise reichen nur deine neuen Algorithmus-Klassen und die Anpassung in `LabyrinthApp`.\n\n\n# Exercise: swissmap\n\n### Correctness\n- In `ModeButton.getInteractiveArea(...)` verwendest du `gui.getMouseX()`/`getMouseY()` als Ursprung des Buttons; damit “wandert” die interaktive Fläche mit der Maus, statt fix an der Button-Position zu liegen, wodurch Hover/Click-Verhalten nicht wie ein Button funktioniert.\n- In `ModeButton.draw(...)` zeichnest du den Button an fixen Koordinaten (z.B. `drawRect(50,20,10,20)`), aber die interaktive Fläche ist an einer ganz anderen Stelle (bei der Maus). Zeichnung und `getInteractiveArea` passen also nicht zusammen.\n- In `ModeButton` ist `map` als `static` deklariert; wenn mehrere Buttons/Maps existieren würden, teilen sie sich diesen Zustand und der Button ist nicht mehr sauber an “seine” Map gebunden (das widerspricht der Idee “Referenz im Objekt speichern” aus der Aufgabenstellung).\n\n### Suggestion\n- Definiere für den Button eine feste Position und feste Größe (Attribute) und verwende exakt diese Werte sowohl beim Zeichnen als auch in `getInteractiveArea(...)`, damit Hover/Click dort auslösen, wo der Button sichtbar ist.\n- Speichere die `SwissMap`-Referenz als normales Instanzfeld (nicht `static`), so wie du es schon über den Konstruktor übergibst; dann arbeitet jeder Button zuverlässig mit der Map, die ihm gegeben wurde.\n\n### Code Style\n- `ModeButton`: Das `if (hovered) ... else ...` zeichnet aktuell in beiden Fällen dasselbe Rechteck; entweder Unterschied sichtbar machen oder die Verzweigung weglassen.\n- `SwissMapApp`: `import ch.trick17.gui.component.Component;` und `import ch.trick17.gui.component.Drawable;` sind nicht nötig (du verwendest `Component` gar nicht, und `Drawable map = new SwissMap();` könnte auch einfach `SwissMap map = new SwissMap();` sein).\n- `InteractiveArea`-Klasse ist unbenutzt und `contains(...)` gibt immer `false` zurück; wenn du bei `Rectangle` bleibst, kannst du diese Klasse entfernen, sonst müsste sie korrekt implementiert und auch verwendet werden.\n- In `City/Lake/Mountain`: Du berechnest `coord.toGuiX(gui)`/`toGuiY(gui)` mehrfach pro `draw`/`getInteractiveArea`; das lässt sich lesbarer machen, indem du die Werte einmal in Variablen speicherst.\n\n\n# Exercise: visualizer\n\n### Correctness\n- `Country` implementiert das `DataPoint`-Interface nicht, damit kannst du die Länder wie in der Aufgabe verlangt nicht im `Visualizer` darstellen.\n- `Processor` implementiert das `DataPoint`-Interface nicht, damit kannst du die Prozessoren wie in der Aufgabe verlangt nicht im `Visualizer` darstellen.\n- Für den Prozessor-Datensatz fehlt damit auch die geforderte Abbildung: x-Achse als kombinierter Wert aus Jahr+Monat und y-Achse als (vereinfachte) effektive Geschwindigkeit `clockRate * cores` sowie die verlangte logarithmische y-Achse (diese ist aktuell nirgends umgesetzt).\n- In `Movie.getGroup()` gruppierst du nach Jahrzehnt; laut Screenshots/Legende ist die Gruppierung bei Filmen typischerweise etwas wie Genre/Studio/… (also eine Eigenschaft, die im Datensatz enthalten ist bzw. zur Legendendarstellung passt). Mit Jahrzehnten weicht die Darstellung von der Spezifikation “Rest aus Screenshots entnehmen” ab.\n\n### Suggestion\n- Schau dir an, welche beiden Achsenwerte für Länder gefordert sind (Literacy vs. GDP per Capita) und mappe diese auf deine `getX()`/`getY()`-Methoden, dann lass `Country` das Interface implementieren.\n- Für Prozessoren: überlege dir eine Zahlendarstellung für “Jahr+Monat als ein Wert” (z. B. als fortlaufender Monatsindex oder als `year + month/12.0`) und berechne die effektive Geschwindigkeit aus den vorhandenen Feldern; das ergibt direkt Werte für `getX()` und `getY()`, danach `Processor implements DataPoint`.\n- Die logarithmische y-Achse muss irgendwo in der Visualisierung passieren (nicht in den CSV-Daten): suche in `Visualizer` die Stelle, wo y in Pixel umgerechnet wird, und überlege dort, wie du eine Log-Skalierung einbauen kannst, ohne das Interface zu verbiegen.\n- Für die Film-Gruppierung: vergleiche Legendeneinträge in den Screenshots mit deinem Gruppenkonzept und prüfe, welche Information aus den Filmzeilen (CSV) sich dafür eignet; falls die CSV keine passende Gruppenspalte hat, ist deine Interface-Idee “group kann null sein” zwar okay, aber dann entspricht die Darstellung nicht der gezeigten Legende.\n\n### Code Style\n- In `Visualizer` sind `DUMMY_DOUBLE` und `DUMMY_STRING` jetzt unbenutzt; entferne sie (und die TODO-Kommentare dazu), damit der Code konsistent bleibt.\n- Der Kommentar `// Beispiel` in `Movie.getX()` wirkt wie ein Überbleibsel/Unsicherheit; entweder begründen (warum budget als x) oder entfernen.\n- Javadoc im `DataPoint` ist sehr kurz; die Aufgabe verlangt, das erwartete Verhalten zu dokumentieren—z. B. ob `getGroup()` `null` sein darf, ob `getDetails()` Zeilenumbrüche enthalten soll, und was `getLabel()` genau repräsentiert.\n",
"status" : "SUCCESS"
}
}