Elemente einer ArrayList von rechts wegnehmen

[Zootlex]

Hey Leute,

ich muss für mein Studium ein kleines Spiel erstellen.
Leider habe ich Probleme Elemente von einer ArrayListe von "rechts" wegzunehmen.



//anzahl = Anzahl der Elemente die weggenommen werden sollen.

			if(seite == 'l') {
				for(int i = 0;i < anzahl;i++) {
					currentMinion = minionsHome.get(i);
					playerMinions.add(currentMinion);
					minionsHome.remove(i);
				}
				showMinions();
				loose();
			}
	
			if(seite == 'r') {
				for(int i=minionsHome.size();i>minionsHome.size()-anzahl;i--) {
					currentMinion = minionsHome.get(i-1);
					playerMinions.add(currentMinion);
					minionsHome.remove(i-1);
				}
				showMinions();
				loose();
			}

Jedes Mal wenn ich also was von rechts wegnehmen will kriege ich folgende Exception:

Exception in thread "main" java.lang.ArrayIndexOutOfBoundsException: -1

Bitte helft mir :<

 

[Ruzmanz]

Sorry, ist beides falsch. Die for-Schleife führt die "if-Bedingung" übrigens bei jedem Durchlauf neu aus. Da du die Größe deiner Liste immer weiter verkleinerst geht das im Prinzip bis "-anzahl" weiter ...

public class DeleteTest {
	public static void main(String[] args) {
		List<String> arrayList = new ArrayList<String>(Arrays.asList("A;B;C;D;E;F;G;H;I;J;K;L;M;N;O;P;Q;R;S;T;U;V;W;X;Y;Z".split(";")));
		deleteLeft(4, arrayList);
		deleteRight(4, arrayList);
	}
	
	public static void deleteLeft(int anzahl, List<String> arrayList) {
		for(int i = 0;i < anzahl;i++) {
			String s = arrayList.get(i);
			arrayList.remove(i);
			System.out.println("Lösche: " + s);
		}
		System.out.println("Ende1");
	}
	
	public static void deleteRight(int anzahl, List<String> arrayList) {
		for(int i=arrayList.size();i>arrayList.size()-anzahl;i--) {
			String s = arrayList.get(i-1);
			arrayList.remove(i-1);
			System.out.println("Lösche: " + s);
		}
		System.out.println("Ende2");
	}
}

Ergebnis:

Lösche: A
Lösche: C
Lösche: E
Lösche: G
Ende1
Lösche: Z
Lösche: Y
Lösche: X
Lösche: W
Lösche: V
Lösche: U
Lösche: T
Lösche: S
Lösche: R
Lösche: Q
Lösche: P
Lösche: O
Lösche: N
Lösche: M
Lösche: L
Lösche: K
Lösche: J
Lösche: I
Lösche: H
Lösche: F
Lösche: D
Lösche: B
Exception in thread "main" java.lang.ArrayIndexOutOfBoundsException: -1
at java.util.ArrayList.elementData(ArrayList.java:418)
at java.util.ArrayList.get(ArrayList.java:431)
at DeleteTest.deleteRight(DeleteTest.java:24)
at DeleteTest.main(DeleteTest.java:10)

 

[Foxei]

Hi Mach es doch so

public class DeleteTest {
    public static void main(String[] args) {
        List<String> arrayList = new ArrayList<String>(Arrays.asList("A;B;C;D;E;F;G;H;I;J;K;L;M;N;O;P;Q;R;S;T;U;V;W;X;Y;Z".split(";")));
        deleteLeft(4, arrayList);
        deleteRight(4, arrayList);
    }
 
    public static void deleteLeft(int anzahl, List<String> arrayList) {
        for(int i = 0;i < anzahl;i++) {
            String s = arrayList.get(i);
            arrayList.remove(i);
            System.out.println("Lösche: " + s);
        }
        System.out.println("Ende1");
    }
 
    public static void deleteRight(int anzahl, List<String> arrayList) {
        for(int i=1;i<=anzahl;i++) {
            String s = arrayList.get(arrayList.size()-i);
            arrayList.remove(arrayList.size()-i);
            System.out.println("Lösche: " + s);
        }
        System.out.println("Ende2");
    }
}

Ausgabe:

Lösche: A
Lösche: C
Lösche: E
Lösche: G
Ende1
Lösche: Z
Lösche: X
Lösche: V
Lösche: T
Ende2

 

[Foxei]

Du weist aber das du damit nicht die ersten 4 Elemente Löschst bzw die letzte. !!!!!!!!!
Um die ersten 4 bzw die letzten 4 zu löschen muss es so aussehen:

public class DeleteTest {
	public static void main(String[] args) {
		List<String> arrayList = new ArrayList<String>(Arrays.asList("A;B;C;D;E;F;G;H;I;J;K;L;M;N;O;P;Q;R;S;T;U;V;W;X;Y;Z".split(";")));
		deleteLeft(4, arrayList);
		deleteRight(4, arrayList);
	}
 
	public static void deleteLeft(int anzahl, List<String> arrayList) {
		for(int i = 0;i < anzahl;i++) {
			String s = arrayList.get(0);
			arrayList.remove(0);
			System.out.println("Lösche: " + s);
		}
		System.out.println("Ende1");
	}
 
	public static void deleteRight(int anzahl, List<String> arrayList) {
		for(int i=0;i<anzahl;i++) {
			String s = arrayList.get(arrayList.size()-1);
			arrayList.remove(arrayList.size()-1);
			System.out.println("Lösche: " + s);
		}
		System.out.println("Ende2");
	}
}

Ausgabe:

Lösche: A
Lösche: B
Lösche: C
Lösche: D
Ende1
Lösche: Z
Lösche: Y
Lösche: X
Lösche: W
Ende2

 

[Ruzmanz]

Du weist aber das du damit nicht die ersten 4 Elemente Löschst bzw die letzte. !!!!!!!!!

War das an mich gerichtet? Bei so "einfachen" Sachen bringt es nichts die Lösung zu posten. Die Leute müssen lernen ihre Probleme selbst zu lösen. Ich habe lediglich seinen Quelltext vereinfacht und genau so implementiert wie der TE. Anhand meiner Vorarbeit sollte man erkennen wo die Ausgabe nicht mit dem Erwartungswert übereinstimmt ...

 

[Foxei]

Okay Sorry
Also wir nehmen jetzt mal so wie Ruzmanz deinen Code verändert hat an du hättest eine Array mit allen 26 Buchstaben des Alphabetes. Und wir möchten nun die die ersten vier und die letzten vier Buchstaben Löschen also A B C D und Z Y X W.

public static void deleteLeft(int anzahl, List<String> arrayList) {
		for(int i = 0;i < anzahl;i++) {
			String s = arrayList.get(i);
			arrayList.remove(i);
			System.out.println("Lösche: " + s);
		}
		System.out.println("Ende1");
	}

Ausgabe:

Lösche: A
Lösche: C
Lösche: E
Lösche: G

Wie hier Auffallen sollte sind das nicht die ersten vier Buchstaben des Alphabets.
Warum das denn nicht? Das ist ziemlich Simpel du beim ersten Durchlauf ist i=0 Damit nimmst du den ersten Buchstaben weg nämlich A. beim zweiten Durchlauf ist i=1 damit nimmst du den zweiten Buchstaben des ohnehin schon kleiner gewordenen Arrays weg :/ also ein C. Das summiert sich dann und führt dazu das du nicht die ersten vier weg nimmst.
Also musst du immer den ersten weg nehmen sprich anstatt i einzusetzen muss da immer 0 rein. Denn wenn du den Nullten Buchstaben weg nimmst also das A wird der Neue Nullte Buchstabe ja automatisch ein B. usw.

public static void deleteLeft(int anzahl, List<String> arrayList) {
		for(int i = 0;i < anzahl;i++) {
			String s = arrayList.get(0);
			arrayList.remove(0);
			System.out.println("Lösche: " + s);
		}
		System.out.println("Ende1");
	}

Ausgabe:

Lösche: A
Lösche: B
Lösche: C
Lösche: D

Nun zu den letzten Vier:

public static void deleteRight(int anzahl, List<String> arrayList) {
		for(int i=arrayList.size();i>arrayList.size()-anzahl;i--) {
			String s = arrayList.get(i-1);
			arrayList.remove(i-1);
			System.out.println("Lösche: " + s);
		}
		System.out.println("Ende2");
	}

Ausgabe:

Lösche: Z
Lösche: Y
Lösche: X
Lösche: W
Lösche: V
Lösche: U
Lösche: T
Lösche: S
Lösche: R
Lösche: Q
Lösche: P
Lösche: O
Lösche: N
Lösche: M
Lösche: L
Lösche: K
Lösche: J
Lösche: I
Lösche: H
Lösche: F
Lösche: D
Lösche: B
Exception in thread "main" java.lang.ArrayIndexOutOfBoundsException: -1
at java.util.ArrayList.elementData(ArrayList.java:418)
at java.util.ArrayList.get(ArrayList.java:431)
at DeleteTest.deleteRight(DeleteTest.java:24)
at DeleteTest.main(DeleteTest.java:10)

Also "i>arrayList.size()-anzahl" Müsste je eigentlich funktionieren wenn du nicht in der schleife die zahl die für arrayList.size() gegeben ist nicht immer dynamisch um eins kleiner machen würdest diese Schleife hat somit ein kein Ende.

Also neu Gedacht wie oben wissen wir ja das wir durch anzahl eine zahl haben nach der wir uns richten können da diese fest ist und in der Schleife nicht verändert wird. Also machen wir uns auch hier eine Schleife wie oben.

for(int i = 0;i < anzahl;i++) {
			
		}

Nun ent spricht die Ausgabe hier auch nicht den letzten vier wir wollen ja immer den letzten wer aus der List Löschen. Mit arrayList.size(); bekommen wir die momentane länge der list da die liste aber mit dem Wert 0 beginnt müssen wir hier noch eins abziehen und haben unser Ergebniss.

public static void deleteRight(int anzahl, List<String> arrayList) {
		for(int i=0;i<anzahl;i++) {
			String s = arrayList.get(arrayList.size()-1);
			arrayList.remove(arrayList.size()-1);
			System.out.println("Lösche: " + s);
		}
		System.out.println("Ende2");
	}

Noch Fragen?