Concurrent Modification Exception - HashMap (kein remove)

[Zelnehlun]

Hallo,
als erstes die Java Code Ausschnitte:

static HashMap<String, WWorld> worlds = new HashMap<String, WWorld>();

public static void onServerTick(ServerTickEvent e){
		if(ticks % 100 == 0){
			synchronize();
		}
		
		if(ticks % 400 == 0){
			spawnMonsters();
			spawnAnimals();
		}
		
		if(ticks % 600 == 0){
			checkSpawner();
		}
		
		ticks++;
	}
	
	// methods
	
	public static void checkSpawner(){
		for(WWorld world:worlds.values()){
			for(CChunk chunk:world.getChunks()){
				chunk.checkSpawner();
			}
		}
	}

public void checkSpawner(){
			if(!isProtected(chunk)){
				BlockState[] states = chunk.getTileEntities();
				
				for(BlockState state:states){
					if(state instanceof CreatureSpawner){
						CreatureSpawner spawner = (CreatureSpawner)state;
						Block block = spawner.getBlock();
						
						for(int i = 0; i < 2; i++){
							if(!isSpawnerCapReached()){
								int modX = Rand.nextInt(3) + 1;
								int modZ = Rand.nextInt(3) + 1;
								block = block.getRelative(modX, -1, modZ);
								
								if(isOpaque(block)){
									block = block.getRelative(BlockFace.UP);
									
									if(isTransparent(block)){
										block = block.getRelative(BlockFace.UP);
										
										if(isTransparent(block)){
											block = block.getRelative(BlockFace.DOWN);
											World world = chunk.getWorld();
											Location loc = block.getLocation();
											EntityType type = spawner.getSpawnedType();
											Entity ent = world.spawnCreature(loc, type);
											int id = ent.getEntityId();
											
											spawnerEntities.put(id, ent);
											SFX.playEffect(spawner.getBlock(), Effect.MOBSPAWNER_FLAMES);
											SFX.playEffect(ent, Effect.MOBSPAWNER_FLAMES);
										}
									}
								}
							}else{
								break;
							}
						}
					}
				}
			}
		}
		
	}

public static class WWorld {
		
		String name;
		HashMap<String, CChunk> chunks = new HashMap<String, CChunk>();
		
		public WWorld(World world){
			this.name = world.getName();
			Chunk[] chunks = world.getLoadedChunks();
			
			for(Chunk chunk:chunks){
				registerChunk(chunk);
			}
		}
		
		public Collection<CChunk> getChunks(){
			return chunks.values();
		}
		
		public void registerChunk(Chunk chunk){
			CChunk cChunk = new CChunk(chunk);
			String name = cChunk.name;
			
			chunks.put(name, cChunk);
		}
		
		public CChunk getChunk(Chunk chunk){
			String name = chunk.getX() + " " + chunk.getZ();
			
			return chunks.get(name);
		}
		
	}
	
	public static class CChunk {
		
		Chunk chunk;
		String name;
		HashMap<Integer, Entity> entities = new HashMap<Integer, Entity>();
		HashMap<Integer, Entity> spawnerEntities = new HashMap<Integer, Entity>();
		
		public CChunk(Chunk chunk){
			this.chunk = chunk;
			name = chunk.getX() + " " + chunk.getZ();
			Entity[] ents = chunk.getEntities();
			
			for(Entity ent:ents){
				if(isLivingEntity(ent)){
					addEntity(ent);
				}
			}
		}

[...]

Hier die Fehlermeldung:

[SEVERE] Could not pass event ServerTickEvent to Alonian
java.util.ConcurrentModificationException
	at java.util.HashMap$HashIterator.nextEntry(HashMap.java:806)
	at java.util.HashMap$ValueIterator.next(HashMap.java:835)
	at zelnehlun.alonian.WorldPopulator.checkSpawner(WorldPopulator.java:114)
	at zelnehlun.alonian.WorldPopulator.onServerTick(WorldPopulator.java:104)
	at zelnehlun.listeners.WorldListener.onServerTick(WorldListener.java:32)
	at sun.reflect.GeneratedMethodAccessor6.invoke(Unknown Source)
	at sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
	at java.lang.reflect.Method.invoke(Method.java:601)
	at org.bukkit.plugin.java.JavaPluginLoader$1.execute(JavaPluginLoader.java:301)
	at org.bukkit.plugin.RegisteredListener.callEvent(RegisteredListener.java:62)
	at org.bukkit.plugin.SimplePluginManager.callEvent(SimplePluginManager.java:461)
	at org.getspout.spout.ServerTickTask.run(ServerTickTask.java:49)
	at org.bukkit.craftbukkit.scheduler.CraftScheduler.mainThreadHeartbeat(CraftScheduler.java:126)
	at net.minecraft.server.MinecraftServer.w(MinecraftServer.java:588)
	at net.minecraft.server.MinecraftServer.run(MinecraftServer.java:506)
	at net.minecraft.server.ThreadServerApplication.run(SourceFile:492)

Hier noch die gesamte Klasse:
[Java] WorldPopulator - Pastebin.com

Nun meine Frage: Die Fehlermeldung tritt in unregelmäßigen Abständen auf, während ich durch die HashMaps iteriere verändere ich die Menge nicht, wie kommt es dann zu dieser Fehlermeldung?

Ich hoffe mir kann jemand helfen :/

Gruß Zelnehlun

 

[Gast2]

Irgendwer (vielleicht nen anderer Thread(?) fuscht in der worlds Liste rum während du darüber iterierst. In den kurzen Stück Code ist kein Fehler zu sehen.

 

[Zelnehlun]

Hm ich verwende keine anderen Threads, da es nur Probleme durch das asynchrone Arbeiten geben würde

Die World List wird ebenfalls ausschließlich von dieser Klasse genutzt, und das ServerTickEvent ist synchron.

Edit:

Die vollständige Klasse ist im ersten Beitrag als Link angehängt.

 

[Marco13]

...während ich durch die HashMaps iteriere verändere ich die Menge nicht,

Entweder du hast einen Fehler in HashMap entdeckt, der dort seit 15 Jahren von zig-tausenden von Entwicklern trotz hunderttausender Programme mit Millionen Codezeielen unentdeckt vor sich hin schlummerte, oder... du veränderst die Menge DOCH. Man könnte jetzt sagen: Joa, nimm' ne ConcurrentHashMap (oder so), aber ... ob das die geeignetste Lösung ist, kann man mangels Code nicht sagen.
Poste mal die "CCHunk#checkSpawner"-Methode.

 

[SlaterB]

zuverlässig ist meistens, die Map (und sei es temporär) durch eine eigene Klasse zu ersetzen/ wrappen, darin nur einige Methoden wie put/ remove überschreiben, alle anderen mit Exception implementieren

und die Änderungen loggen, dann sieht man ob was beim Durchlauf passiert, falls dieser auch mit Anfang/ Ende geloggt wird

static HashMap<String, WWorld> worlds = new HashMap<String, WWorld>();
->
static Map<String, WWorld> worlds = new MyWorldMap();
oder gar
static MyWorldMap worlds = new MyWorldMap();

Variablen eh immer vom Interface-Typ, hier Map oder auch List statt ArrayList,
das merke dir, recht nützlich