Discussion :

[Thixomag]

Bonjour,

J'ai un problème dans le cadre d'une application qui sert de serveur de tchat.
En gros il y a deux class principales RomuServer et AquaClientThread.

RomuServer contient le main, un serverSocket, et un Vector de la liste des clients.
Le serverSocket écoute et lorsqu'une connexion est acceptée, un nouveau AquaClientThread est créé et le stocke dans le :
static Vector clientList= new Vector(); de la class RomuServer.

Le problème est que le programme "plante", à certain un moment.
C'est très aléatoire, il peut tenir 2 heures comme 2 jours avec 10 connectés ou 60.

Donc j'ai essai de savoir d'où venait le problème en faisant des println partout à chaque début et fin de fonction et même plus.
Et j'ai reussi à voir que le problème se créait dans cette fonction :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
    static synchronized void sendToAllClients(String message) {
 
        AquaClientThread foo;
        Iterator client_it = clientList.iterator();
        try {
            while(client_it.hasNext()) {
                foo = (AquaClientThread)client_it.next();
                if (foo!=null) {
	  	    // dernier println possible lorsque le problème survient
                    foo.send(message,"MESSAGE");
                }
            }
        } catch(Exception ex) { echo("plantage sendToAllClients "+ex); }
    }

Donc cette fonction est dans la class principale RomuServer, et parcoure le Vector pour envoyer à tous les clients un message.
Elle fonctionne la plus part du temps correctement, mais parfois la class RomuServer "freeze", le programme continue de tourner et les threads font encore leur boulot.
Quand cela arrive, plus rien ne se passe sur RomuServer, et la dernièr action qu'il arrive à faire est un println juste avant le food.send().
La fonction send ne se lance pas d'ailleurs dans le thread concerné, quand le problème survient.

Je rappelle que la plupart du temps que la fonction sendToAllClients fonctionne correctement, mais qu'au bout d'un certain temps, très aléatoire, et à un certain moment dans la boucle, un foo.send fait freezer RomuServer, et je suis obligé d'arrêter le programme et de le relancer.

Merci de m'aider, ou de donner des pistes s'il vous plait.

[adiGuba]

Salut,


Et à quoi correspond cette méthode send() ???

a++

[Thixomag]

Comme son nom l'indique elle sert à envoyer un message :

Code :

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

1
2
3
4
5
	public void send(String message, String appelle){
                // n'apparait pas lors que RomuServer freeze
		out.println(message+'\u0000');
		out.flush();
	}

Si je fait un screenOut.println juste au début de la fonction, il n'apparait pas, quand le problème survient bien sûr. Ça ne vient pas de send.

[dinobogan]

Donne le code de la méthode qui appelle "sendToAllClients".

[Thixomag]

sendToAllClients est appelé dans pas mal d'endroit :
- Une fois dans une fonction de AquaClientThread qui interprète les messages reçu. Si c'est un message texte qu'il faut envoyer à tout le monde : RomuServer.sendToAllClients(Pseudo+" : "+Msg);
- Puis à plusieurs endroit dans RomuServer pour certaine requête spécial.

Edit : J'ai fait beaucoup de teste, et je ne pense pas que ça dépende de la fonction qui appel sendToAllClients, mais là je vais tester une version qui me permet de savoir qu'elle est la fonction qui a appelé sendToAllClients au moment ou ça plante. J'ai plus qu'à attendre que ça plante.

[gifffftane]

Bon heu... freezer ça veut dire quoi ?

Je subodore qu'il s'agiréssasse d'un blocage du programme qui ne ferait plus rien : pas de stack trace, pas de fin de programme, plus même de println - la galère -, rien.

Je pense que cela vient de threads qui attendent un déblocage de synchronized qui ne vient jamais. Il faudrait que tu nous établisses une sorte de schéma de tes synchronized ; quelles sont les variables partagées, quelles sont les portions de code verrouillées, etc.