{
"llm" : {
"feedback" : "# Exercise: todo\n\n### Correctness\n- In `listTasks` iterierst du immer über die gesamte Array-Länge und greifst dabei auf `taskarray[i]` zu, auch wenn `taskarray` noch `null` ist oder wenn einzelne Einträge `null` sein könnten; das führt beim ersten `list all` (ohne vorheriges `add`) bzw. bei `null`-Elementen zu einem Fehler statt zu `(No tasks)`.\n- `listTasks` gibt nie `(No tasks)` aus, wenn keine Tasks ausgegeben werden (z.B. leere Liste oder bei `list to do`, wenn alle erledigt sind), obwohl das gefordert ist.\n- Die Aufgabenbeschreibung verlangt, dass ein einzelner Task mit `IO.println(task);` ausgegeben wird und dafür `Task.toString()` implementiert wird; du druckst stattdessen manuell zusammengebauten Text und hast keine `toString()`-Implementierung.\n- `markTaskDone` prüft nicht, ob die angegebene `id` überhaupt existiert; laut Anforderungen soll bei ungültiger ID `\"Task with ID XX not found\"` ausgegeben werden.\n\n### Suggestion\n- Überlege dir in `listTasks`, wie du erkennst, ob überhaupt ein Task ausgegeben wurde (z.B. über eine boolean-Variable) und gib danach nur dann `(No tasks)` aus, wenn keiner gedruckt wurde.\n- Denke daran, dass `listTasks` auch funktionieren muss, wenn noch gar kein Task hinzugefügt wurde; prüfe also vor der Schleife, ob es überhaupt etwas zum Durchlaufen gibt.\n- Implementiere in `Task` eine `toString()` so, dass `IO.println(task)` genau die gewünschte Darstellung liefert (inkl. Done-Markierung/Format), und nutze diese Ausgabe dann wirklich in `listTasks`.\n- In `markTaskDone`: prüfe, ob `id` innerhalb des gültigen Bereichs liegt (und ob an dieser Stelle wirklich ein Task vorhanden ist), bevor du `done` setzt; falls nicht, gib die geforderte Meldung aus.\n\n### Code Style\n- Du mischst `IO.println`/`IO.print` mit `System.out.println`; entscheide dich für eine Variante (in diesem Übungsgerüst ist es naheliegend, konsequent `IO` zu verwenden).\n- `taskarray` sollte direkt initialisiert werden (z.B. feste Größe wie im Auftrag erwähnt), statt bei jedem `addTask` das Array zu kopieren; das macht die Logik einfacher und näher an der Aufgabenstellung.\n- In `listTasks` ist die Schleifenbedingung `i <= taskarray.length-1` unnötig kompliziert; `i < taskarray.length` ist lesbarer.\n- Auskommentierter, unbenutzter Code in `Task` (`taskInfo`) kann entfernt werden, um die Klasse übersichtlich zu halten.\n\n\n# Exercise: energymeter\n\n### Correctness\n- Deine Methoden und Attribute sind `public`, im Beispiel (und sehr wahrscheinlich in den Unit-Tests erwartet) sind Konstruktor und Methoden package-private (ohne `public`); dadurch können Tests fehlschlagen, wenn sie genau diese Signaturen/Modifier prüfen.\n- In `fill` hast du einen Fall `else if (level+energie < 0)` eingebaut, obwohl negative `energie` bereits per Exception ausgeschlossen ist; damit entspricht die Logik nicht der geforderten Spezifikation „nur auffüllen bis max. capacity“ (kein zusätzlicher Clamp nach unten in `fill`).\n\n### Suggestion\n- Schau dir an, wie die vorgegebenen Tests die Klasse instanziieren und Methoden aufrufen: Wenn sie ohne `public` geschrieben sind (z. B. `new EnergyMeter(...)` im selben Package), kann ein anderer Sichtbarkeits-Modifier als erwartet zu einem Mismatch führen. Passe die Sichtbarkeit so an, dass sie zu den Tests passt.\n- Überlege bei `fill`, welche Zustandsänderungen laut Aufgabenbeschreibung nötig sind: Energie auffüllen und dabei höchstens auf `capacity` begrenzen. Alles, was den Level in `fill` nach unten korrigiert, ist vermutlich nicht Teil der Anforderungen.\n\n### Code Style\n- Einheitliche Sprache bei Bezeichnern: Du mischst Deutsch (`kapazitaet`, `energie`, `intensitaet`, `dauer`) mit englischen Attributnamen (`capacity`, `level`). Entscheide dich für eine Variante, das macht den Code konsistenter.\n- `level` wird explizit mit `= 0` initialisiert; bei Instanzfeldern ist das in Java ohnehin der Default-Wert – du könntest das weglassen, wenn du es nicht bewusst zur Betonung brauchst.\n- In `fill` ist der `else if (level+energie < 0)`-Zweig faktisch toter Code (wegen Exception bei negativer Energie) und macht die Methode unnötig komplex.\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"
}
}