Discussion :

[pazze]

Bonjour à tous !

Je dois créer un petit jeu multi-joueurs en temps réel (max 10 connections). Pour ce faire j'ai créé un serveur multi-thread.

J'ai un problème lors de la déconnection d'un client. Le serveur boucle sans fin pour afficher le dernier message reçu.

Voila le thread pour chaque client

Code :

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

1
2
3
4
5
6
7
8
9
10
void* reception(void* arg) {
 
    char buffer[4096];
    while(recevoir((int)arg, buffer)) {
        cout << buffer << endl;
 
    }
	close((int)arg);
    return (void*)1;
}

Le client :

Code :

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

1
2
3
4
    char buf[4096]= "hello";
    envoyer (socket, buf);
    sleep(2);
    envoyer (socket, buf);

Lorsqu'on quitte le client, que ce soit proprement ou par ctrl-c. Le serveur n'arrête pas d'afficher "hello". Je pense que ça doit être un problème de socket non fermé mais je n'en suis pas sûr !

EDIT: Comment arrêter le thread lorsqu'un client se déconnecte ?

[Invité(e)]

Bonjour,

Peux tu poster le code de la fonction recevoir ?

[pazze]

Voila

Code :

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

1
2
3
4
5
6
7
int recevoir(int socket, char buffer[4096]) {
    if (recv(socket, buffer, 4096, 0) == -1) {
        perror("Server: recv");
        return EXIT_FAILURE;
    }
    else return 4096;
}

En la modifiant comme ceci :

Code :

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

1
2
3
4
5
6
7
int recevoir(int socket, char buffer[4096]) {
    if (recv(socket, buffer, 4096, 0) == -1) {
        perror("Server: recv");
        return -1;
    }
    else return 1;
}

J'ai toujours le même problème

[Invité(e)]

En la modifiant comme ceci :[...]
J'ai toujours le même problème

Normal, tu ne sortira de ta boucle while que si recevoir renvoie 0.

Or EXIT_FAILURE est souvent différent de 0.

Dans ta fonction recevoir, garde le 'return -1' en, cas d'erreur, mais change la boucle while par :

Code :

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

while(recevoir(...) != -1) {...}

[jeroman]

Mélanger du C et du C++, c'est pas propre, et en plus ça oblige à compiler en C++. Ceux sont deux langages à part entière. Et d'ailleurs il y a des choses écrites en C qui ne compilent pas en C++ (je parle pas de ton code mais en général).

[pazze]

Même en changeant ma boucle while( ..!= -1), j'ai encore le même problème...

Je suis d'accord pour le mélange C et C++ mais comment bien différencier les deux lors d'un programme réseau (écrit en c++ lui) ?

[Invité(e)]

Même en changeant ma boucle while( ..!= -1), j'ai encore le même problème...

Ajoute des traces dans ton code pour mieux comprendre ce qui se passe et où (tu peux aussi utiliser un débogueur, mais en contexte client/serveur / multithread, ce n'est pas facile).

[pazze]

En fait, après quelques tests, j'ai ce problème à la fermeture du socket côté client.

Avec ce client, j'ai le problème après un ctrl-c

Code :

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

1
2
3
4
	char buff[4096] = "quit";
	envoyer (socket, buff);
 
	while(1) {}

alors que j'ai directement le problème ici

Code :

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

1
2
3
4
	char buff[4096] = "quit";
	envoyer (socket, buff);
	close(socket);
	while(1) {}

Pourtant la fermeture se fait sans soucis.

EDIT: Côté client, le socket se ferme. Et côté serveur, la réception continue comme si le socket était toujours rempli.


EDIT2: Problème résolu, recv() renvoie 0 quand le socket sur lequel il veut lire est fermé. Un test en plus dans ma fonction recevoir() et c'est résolu