Modification d'une liste lors d'une itération

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:

---

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 ?

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

Code:

---

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

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

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:

---

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:

---

1 2 3 4 5 6 7

void RemoveUser(User user) { lock(UsersList) { UsersList.Remove(user); } }

---

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

Une petite question subsidiaire :

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

Code:

---

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 ?

Ca marchera uniquement si tu n'appelle jamais la fonction2 directement, ce qui implique logiquement de la rendre privée.

Merci beaucoup. Effectivement, là c'était pour l'exemple. Dans mon code toutes les fonctions sont privées et la fonction 2 n'est jamais appelée directement. Merci encore à tous !