Discussion :

[Soufiane_]

Bonjour, j'ai codé un client et un serveur pour une messagerie instantanée qui fait partie d'un projet que je dois rendre pour mardi, sauf que j'ai l'impression que les deux refusent de marcher ensemble.
Le client comme le serveur compilent niquel mais ils ne remplissent pas leur fonction on dirait...

Je lance d'abord le serveur avant le client, mais au lancement du client il est sensé se connecter au serveur et le serveur afficher un message comme quoi la connexion a été acceptée, sauf que ce message ne s'affiche jamais, et les socket envoyées depuis le client ne sont jamais traitées par le serveur...

Voici les codes en question :

Boucle de traitement du serveur :

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
 
while (true)
	{
		std::cout << "OK" << std::endl;
		sockaddr_in from = {0};
		int addrlen = sizeof(from);
		CLIENT newClient(accept(server, (SOCKADDR*)(&from), &addrlen), from);
 
		std::cout << "Client connecté" << std::endl;
 
		if (newClient.m_connected) // la socket est connectée
		{
			std::thread processing(Process, newClient);
			processing.detach();
		}
	}

Côté client, un peu + de code :

La boucle de gestion du client :

Code :

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

1
2
3
4
5
6
7
8
9
 
std::thread sending(sendMessage, server);
	std::thread receiving(recvMessage, server);
 
	sending.detach();
	receiving.detach();
 
	while (true)
	{ }

Les deux fonctions lancées en threads :

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
 
void sendMessage(SOCKET s)
{
	while (true)
	{
		std::string buff;
		std::cout << "[Vous] ";
		std::getline(std::cin, buff);
		std::cout << std::endl;
 
		int Bytes = send(s, buff.c_str(), 1024, 0);
		if (Bytes <= 0)
		{
			std::cout << "[ERREUR] Interruption de la connexion avec le serveur" << std::endl;
			break;
		}
	}
}

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
 
void recvMessage(SOCKET s)
{
	while (true)
	{
		char buffer[1025];
		int Bytes = recv(s, buffer, 1024, 0);
		if (Bytes <= 0)
		{
			std::cout << "[ERREUR] Interruption de la connexion avec le serveur" << std::endl;
			break;
		}
 
		std::cout << "[Message] " << buffer << std::endl;
	}
}

Merci d'avance pour l'aide que vous pourriez m'apporter !

[Bousk]

Avec juste des morceaux de code qui n'ont pas grand sens, je vois pas trop par quel magie on pourrait t'aider.
Pour le reste, (re)lis ton cours et tu as aussi un cours disponible dans ma signature. Un code ne refuse pas de marcher, il fonctionne tel que tu l'implémentes.
Mettre des threads de partout en étant débutant est le meilleur moyen de se planter.

[Wx234]

Même si je suis d'accord avec ce qui est dit au dessus je vois immédiatement plusieurs problèmes potentiels avec ton code :

Code :

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

CLIENT newClient(accept(server, (SOCKADDR*)(&from), &addrlen), from);

form en tant que second argument donné à la fonction est probablement copié, hors il ne faut pas compter sur l'ordre d'évaluation des arguments de fonction, pense à appeler accept avant d'appeler cette fonction

Code :

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

int Bytes = send(s, buff.c_str(), 1024, 0);

l'argument len de send est la longueur exacte du message que tu dois transmettre, essaye buff.size() ou buff.size() + 1 plutôt que 1024, pour avoir plus d'informations sur son fonctionnement les pages du manuel sont ton ami (et https://linux.die.net/man/2/send aussi)