Aide pour un débutant RMI

Version imprimable

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:

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???

12/02/2012, 21h20
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:

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...

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

Code:

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???

A priori, çà se corrige ainsi :

Code:

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);
    }
}

12/02/2012, 21h53
adrian07

Merci pour ton aide :ccool::ccool::ccool::ccool: