Discussion :

[france38]

Bonjour

Voila j'ai fait une procédure qui copie un fichier et qui ajoute +1 à chaque caractère du fichier, en le copiant dans un autre fichier.
Seulement j'ai un problème c'est que à la fin du fichier il m'ajoute toujours 1 caractère en plus, alors qu'il n'est pas dans le fichier orignal.

Cela m'arrive aussi quand je fais une copie intacte d'un fichier.

Voici le 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
24
25
26
27
28
29
30
31
32
33
34
35
 
void cryptage()
{
 FILE *fichier=NULL;
 FILE *fichierCrypter=NULL;
 char caractere;
 
 fichier=fopen("ess.txt","r");
 
 
 if(fichier!=NULL)
    {
    fichierCrypter=fopen("crypter.txt","w");
    if(fichierCrypter!=NULL)
        {
           while(!feof(fichier))
           {
            fread(&caractere,sizeof(caractere),1,fichier);
            caractere++;
            fwrite(&caractere,sizeof(caractere),1,fichierCrypter);
           }
 
        }
        else
        {
         printf("Erreur sur fichierCrypter\n");
        }
    }
    else
    {
     printf("Erreur sur fichier\n");
    }
    fclose(fichier);
    fclose(fichierCrypter);
}

Merci d'avance.

[Médinoc]

Bonjour,

1. Mauvaise utilisation de feof(). Tu dois lire tant que fread() n'échoue pas, et ne tester feof() qu'une fois que fread() a échoué.
2. Attention: si tu veux pouvroi crypter n'importe quel fichier, tu dois faire une lecture et écriture binaire, pas texte: Ajoute la lettre "b" au second paramètre de fopen(), dans les deux appels.

[france38]

1.Excuse moi mais il y a un petit point que je ne comprend pas bien, c'est que si je lit jusqu'à ce que fread échoue, cela veux dire que je suis à la fin du fichier non? est si c'est le cas à quoi cela sert que je reteste que je suis bien à la fin?

Est-ce que je peut avoir un petit exemple?

2.Oui bien sur la je fais juste un petit essai sur les fichiers texte je connais pas trop tous ce qui est cryptage mais merci pour l'information.

Merci

[Médinoc]

En réponse à ta question 1: Ça sert à vérifier que fread() n'a pas échoué pour une autre raison (quelle qu'elle soit).

En, clair, après que fread() échoue:

1. si feof(), OK
2. sinon, on gueule.

[france38]

Si j'ai bien compris ce que tu ma dit le code devrais donnais un truc dans le genre la. Il a l'air de fonctionner.

Code :

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

1
2
3
4
5
6
7
8
9
 
while(fread(&caractere,sizeof(caractere),1,fichier)!=0)
           {
               if (!feof(fichier))
                  {
                     caractere++;
                     fwrite(&caractere,sizeof(caractere),1,fichierCrypter);
                  }
           }

[nicolas.sitbon]

C'est une FAQ!

[Médinoc]

Plutôt comme ça:

Code C :

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

1
2
3
4
5
6
7
8
9
	while(fread(&caractere,sizeof(caractere),1,fichier)!=0)
	{
		caractere++;
		fwrite(&caractere,sizeof(caractere),1,fichierCrypter);
	}
	if(!feof(fichier))
	{
		puts("Erreur en lecture: fread() s'est arrete avant la fin.");
	}

PS: Si tu fais tout caractère par caractère, tu peux remplacer avantageusement fread() et fwrite() par fgetc() et fputc(). Attention toutefois, fgetc() retourne un int.

[france38]

Merci bien pour toute c'est réponse.