Multiple envoie socket

Version imprimable

Eu...c'est encore moi, je vais me faire taper à force ;)
Mais bon, ça peut en aider d'autre aussi hein...hein? :?

Bon voila, j'ai 2 prog, un client et un serveur (jusque là tout est normal), le client se connecte au serveur et lui envoi un premier message avec le md5 du fichier qui'il va envoyer par la suite, le serveur enregistre ce md5 en memoire, effectue un traitement avec et renvoi une réponse, le client doit renvoyer un autre message jusqu'à lui envoyer le message "END" pour terminer la transmission. Le problème c'est que j'ai ce code mais ça ne vas pas... et je ne pense pas qu'il faille se deconnecter à chaque fois puis se reconnecter, j'ouvre ma connexion avec

Code:

connect(sock, (SOCKADDR *)&sin, sizeof(sin));

puis je peux envoyer autant de truc que je veux non?

Voici mon code :
Coté client :
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 bool verifMd5(string md5File, string addIp, int port, string ucName) { WSADATA WSAData; WSAStartup(MAKEWORD(2,0), &WSAData); SOCKET sock; SOCKADDR_IN sin; sock = socket(AF_INET, SOCK_STREAM, 0); sin.sin_addr.s_addr = inet_addr(addIp.c_str()); sin.sin_family = AF_INET; sin.sin_port = htons(port); connect(sock, (SOCKADDR *)&sin, sizeof(sin)); send(sock, ucName.c_str(), (int)strlen(ucName.c_str()), 0); //pour gerer le retour du serveur je penser faire ici un recv(csock, recvbuf, recvbuflen, 0); ... je pense que ça devrait aller system("PAUSE"); send(sock, md5File.c_str(), (int)strlen(md5File.c_str()), 0); closesocket(sock); WSACleanup(); }
Coté serveur :
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 int main(void) { int port = 1563; char *nameFile; char recvbuf[DEFAULT_BUFLEN] = ""; int recvbuflen = DEFAULT_BUFLEN; WSADATA WSAData; WSAStartup(MAKEWORD(2,0), &WSAData); SOCKET sock; SOCKET csock; SOCKADDR_IN sin; SOCKADDR_IN csin; sock = socket(AF_INET, SOCK_STREAM, 0); sin.sin_addr.s_addr = INADDR_ANY; sin.sin_family = AF_INET; sin.sin_port = htons(port); bind(sock, (SOCKADDR *)&sin, sizeof(sin)); listen(sock, 0); int sinsize = sizeof(csin); if((csock = accept(sock, (SOCKADDR *)&csin, &sinsize)) != INVALID_SOCKET) { do { //ça va pas du tout ça car en bouclant comme ça il m'affiche ne console tout le temps le premier envoi effectuer avec le client...en boucle... recv(csock, recvbuf, recvbuflen, 0); cout << recvbuf << endl; closesocket(sock); WSACleanup(); } while (recvbuf != "END"); } system("PAUSE"); return 0; }
Voili voilou...
Ha si, aussi, je voulais essayer de voir pour une gestion de timeout, genre le client arrive à se connecter au serveur, envoi le premier message mais la réponse met trop de temps à arriver... pareil coté serveur... Si vous avez des idées...

Merci à tous !:D:D:D

Mais si, mais si... un jour, quand je posterais, ce sera un code fini, propre, commenté et tout pour aider les gens et pas pour leur faire fondre le cerveau... Mais si, mais si... :oops:

25/10/2007, 17h16
BainE

Salut,

et il est ou le probleme ?

Tu veux savoir s il faut deconnecter a chaque emission ?
non, tu conserves ta socket tant que tu comptes communiquer avec.
Par contre ca a l air d'etre en mode bloquant (le process s'arrete tant que recv ne recoit rien).

De plus tit commentaire constructif :
- lors de ton send( maString.c_str() ) pas sur que l envoi soit direct, pas de marqueur de fin de chaine, j'ai peur que ce soit bufferisé.
25/10/2007, 17h25
ram-0000
Citation:

Envoyé par Verbalinsurection

le client se connecte au serveur et lui envoi un premier message avec le md5 du fichier qui'il va envoyer par la suite

Tu dis qu'il envoie d'abord le MD5 puis ensuite le fichier

Si je regarde le code du client, il envoie d'abord le fichier et en second le MD5

Si je regarde le code du client, je ne voie jamais l'envoie de la balise "END"

C'est pas très grave, mais strlen(ucName.c_str()), c'est pas très joli et encore moins efficace. ucName.length() est nettement mieux.

Si je regarde le code du serveur, le programme boucle tant que la balise "END" n'est pas reçue.
Code:

1 2 while (recvbuf != "END");
Ce code ne marchera jamais car dans ce cas, tu regarde si l'adresse de ton buffer de reception (recvbuf) est égal au pointeur "END". Cela ne sera jamais vrai. Le bon test c'est
Code:

1 2 while (memcmp(recvbuf, "END", strlen("END") != 0);
Je ne voie pas dans ton code comment le serveur sais qu'il a finit de lire le fichier qu'il va lire le MD5

Dans le code du serveur, la fermeture des socket (d'ailleurs, tu oublies de fermer le socket client) et le WSACleanup() se font à l'issue de la réception du "END" et pas dans la boucle do...while

Il y a encore du travail !!

Raymond
25/10/2007, 17h36
Verbalinsurection

Et oui, il y a encore du travail, mais je suis là pour ça ;)

Merci en tout cas pour vos conseils je m'y met de suite !

Ok, alors en cette belle journée de froid (2° !!!!!!8O) j'ai repris mon code, alors :
J'ai bien fait les modifs que tu m'as donné ram_0000, alors voila mon nouveau code :

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
bool verifMd5(string md5File, string addIp, int port, string ucName) {
    WSADATA WSAData;
    WSAStartup(MAKEWORD(2,0), &WSAData);
    SOCKET sock;
    SOCKADDR_IN sin;
    sock = socket(AF_INET, SOCK_STREAM, 0);
    sin.sin_addr.s_addr = inet_addr(addIp.c_str());
    sin.sin_family = AF_INET;
    sin.sin_port = htons(port);
string end = "END";
    connect(sock, (SOCKADDR *)&sin, sizeof(sin));
system("PAUSE");
    send(sock, ucName.c_str(), ucName.length(), 0);
system("PAUSE");
    send(sock, md5File.c_str(), md5File.length(), 0);
system("PAUSE");
    send(sock, end.c_str(), end.length(), 0);
 
    closesocket(sock);
    WSACleanup();
}

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
int main(void) {
    int port = 1563;
    char *nameFile;
 
    char recvbuf[DEFAULT_BUFLEN] = "";
    int recvbuflen = DEFAULT_BUFLEN;
 
    WSADATA WSAData;
    WSAStartup(MAKEWORD(2,0), &WSAData);
    SOCKET sock;
    SOCKET csock;
    SOCKADDR_IN sin;
    SOCKADDR_IN csin;
    sock = socket(AF_INET, SOCK_STREAM, 0);
    sin.sin_addr.s_addr	= INADDR_ANY;
    sin.sin_family = AF_INET;
    sin.sin_port = htons(port);
    bind(sock, (SOCKADDR *)&sin, sizeof(sin));
    listen(sock, 0);
    int sinsize = sizeof(csin);
    if((csock = accept(sock, (SOCKADDR *)&csin, &sinsize)) != INVALID_SOCKET) {
        do {
            recv(csock, recvbuf, recvbuflen, 0);
            cout << recvbuf << endl;
        } while (memcmp(recvbuf, "END", strlen("END")) != 0);
        closesocket(sock);
        WSACleanup();
    }
 
    //attenteFichier(port, nameFile);
    system("PAUSE");
	return 0;
}

Bon alors, mon cas s'arrange, voila ce que j'obtiens sur la console coté serveur :

Citation:

THOMAS
e326ff28edaf71e2c1246a6cae0750e8
END6ff28edaf71e2c1246a6cae0750e8

Comme vous pouvez le voir, je peux bien envoyé plusieur pacquet sans problème mais comme vous pouvez le voir aussi, quad j'envoi mon END il me remet le md5 derrière...Mais euuu!!!!
Une ptite idée?

edit : Ha et puis, comme c'est du tcp, je dois pouvoir envoyer une réponse à partir du serveur, mais comment....?

Citation:
Envoyé par Verbalinsurection
Ok, alors en cette belle journée de froid (2° !!!!!!8O) j'ai repris mon code, alors :
J'ai bien fait les modifs que tu m'as donné ram_0000, alors voila mon nouveau code :
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 bool verifMd5(string md5File, string addIp, int port, string ucName) { WSADATA WSAData; WSAStartup(MAKEWORD(2,0), &WSAData); SOCKET sock; SOCKADDR_IN sin; sock = socket(AF_INET, SOCK_STREAM, 0); sin.sin_addr.s_addr = inet_addr(addIp.c_str()); sin.sin_family = AF_INET; sin.sin_port = htons(port); string end = "END"; connect(sock, (SOCKADDR *)&sin, sizeof(sin)); system("PAUSE"); send(sock, ucName.c_str(), ucName.length(), 0); system("PAUSE"); send(sock, md5File.c_str(), md5File.length(), 0); system("PAUSE"); send(sock, end.c_str(), end.length(), 0); closesocket(sock); WSACleanup(); }
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 int main(void) { int port = 1563; char *nameFile; char recvbuf[DEFAULT_BUFLEN] = ""; int recvbuflen = DEFAULT_BUFLEN; WSADATA WSAData; WSAStartup(MAKEWORD(2,0), &WSAData); SOCKET sock; SOCKET csock; SOCKADDR_IN sin; SOCKADDR_IN csin; sock = socket(AF_INET, SOCK_STREAM, 0); sin.sin_addr.s_addr = INADDR_ANY; sin.sin_family = AF_INET; sin.sin_port = htons(port); bind(sock, (SOCKADDR *)&sin, sizeof(sin)); listen(sock, 0); int sinsize = sizeof(csin); if((csock = accept(sock, (SOCKADDR *)&csin, &sinsize)) != INVALID_SOCKET) { do { recv(csock, recvbuf, recvbuflen, 0); cout << recvbuf << endl; } while (memcmp(recvbuf, "END", strlen("END")) != 0); closesocket(sock); WSACleanup(); } //attenteFichier(port, nameFile); system("PAUSE"); return 0; }
Bon alors, mon cas s'arrange, voila ce que j'obtiens sur la console coté serveur :

Comme vous pouvez le voir, je peux bien envoyé plusieur pacquet sans problème mais comme vous pouvez le voir aussi, quad j'envoi mon END il me remet le md5 derrière...Mais euuu!!!!
Une ptite idée?

edit : Ha et puis, comme c'est du tcp, je dois pouvoir envoyer une réponse à partir du serveur, mais comment....?
Non, il ne remet pas le MD5 derrière, ton problème est coté serveur. tu ne termine pas ta string pour de l'affichage après la réception

Normallement, la fonction recv() te dis combien d'octets tu as recu
to code devrait ressemble à quelquechose comme cela
Code:

1 2 3 4 5 6 7 8 9 10 11 int nb_recv = recv(csock, recvbuf, recvbuflen, 0); if(nb_recv != -1) { recvbuf[nb_recv] = 0; } else { /* erreur de reception */ ... }
Raymond

26/10/2007, 10h22
Verbalinsurection

Ok merci, et pour pouvoir envoyer une réponse ?
Je sais pas, peut etre esce une betise mais en vb quand on reçoit un paquet tcp on peut renvoyer direct une réponse sans devoir remettre adresse ip etc...
C'est possible en c++?
26/10/2007, 10h34
ram-0000

Citation:

Envoyé par Verbalinsurection

Ok merci, et pour pouvoir envoyer une réponse ?
Je sais pas, peut etre esce une betise mais en vb quand on reçoit un paquet tcp on peut renvoyer direct une réponse sans devoir remettre adresse ip etc...
C'est possible en c++?

Oui, utilise la fonction send()

Raymond
26/10/2007, 11h01
Verbalinsurection

Désolé mon cerveau etait sur shutdown...
Merci encore !