Serveur multithread qui marche un coup sur deux

Sujet résolu

Anonyme

30 mars 2008 à 19:13:15

Salut à tous
je suis entrain de programmer un chat avec serveur multi-thread mais cependant je suis confronté à un problème que je n'arrive pas à résoudre.
Voici mon problème :

Une fois que mon serveur est en écoute, et qu'il reçoit un client celui-ci est envoyé sur un thread qui écoute les messages qu'il envoit. Cependant il en reçoit une fois sur deux !
voici le code du thread:

void *Serveur::thfonc2(int id)
{
SOCKET soc;
soc = users[id].soc;
while (recevoir(soc)!="/stop") // dès qu'on reçoit /stop on coupe
{
cout<<recevoir(soc)<<endl; //on affiche ce que le client a envoyé
}
users[id].marche = 0;
cout << "Un client c'est déconnecté de la socket "<<soc<<" de "<<inet_ntoa (m_sin.sin_addr)<<":"<<htons (m_sin.sin_port)<<endl;
}

enfin je sais vraiment pas d'où le problème peut venir, si vous voulez plus de mon code demander moi.

chlablak

30 mars 2008 à 19:17:51

je sais pas se que fais recevoir(), mais je ferais la boucle ainsi:

for(;;)
{
std::string Recu = recevoir(soc);
if(Recu == "/stop") // dès qu'on reçoit /stop on coupe
break;
cout<< Recu <<endl; //on affiche ce que le client a envoyé
}

C++ Références | Idiomes | FAQ

Anonyme

30 mars 2008 à 19:24:19

salut, merci d'avoir été si rapide pour répondre,recevoir retourne ce qu'envoie le client,en string.
voici le code :

string Serveur::recevoir(SOCKET a)
{
string message;
char buffer[512];
memset(buffer,0,sizeof(buffer));//Ici, c'est la procédure pour initialiser
if((recv(a, buffer, sizeof(buffer), 0)) != -1)
{
message = buffer; //on met le char dans le string
}
else //si le client c'est déconnecté
{
message="/stop";
}
delete [] buffer;
return message;
}

ha! merci ton code fonctionne bien, Est ce que tu pourrait m'expliquer ?

chlablak

30 mars 2008 à 19:32:55

enfaite ton erreur dans ta boucle c'est que tu appelais deux fois recevoir() mais que tu ne stockais pas son resultat.

et pour ta fonction recevoir(), quelques remarques:

string Serveur::recevoir(SOCKET a)
{
char buffer[512];
memset(buffer,0,sizeof(buffer));
// tu peux remplacer les deux lignes ci-dessus par
char buffer[512] = {0};
int recu_taille = 0;
if((recu_taille = recv(a, buffer, sizeof(buffer), 0)) != -1)
{
// il faut faire finir le buffer par un '\0'
buffer[recu_taille] = '\0';
}
else //si le client c'est déconnecté
{
strcpy(buffer, "/stop");
}
delete [] buffer; // tu n'alloues jamais rien, donc inutile
return buffer; // tu peux directement retourner le buffer
}

edit: j'ai edité mon code qui avais une petite erreur -> http://www.siteduzero.com/forum-83-258033-2332165-serveur-multithread-qui-marche-un-coup-sur-deux.html#r2332165

C++ Références | Idiomes | FAQ

Anonyme

30 mars 2008 à 19:40:09

Merci beaucoup par l'aide et les conseil

tout marche bien maintenant.
Je vais pouvoir continuer à coder
A+