Die unerklärliche Fehlermeldung

[Almanex]

Guten Morgen allerseits,

seit Tagen beschäftige ich mich mit der Bounds-Exception. Ich habe schon viele Ansätze probiert, aber kein Ansatz hat mich weiter gebracht.

Die Liste enthält 16 Element, über die Methode remove() übergebe ich einen Ganzzahl-Parameter und möchte diese Elemente aus der Liste löschen. Die gelöschte Elemente in einer anderen Listen speichern. Wenn ich 8 von 16 Elemente bzw. <= 8 löschen möchte, dann funktioniert es problemlos. Sobald ich Elemente > 8 löschen möchte, wirft Eclipse die Bounds-Fehlermeldung. Wer kann mir helfen?

Ich habe euch die betroffenen Stellen rot markiert.

Mit freundlichen Grüßen

Almanex

 

[kneitzel]

Du hast die falsche Zeile markiert - die Zeile, in der der Fehler aufgetreten ist, ist 2 Zeilen tiefer wo Du Element 16 von ballListOfPool abfragst (Zeile 102 - wie auch im Stacktrace vermerkt).

Und wir sehen nichts zu ballListOfPool, daher können wir Dir nicht sagen, wieso Da nur 14 Elemente drin sind und nicht 17, wie Du anscheinend annimmst.

Anmerkung: Codem stacktrace und so bitte in Code-Tags posten! Also über den Knopf </> links oberhalb des Eingabebereichs.

 

[mihe7]

Das Problem dürfte sein, dass Du über die Indizes 1 bis n iterierst, in der Schleife dann das betreffende Element löscht. Durch das Löschen verschieben sich die nachfolgenden Elemente nach vorne.

 

[Almanex]

[CODE lang="java" title="Die Liste"]List<BallPool> ballListOfPool = new ArrayList<>();
ballListOfPool.add(new BallPool(Color.RED, build2));
ballListOfPool.add(new BallPool(Color.WHITE, build2));
ballListOfPool.add(new BallPool(Color.YELLOW, build2));
ballListOfPool.add(new BallPool(Color.RED, build2));
ballListOfPool.add(new BallPool(Color.WHITE, build2));
ballListOfPool.add(new BallPool(Color.YELLOW, build2));
ballListOfPool.add(new BallPool(Color.RED, build2));
ballListOfPool.add(new BallPool(Color.WHITE, build2));
ballListOfPool.add(new BallPool(Color.YELLOW, build2));
ballListOfPool.add(new BallPool(Color.RED, build2));
ballListOfPool.add(new BallPool(Color.WHITE, build2));
ballListOfPool.add(new BallPool(Color.YELLOW, build2));
ballListOfPool.add(new BallPool(Color.RED, build2));
ballListOfPool.add(new BallPool(Color.WHITE, build2));
ballListOfPool.add(new BallPool(Color.YELLOW, build2));
ballListOfPool.add(new BallPool(Color.RED, build2));[/CODE]

[CODE lang="java" title="Remove-Methode"]public List<? extends BilliardBall> removeBalls(int numberOfRemoveBalls) {

List<BilliardBall> listOfremoveBalls = new ArrayList<>();

if (numberOfRemoveBalls <= listOfBalls.size()) {
for (int i = 1; i <= numberOfRemoveBalls; i++) {
listOfremoveBalls.add(listOfBalls.get(i - 1));
listOfBalls.remove(i - 1);
}
}
return listOfremoveBalls;
}[/CODE]

[CODE lang="java" title="Aufruf"]System.out.println(pool.removeBalls(14));[/CODE]

Habe den Code danach bisschen angepasst, aber jetzt ist das Bild auf dem aktuellsten Stand
Die Fehlermeldung:

 

[mihe7]

Nein, es geht um die Schleife selbst.

Wenn Du 2 Elemente in der Liste hast, dann lässt Du i von 1 bis inkl. 2 laufen. Jetzt löscht Du im ersten Durchlauf das Element mit Index 0. Dadurch hast die Liste nur noch ein Element mit Index 0; i wird um 1 erhöht beim nächsten Durchlauf krachts natürlich.

Es gibt zwei Möglichkeiten: entweder löschst Du immer das Element an Index 0 oder aber Du zählst rückwärts.

 

[Almanex]

Vielen Dank, mihe7!

Jetzt leuchtet es ein 😅

Die Lösung:
[CODE lang="java" title="Remove-Methode"]public List<? extends BilliardBall> removeBalls(int numberOfRemoveBalls) {

List<BilliardBall> listOfremoveBalls = new ArrayList<>();

if (numberOfRemoveBalls <= listOfBalls.size()) {
for (int i = numberOfRemoveBalls - 1; i >= 0; i--) {
listOfremoveBalls.add(listOfBalls.get(i));
listOfBalls.remove(i);
}
}
return listOfremoveBalls;
}[/CODE]