{
"llm" : {
"feedback" : "# Exercise: todo\n\n### Correctness\n- Die Aufgabe verlangt explizit ein **Array für Task-Objekte** in `ToDoApp` (und das Finden des ersten freien Platzes im Array als ID). Du verwendest stattdessen eine `ArrayList<Task>`, damit erfüllst du diese Anforderung nicht.\n- Das Grundgerüst erwartet, dass die App über die vorgegebene Struktur (inkl. Klasse/Methoden im bestehenden `ToDoApp`-Gerüst) lauffähig ist; in deinem `ToDoApp.java` ist `ArrayList<Task> taskList= ...` und `void main()` so, wie es dasteht, **nicht** in eine Klasse eingebettet und `main` ist nicht `static` – so kompiliert/läuft es in Java typischerweise nicht wie gefordert.\n\n### Suggestion\n- Schau dir die TODO-Kommentare an: Dort steht, dass du ein `Task[]` anlegen sollst. Überlege, wie du die Liste-Funktionalität, die du mit `ArrayList` gelöst hast, mit einem festen Array plus einer Hilfsvariable (z. B. “wie viele Plätze sind belegt?” oder “erster freier Index”) nachbauen kannst.\n- Prüfe die Signatur/Platzierung von `main`: In Java muss `main` in einer Klasse stehen und als Einstiegspunkt üblicherweise `public static void main(String[] args)` sein. Orientiere dich am vorgegebenen Gerüst (Package, Klasse, Methodenheader), damit es mit dem restlichen Code zusammenpasst.\n\n### Code Style\n- In `listTasks` rufst du `a.toString()` explizit auf; idiomatischer ist `IO.println(a)`, weil Java automatisch `toString()` nutzt (und genau das steht auch im Kommentar).\n- In `Task` sind die Felder package-private/`public` gemischt (`public boolean done`, der Rest ohne Modifier). Einheitlicher wäre hier ein konsistenter Sichtbarkeits-Entscheid (und typischerweise eher `private` + Zugriffsmethoden, wenn ihr das schon behandelt habt).\n- `StringBuilder` in `toString()` ist hier nicht zwingend nötig; es macht den Code etwas schwerer lesbar als eine einfache String-Zusammenstellung (bei so kurzen Strings).\n- In `ToDoApp.java` fehlen in deinem Ausschnitt Imports (`ArrayList`, `Scanner`) und die Klassen-/Package-Deklaration; achte darauf, dass die Datei vollständig und konsistent zum Projektaufbau ist.\n\n\n# Exercise: energymeter\n\n### Correctness\n- Im Konstruktor wird `level` nicht explizit auf `0` gesetzt; falls die Unit-Tests erwarten, dass der Startzustand klar initialisiert wird, könnte das zu einem Fehlschlag führen.\n\n### Suggestion\n- Schau dir an, welchen Anfangsfüllstand ein frisch erzeugtes `EnergyMeter` haben soll, und stelle sicher, dass dieser Zustand im Konstruktor bewusst gesetzt wird (nicht nur “zufällig” durch Default-Werte).\n\n### Code Style\n- `throw new IllegalArgumentException(/* Fehlermeldung */);`: Die Kommentar-Platzhalter wirken wie unaufgeräumter Code; entweder eine sinnvolle Nachricht angeben oder den parameterlosen Konstruktor verwenden.\n- In `consume` prüfst du `intensity < 0 || duration < 0` in einer gemeinsamen Bedingung; lesbarer (und leichter zu debuggen) wäre es, beide Parameter getrennt zu validieren, damit klar ist, welcher ungültig war.\n\n\n# Exercise: pong\n\nLlm Evaluation ist für diese Aufgabe deaktiviert. Entferne die .llmignore Datei vom Package der Aufgabe.\n\n\n# Exercise: stepstats\n\n### Correctness\n- Die Attributnamen entsprechen nicht der geforderten Verwendung: Im Auftrag steht `averageSteps`, `minSteps`, `maxSteps`, aber für den Erfolg wird explizit `successDays` gezeigt; zusätzlich wird im Beispiel `avgSteps` verwendet. Entscheidend ist: In der Aufgabenbeschreibung wird u. a. `double avgSteps = stats.averageSteps;` gezeigt, dein Code liefert zwar `averageSteps`, aber falls die Tests/Client-Code strikt auf `avgSteps` oder exakt die vorgegebenen Feldnamen prüfen, könnte das fehlschlagen.\n- In `StepTracker` ist die Signatur `void main()` nicht die übliche/erwartete Java-Entry-Point-Signatur (typisch `public static void main(String[] args)`). Wenn das Übungsprojekt das wirklich als normales Java-Programm ausführen will, kann es so nicht starten.\n\n### Suggestion\n- Vergleiche die in der Aufgabenstellung gezeigte Nutzung Zeile für Zeile mit deinen Feldnamen. Falls automatisierte Tests laufen, orientieren die sich oft exakt an diesen Namen.\n- Prüfe, wie `StepTracker` im Template ursprünglich aufgebaut war (Package, Imports, `public class StepTracker`, `public static void main(String[] args)`). Übernimm die erwartete Main-Signatur, damit das Programm in der vorgesehenen Umgebung gestartet werden kann.\n\n### Code Style\n- `totalSteps` ist als `public` Feld gespeichert, wird aber ausserhalb der Klasse nicht verwendet; das ist unnötiger Zustand. Üblicher wäre: lokal im Konstruktor rechnen und nur Resultate als Felder behalten.\n- Du nutzt `this.` teils konsistent, teils nicht (z. B. bei `totalSteps` in der Durchschnittsberechnung). Einheitlich bleiben verbessert Lesbarkeit.\n- In `StepTracker` fehlen (im gezeigten Code) `package`-Deklaration und Imports für `Scanner`/`IO`; falls das im echten Projekt nicht schon implizit vorhanden ist, sollte das sauber im File stehen.\n",
"status" : "SUCCESS"
}
}