Discussion :

[Kaneda Shotaro]

Bonsoir
Voici mon soucis : j'ai un serveur. Ce dernier stocke dans une liste tous les utilisateurs connectés.
Un utilisateur envoie des données pouvant parfois être un peu lourdes au serveur et ce dernier envoie ces paquets à tous les autres utilisateurs.
J'ai une fonction qui contient en gros :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
foreach (User user in UsersList)
{
//Envoi des données
}

Seulement si un utilisateur se déconnecte à ce moment là la liste est changée et le serveur plante. Comment puis-je améliorer ça ?

[tomlev]

Fais une copie de la liste avant d'énumérer:

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
foreach (User user in UsersList.ToList())
{
    //Envoi des données
}

Et ajoute un try/catch autour de l'envoi des données, au cas où un client qui était encore dans ta liste quand tu as commencé s'est déconnecté depuis

[Kaneda Shotaro]

C'est tellement bête et je n'y ai pas pensé !
MErci beaucoup !

[tomlev]

Tout bien réfléchi, ce n'est pas suffisant... la liste peut toujours être modifiée par un autre thread pendant l'exécution de ToList(). Pour bien faire, il faudrait faire un lock :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
List<User> users;
lock(UserList)
{
    users = UsersList.ToList();
}
foreach(User user in users)
{
    //Envoi des données
}

Et bien sûr, pour que ça serve à quelque chose, il faut que le code qui modifie la liste fasse aussi ce lock...

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
void RemoveUser(User user)
{
    lock(UsersList)
    {
        UsersList.Remove(user);
    }
}

[Kaneda Shotaro]

Encore merci ! Ca par contre je n'y aurai pas pensé du tout !

[Kaneda Shotaro]

Une petite question subsidiaire :

Est-ce que ça peut fonctionner si je fais ça :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
public void fonction1()
{
   lock(maliste)
   {
      fonction2(2);
   }
}
public void fonction2(int index)
{
   maliste.RemoveAt(index);
}

?
Ou est-ce que le lock de la fonction 1 va bloquer la fonction 2 ? Les deux faisant partie du même thread cela ne devrait pas poser de problème, si ?