Discussion :

[afrikha]

Bonjour tout le monde ,
J'ai écrit un bète programme qui consiste à copier un fichier. Tout se passe bien sauf que le dernier enregistrement est copié deux fois.
Voici mon code :

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
#include <stdio.h>
#include <stdlib.h>
 
int main(int argc, char *argv[])
{
    FILE *fichier1,*fichier2;
    int matricule;
    char nom[10],prenom[10];
    /*ouverture des fichiers*/
    fichier1=fopen("c:\\fichierC.txt","r");
    fichier2=fopen("c:\\fichierC2.txt","w");
    while(!feof(fichier1)){
                          fscanf(fichier1,"%i %s %s",&matricule,nom,prenom);
                          fprintf(fichier2,"%i %s %s \n",matricule,nom,prenom);
                          }
    /*fermeture des fichiers*/
    fclose(fichier1);
    fclose(fichier2);
 
 
  system("PAUSE");	
  return 0;
}

P.S: je ne sais pas si fscanf est dépréciée mais je suis contraint de l'utiliser car c'est un exercice.

Merci d'avance pour votre aide

[Franck.H]

Pourquoi pas utiliser la méthode classique s'il s'agit de copier tout simplement un fichier source vers un fichier destination ?

Cette méthode consiste en l'ouverture en mode binaire sur la source et la destination, la lecture fread depuis la source et l'écriture fwrite vers la destination !

Voir la FAQ: Comment copier un fichier ?

[afrikha]

J'aurais bien essayé mais je suis obligé de passer par fprintf et fscanf, de plus j'aimerais comprendre l'erreur dans mon programme.
Merci quand même d'avoir tenté de m'aider.

[Franck.H]

J'aurais bien essayé mais je suis obligé de passer par fprintf et fscanf

C'est une contrainte imposée ? Ou bien c'est par rapport à ce que contient ton fichier source ou meme son formattage (dans ce cas tu peux utiliser la méthode binaire) ?

[Franck.H]

Pour ton cas ici présent, le mieux serais de tester et faire l'appel directement dans la condition de ta boucle et de tester avec feof après la boucle, là c'est une meilleure utilisation de cette fonction !

[Pragmateek]

Code :

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

while(!feof(fichier1))

"feof" permet de tester si le flag "EOF" du flux a été levé par une fonction de lecture; il est généralement utilisé après l'arrêt de la lecture pour vérifier que l'arrêt est bien dû à l'arrivée à la fin du flux et non pas à une erreur de lecture.
De plus il faudrait vérifier le retour de "fscanf" pour vérifier que tous les éléments ont été lus.

[gege2061]

P.S: je ne sais pas si fscanf est dépréciée mais je suis contraint de l'utiliser car c'est un exercice.

scanf n'est pas obsolète, juste compliqué à utiliser. Pour exemple, une meilleur utilisation serait :

Code :

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

1
2
3
4
if (fscanf(fichier1,"%i %9s %9s",&matricule,nom,prenom) == 3)
{
  /*  Conversions réussies, on recopie */
}

Mais comment faire en cas d'erreur ? Si tu ne fais rien, tu tombe sur une boucle infinie

fopen peut échouer pense à tester la valeur de retour !