{
"llm" : {
"feedback" : "# Exercise: todo\n\n### Correctness\n- In `listTasks` iterierst du über `taskarray.length`, auch wenn `taskarray` noch `null` ist (direkt nach Start, vor dem ersten `add`) – damit crasht `list all`/`list to do`, statt `(No tasks)` zu drucken.\n- In `listTasks` wird `(No tasks)` nie ausgegeben, selbst wenn keine Tasks vorhanden sind oder (bei `list to do`) alle Tasks bereits erledigt sind.\n- In `listTasks` greifst du ohne Prüfung auf `taskarray[i]` zu; falls ein Eintrag mal `null` wäre, führt das zu einem Fehler.\n- Du gibst Tasks nicht wie gefordert mit `IO.println(task);` aus und hast in `Task` keine `toString()`-Methode implementiert, obwohl das im Kommentar ausdrücklich verlangt wird.\n- `markTaskDone` behandelt den Fall nicht, dass die ID ungültig ist oder kein Task unter dieser ID existiert; laut Aufgabe soll dann `\"Task with ID XX not found\"` ausgegeben werden (bei dir kommt sonst eine Exception).\n\n### Suggestion\n- Überlege dir, wie du in `listTasks` zuerst feststellen kannst, ob überhaupt ein Array existiert bzw. ob überhaupt etwas ausgegeben wurde, damit du danach gezielt `(No tasks)` ausgeben kannst.\n- Für `list to do`: Zähle/merke während des Durchlaufens, ob mindestens ein passender Task gedruckt wurde; wenn nicht, kommt die `(No tasks)`-Meldung.\n- Implementiere `toString()` in `Task` so, dass `IO.println(task)` genau die gewünschte Darstellung liefert (inkl. Done-Markierung und ID, falls das im Output gefordert ist) – dann musst du in `listTasks` nicht selbst Strings zusammenbauen.\n- In `markTaskDone`: prüfe vor dem Zugriff, ob `id` im gültigen Bereich liegt und ob unter dieser ID wirklich ein Task gespeichert ist; nur dann auf `done=true` setzen, sonst die Fehlermeldung drucken.\n\n### Code Style\n- In `listTasks` verwendest du `System.out.println`, obwohl der Rest mit `IO.println/IO.print` arbeitet; bleib konsistent bei einer Ausgabemethode.\n- Die Schleife `for (int i = 0; i <= taskarray.length-1; i++)` ist unnötig kompliziert; das lässt sich lesbarer formulieren.\n- In `Task` sind auskommentierte Methoden (`taskInfo`) im finalen Code störend; entferne sie, wenn sie nicht mehr gebraucht werden.\n- `taskarray` als Feldname ist schwer lesbar; ein klarer Name wie `tasks` passt besser zum Kontext.\n- Du nutzt `Arrays.copyOf`, aber im gezeigten Code fehlt der Import (`java.util.Arrays`); achte darauf, dass alles Nötige importiert ist.\n\n\n# Exercise: energymeter\n\n### Correctness\n- Deine Methoden und der Konstruktor sind als `public` deklariert; in der Aufgabenstellung/Unit-Tests kann aber erwartet werden, dass sie package-private (ohne Modifier) sind – das kann zu Test-Fehlschlägen führen, wenn die Signaturen exakt geprüft werden.\n\n### Suggestion\n- Schau dir genau an, wie die Tests auf die Klasse zugreifen (gleiches Package oder nicht) und ob sie die Sichtbarkeit/Signatur der Member exakt voraussetzen; passe dann die Modifier entsprechend an.\n\n### Code Style\n- In `fill` hast du einen Zweig `else if (level+energie < 0)`, der durch den vorherigen Check `energie < 0` nie erreichbar ist; das macht den Code unnötig komplex.\n- Einheitliche Sprache bei Bezeichnern (entweder Deutsch oder Englisch) verbessert Lesbarkeit; aktuell mischst du `capacity/level` mit `kapazitaet/energie/intensitaet/dauer`.\n- Die Logik in `fill`/`consume` lässt sich jeweils auf “erst rechnen, dann auf [0..capacity] begrenzen” vereinfachen, was die Anzahl der Fälle reduziert.\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\nLlm Evaluation ist für diese Aufgabe deaktiviert. Entferne die .llmignore Datei vom Package der Aufgabe.\n",
"status" : "SUCCESS"
}
}