{
"llm" : {
"feedback" : "# Exercise: parking\n\nLlm Evaluation ist für diese Aufgabe deaktiviert. Entferne die .llmignore Datei vom Package der Aufgabe.\n\n\n# Exercise: labyrinth\n\n### Correctness\n- Du hast die vorgegebenen `MAPS` in `LabyrinthApp` verändert (andere Anzahl Levels, andere Layouts). Damit löst du nicht mehr die gestellte Aufgabe auf den gelieferten Levels.\n- In deiner Abgabe fehlen die geforderten Klassen `StupidAlgorithm` und `TryStraightFirst` (bzw. sie sind nicht vorhanden im Attempt). Die Aufgabe verlangt explizit diese beiden zusätzlich zu `BacktrackingAlgorithm`.\n- `BacktrackingAlgorithmWithMemory` merkt sich „besuchte Positionen“ nur über `row/col`, ignoriert aber die Blickrichtung (`dir`). Dadurch kann dein Algorithmus in manchen Labyrinthen zu früh abbrechen, obwohl dieselbe Zelle aus einer anderen Richtung noch unerkundete Wege zulässt.\n- In `BacktrackingAlgorithmWithMemory`: Wenn eine Position schon gesehen wurde, `return`st du sofort, ohne dabei den aktuellen Rekursionsschritt sauber „zurückzuspulen“ (Backtracking). Das kann dazu führen, dass du nach dem Rekursionsaufruf nicht zuverlässig am gleichen Ort/mit gleicher Richtung bist, wie du es für die weitere Exploration erwartest.\n- In `BacktrackingAlgorithm` (erste aktive Version): Beim „Zurückgehen“ nach einem fehlgeschlagenen Rekursionsaufruf machst du `moveForward()` statt einer echten Umkehrbewegung. Das ist kein korrektes Backtracking und kann dich in eine falsche Zelle bewegen oder sogar crashen, wenn vorne kein Pfad ist.\n\n### Suggestion\n- Verwende in `LabyrinthApp` wieder exakt die vorgegebenen `MAPS` aus der Vorlage und teste deine Algorithmen darauf, nicht auf vereinfachten eigenen Karten.\n- Implementiere wirklich je eine Klasse `StupidAlgorithm` und `TryStraightFirst` mit dem in der Aufgabenstellung beschriebenen Verhalten, und wähle dann in `main` jeweils den passenden Algorithmus aus (erst Stupid, dann TryStraightFirst, dann Backtracking).\n- Überlege dir für die „Memory“-Variante, ob ein Zustand nicht eher aus **Position + Richtung** bestehen sollte, statt nur aus `row/col`. Sonst markierst du evtl. einen Knoten als „fertig“, obwohl du ihn noch nicht in allen relevanten Orientierungen geprüft hast.\n- Achte beim Backtracking darauf, dass nach einem rekursiven „Vorwärts-Schritt“ und erfolglosem Finden des Ziels der **exakte inverse Schritt** erfolgt (inkl. notwendiger Drehungen), sodass du garantiert wieder im ursprünglichen Zustand landest, bevor du die nächste Richtung ausprobierst.\n- Wenn du bei „seenPositions enthält …“ abbrichst: prüfe, ob du an dieser Stelle wirklich abbrechen willst, oder ob du eher „diesen Zweig nicht weiterverfolgen“ willst, aber trotzdem sicherstellen musst, dass der Aufrufer nach dem Rekursionsaufruf wieder korrekt zurückgesetzt ist.\n\n### Code Style\n- Du hast mehrere komplette „Versionen“ als auskommentierten Code in `BacktrackingAlgorithm`. Das macht die Abgabe schwer lesbar; behalte nur eine finale Variante und nutze Versionsverwaltung/Commits für Alternativen.\n- `BacktrackingAlgorithmWithMemory` kodiert Positionen als String (`\"col/row\"`). Das ist unnötig fehleranfällig und schwer zu warten; eine kleine Hilfsklasse/Record für Koordinaten (und ggf. Richtung) wäre klarer.\n- In `LabyrinthApp` hast du die Aufgaben-Vorlage stark umgebaut (andere Karten). Für Übungsabgaben ist es üblicherweise besser, nur die TODO-Stellen und geforderten neuen Klassen zu ändern, damit Korrektur/Tests vergleichbar bleiben.\n\n\n# Exercise: swissmap\n\n### Correctness\n- In `SwissMapApp` ist die `main`-Methode nicht als Java-Entry-Point definiert (`void main()` statt `public static void main(String[] args)`); so startet das Programm in der Regel nicht wie gefordert über `main`.\n- Die Hover-Anforderung „Beschreibung anzeigen“ ist in `City`, `Lake` und `Mountain` nicht umgesetzt: Beim Hover änderst du zwar teils die Darstellung, aber du zeichnest nirgends die Beschreibung (z. B. `toString()` oder Text mit Name/Infos).\n- `getInteractiveArea(...)` nutzt bei `City`/`Lake`/`Mountain` Werte wie `area` bzw. `height` direkt als Breite/Höhe des `Rectangle`; diese Werte sind in km² bzw. Metern und passen nicht zur GUI-Pixel-/Skalierungslogik der Karte. Dadurch ist der interaktive Bereich sehr wahrscheinlich riesig oder völlig unpassend.\n- In `City.draw(...)` verwendest du `gui.fillCircle(...)`, ohne zuvor eine Farbe zu setzen; je nach Default kann das Ergebnis von anderen Komponenten/Draw-Aufrufen abhängen und ist evtl. nicht wie beabsichtigt sichtbar/kontrastreich (insbesondere auf dem Kartenhintergrund).\n\n### Suggestion\n- Prüfe, wie eine Java-`main`-Methode exakt signiert sein muss, damit sie als Einstiegspunkt erkannt wird, und passe deine `SwissMapApp` entsprechend an.\n- Nutze dein `hovering`-Flag nicht nur zum „größer/anderes Bild“, sondern auch, um beim Zeichnen zusätzlich Text an einer sinnvollen Position (z. B. neben dem Symbol oder oben links) auszugeben; als Basis kannst du die vorhandenen Daten bzw. `toString()` verwenden.\n- Überlege dir für `getInteractiveArea(...)` eine Pixel-basierte Fläche, die zur tatsächlich gezeichneten Größe des Symbols passt (z. B. ein kleines Rechteck um den Icon-Mittelpunkt). Orientiere dich dabei an den GUI-Koordinaten (`toGuiX/toGuiY`) und einer festen (oder skalierten) Icon-Größe statt an `area`/`height`.\n- Setze in den `draw(...)`-Methoden explizit eine Farbe (und ggf. Schriftgröße), bevor du primitives (Kreis/Text/Rechteck) zeichnest, damit die Darstellung stabil und gut sichtbar ist.\n\n### Code Style\n- In `ModeButton` sollten `WIDTH` und `HEIGHT` als Konstanten `static final` benannt werden (z. B. `BUTTON_WIDTH/BUTTON_HEIGHT`), da sie nicht pro Instanz variieren.\n- In `ModeButton` könntest du `swissMap` als `final` deklarieren, da es nach dem Konstruktor nicht mehr geändert wird.\n- In `ModeButton.onLeftClick(...)` ist die zusätzliche Prüfung über `onButton` eher redundant, weil der Click normalerweise ohnehin nur bei Treffer der `InteractiveArea` weitergereicht wird; das macht die Logik unnötig kompliziert.\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"
}
}