{
"llm" : {
"feedback" : "# Exercise: todo\n\n### Correctness\n- Deine `toString()`-Ausgabe in `Task` entspricht nicht dem geforderten Format aus dem Beispiel (mit Einrückung, optionalem ✓, und `[id] description`), dadurch sehen die Listen-Ausgaben anders aus als erwartet.\n- In `markTaskDone(int id)` prüfst du nur `id > taskList.length - 1`, aber nicht `id < 0`; negative IDs würden damit nicht die geforderte “not found”-Meldung auslösen.\n- In `markTaskDone(int id)` fehlt die Prüfung auf “kein Task mit dieser ID gespeichert” in dem Sinn, dass ein Array-Platz auch `null` sein könnte (die Aufgabenbeschreibung spricht explizit von “falls kein Task mit dieser ID gespeichert ist…”). Deine Implementierung setzt voraus, dass jeder Index bis `length-1` immer belegt ist.\n\n### Suggestion\n- Orientiere dich bei `Task.toString()` an der Beispielausgabe: überlege, welche Teile variieren (✓ ja/nein) und welche immer gleich sind (`[id]` und Beschreibung), und baue die String-Repräsentation genau darauf auf.\n- Ergänze in `markTaskDone` eine Abfrage für ungültige IDs nach unten (kleiner als 0), nicht nur nach oben.\n- Überlege dir, wie du “nicht gespeichert” erkennst, wenn IDs an Array-Positionen gebunden sind: selbst wenn der Index im Bereich liegt, kann der Eintrag fehlen (z.B. `null`). Prüfe das, bevor du `done` setzt.\n\n3. Code Style:\n- In `listTasks` ist `IO.println(task.toString())` unnötig explizit; `IO.println(task)` reicht, wenn `toString()` korrekt implementiert ist (und ist auch so in der Aufgabenbeschreibung erwähnt).\n- `Task`-Felder sind alle `public`; kapsle sie typischerweise (z.B. `private`) und biete gezielte Methoden an (z.B. zum “done setzen”), statt von außen direkt zu schreiben.\n- In `ToDoApp` verwendest du `Arrays.copyOf`, aber im gezeigten Code fehlt der passende Import (`java.util.Arrays`), das führt sonst zu einem Compile-Fehler.\n- `Task[] taskList = new Task[0];` und ständiges Kopieren funktioniert, ist aber für eine Übung mit “Array für Task-Objekte erstellen” eher unüblich; meist nimmt man ein fixes Array und verwaltet zusätzlich, wie viele Plätze benutzt sind.\n\n\n# Exercise: energymeter\n\n### Correctness\n- Der Konstruktor setzt den `level` nicht explizit auf `0` (auch wenn `double` standardmässig `0.0` ist, kann das in Tests/Requirements als erwartet explizites Initialisieren gelten).\n\n### Suggestion\n- Schau dir im Konstruktor an, welchen Startzustand der Meter haben soll (insbesondere der aktuelle Füllstand) und initialisiere ihn bewusst zusammen mit der Kapazität.\n\n### Code Style\n- Die Fehlermeldung `\"Input has to be positive\"` ist inhaltlich leicht irreführend, da `0` bei dir erlaubt ist (es geht um „nicht negativ“). Einheitlicher und präziser formulieren.\n- In `consume` prüfst du beide Parameter in einer Bedingung; lesbarer wird’s oft, wenn du die Checks getrennt hältst (vor allem, falls du später unterschiedliche Meldungen willst).\n- Du verwendest mal `capacity` und mal `this.capacity` im gleichen Stil (z.B. in `percentFull`); einheitlich bleiben.\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- In der Aufgabenbeschreibung ist das Attribut als `averageSteps` vorgegeben; im Beispiel/Verwendungstext steht aber `averageSteps` *und* im Beispielcode `avgSteps`. Wenn eure Vorlage/Tests strikt nach dem vorgegebenen Namen erwarten, kann es sein, dass sie `averageSteps` **nicht** akzeptieren (oder umgekehrt).\n- `StepTracker` ist nicht im Package `ch.fhnw.prog1.exercise.stepstats` und hat auch keine `public static void main(String[] args)`-Signatur wie im Beispiel. Falls die Aufgaben-Umgebung genau diese Struktur verlangt, startet das Programm so ggf. nicht wie erwartet.\n\n### Suggestion\n- Schau in der Aufgabenstellung bzw. in den Tests/der Vorlage nach, **welcher Feldname** für den Durchschnitt tatsächlich erwartet wird (`averageSteps` vs. `avgSteps`) und benenne das Attribut entsprechend.\n- Vergleiche deine `StepTracker`-Datei mit der vorgegebenen Projektstruktur: prüfe, ob ein `package ...;`-Header erwartet ist und ob die Methode wirklich als Java-Entry-Point (`public static void main(String[] args)`) gebraucht wird.\n\n### Code Style\n- Die Felder sind alle `public` und teilweise explizit mit `this.` verwendet; das ist ok für die Übung, aber üblich wäre eher: Felder kapseln (`private`) und ggf. Getter anbieten (falls nicht explizit anders verlangt).\n- Uneinheitliche Namensgebung zwischen Aufgabe/Beispiel (`avgSteps`) und deiner Lösung (`averageSteps`) kann verwirren – auch wenn es technisch funktioniert, ist Konsistenz wichtig.\n- In `StepTracker` fehlen sichtbare Imports für `Scanner` und ggf. die `IO`-Klasse; falls das nicht automatisch durch die Vorlage kommt, solltest du die Abhängigkeiten klar machen (Imports/Package).\n",
"status" : "SUCCESS"
}
}