ArrayList und Threads

[Campino]

Hi,
Ich greife in einem Thread auf eine ArrayList zu, indem ich einen Iterator erstelle. Dann nehme ich mir nacheinander Objekte aus einer LinkedList (moten), index 0-99 weiße ihnen über die getInput function die Elemente aus der ArrayList zu (Dabei ist eigentlich egal, welches Objekt aus moten welches aus inputs erhält). Das ganze sieht so aus:

public void run(){
        while(true){
            Iterator inp=inputs.listIterator();
            for(int t=0;inp.hasNext();t++){
                if(t>99){
                    t=0;
                }
                Mote m=(Mote)moten.get(t);
                Input i=(Input)inp.next();  //line 62
                m.getInput(i);
            }
        }
    }

Der Debugger meldet:

Exception in thread "Thread-0" java.util.ConcurrentModificationException
        at java.util.AbstractList$Itr.checkForComodification(AbstractList.java:449)
        at java.util.AbstractList$Itr.next(AbstractList.java:420)
        at Matrix.run(Matrix.java:62)

Woran liegt das?? Danke schon mal...

 

[foobar]

ArrayList ist nicht Threadsave

Note that this implementation is not synchronized. If multiple threads access an ArrayList instance concurrently, and at least one of the threads modifies the list structurally, it must be synchronized externally. (A structural modification is any operation that adds or deletes one or more elements, or explicitly resizes the backing array; merely setting the value of an element is not a structural modification.) This is typically accomplished by synchronizing on some object that naturally encapsulates the list. If no such object exists, the list should be "wrapped" using the Collections.synchronizedList method. This is best done at creation time, to prevent accidental unsynchronized access to the list:

http://java.sun.com/j2se/1.3/docs/api/java/util/ArrayList.html

 

[Campino]

*Fluch* Was soll ich denn nehmen? Gibt es eine Threadsichere variante?

 

[dotlens]

kannsts ja einfach snchronyzen, nicht?

 

[Campino]

Wie geht das den ???:L ???:L

 

[Beni]

So geht das:

public void run(){
        while(true){
            synchronized( inputs ){
                Iterator inp=inputs.listIterator();
                for(int t=0;inp.hasNext();t++){
                    if(t>99){
                        t=0;
                    }
                    Mote m=(Mote)moten.get(t);
                    Input i=(Input)inp.next();  //line 62
                    m.getInput(i);
                }
            }
        }
    }

 

[bygones]

nimm Vector - nix anderes als synchronisierte ArrayList...

oder List list = Collections.synchronizedList(new ArrayList(...));