Copier un repertoire avec ses sous répertoire. Fonction qui a l'air ok INSIDE. aRésolu

Question

Bonjour,

Je suis d&#233;butant en C. Je cherchais une fonction capable de copier un repertoire avec ses sous repertoire.
J'ai cr&#233;e ma propre fonction, qui &#224; l'air de marcher. Pour progresser, si vous avez la moindre remarque ou am&#233;lioration faites le moi savoir pour que j'adopte d&#232;s le d&#233;but la bonne facon de programmer

//fonction qui va copier le repertoire szPathSource vers szPathDest
int CopyDir (char* szPathSource, char* szPathDest)
{
 char* szMyPathSource = NULL;
 char* szMyPathDest  = NULL;
 char* szBuffer    = NULL;
 WIN32_FIND_DATA    MyFile;  
 HANDLE        Myliste;


 szMyPathSource = (char*) malloc( sizeof(char)*(strlen(szPathSource)+2) );
 szMyPathDest  = (char*) malloc( sizeof(char)*(strlen(szPathDest)+2) );


 lstrcpy(szMyPathSource,szPathSource);
 lstrcpy(szMyPathDest,szPathDest);


 if (szMyPathSource [strlen(szMyPathSource) -1] != '\')
  lstrcat(szMyPathSource,"\");
 if (szMyPathDest [strlen(szMyPathDest) -1] != '\')
  lstrcat(szMyPathDest,"\");


 szBuffer = (char*) malloc( sizeof(char)*(strlen(szMyPathSource)+2) );
 lstrcpy(szBuffer,szMyPathSource);
 lstrcat(szBuffer,"*");


  //On copie colle le contenu du repertoire dans notre repertoire de travail
 Myliste = FindFirstFile(szBuffer, &MyFile);
 while(FindNextFile(Myliste, &MyFile)) {
  char* szSousBufferSource =  NULL;
  char* szSousBufferDest  =  NULL;
  
  //si c'est le repertoire courant on ne fait rien
  if ( !strcmp(MyFile.cFileName, "..") || !strcmp(MyFile.cFileName, ".") )
   continue;


  szSousBufferSource = (char*) malloc( sizeof(char)*(strlen(szMyPathSource)+strlen(MyFile.cFileName)+1) );
  szSousBufferDest  = (char*) malloc( sizeof(char)*(strlen(szMyPathDest)+strlen(MyFile.cFileName)+1) );


   lstrcpy(szSousBufferSource,szMyPathSource);
  lstrcat(szSousBufferSource,MyFile.cFileName);


  lstrcpy(szSousBufferDest,szMyPathDest);
  lstrcat(szSousBufferDest,MyFile.cFileName);


  // C'est un repertoire on applique la fonction r&#233;cursive
  if( MyFile.dwFileAttributes & FILE_ATTRIBUTE_DIRECTORY )
  {
   // on cr&#233;e le repertoire
   if (mkdir(szSousBufferDest))
    return 1;
   if (CopyDir (szSousBufferSource,szSousBufferDest))
    return 1;
  }
  else
   // C'est un fichier, on copie/colle
   if (!CopyFile(szSousBufferSource, szSousBufferDest, FALSE))
    return 1;


  //on d&#233;salloue les buffer 
  szSousBufferSource = NULL;
  szSousBufferDest  = NULL;


  free(szSousBufferSource);
  free(szSousBufferDest);
 }
 FindClose (Myliste); 


 szMyPathSource = NULL;
 szMyPathDest  = NULL;
 szBuffer    = NULL;


 free(szMyPathSource);
 free(szMyPathDest);
 free(szBuffer);


 return 0;
}

ymca2003 · Accepted Answer

la d&#233;sollacation est mal faite
//on d&#233;salloue les buffer 
  szSousBufferSource = NULL;
  szSousBufferDest  = NULL; 
  free(szSousBufferSource);
  free(szSousBufferDest);

=> la fonction free est appel&#233;e avec des pointeurs NULL

il faut appel free avant de mettre &#224; NULL


  free(szSousBufferSource);
  free(szSousBufferDest);


  szSousBufferSource = NULL;
  szSousBufferDest  = NULL; 

idem pour
free(szMyPathSource);
 free(szMyPathDest);
 free(szBuffer);

cs_opala · Answer

ok merci

dans mon application principale, j'ai :

char * szNomConfigXML    = "Lot_config.xml";

la commande free(szNomConfigXML); fait planter mon appli, est ce normal?

Est t'il de faire un = NULL apres un free?
Exemple:
free(szMachaine);
szMachaine = NULL;

Taron31 · Answer

Nan

___________________
MVS - Most Valuable Student ( Microsoft)

ymca2003 · Answer

c'est normal parce que le pointeur n'a pas &#233;t&#233; allou&#233; avec malloc (c'est une chaine constante statique en m&#233;moire)
mettre &#224; NULL les variables apr&#232;s les avoir lib&#233;rer permet de s'assurer qu'on va pas les utiliser par m&#233;garde (un test &#224; NULL permettra alors de pas toucher &#224; la variable).

Copier un repertoire avec ses sous répertoire. Fonction qui a l'air ok INSIDE. a

4 réponses

Votre réponse

Discussions similaires