{
"llm" : {
"feedback" : "# Exercise: todo\n\n### Correctness\n- Die Aufgabe verlangt explizit ein **Array für Task-Objekte** in `ToDoApp`; du verwendest stattdessen eine `ArrayList`.\n- In `ToDoApp` importierst du `ch.fhnw.prog1.exercise.todo.task`, aber deine Klasse heißt `task` und liegt im Package `ch.fhnw.prog1.exercise.todo`; je nach tatsächlicher Projektstruktur/Dateinamen kann das so **nicht kompilieren** (insbesondere auch wegen Groß-/Kleinschreibung und ob die Klasse wirklich im Subpackage `.task` liegt).\n- Die vorgegebene Grundstruktur sieht eine `main(String[] args)`-Methode vor; dein `void main()` entspricht dem in Java normalerweise nicht und könnte dazu führen, dass das Programm **nicht startbar** ist.\n\n### Suggestion\n- Ersetze die `ArrayList` durch ein festes `Task[]` und verwalte zusätzlich, **wie viele Plätze bereits benutzt** sind (oder finde den ersten `null`-Slot), damit die ID wirklich dem Array-Index entspricht wie gefordert.\n- Prüfe, ob dein `import` wirklich auf den korrekten Typ/Package zeigt: Achte darauf, dass **Klassenname, Dateiname und Package** zusammenpassen (und nutze die korrekte Großschreibung).\n- Schau dir die Signatur der Einstiegsmethode an: In Java ist es typischerweise `public static void main(String[] args)`. Passe das so an, dass der Runner/IDE dein Programm auch wirklich als Java-Programm erkennt.\n\n### Code Style\n- Klassen in Java schreibt man üblicherweise **PascalCase**: `Task` statt `task`. Das reduziert Verwirrung (und verhindert leicht Import-/Typfehler).\n- In `listTasks` rufst du explizit `a.toString()` auf; idiomatischer ist `IO.println(a)`, dann wird `toString()` automatisch verwendet (und entspricht auch dem Kommentar in der Aufgabenstellung).\n- Deine Variable `StringBuilder a` in `toString()` heißt wie dein Loop-Element `for(task a : ...)`; unterschiedliche, sprechende Namen würden die Lesbarkeit deutlich verbessern.\n- `ArrayList<task> taskList= new ArrayList<task>();` sollte in einer Klasse als Feld stehen (mit Sichtbarkeit wie `private`) und nicht „lose“ im File ohne umgebende Klasse/Methode.\n\n\n# Exercise: energymeter\n\n### Correctness\n- Der Konstruktor setzt den Startwert von `level` nicht explizit auf `0` (die Tests könnten erwarten, dass der Startzustand klar initialisiert wird, unabhängig von Default-Werten).\n\n### Suggestion\n- Überlege, ob du im Konstruktor neben `capacity` auch den initialen Füllstand (`level`) bewusst setzen solltest, damit der Anfangszustand eindeutig ist.\n\n### Code Style\n- In `throw new IllegalArgumentException(/* Fehlermeldung */);` stehen Kommentar-Platzhalter in den Klammern; entweder entferne die Klammern komplett oder gib eine echte Fehlermeldung als String an.\n- Einheitlichkeit: Du verwendest teils `this.capacity` und teils `capacity` im selben Methodenblock; entscheide dich für einen Stil und zieh ihn durch.\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 geforderte Verwendung nennt die Attribute `averageSteps`, `minSteps`, `maxSteps`, `successDays` ohne Casts; bei dir sind `minSteps` und `maxSteps` zwar `int`, aber im Client castest du trotzdem auf `int` – das passt nicht zur geforderten Zugriffsidee (und ist unnötig).\n- In der Aufgabenbeschreibung ist der Attributname `avgSteps` vorgegeben (im Code-Snippet: `double avgSteps = stats.averageSteps;` ist widersprüchlich, aber der Beispiel-Client verwendet `stats.avgSteps`). Dein Attribut heißt `averageSteps`; je nach Tests/Erwartung kann das als falsch gewertet werden, wenn exakt `avgSteps` erwartet wird.\n- `StepTracker` ist nicht wie im Beispiel als vollständige Java-Klasse mit `public static void main(String[] args)` umgesetzt (bei dir: `void main()` ohne Klasse/Package) – wenn die Vorlage/Abgabe das so verlangt, kompiliert das nicht.\n\n### Suggestion\n- Schau dir genau an, welche Attributnamen der neue Client-Code bzw. die Tests erwarten (insbesondere `avgSteps` vs. `averageSteps`) und richte die öffentliche API der Klasse daran aus.\n- Wenn `minSteps`/`maxSteps` bereits `int` sind, prüfe, ob du im Ausgabecode wirklich noch casten musst oder ob du direkt die Felder ausgeben kannst.\n- Vergleiche deinen `StepTracker` mit der bisherigen/provided Struktur in der Vorlage: ist dort eine Klasse + `public static void main(String[] args)` gefordert und ein `package`-Statement nötig?\n\n### Code Style\n- `totalSteps` ist als `public` Feld nach außen sichtbar, wird aber im Client gar nicht gebraucht; besser als lokale Variable im Konstruktor halten oder mindestens kapseln (unnötige öffentliche Zustände vermeiden).\n- Verwende konsistente Namensgebung: Entweder an die vorgegebene API halten (`avgSteps`) oder überall konsequent einen Namen verwenden (und Client entsprechend).\n- In `StepTracker` fehlen im Attempt die üblichen Imports (`java.util.Scanner`) und eine klare Klassenstruktur; selbst wenn die Umgebung “Script-Style” erlaubt, ist es in normalem Java unüblich/fehleranfällig.\n",
"status" : "SUCCESS"
}
}