{
"llm" : {
"feedback" : "# Exercise: todo\n\n### Correctness\n- Du implementierst kein Task-Array wie in der Aufgabenstellung verlangt, sondern verwendest eine `List<Task>`; damit erfüllst du die Vorgabe „in ToDoApp ein Array für Task-Objekte zu erstellen“ nicht.\n- In `listTasks` soll die Ausgabe eines einzelnen Tasks explizit über `IO.println(task);` passieren (und damit über `Task.toString()`); du gibst stattdessen manuell `id`/`description` aus und nutzt `System.out.println` statt `IO.println`.\n- In `Task` fehlt eine `toString()`-Implementierung, obwohl die Aufgabenstellung das für die Ausgabe fordert.\n- In `markTaskDone` gibst du keine Meldung `\"Task with ID XX not found\"` aus, wenn keine Task mit dieser ID existiert (oder die ID ungültig ist).\n- In `listTasks(false)` soll `(No tasks)` erscheinen, wenn es keine auszugebenden (nicht erledigten) Tasks gibt; bei dir erscheint `(No tasks)` nur, wenn die Liste komplett leer ist, nicht wenn alle Tasks bereits `done` sind.\n\n### Suggestion\n- Schau dir die TODO-Kommentare im Gerüst an: dort wird explizit ein fester Speicher (`Task[]`) erwartet; überlege, wie du „freier Platz im Array“ und „ID entspricht Array-Index“ damit umsetzt.\n- Implementiere die Ausgabe so, dass wirklich das `Task`-Objekt gedruckt wird (nicht zusammengesetzte Strings); dafür brauchst du eine passende `toString()` in `Task`, die auch den Done-Status in der Darstellung berücksichtigt.\n- Für `markTaskDone`: überlege dir eine klare Bedingung, wann „nicht gefunden“ gilt (z.B. ID außerhalb des belegten Bereichs oder `null` im Array) und gib dann exakt die geforderte Meldung aus.\n- Für `listTasks(false)`: zähle/merke dir, ob du mindestens eine Task ausgegeben hast; wenn nicht, gib `(No tasks)` aus (auch wenn intern Tasks existieren, aber eben keine passenden).\n\n### Code Style\n- Du mischst `IO.println`/`IO.print` mit `System.out.println`; bleib konsequent bei der im Gerüst vorgesehenen IO-Klasse.\n- `import java.sql.SQLOutput;` ist unbenutzt und thematisch falsch (kann weg).\n- Es fehlen Imports für `List`/`ArrayList`/`Scanner` im gezeigten Code; achte darauf, nur benötigte Imports zu verwenden und alle nötigen tatsächlich zu haben.\n- In `markTaskDone` könntest du nach dem Finden der Task frühzeitig abbrechen (statt weiter zu iterieren), um unnötige Durchläufe zu vermeiden.\n\n\n# Exercise: energymeter\n\n### Correctness\n- Im Konstruktor wird `level` nicht explizit auf `0` gesetzt (verlass dich nicht darauf, dass der Defaultwert immer genügt; die Tests könnten ein definiertes Initialisieren erwarten).\n- In `consume(...)` setzt du `level` auf `0`, wenn `toConsume > capacity` ist. Der Verbrauch hängt aber vom aktuellen `level` ab, nicht davon, ob die benötigte Energiemenge größer als die *Kapazität* ist. Dadurch kann Energie fälschlicherweise auf `0` fallen, obwohl noch genug `level` vorhanden wäre.\n\n### Suggestion\n- Überlege im Konstruktor, welchen Startzustand das Meter laut Aufgabenbeschreibung haben soll, und setze den Füllstand bewusst entsprechend.\n- In `consume(...)`: Prüfe nur, ob der berechnete Verbrauch den aktuellen `level` unter `0` drücken würde. Die `capacity` ist hier eher eine obere Grenze beim **Füllen**, nicht beim **Verbrauchen**.\n\n### Code Style\n- Attribute und Methoden sind bei dir `public`. Üblicherweise kapselt man den Zustand (`capacity`, `level`) und macht sie `private`, damit nur die Methoden das Meter verändern können.\n- Einheitliche Klammer- und Leerzeichenformatierung (z. B. `if ( ... ) { ... }`) würde die Lesbarkeit verbessern.\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"
}
}