{
"llm" : {
"feedback" : "# Exercise: floodfill\n\n### Correctness\n- In `DrawingApp` wird beim Rechtsklick immer `currentAlgorithm` verwendet, aber dieses Feld wird nie aktualisiert, wenn du mit ↑/↓ den `ModusIndex` änderst; dadurch kann man faktisch nicht zwischen den Algorithmen umschalten.\n- `FillAlgorithm` deklariert zusätzlich `String toString();` im Interface; gefordert war ein Interface, das die `fill(...)`-Methode deklariert (die Anzeige soll über das normale `toString()` laufen, nicht als Interface-Vertrag).\n- `RecursiveFloodFill` implementiert kein Flood-Fill-Verhalten: Du vergleichst gegen `newColor` statt gegen die Start-/Old-Color, dadurch wird in typischen Fällen nicht sinnvoll begrenzt, was geflutet werden soll.\n- `RecursiveFloodFill` prüft die Bildgrenzen erst **nachdem** `img.getPixel(x, y)` aufgerufen wird; bei Randrekursion führt das zu Out-of-Bounds-Zugriffen.\n- `RecursiveFloodFill`: Deine Abbruchlogik verhindert Endlosrekursion nicht zuverlässig (du färbst zwar um, aber der Basisfall ist an die falsche Farbe gekoppelt); außerdem wird der Rückgabewert (`boolean`) nirgends sinnvoll genutzt.\n- `IterativFloodFill`: Du prüfst `equals(newColor)` statt “gesuchte alte Farbe” (Farbe des Startpixels). Damit wird meistens entweder gar nichts gefüllt oder du füllst nur bereits neue Farbe.\n- `IterativFloodFill`: Deine Nachbar-Grenzprüfungen sind off-by-one (`<= img.getHeight() - 1` und dann `y + 1` etc.), dadurch können Koordinaten `== height` bzw. `== width` in die Worklist kommen → Out-of-Bounds beim `getPixel`.\n- `IterativFloodFill` nutzt `ArrayList.getLast()/removeLast()`; das gibt es bei `ArrayList` nicht (nur z.B. bei `LinkedList` oder seit neueren JDKs bei `SequencedCollection`-Views, aber `ArrayList` direkt typischerweise nicht in eurem Setup). Das wird sehr wahrscheinlich nicht kompilieren.\n\n### Suggestion\n- Statt ein festes `currentAlgorithm = Modus.get(2)` zu speichern, überlege dir, wie du bei jedem Rechtsklick den Algorithmus anhand von `ModusIndex` auswählst (analog zur Farbauswahl über `colorIndex`).\n- Lass `FillAlgorithm` nur das deklarieren, was wirklich extrahiert werden sollte: die `fill(...)`-Signatur. Für die Anzeige in der Statuszeile reicht es, in den Klassen `toString()` zu überschreiben (ohne es im Interface zu fordern).\n- Beim Flood-Fill brauchst du immer zwei Farben: die **oldColor** (vom Startpixel) und die **newColor** (Ziel). Überlege, woher du `oldColor` bekommst und wie du sie in der Rekursion/Iteration “mitgibst”.\n- Achte beim rekursiven Basisfall auf die Reihenfolge: zuerst Grenzen prüfen, dann Pixel-Farbe lesen, dann ggf. abbrechen. Sonst knallt es am Rand.\n- Für die iterative Variante: prüfe für jeden entnommenen Punkt, ob er noch die `oldColor` hat; nur dann färben und Nachbarn hinzufügen. Das verhindert auch, dass du Pixel mehrfach verarbeitest.\n- Bei den Nachbarn: Wenn du `y+1` erzeugst, muss vorher gelten `y < height-1` (nicht `<=`). Genauso bei `x+1` mit `x < width-1`. Für `y-1` entsprechend `y > 0`, etc.\n- Wenn du eine Stack-artige Worklist willst, nimm entweder eine Struktur, die “am Ende” effizient entfernen kann, oder arbeite bei `ArrayList` mit `size()-1` als Index.\n\n### Code Style\n- Java-Konventionen: Variablen/Felder wie `Modus`, `ModusIndex` besser klein anfangen (`modus`, `modusIndex`), Klassen/Typen groß.\n- In `Coord` sind Felder und Konstruktor package-private; meist macht man sie `private final` und bietet Getter an (oder gleich als `record`), damit du nicht überall direkt `current.x` verwendest.\n- In `RecursiveFloodFill` ist der `boolean`-Rückgabewert unnötig/verwirrend, weil du ihn nicht nutzt; eine rekursive Flood-Fill-Hilfsmethode ist typischerweise `void`.\n- Kommentar “hier das Interface verwenden!” ist ok zum Lernen, aber der Code sollte die Idee dann auch durchziehen (kein festes `currentAlgorithm`, sondern wirklich “aktuelles” anhand Index).\n\n\n# Exercise: flashcard\n\nLlm Evaluation ist für diese Aufgabe deaktiviert. Entferne die .llmignore Datei vom Package der Aufgabe.\n\n\n# Exercise: imagestats\n\nLlm Evaluation ist für diese Aufgabe deaktiviert. Entferne die .llmignore Datei vom Package der Aufgabe.\n",
"status" : "SUCCESS"
}
}