Discussion :

[mkacho]

bonjour, pour un programme de sauvegarde de fichier SOUS WINXP, j'ai ecris une fonction de copie de fichier en c qui bug pendant l'execution.Dapres mes recherche mon problem vie de ma fonction fwrite mais je ne vois pa ce qui cloche. Ma fonction doit pouvoir utiliser pour fonctionnè deux chaines de caractaire declarer en variable globale au debut du programme. Quellequ'u pourait bien corriger l'erreur SVP. merci pour votre aide.

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
19
20
21
22
23
24
25
26
27
#include <stdio.h>
#include <stdlib.h>
#include <string.h>
 
static char * sources ="C:\\Documents and Settings\\Administrateur\\Bureau\\ibrahim ahamadi\\prog\\Projet_Prog\\versionC\\ft_copie\\Premier.url";
static char * destination="C:\\Documents and Settings\\Administrateur\\Bureau\\ibrahim ahamadi\\prog\\Projet_Prog\\versionC\\ft_copie\\caca";
 
 
void  copieFile(char * fileSource, char * fileDestination){
 
  int n;
  FILE * lacopie;
  FILE * lasource;
 
  lasource = fopen(fileSource, "rb");
  lacopie= fopen(fileDestination, "wb");
 
  do{
    fread(&n, sizeof(int), 1, lasource);
    if(!feof(lasource))  {fwrite(&n, sizeof(int), 1, lacopie);}
  }
  while(!feof(lasource));
}
int
main(void){
  copieFile(sources,destination); return  0;
}

[DaZumba]

Plusieurs imperfections dans ce code:
- verifier que les fopen() n'ont pas echoue.
- verifier que fread() et fwrite() ont bien lu/ecrit le nombre d'elements demandes. feof() ne doit etre appele qu'apres la detection d'une erreur.
- es-tu bien sur que la taille de ton fichier est un multiple de sizeof(int) ? En general, non: il faut donc changer ta facon de proceder.

[Pragmateek]

Lire de plus gros blocs de données ne serait il pas plus performant?

[Trap D]

La boucle de lecture/écriture devrait s'écrire ainsi

Code :

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

1
2
3
// ici il n'y a pas de test sur l'écriture puisque le retour de fwrite n'est pas testé
while ((nb = fread(&n, sizeof(int), 1, lasource)) > 0)
    fwrite(&n, sizeof(int), nb, lacopie);

[DaZumba]

Lire de plus gros blocs de données ne serait il pas plus performant?

Si. En general, on lit les fichiers dans un tampon de char de taille BUFSIZ (stdio.h):

Code :

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

1
2
 
char buffer[BUFSIZ];

[Pragmateek]

La boucle de lecture/écriture devrait s'écrire ainsi

Puis tester avec "feof()" la cause de l'arrêt de la boucle de lecture:

Code :

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

1
2
3
4
5
6
if(feof(lasource)){
/*la fin de fichier a été atteinte*/
}
else{
/*une erreur de lecture s'est produite*/
}

[Médinoc]

Si. En general, on lit les fichiers dans un tampon de char de taille BUFSIZ (stdio.h):

Code :

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

1
2
 
char buffer[BUFSIZ];

Tu es sûr ? Il me semblait que cette constante n'avait qu'un seul but: Définir la taille du buffer INTERNE des fonctions de stdio.h parce qu'il est possible de le changer.
Mais c'est vrai qu'envisagé sous cet angle, une taille multiple de BUFSIZ peut être optimale.

Puis tester avec "feof()" la cause de l'arrêt de la boucle de lecture:

Je mets l'accent sur le puis.

PS: mkacho : Déclare tes char * en "static const char * const source = ...". Le premier const est le plus important, il t'évitera de faire des bétises avec tes noms de fichiers.

[DaZumba]

Tu es sûr ? Il me semblait que cette constante n'avait qu'un seul but: Définir la taille du buffer INTERNE des fonctions de stdio.h parce qu'il est possible de le changer.
Mais c'est vrai qu'envisagé sous cet angle, une taille multiple de BUFSIZ peut être optimale.

C'est exactement ca. On pourrait utiliser n'importe quoi, mais puisque BUFSIZ existe, autant en profiter. Ce sera plus performant ou non, peu importe. La doc de la glibc ecrit a ce sujet:

Sometimes people also use BUFSIZ as the allocation size of buffers used for related purposes, such as strings used to receive a line of input with fgets [...]. There is no particular reason to use BUFSIZ for this instead of any other integer, except that it might lead to doing I/O in chunks of an efficient size.