Code Client/Server et envoi plusieurs fichiers ...

Question

Bonjour,

Apr&#232;s des efforts j'ai r&#233;alis&#233; une petite application Client/Server qui sert &#224;
transf&#233;rer plusieurs fichiers du client vers le serveur. Ca marche impeccable,
mais pas toujours. Parfois le client ne re&#231;oit pas le fichier complet !

Si j&#146;enl&#232;ve le Sleep(2000) alors un client sur le r&#233;seau
et pas sur le m&#234;me PC ne re&#231;oit pas les memes bytes qu'il doit recevoir ... ou es le probleme dans mon code ?



----- Code Serveur -----




















//variables globales

Char FICHIERS[100][50] ; // tableau des noms des
fichiers

int N ;//nombre des fichiers &#224; transmettre en entier

Char N2[3] ; //nombre des fichiers &#224; transmettre en
char

int TailleFichier;//taille du fichier en entier

char T[10];//taille du fichier en char

Char *buffichier; // buffer memoire, va contenir le fichier

FILE *fich; // Pointeur fichier Source 





/* fonction qui lit le continue d&#146;un r&#233;pertoire et remplit
un tableau nomm&#233; char FICHIERS par les noms des fichiers, et une variable N par
le nombre des fichiers. */






ReadDirectory("C:\Dev-Cpp\txt\"); 









//Envoi du nombre de fichiers ...

sprintf(N2,"%d",N);//Conversion de type

if (send(sock2,N2,sizeof(N2),0) < 0) Erreur("Erreur
Transmission du Serveur") ;
















 int i=0 ;

while (i < N)

{

//Envoi le nom du fichier

if (send(sock2,fichiers[i],sizeof(fichiers[i]),0) < 0)
Erreur("Erreur Transmission du Serveur !"); 













if ((buffichier (char *)malloc(10000)) NULL)
Erreur("Reservation Memoire Impossible !");

if ((fich fopen(fichiers[i],"rb")) NULL)
Erreur("Ouverture Fichier Impossible !");

TailleFichier = fread(buffichier,1,10000,fich); // Lecture
dans buffichier

sprintf(T,"%d",TailleFichier);//conversion du int
en char 







//Envoi de la taille du fichier

if (send(sock2,T,sizeof(T),0) < 0) Erreur("3:Erreur
Transmission du Serveur !");


















 i++ ;//incrementation du conteur

Sleep(2000);

if (send(sock2,buffichier,TailleFichier,0) < 0)
Erreur("Erreur Transmission du Serveur !");

_fcloseall();//
Fermeture Fichier

free(buffichier);//
Lib&#232;re m&#233;moire

}






 ----- Code Client -----








 //reception du nombre de fichiers

recv(sock,N2,sizeof(N2),0);






N = atoi(N2); //conversion du char en integer














while (i++ < N)

{

//reception du nom
du fichier

char nomfichier[50];

recv(sock, nomfichier,sizeof(nomfichier),0); 







if ((fich fopen(nomfichier,"wb"))
 NULL) Erreur("Ouverture Fichier Impossible !"); 









//reception de la taille du fichier

recv(sock,T,sizeof(T),0);









TailleFichier = atoi(T) ; //conversion du char en
integer

if ((buffichier (char *)malloc(TailleFichier)) NULL)
Erreur("Reservation Memoire Impossible !");










//reception du fichier

int receved ;

if ((receved = recv(sock,buffichier, TailleFichier,0)) <
0) Erreur("Erreur Reception du Serveur !");


































if(receved !=
TailleFichier)

{

printf("%d != %d", buffichier, TailleFichier);

Erreur("Erreur Lors du transfert !");

}

fwrite(buffichier,1, TailleFichier,fich);// Ecriture dans FICHIER

free(buffichier);//
Lib&#232;re m&#233;moire

_fcloseall();//
Fermeture Fichier

}

closesocket(sock);// Fermeture Socket

WSACleanup();//
LIBERATION DLL WINSOCK

meech · Answer

Salut,

Je pense que &#231;a mer[...]ouille &#224; cause de la ligne o&#249; tu lis le fichier avec la fonction fread :

TailleFichier = fread(buffichier,1,10000,fich); 

Le troisi&#232;me param&#232;tre de fread sp&#233;cifie le nombre de blocs &#224; lire dans le fichier. Donc, si la taille du fichier est sup&#233;rieure &#224; environ 10 000 octets (environ 10 kilo-octets, ce qui est peu), tu ne lis que les 10 premiers kilo-octets et tu abandonnes le reste...

En tout cas, ta variable TailleFichier ne retourne la "bonne" taille du fichier seulement si celle-ci est inf&#233;rieure ou &#233;gale &#224; 10 000 blocs (la fonction fread retournant, sauf en cas d'erreur, le nombre de blocs de donn&#233;es r&#233;ellement lus).

Ensuite, tu adresses au client le bloc de 10 ko au maximum, ce qui explique que le client ne re&#231;oives pas toujours compl&#232;tement le contenu du fichier.

La solution consiste &#224; r&#233;aliser une boucle o&#249; :
1. tu stockes dans un buffer la partie du fichier &#224; lire avec fread ;
2. tu envoies la partie lue au client avec send.

Exemple (en s'adaptant vite fait &#224; ton code)
#define BUFFER_SIZE 1024

// (...)

int length;
while((length = fread(buffichier,1,BUFFER_SIZE,fich)) != 0) 
{
   if (send(sock2,buffichier,length,0) < 0) 
            Erreur("Erreur Transmission du Serveur !");
}

En esp&#233;rant t'avoir aid&#233; un peu, 
Ciao.

PS. Documentation sur la fonction fread.

QuakeCoder · Answer

merci pour ta reponse.

bon.le probleme ne viens pas du fread, car noremalent j'ai mis dans le
code (#define FICH_MAX_SIZE 9000000    // Taille Maximum
de Fichier (octets)) 9 mega.

Et tous mes fichiers < 9megas. en plus le programe marche nickel si
je le lance dans le meme poste avec le serveur. j'ai le probleme juste
avec les clients distants !

meech · Answer

Sans vouloir te vexer, tu peux
augmenter la taille de ton buffer &#224; 600 millions si tu veux, ce n'est
pas ce qu'il y a de mieux au niveau du code... je te conseille la
boucle avec un buffer plus petit : tu envoies ainsi de petites bribes
de fichier sans saturer ni le client, ni le serveur... Bref, fais comme
tu veux mais, un moment o&#249; un autre, tu seras emb&#234;ter par cela.



C&#244;t&#233; client, c'est le m&#234;me genre de probl&#232;me : 



En fait, rien ne te force &#224; bricoler des histoires de r&#233;cup&#233;ration de
taille de fichier : la fonction recv() n'a pas besoin de cela pour
rep&#233;rer quand elle ne trouve plus aucune donn&#233;e en provenance du
serveur. M&#234;me solution une boucle : ici, tant que le client re&#231;ois
quelque chose c'est &#224; dire tant que :


receved = recv(sock,buffichier, TailleFichier,0))== 0




Fais comme tu veux, mais tu ne trouveras aucun exemple (ou ils ne
seront pas terribles) o&#249; le transfert se fait d'un seul bloc. Ce qui
est certain, c'est que des serveurs HTTP (comme Apache ou Microsoft
IIS) n'adresse pas la r&#233;ponse en un seul morceau... c'est &#233;galement le
cas des serveurs FTP ou de tout ce qui fonctionne en mode connect&#233; (ie.
socket).



Ciao & bon courage.

QuakeCoder · Answer

oui voila :) . je vais suivre ta procedure et j te repondrai d&#233;s que je finis ... merci bq

QuakeCoder · Answer

Bonjour. voila j'ai fais pour le transfert des fichiers ce que vous
m'avez dis.Maintenant c'est bon ... mais ya encore un probleme, parfois
et pas toujours, quand le serveur envoi le nom du fichier &#224; transferer
le client se bug et il recoit une parti du nom et recoit l 'autre parti
quand il recoit la taille du fichier.( voir le code ).



voila.j c pas ou le probleme. dois je faire une boucle aussi pour le transfert du nom du fichier et sa taille ??

meech · Answer

Salut,



Je ne pense pas non. Envoie ton code ! 



PS. Pas la peine de me vouvoyez !!

Code Client/Server et envoi plusieurs fichiers ...

6 réponses

Votre réponse

Discussions similaires