Fehler beim AsyncPlayerChatEvent

[Darkblader24]

Hallo

Ich bekomme in meinem Plugin jeden Tag ein paar mal folgenden Fehler:

Spoiler: Error in der Console

2013-05-06 18:24:47 [SEVERE] Could not pass event AsyncPlayerChatEvent to PPS v3.0
org.bukkit.event.EventException
	at org.bukkit.plugin.java.JavaPluginLoader$1.execute(JavaPluginLoader.java:427)
	at org.bukkit.plugin.RegisteredListener.callEvent(RegisteredListener.java:62)
	at org.bukkit.plugin.TimedRegisteredListener.callEvent(TimedRegisteredListener.java:26)
	at org.bukkit.plugin.SimplePluginManager.fireEvent(SimplePluginManager.java:479)
	at org.bukkit.plugin.SimplePluginManager.callEvent(SimplePluginManager.java:461)
	at net.minecraft.server.v1_5_R3.PlayerConnection.chat(PlayerConnection.java:900)
	at net.minecraft.server.v1_5_R3.PlayerConnection.a(PlayerConnection.java:840)
	at net.minecraft.server.v1_5_R3.Packet3Chat.handle(Packet3Chat.java:44)
	at org.spigotmc.netty.NettyNetworkManager$2.run(NettyNetworkManager.java:113)
	at java.util.concurrent.Executors$RunnableAdapter.call(Executors.java:471)
	at java.util.concurrent.FutureTask$Sync.innerRun(FutureTask.java:334)
	at java.util.concurrent.FutureTask.run(FutureTask.java:166)
	at java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1110)
	at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:603)
	at java.lang.Thread.run(Thread.java:722)
Caused by: java.util.ConcurrentModificationException
	at java.util.ArrayList$Itr.checkForComodification(ArrayList.java:819)
	at java.util.ArrayList$Itr.next(ArrayList.java:791)
	at de.PPS.Chat.Chat.sendChatroomMessage(Chat.java:547)
	at de.PPS.Chat.Chat.Chat(Chat.java:504)
	at sun.reflect.GeneratedMethodAccessor213.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:425)
	... 14 more

Daraufhin stüzt der Server ab und es werden sämtliche Spieler gekickt. Und nach ca 20 Sekunden kann man dann einfach wieder drauf joinen, als wäre nix gewesen.

Mein Code auf Zeile 504 ist folgender

sendChatroomMessage("Global", nachricht);

In der Methode, die da aufgerufen wird, steht folgendes:

public static void sendChatroomMessage(String chatraum,String nachricht){
	if(chatraum.equalsIgnoreCase("Global")){
		for (Player GlobaleP : GlobalChat) {
			GlobaleP.sendMessage(nachricht);
		}
	}
	if(chatraum.equalsIgnoreCase("Clan")){
		for (Player ClanP : ClanChat) {
			ClanP.sendMessage(nachricht);
		}
	}
}

Wo liegt hier der Fehler?
Das komische ist, dass das nur manchmal passiert.

Vielen Dank im Vorraus

 

[games6471]

Dein GlobalChat oder ClanChat wird während dem Iterieren verändert.

ConcurrentModificationException

Die nervigste Exception welche auftritt beim Arbeiten mit mehren Threads.

 

[Darkblader24]

Stimmt! Es werden ja dauernd Leute in der Liste verändert, da ja ständig welche joinen oder quitten.
Aber wie kann ich dann soetwas beheben?

 

[Crafter6432]

In dem du den Sheduler nutzt um die nachrichten über den Hauptthread zu verschicken.
http://jd.bukkit.org/rb/apidocs/org....html#callSyncMethod(org.bukkit.plugin.Plugin, java.util.concurrent.Callable)

 

[ChristianG]

Faustregel:
Nicht auf die Bukkit API ausm Sheduler direkt zugreifen!