Discussion :

[Tellmarch]

Bonjour, je n'ai pas trop l'habitude de manipuler des listes en java.
Mon problème intervient avec ce code :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
 
    private List<ConnectedUser> users;
 
    ... (dans le constructeur)
    this.users = new ArrayList<ConnectedUser>();
    ...
 
    public void removeUser(String userName) {
        for(ConnectedUser u:users) {
            if(u.getUsername().equals(userName)) {
                users.remove(u);
            }
        }
    }

Ce code génère une

Code :

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

Exception in thread "ServerThread"java.util.ConcurrentModificationException

Si j'ai bien compris, c'est parceque quand on itère sur la liste dans le for, on ne peut pas la modifier. Quelle est alors la bonne façon d'enlever un élément de la liste? Ma première idée et de faire une copie de la liste pour itérer, mais ça parait coûteux en temps... Y a-t'il une meilleur solution?

[yann2]

Bonjour

3 solutions

1 - Utiliser un itérateur (Iterator qu'on obtient avec myList.iterator()) et utiliser l'opération remove() de l'opérateur

2 - Copier la liste avant d'itérer for(ConnectedUser u: new ArrayList<ConnectedUser>(users)) {

3 - Placer les éléments à supprimer dans une liste usersToRemove et faire un users.removeAll(usersToRemove).

J'ai une préférence pour la solution 3 mais, c'est à vous de voir

Yann

[Tellmarch]

Merci beaucoup pour cette réponse claire et précise