{
"llm" : {
"feedback" : "# Exercise: prime\n\n### Correctness\n- `isPrime` liefert aktuell immer `false` zurück und prüft `x` nicht auf Primzahl-Eigenschaften wie in der Aufgabe verlangt.\n\n### Suggestion\n- Überlege dir eine Sequenz von Kandidaten `t`, die als Teiler in Frage kommen (gemäss Definition: `t > 1` und `t < x`) und prüfe, ob es irgendein `t` gibt, für das `x % t == 0` gilt.\n- Denke auch daran, was passieren soll, wenn kein solcher Teiler gefunden wird: Was bedeutet das für den Rückgabewert?\n\n### Code Style\n- Der `// TODO`-Kommentar ist ok, aber im finalen Stand sollte er entfernt/ersetzt werden, sobald die Methode implementiert ist.\n\n\n# Exercise: binsearchfirst\n\n### Correctness\n- `binSearch` ist nicht implementiert und liefert immer `0`, dadurch werden die geforderten Resultate (z.B. Suche nach 14 → 7, 24 → 10, 17 → 9, …) nicht erfüllt.\n- Die Anforderungen „erstes Vorkommen bei Duplikaten“ und „Einfügeposition, falls nicht vorhanden“ werden aktuell gar nicht umgesetzt.\n\n### Suggestion\n- Baue eine echte binäre Suche mit zwei Grenzen (links/rechts) auf und entscheide in der Schleife so, dass du am Ende direkt die Einfügeposition ablesen kannst.\n- Achte bei der Fallunterscheidung darauf, dass bei `value` gleich einem Array-Element nicht sofort zurückgegeben wird, sondern die Suche so weiterläuft, dass du beim *ersten* passenden Index landest (Gedanke: Grenze so verschieben, dass links nur „zu kleine“ Werte liegen).\n- Überlege dir ein Schleifen-Invariant: Welche Werte sollen garantiert links liegen, welche rechts? Daraus ergibt sich, ob du bei `data[mid] < value` oder bei `data[mid] >= value` welche Grenze verschiebst.\n- Wenn die Schleife endet, sollte der zurückgegebene Index sowohl für „nicht gefunden“ (Einfügeposition) als auch für „gefunden“ (erster Treffer) funktionieren.\n\n### Code Style\n- Der `TODO`-Kommentar ist ok, aber aktuell ist die Methode ein Stub; entferne/ersetze ihn, sobald die Logik implementiert ist.\n\n\n# Exercise: tripleseqsearch\n\n### Correctness\n- Wenn keine `7` im Array vorkommt, wird nach der ersten Schleife trotzdem `i++` ausgeführt und dadurch „als ob“ eine 7 gefunden worden wäre weitergesucht; damit kann die Methode fälschlicherweise `true` liefern.\n- Wenn die `7` erst am letzten Index steht, führt das anschließende `i++` dazu, dass die Suche nach der negativen Zahl gar nicht mehr stattfinden kann; hier sollte die Methode eindeutig `false` liefern, statt einfach mit „Index hinter dem Array“ weiterzuarbeiten.\n- Wenn nach der `7` keine negative Zahl mehr vorkommt, wird trotzdem `i++` ausgeführt und danach wird nach einer geraden Zahl gesucht, obwohl die negative Zahl gar nicht gefunden wurde; das kann ebenfalls zu falschen `true`-Resultaten führen.\n- Wenn nach der negativen Zahl keine gerade Zahl mehr vorkommt, wird trotzdem `i++` ausgeführt; damit prüfst du nicht sauber, ob tatsächlich eine gerade Zahl gefunden wurde (du verschiebst den Index unabhängig vom Sucherfolg).\n\n### Suggestion\n- Prüfe nach jeder der drei Such-Phasen explizit, ob das gesuchte Element wirklich gefunden wurde (z.B. ob du wegen „Treffer“ oder wegen „Ende des Arrays“ aus der Schleife raus bist), bevor du mit der nächsten Phase weitermachst.\n- Überlege, ob du das `i++` wirklich immer pauschal nach jeder Schleife brauchst: Es reicht meist, ab dem gefundenen Index weiterzusuchen (oder gezielt ab „nächstem Index“), aber nur dann, wenn du wirklich etwas gefunden hast.\n- Achte darauf, dass „7, negative Zahl, gerade Zahl“ in dieser Reihenfolge gefunden werden müssen; wenn eine Stufe nicht gefunden wird, sollte sofort `false` zurückkommen, statt mit dem nächsten Kriterium weiterzusuchen.\n- Teste gedanklich Randfälle: leeres Array, Array ohne 7, 7 ganz am Ende, 7 vorhanden aber keine negative danach, negative vorhanden aber keine gerade danach.\n\n### Code Style\n- Die `// TODO`-Kommentarzeile ist nicht mehr passend, da du bereits implementiert hast; entferne oder ersetze sie.\n- Die Logik ist aktuell schwer nachvollziehbar, weil `i++` dreimal „blind“ nach den Schleifen passiert; lesbarer wäre es, die Absicht (Startindex für nächste Suche) klarer auszudrücken und nur bei Erfolg weiterzuschalten.\n\n\n# Exercise: sqrt\n\n### Correctness\n- `squareRoot(double x)` liefert immer `0` zurück und berechnet keine Quadratwurzel per binärer Suche, wie in der Aufgabenstellung gefordert.\n- Die geforderte Abbruchbedingung der Suche („wenn zwischen `low` und `high` keine weitere darstellbare Zahl mehr existiert“, z.B. `low == mid || high == mid`) ist nicht implementiert.\n- Die Initialisierung der Startgrenzen (`low`, `high`) abhängig davon, ob `x > 1` oder `x < 1` ist, fehlt.\n- Die Entscheidung, ob `mid` zur unteren oder oberen Grenze wird (Vergleich von `mid*mid` mit `x`), fehlt.\n- Die Rückgabe der besseren der beiden Grenzen anhand der kleineren Abweichung (`|low^2 - x|` vs. `|high^2 - x|`) ist nicht umgesetzt.\n\n### Suggestion\n- Überlege dir zuerst, wie du `low` und `high` so setzt, dass die Wurzel sicher dazwischen liegt (die Aufgabe gibt dir dafür direkt die Ungleichungen für `x > 1` und `x < 1`).\n- Implementiere dann eine Schleife, die `mid` aus `low` und `high` berechnet (achte auf die in der Aufgabe angegebene Formel) und anschließend anhand von `mid*mid` entscheidet, welche Grenze ersetzt wird.\n- Nutze als Stoppkriterium nicht eine feste Anzahl Iterationen, sondern genau den Fall, dass `mid` nicht mehr „zwischen“ den Grenzen liegt (`low == mid` oder `high == mid`), weil dann keine darstellbare Zahl mehr dazwischen existiert.\n- Wenn die Schleife beendet ist, vergleiche, welche der beiden Grenzen das Quadrat näher an `x` bringt, und gib diese Grenze zurück.\n\n### Code Style\n- Der `// TODO`-Kommentar ist noch vorhanden, obwohl hier die geforderte Logik implementiert werden soll.\n",
"status" : "SUCCESS"
},
"unitTest" : {
"tests" : [ {
"name" : "testTooSmallInteger()",
"status" : "PASSED",
"message" : null
}, {
"name" : "testTooLargeInteger()",
"status" : "FAILED",
"message" : "expected: <5> but was: <0>"
}, {
"name" : "testZeroLengthArray()",
"status" : "PASSED",
"message" : null
}, {
"name" : "testNonExistingInteger()",
"status" : "FAILED",
"message" : "expected: <4> but was: <0>"
}, {
"name" : "testEqualInteger()",
"status" : "FAILED",
"message" : "expected: <11> but was: <0>"
} ]
}
}