{
"llm" : {
"feedback" : "# Exercise: todo\n\nLlm Evaluation ist für diese Aufgabe deaktiviert. Entferne die .llmignore Datei vom Package der Aufgabe.\n\n\n# Exercise: energymeter\n\n### Correctness\n- Der Konstruktor setzt `level` nicht explizit auf `0`. Das kann in Tests relevant sein, wenn davon ausgegangen wird, dass der Start-Füllstand immer 0 ist (unabhängig von Java-Defaultwerten).\n- In `consume(double capacity, double duration)` heisst der erste Parameter `capacity`, obwohl laut Aufgabe (und Tests) hier die Verbrauchs-Intensität gemeint ist. Das ist zwar funktional ähnlich, kann aber zu Missverständnissen führen, wenn du später mit `this.capacity` arbeitest oder Tests/Leser etwas anderes erwarten (Parameterbedeutung).\n\n### Suggestion\n- Setze im Konstruktor den Startzustand des Meters bewusst/verständlich (insbesondere den Start-`level`) statt dich auf Defaultwerte zu verlassen.\n- Benenne den ersten Parameter von `consume` so, dass klar ist, dass es um die Intensität (Verbrauch pro Zeit) geht und nicht um die Meter-Kapazität; dann wird auch klarer, warum du mit `duration` multiplizierst.\n\n### Code Style\n- Die Attribute `capacity` und `level` sind `public`. Üblicherweise kapselt man den Zustand (z. B. `private`) und lässt Zugriff nur über Methoden zu, damit niemand von aussen beliebige Werte setzen kann.\n- Vermeide Parameter-Namen, die Feldnamen/Bedeutungen „überladen“ (z. B. `consume(double capacity, ...)` vs. `this.capacity`). Das macht den Code schwerer zu lesen.\n- Du mischst `IllegalArgumentException` und `java.lang.IllegalArgumentException` in verschiedenen Methoden—einheitlich bleiben (das `java.lang.` ist hier nicht nötig).\n- Die `// TODO`-Zeile ist nach der Implementierung nicht mehr nötig.\n\n\n# Exercise: pong\n\n### Correctness\n- Es fehlen die Klassen `Player` und `Ball` komplett (inkl. der geforderten Attribute wie Position, Balkenlänge/Punkte bzw. Geschwindigkeit beim Ball).\n- In `PongGame` werden keine zwei `Player`-Objekte und kein `Ball`-Objekt erzeugt/initialisiert (Ball Mitte + zufällige Startgeschwindigkeit ist nicht umgesetzt).\n- In der Spielschleife findet keine Bewegung statt: Tastatureingaben werden zwar abgefragt, aber nicht genutzt, um die Balken zu bewegen; der Ball wird ebenfalls nicht anhand `(vx, vy)` fortbewegt.\n- Kollisionen oben/unten (Ball abprallen, Player im Feld halten) sind nicht implementiert.\n- Kollisionen Ball–Balken sowie Seitenwand-Treffer mit Punktevergabe und Ball-Reset sind nicht implementiert.\n- Punktestand wird nicht gespeichert und nicht gezeichnet.\n- Erweiterung auf mehrere Bälle (Ball-Array, periodisches Hinzufügen, Entfernen beim Verlassen) ist nicht umgesetzt.\n\n### Suggestion\n- Lege zuerst zwei kleine Klassen an und überlege dir, welche Felder wirklich dauerhaft gespeichert werden müssen (Positionen, Größen, Punkte; beim Ball zusätzlich Geschwindigkeiten).\n- Initialisiere im `main` vor der Schleife zwei Spieler links/rechts und den Ball in der Mitte; für „zufällige Geschwindigkeit“ brauchst du Vorzeichen und Betrag (und achte darauf, dass `vx` nicht 0 wird).\n- Nutze die `isKeyPressed`-Booleans nicht nur zum Abfragen, sondern um pro Frame `py` der Spieler zu verändern (für beide Richtungen brauchst du je zwei Tasten).\n- Baue die Bewegung/Physik in eine feste Reihenfolge: erst Input → dann Positionsupdate → dann Kollisionen/Scoring → dann Zeichnen.\n- Für Wandkollisionen oben/unten: beim Ball reicht es, die y-Geschwindigkeit umzudrehen, sobald der Ball-Radius die Grenze überschreitet; bei Spielern clampst du `py` in `[0, HEIGHT - paddleHeight]`.\n- Für Ball–Schläger-Kollisionen hilft es, mit einfachen Rechteck-Überlappungen zu arbeiten (Ball als Bounding-Box/Radius vs. Paddle-Rechteck) und bei Treffer `vx` umzudrehen und den Ball minimal aus dem Schläger herauszuschieben.\n- Für mehrere Bälle: Überlege dir eine Invariante wie „alle aktiven Bälle liegen in `balls[0..active-1]`“; dann kannst du beim Entfernen durch „mit letztem aktivem Ball überschreiben“ effizient bleiben und beim Hinzufügen einfach am Ende anhängen. Ein Frame-Zähler kann das „alle 1–2 Sekunden“ steuern.\n\n### Code Style\n- In deinem Versuch ist aktuell fast nur die Vorlage mit TODO-Kommentaren; sobald du implementierst: lagere Logik wie `move()`, `draw()` und Kollisions-/Bounding-Box-Helfer lieber in die jeweiligen Klassen oder kleine Hilfsmethoden aus, statt alles in die `while`-Schleife zu packen.\n- Die Variablen `p1Up` und `p2Up` werden derzeit nicht verwendet; entweder nutzen (für Bewegung) oder entfernen, sobald du die Bewegung direkt über `if (gui.isKeyPressed(...))` machst.\n\n\n# Exercise: stepstats\n\nLlm Evaluation ist für diese Aufgabe deaktiviert. Entferne die .llmignore Datei vom Package der Aufgabe.\n",
"status" : "SUCCESS"
}
}