Discussion :

[adrian07]

Salut à tous,
Je viens de réaliser une petite application de chat client/serveur avec RMI...
Bon, le serveur marche bien mais j'ai un pb avec le client...en fait, lorsque je demande la liste des messages, il génère une exception

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
 
Exception in thread "main" java.util.ConcurrentModificationException
	at java.util.AbstractList$Itr.checkForComodification(AbstractList.java:372)
	at java.util.AbstractList$Itr.next(AbstractList.java:343)
	at com.esprit.chat.server.service.ChatServiceImpl.getMessages(ChatServiceImpl.java:23)
	at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
	at sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:39)
	at sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:25)
	at java.lang.reflect.Method.invoke(Method.java:597)
	at sun.rmi.server.UnicastServerRef.dispatch(UnicastServerRef.java:305)
	at sun.rmi.transport.Transport$1.run(Transport.java:159)
	at java.security.AccessController.doPrivileged(Native Method)
	at sun.rmi.transport.Transport.serviceCall(Transport.java:155)
	at sun.rmi.transport.tcp.TCPTransport.handleMessages(TCPTransport.java:535)
	at sun.rmi.transport.tcp.TCPTransport$ConnectionHandler.run0(TCPTransport.java:790)
	at sun.rmi.transport.tcp.TCPTransport$ConnectionHandler.run(TCPTransport.java:649)
	at java.util.concurrent.ThreadPoolExecutor$Worker.runTask(ThreadPoolExecutor.java:885)
	at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:907)
	at java.lang.Thread.run(Thread.java:619)
	at sun.rmi.transport.StreamRemoteCall.exceptionReceivedFromServer(StreamRemoteCall.java:255)
	at sun.rmi.transport.StreamRemoteCall.executeCall(StreamRemoteCall.java:233)
	at sun.rmi.server.UnicastRef.invoke(UnicastRef.java:142)
	at java.rmi.server.RemoteObjectInvocationHandler.invokeRemoteMethod(RemoteObjectInvocationHandler.java:178)
	at java.rmi.server.RemoteObjectInvocationHandler.invoke(RemoteObjectInvocationHandler.java:132)
	at $Proxy0.getMessages(Unknown Source)
	at com.esprit.chat.client.Test.main(Test.java:33)

Je vous transmets le code du client et celui du serveur en pièce jointe...

Est ce que quelqu'un peut m'aider???

[Pastis]

Bonjour,

je n'ai pas ouvert votre code car l'erreur paraît claire.

java.util.ConcurrentModificationException signifie que vous modifiez la liste pendant que vous itérez dessus. Pour supprimer un élément d'une liste au fur et à mesure qu'on itère dessus, il faut faire :

Code :

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

1
2
3
4
5
6
7
8
 
Iterator it = list.iterator() ;
while ( it.hasNext() ) {
    Object o = it.next() ;
    if ( supprimable( o ) ) {
       it.remove() ;
    }
}

Notez qu'on supprime sur l'itérateur et non sur la liste.
Autre solution : travailler sur une copie de la liste parcourue...

[adrian07]

Je ne suis pas sur d'avoir bien compris ce que vous dites...
Voici le code du service chat:

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
package com.esprit.chat.server.service;

import java.rmi.RemoteException;
import java.rmi.server.UnicastRemoteObject;
import java.util.ArrayList;
import java.util.HashSet;
import java.util.LinkedList;
import java.util.List;
import java.util.Set;
import java.util.Vector;

import com.esprit.chat.domain.Contact;
import com.esprit.chat.domain.Message;

public class ChatServiceImpl extends UnicastRemoteObject implements ChatService{

	private  Vector<Message> messagesPool=new Vector<Message>();
	private Vector<Contact> contacts=new Vector<Contact>();
	
	public Vector<Message> getMessages(Contact client){
		 Vector<Message> messages=new Vector<Message>();
		
		for (Message m:messagesPool){
			if (m.getTo().getPseudo().equals(client.getPseudo())){
				messages.add(m);
				messagesPool.remove(m);
			}
		}
		
		return messages;
		
		
		
	}
	
	
	public void envoyer(Message message){
		
		messagesPool.add(message);
		
		
		System.out.println(message.getContent());
	}
	public ChatServiceImpl() throws RemoteException {
		super();
		
		Contact contact=new Contact();
		contact.setPseudo("foulen");
		contacts.add(contact);
	}


	
	public Vector <Contact> getContacts() throws RemoteException {
		// TODO Auto-generated method stub
		return contacts;
	
	}

D'après ce que j'ai compris, le pb est dans les deux lignes que j'ai mis en rouge???

[Pastis]

A priori, çà se corrige ainsi :

Code :

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

1
2
3
4
5
6
7
8
9
 
Iterator<Message> it = messagesPool.iterator() ;
while ( it.hasNext() ){
    Message m = it.next() ;
    if (m.getTo().getPseudo().equals(client.getPseudo())){
	messages.add(m);
	it.remove(m);
    }
}

[adrian07]

Merci pour ton aide