Konkurierender Zugriff auf Iterierende Liste

[Tokka]

Hallo!

Ich habe ein Problem: In meinem Programm habe ich eine Liste, die Ich über einen Iterator durchlaufe

for (ClientInterface client : gamePlayers)
		{
			try
			{
				Gambler gambler = this.localGambler.get(client);
				GameState gs = getGamblerGameState(gambler);
				client.onEndGame("Runde zu ende", gs);
			}
			catch (RemoteException e)
			{
				localDisconnect(client);
			}
		}

Das Problem ist nur, das es vorkommen kann, das der Iterator gerade das Element liest, welches gerade geändert wird.
Es kommt zu einem Konkurierenden Zugriff und Java meldet sich mit einer Fehlermeldung.

Wie kann ich dieses Problem umgehen??? Derzeit kopiere ich die liste, bevor ich sie Iteriere, aber das finde ich eine nicht so tolle Lösung.

Welche Vorschläge habt Ihr??

Gruß & Danke

Tokka

 

[Lim_Dul]

Kennst du andern Stellen, von wo drauf zugegriffen wird?

Wenn ja, dann kannst du folgendes machen:

synchronized(gamePlayers) {
for (ClientInterface client : gamePlayers)
      {
         try
         {
            Gambler gambler = this.localGambler.get(client);
            GameState gs = getGamblerGameState(gambler);
            client.onEndGame("Runde zu ende", gs);
         }
         catch (RemoteException e)
         {
            localDisconnect(client);
         }
      }
}

Und gleiches auch an jeder Stelle, wo auf was geändert werden könnte.

 

[Guest]

Da die RMI Zugriffe länger dauern als das Erstellen der Kopie,
würde ich folgende Lösung vorziehen.

ClientInterface clientList[];
synchronized(gamePlayers)
{
  clientList = gamePlayers.toArray(new ClientInterface[gamePlayers.size()]);
}
for (ClientInterface client : clientList)
{
  try
  {
    Gambler gambler = this.localGambler.get(client);
    GameState gs = getGamblerGameState(gambler);
    client.onEndGame("Runde zu ende", gs);
  }
  catch (RemoteException e)
  {
    localDisconnect(client);
  }
}

Mach dir keine Gedanken über das Erstellen der Arrays. So viele Clients hast
du sicherlich gar nicht, dass dies zum (Speicher)Problem werden könnte.

 

[Tokka]

Danke Lim_Dul, hat geklapt

Gruß
Tokka