ConcurrentModificationException in HashMap

[huckleberry]

Huhuu Forum,

Ich habe folgende HashMap. Objekte mit einem älteren Zeitstempel als TIMEOUT_LOCATION_TABLE sollen rausgeworfen werden:

        private HashMap<String, SpecialObj> myHashMap = new private HashMap<String, SpecialObj>();
        // bla HashMap voll machen

	public void updateHashMapEntries(long cTime, LogFileWriter l) {
		Iterator<SpecialObj> it = this.myHashMap.values().iterator();
		boolean mode = false;
		SpecialObj v;
		while ( it.hasNext() ){
			v = it.next();
			long vts = v.getTimeStamp();
			if( vts + Bla.TIMEOUT_LOCATION_TABLE < cTime ) {
				if (!mode) {
					l.getLogRibbon();
					l.writeLog("[UPDATE] TimeOut expired! Update Location Table!");
					mode = true;
				}
				String s = v.getId();
				this.myHashMap.keySet().remove( s );
				l.writeLog(	"      [-] "+s+") with "+this.myHashMap.size()+
						" entries | "+TimeUnit.NANOSECONDS.toMillis(vts)+
						" + "+TimeUnit.NANOSECONDS.toMillis(Bla.TIMEOUT_LOCATION_TABLE)+
						" = "+TimeUnit.NANOSECONDS.toMillis(vts + Bla.TIMEOUT_LOCATION_TABLE) );
			}
		}
	}

und bekomme folgende Exception

java.util.ConcurrentModificationException
	at java.util.HashMap$HashIterator.nextEntry(HashMap.java:793)
	at java.util.HashMap$ValueIterator.next(HashMap.java:822)

geworfen in der Zeile 10 mit

v = it.next();

Hat jemand Rat? Ich danke im voraus.

Mfg Huck

 

[huckleberry]

Alles klar, ich lösche in einer iterierten HashMap ;(

Soll ich eine neue temporäre anlegen und aus dieser löschen und am schluss diese in die eigentliche kopieren? Oder geht das auch effizienter?

public void updateHashMapEntries(long cTime, LogFileWriter l) {
        HashMap<String, SpecialObj> tHM = new HashMap<String, SpecialObj>();
        tHM  = this.myHashMap;
        Iterator<SpecialObj> it = tHM.values().iterator();           
        boolean mode = false;
        SpecialObj v;
        while ( it.hasNext() ){
            v = it.next();
            long vts = v.getTimeStamp();
            if( vts + Bla.TIMEOUT_LOCATION_TABLE < cTime ) {
                if (!mode) {
                    l.getLogRibbon();
                    l.writeLog("[UPDATE] TimeOut expired! Update Location Table!");
                    mode = true;
                }
                String s = v.getId();
                tHM.keySet().remove( s );
                l.writeLog( "      [-] "+s+") with "+tHM.size()+
                        " entries | "+TimeUnit.NANOSECONDS.toMillis(vts)+
                        " + "+TimeUnit.NANOSECONDS.toMillis(Bla.TIMEOUT_LOCATION_TABLE)+
                        " = "+TimeUnit.NANOSECONDS.toMillis(vts + Bla.TIMEOUT_LOCATION_TABLE) );
            }
        }
     this.myHashMap = tHM;
    }

 

[SlaterB]

it.remove();
ist effizienter

oder nur die Keys in einer Liste sammeln und nach der Schleife löschen

siehe auch
http://www.java-forum.org/java-basi...ncurrentmodificationexception.html#post613508