{
"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- In `TryStraightFirst` fehlt der geforderte Fall „wenn weder vorne/links/rechts geht, dann rechtsum kehrt machen“; aktuell kann die Figur in einer Sackgasse steckenbleiben und die Schleife wird nie fertig.\n- In `TryStraightFirst` drehst du bei `pathToTheLeft()`/`pathToTheRight()` nur, machst aber in diesem Schleifendurchlauf keinen Schritt nach vorne; das entspricht nicht der Aufgabenbeschreibung („links oder rechts versuchen“ → drehen **und** gehen, wenn möglich).\n- In `BacktrackingAlgorithm.navigate` initialisierst du `besucht` mit `new boolean[col][row]`; `col`/`row` sind aber aktuelle Positionen, nicht die Labyrinth-Grösse. Dadurch ist das Array oft zu klein und `besucht[row][col]` kann zu `ArrayIndexOutOfBoundsException` führen.\n- In `checkPath` ist die Backtracking-Logik inkonsistent: Nach dem „ahead“-Zweig stellst du die Position wieder zurück (durch Umdrehen + `moveForward()`), aber nach den „left“/„right“-Zweigen machst du kein entsprechendes Zurückgehen zur Verzweigung. Dadurch kann die Rekursion an einer ganz anderen Stelle weiterlaufen als gedacht.\n- In `checkPath`: Wenn kein Pfad existiert, drehst du zweimal rechts und rufst `checkPath` erneut auf, ohne dich zu bewegen. Das ändert nur die Richtung, aber nicht `row/col`; da `besucht[row][col]` bereits `true` ist, brichst du dann sofort ab und gehst nicht wirklich zurück.\n\n### Suggestion\n- Für `TryStraightFirst`: Überlege dir einen klaren „else“-Zweig für den Sackgassen-Fall. Die Aufgabenformulierung nennt explizit „rechtsum kehrt“, also eine 180°-Drehung, bevor du weiter entscheidest.\n- Für `TryStraightFirst`: Wenn du dich entscheidest, links oder rechts „zu versuchen“, denk daran, dass ein „Versuch“ typischerweise aus *Drehen + Vorwärts* besteht (so wie beim Geradeaus-Fall auch).\n- Für `BacktrackingAlgorithm`: Das „besucht“-Array sollte sich an der Labyrinth-Dimension orientieren, nicht an der Startposition. Da du über `Figure` keine `rows()/cols()` bekommst, brauchst du entweder eine andere Datenstruktur (z.B. Set von Koordinaten) oder du musst die nötigen Dimensionen anders verfügbar machen.\n- Für `BacktrackingAlgorithm`: Backtracking klappt am einfachsten, wenn du für jeden rekursiven Schritt garantierst: „Ich komme nach dem rekursiven Aufruf wieder exakt auf das Feld und in die Richtung zurück, die ich vor dem Schritt hatte.“ Prüfe deine drei Richtungs-Zweige darauf, ob sie diese Eigenschaft alle einhalten.\n- Für Sackgassen im Backtracking: Eine 180°-Drehung allein reicht nicht – du musst typischerweise auch wirklich einen Schritt zurück machen (wenn möglich), damit du zur letzten Verzweigung zurückkommst.\n\n3. Code Style:\n- In `BacktrackingAlgorithm.navigate` sind auskommentierte Codeblöcke geblieben; die würde ich entfernen, damit klar ist, welche Strategie du wirklich verfolgst.\n- Variablennamen wie `besucht` sind ok, aber `col`/`row` werden mehrfach neu deklariert (in `navigate` und `checkPath`); das kann verwirren. Lieber bewusst nur dort holen, wo du sie brauchst, und Wiederholungen minimieren.\n- In `TryStraightFirst` gibt es keinen abschliessenden `else`-Fall; selbst wenn es funktional wäre, hilft ein expliziter finaler Zweig der Lesbarkeit (zeigt: alle Fälle sind bedacht).\n\n\n# Exercise: swissmap\n\n### Correctness\n- Deine `City`, `Lake`, `Mountain` und `ModeButton` verwenden Bildpfade wie `swissmap_img/...`, während in der Aufgabe explizit auf die Bilder im `resources`-Ordner verwiesen wird (und `SwissMap` im Starter/Beispiel `swissmap/...` nutzt). Wenn diese Pfade nicht zu den tatsächlich mitgelieferten Resources passen, werden die Objekte nicht (korrekt) gezeichnet.\n- Der `ModeButton` hat als interaktive Fläche (`getInteractiveArea`) ein `Rectangle` von `15x15`, wird aber in `draw` als `65x25` gezeichnet. Dadurch ist nur ein kleiner Teil des sichtbaren Buttons tatsächlich klick-/hoverbar.\n- `Lake` und `Mountain` skalieren ihre Icons mit `WIDTH * scale(gui) / BG_PIXEL_WIDTH` (also mit der Kartenbreite). Das ist sehr wahrscheinlich nicht die beabsichtigte Skalierung für ein kleines Icon und führt dazu, dass die Icons extrem gross werden bzw. nicht wie gefordert „als Marker“ aussehen.\n\n### Suggestion\n- Prüfe, welche Resource-Pfade in der vorgegebenen Projektstruktur wirklich existieren (Ordnername in den Resources, exakter Dateiname) und verwende genau diese Strings in `drawImage(...)`.\n- Leite die Grösse deines `getInteractiveArea(...)` direkt aus der tatsächlich gezeichneten Button-Grösse ab (gleiche Position und gleiche Breite/Höhe wie in `draw`), damit Hover/Click zuverlässig auf der ganzen Fläche funktioniert.\n- Verwende für Seen/Berge eine feste (oder moderat skalierende) Icon-Grösse, die sich an der Marker-Darstellung orientiert, statt die Kartenbreite als Skalierungsbasis zu nehmen. Als Check: Wenn du das Fenster grösser machst, sollte der Marker zwar ggf. leicht mitskalieren, aber nicht „kartenfüllend“ werden.\n\n3. Code Style:\n- Entferne Debug-Ausgaben wie `System.out.println(satelliteMode)` in `SwissMap.draw()` und `System.out.println(getInhabitants())` in `City.onMouseEnter()`, da `draw()` sehr häufig aufgerufen wird und die Konsole sonst zugespammt wird.\n- In `Mountain` und `SwissMap` sind Imports/Code für `ImageIO`, `BufferedImage`, `Objects`, `IOException` drin, werden aber nicht verwendet; das macht die Klassen unnötig unübersichtlich.\n- `ModeButton` speichert `hovered` und `clicked`, nutzt die Flags aber im `draw()` nicht für eine sichtbare Zustandsänderung; entweder Zustände wirklich visuell nutzen oder die Variablen weglassen.\n\n\n# Exercise: visualizer\n\n### Correctness\n- Das `DataPoint`-Interface ist noch leer, obwohl die Aufgabe verlangt, dass du dort die benötigten Methoden (inkl. Javadoc) definierst.\n- In `Visualizer` sind die Dummy-Werte (`DUMMY_DOUBLE`, `DUMMY_STRING`) noch überall im Einsatz und wurden nicht durch Aufrufe auf das jeweilige `DataPoint`-Objekt ersetzt.\n- Keine der Datensatz-Klassen (`Movie`, `Country`, `Processor`) implementiert `DataPoint`, dadurch kann `Visualizer` sie nicht wie gefordert polymorph verwenden.\n- In `VisualizerApp` wird weiterhin `new DataPoint[0]` visualisiert statt die geladenen Daten (Movies/Countries/Processors) zu übergeben, wie gefordert.\n- Die geforderten Achsen-Zuordnungen (Movies: Budget→x, Rating→y; Countries: Literacy→x, GDP/capita→y; Processors: Veröffentlichungsmonat→x, clockRate*cores (log)→y) sind nirgends umgesetzt.\n\n### Suggestion\n- Schau im `Visualizer` nach, welche Informationen er pro Punkt wirklich braucht: überall wo aktuell `DUMMY_DOUBLE` steht, brauchst du vermutlich numerische Werte (z.B. x/y-Koordinate), und wo `DUMMY_STRING` steht, eher Text (z.B. Gruppierung/Label/Detailtext).\n- Leite daraus Methoden im Interface ab: eine Methode für den x-Wert, eine für den y-Wert, eine für die Legenden-Gruppe/Farbe, sowie etwas für den Text in der Hover-Box (Titel + mehrzeiliger Detailtext).\n- Wenn du `Visualizer` umstellst: ersetze die Dummy-Werte immer mit Aufrufen auf *das aktuelle* `DataPoint point` (z.B. in den Schleifen) bzw. auf `closest` in `drawHighlighted`.\n- Sobald `Movie/Country/Processor` `DataPoint` implementieren, kannst du in `VisualizerApp` statt `new DataPoint[0]` die Arrays aus den `load...()`-Methoden übergeben (achte darauf, dass der Rückgabetyp dann als `DataPoint[]` verwendbar ist).\n- Für Prozessoren: überlege dir eine sinnvolle Zahl für “Jahr+Monat” als x-Wert (die Sortierung muss zeitlich stimmen), und für y den Wert aus `clockRateKhz * cores`; die Log-Skalierung muss irgendwo berücksichtigt werden (entweder im zurückgegebenen y oder im Visualizer-Konzept).\n\n### Code Style\n- In deinem aktuellen Stand sind sehr viele `TODO`/Dummy-Platzhalter übrig; sobald du implementierst, entferne die `DUMMY_*` Konstanten konsequent, damit keine “Scheinwerte” im Code bleiben.\n- `VisualizerApp.main()` ist als `void main()` deklariert; prüfe, ob in eurer Umgebung nicht ein `public static void main(String[] args)` erwartet wird (sonst startet das Programm evtl. nicht wie üblich).\n",
"status" : "SUCCESS"
}
}