Problem beim Löschen von elementen aus einer ArrayList

[juligo]

Hi,
ich habe ein Problem mit meiner ArrayList.
Ich erstelle eine ArrayList „al“ mit 5 Zufallszahlen zwischen 1 und 6.
Der Benutzer darf sich nun aussuchen wie viele Elemente er entfernen möchte und dann werden von ihm die Indizes dieser Elemente angegeben und in eine neue ArrayList „al2“ geschrieben. Soweit so gut.
Allerdings möchte ich die von dem Benutzer ausgewählten Indizes aus der Liste „al“ löschen.
Mache ich das mit remove() direkt in der Schleife stimmen die Indizes des Benutzers natürlich nicht mehr überein da die anderen Elemente ja nach „link rücken“.
Also habe ich mir überlegt die Werte einfach mit „0“ zu überschreiben und anschließend zu überprüfen ob die ArrayList „al“ die Werte 0 enthält und wenn ja diese zu löschen.
Allerdings bekomme ich genau dies nicht hin.
Meine Überlegung soweit:

		if (al.contains(0))
		{			
			al.remove(indexOf(0));
		}

Allerdings glaub ich das der Aufruf al.contains(0) mir nicht alle Nullen wieder gibt sondern nur das Erste Element?
Und al.remove(indexOf(0)) ist syntaktisch falsch.
Hat da wer andere Ansätze für mich, wie ich das Löschen hinbekomme?

Zum Überblick mal mein bisheriger Code.

import java.util.*;

class liste
{
	public static void main (String[] args)
	{
		Scanner sc = new Scanner (System.in);
		
		ArrayList al = new ArrayList();
		for (int i = 0; i<5; i++)
		{
			int k = (int)(Math.random()*6)+1;
			al.add(i, k);
		}
		System.out.println(al);
		int index = sc.nextInt();
		int index2;
		ArrayList al2 = new ArrayList();
		for (int i = 0; i<index; i++)
		{
			index2 = sc.nextInt();
			al2.add(i, al.get(index2));
			al.set(index2, 0);
		}
		if (al.contains(0))
		{			
			al.remove(indexOf(0));
		}
			
		System.out.println(al2);
		System.out.println(al);
	}
}

 

[Schandro]

for(int i=0;i<myList.size();++i){
   if(myList.get(i)==0){
      myList.remove(i);
      --i; // EDIT
   }
}

myList.remove löscht dabei das Element mit dem index i, nicht das Element mit dem Wert von i !!

 

[SlaterB]

public class Test {
    public static void main(String[] args)  {
        List<Integer> myList = new ArrayList<Integer>();
        myList.add(0);
        myList.add(0);
        for (int i = 0; i < myList.size(); ++i)
        {
            if (myList.get(i) == 0)
            {
                myList.remove(i);
            }
        }
        System.out.println("size: "+myList.size());
    }
}

Ausgabe: 1


auch nicht zu verachten:

public class Test {
    public static void main(String[] args) {
        List<Integer> myList = new ArrayList<Integer>();
        myList.add(0);
        myList.add(0);

        Integer zero = Integer.valueOf(0);
        while (myList.contains(zero))
        {
            myList.remove(zero);
        }
        System.out.println("size: " + myList.size());
    }
}

 

[juligo]

Dankeschön
Das ich die get()-Methode irgendwie hätte verwenden können hätte mir eigentlich auch wohl einfallen können. Aber nun ja. Das kommt bestimmt mit der Zeit hoffe ich.
Aber deine Zweite Lösung SlaterB find ich ziemlich elegant.
Funktioniert tadellos.
Danke nochmal

 

[SlaterB]

also das mit dem get klappt nicht unbedingt, wollte ich freundlich sagen,
es bleibt am Ende ein Element über weil jeder Index nur einmal geprüft wird, nach dem Löschen aber eine 0 auf den aktuellen Index nachrücken kann,

kann man auch korrigieren, aber contains ist schon ganz gut

 

[Schandro]

ach verdammt, hab das

--i;

in der Schleife im if-Teil vergessen, stimmt..