pb reception de fuchiers

Question

Bonjour!

Apr&#233;s avoir gal&#233;r&#233; pas mal pour envoiyer un fichier par TCP en local,
 
j'ai aussi un probleme concernant la reception de ce fichier,car il me manque un morceau!! sur un 700Mo j'en perds 20, sur un MP3 &#231;a ne se voit pas et je peux donc le lire correctment. J'ai remarqu&#233; que je ne re&#231;cois pas le meme nombre de paquet mais on m'a precis&#233; que 1 send != 1 receiv .

Je vous mets un peu de code:

////******************************
case FD_READ:
    if( sockfichierR == wParam )
    {
     ReceptionFichier();
    }
    else
     ReceiveData();
    break;

case FD_CLOSE:
    {
    if( sockfichierR == wParam )
    {
     lstrcpy(ListeDl[IndexCurrentDl].etat, "Terminer");
     ListeDl[IndexCurrentDl].accept = TRUE;
     
     CloseHandle(NewFile);
     
     UpdateListDl();
    }
    }
    break;

////******************************
DWORD WINAPI SendFile(LPVOID lpParameter) // Fonctionne correctement !
{
 for( int i = 0 ; i<=NbDl ; i++ )
 {
  if( CurrentFile == ListeDl[i].ID )
  {
   IndexCurrentUp = i;
   break;
  }
 }
 i--;


 HANDLE hFile;
 char dataFile[TaillePak];
 DWORD len;


 ListeDl[i].currentNbPak = 0;
 hFile = CreateFile(ListeDl[i].chemin, GENERIC_READ, FILE_SHARE_READ, NULL, OPEN_EXISTING, NULL , NULL);


 if(hFile == INVALID_HANDLE_VALUE)
  return TRUE;


 BOOL End = FALSE;
 
 for( ; End == FALSE ; )
 {
  len = 0;
  ZeroMemory(dataFile, TaillePak);
  ReadFile(hFile, dataFile, TaillePak, &len, NULL);
  send(sockfichierE, dataFile, len, NULL);
  ListeDl[i].currentNbPak++;
  if( len < TaillePak )
   End = TRUE;
 }


 CloseHandle(hFile);


 lstrcpy(ListeDl[i].etat, "Termin&#233;");
 ListeDl[i].accept = FALSE;
 
 shutdown(sockfichierE, SD_BOTH);
 closesocket(sockfichierE);


 MyThread = CreateThread(NULL,NULL,ThreadProc,NULL,NULL,&ThreadID);


 return TRUE;
}

////******************************Et voila la fonction bugg&#233;e!!
BOOL ReceptionFichier()
{
 BOOL Ecrire = TRUE;
 char Sdata[TaillePak];
 ZeroMemory(Sdata, TaillePak);
 int i = 0;


 recv(sockfichierR, Sdata, sizeof(Sdata), NULL);
 
 if( NewFile == NULL )
 {
  for( i = 0 ; i<=NbDl ; i++ )
  {
   if( ListeDl[i].ID == CurrentFile )
   {
    IndexCurrentDl = i;
    break;
   }
  }
  NewFile = CreateFile(ListeDl[IndexCurrentDl].nom, GENERIC_WRITE, FILE_SHARE_READ, NULL, CREATE_ALWAYS, FILE_ATTRIBUTE_NORMAL, NULL);
  if( NewFile == INVALID_HANDLE_VALUE )
   return FALSE;
 }


 DWORD Verif;
 SetFilePointer(NewFile, 0, 0, FILE_END);
 WriteFile(NewFile, Sdata, TaillePak, &Verif, NULL);
 ListeDl[i].currentNbPak++;


 return TRUE;
}

Voila, desol&#233; si j'en ai mis un peu trop!
O&#249; est la boulette ? (ou les boulettes plutot !!!!)
Merci &#224; tous!

Melkior_le_necrarque

BruNews · Answer

Je regarde juste ReceptionFichier():

pourquoi tu Write inconditionnellement TaillePak octets ???

int n = recv(...);

WriteFile(NewFile, Sdata, n, &Verif, NULL);

ne te semble pas ? si oui voir le reste sur ce principe.

ciao...
http://dev.winsysdev.com
BruNews, MVP VC++

melkiorlenecrarque · Answer

OK pour cette erreur, j'ai modifi&#233; la fonction comme ceci:
BOOL ReceptionFichier()
{
 char Sdata[TaillePak];
 ZeroMemory(Sdata, TaillePak);
 
 int n = recv(sockfichierR, Sdata, sizeof(Sdata), NULL);
 
 if( NewFile == NULL )
 {
  int i = 0;
  for( i = 0 ; i<=NbDl ; i++ )
  {
   if( ListeDl[i].ID == CurrentFile )
   {
    IndexCurrentDl = i;
    break;
   }
  }
  NewFile = CreateFile(ListeDl[IndexCurrentDl].nom, GENERIC_WRITE, FILE_SHARE_READ, NULL, CREATE_ALWAYS, FILE_ATTRIBUTE_NORMAL, NULL);
  if( NewFile == INVALID_HANDLE_VALUE )
   return FALSE;
 }


 if( NewFile == INVALID_HANDLE_VALUE )
  return FALSE;


 DWORD Verif;
 SetFilePointer(NewFile, 0, 0, FILE_END);
 WriteFile(NewFile, Sdata, n, &Verif, NULL);
 ListeDl[IndexCurrentDl].currentNbPak++;


 return TRUE;
}

Mais le probl&#232;me est toujours pr&#233;sent.

Pb reception de fuchiers

2 réponses

Votre réponse

Discussions similaires