Discussion :

[lappin]

Bonjour à tous,

J'ai créé à l'aide de code::blocks un programme qui copie les lignes d'un fichier en y ajoutant quelques caractère "au bout" dans un autre fichier. Le problème est le suivant : l'écriture s'arrête inexplicablement lorsque le fichier copié dépasse 4Ko. Alors que la fonction qui copie parcours bien tout le fichier.

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
36
37
38
39
40
41
42
43
44
 
#include <stdio.h>
#include <stdlib.h>
 
int main()
{
    FILE* fichier1 = NULL;
    FILE* fichier2 = NULL;
    fichier1= fopen("b3.txt","r");
    fichier2= fopen("b4.txt","w+");
    char ajout[] = "</br>\n";
    char chaine[10000]="";
    long i=0;
 
    if(fichier1==NULL)
        {
        printf("fichier non trouve");
        }
    else{
 
         do
         {
          do{i++;}while(fgetc(fichier1)!='\n');
 
fseek(fichier1,-i-1,SEEK_CUR);
fgets(chaine,i,fichier1);
 
fputs(chaine,fichier2);
fprintf(fichier2,"</br>\n");
 
 
printf("%d  %d\n",ftell(fichier1),ftell(fichier2));
 
i=0;
         }while(fgetc(fichier1)!='\0');
 
 
   fclose(fichier1);
   fclose(fichier2);
    }
 
 
    return 0;
}

Merci d'avance

[Sve@r]

Salut
J'ai recopié tel quel ton code chez-moi et il m'a parfaitement recopié un fichier de 188ko. Toutefois le programme ne s'est pas arrêté et j'ai du l'interrompre manuellement

Concernant ton code: outre qu'il est indenté comme un porc, je ne pige pas:

• cette première boucle pour récupérer un '\n' et surtout ce fseek(..., -i-1) !!! Si tu as envie de revenir au début, pourquoi ne pas utiliser un fseek(fichier, 0, SEEK_SET) ???
• Cette recherche de '\0' dans le fichier. Ce n'est pas que ce soit impossible de placer un '\0' dans un fichier texte mais déjà ce n'est plus un vrai fichier texte et surtout on a plutôt l'habitude de voir le '\0' comme flag de fin de chaine et non flag de fin de fichier...
• ce "i=0" en fin de boucle alors que i sera utilisé comme nombre d'octets à lire dans le fgets() suivant...

Sinon voici ton code réindenté proprement. C'est quand-même un minimum quand on demande aux autres de lire ses oeuvres qu'elles soient un peu lisibles...

Code c :

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
36
37
38
39
40
41
 
#include <stdio.h>
#include <stdlib.h>
 
int main()
{
	FILE* fichier1 = NULL;
	FILE* fichier2 = NULL;
	fichier1= fopen("b3.pdf","r");
	fichier2= fopen("b4.pdf","w+");
	char ajout[] = "</br>\n";
	char chaine[10000]="";
	long i=0;
 
	if(fichier1==NULL) {
		printf("fichier non trouve");
		// Jamais tu fermes fichier 2 ???
	}
	else {
		do {
			do {
				i++;
			} while(fgetc(fichier1)!='\n');
 
			fseek(fichier1,-i-1,SEEK_CUR);
			fgets(chaine,i,fichier1);
 
			fputs(chaine,fichier2);
			fprintf(fichier2,"</br>\n");
			// Ces deux lignes en une seule: fprintf(fichier2,"%s</br>\n", chaine);
 
			printf("%d  %d\n",ftell(fichier1),ftell(fichier2));
 
			i=0;
		} while(fgetc(fichier1)!='\0');
 
		fclose(fichier1);
		fclose(fichier2);
	} 
	return 0;
}

[lappin]

Merci d'avoir jeté et excuse-moi pour avoir oublié d'indenté.
Si cela peut éclairer, il s'agite de recopier un fichier, non généré par moi, de la forme :
ligne1 \n
ligne2 \n
ligne3 \n
...
Et d'y ajouter </br> à la fin de chaque ligne. i représente en fait la longueur de la ligne à chaque étape. Le déplacement de -i-1 a pour but de revenir à la fin de la ligne une fois que j'en connais la longueur (je n'utilise pas la fonction strlen car il n'y a pas le caractère de fin de chaine à chaque ligne).
Si je recherche '\0' dans le fichier et que je termine par ce même caractère c'est parce que dans mon fichier source je me suis aperçut qu'il s'agissait du dernier caractère du fichier.
Si je remet i à 0, c'est pour réinitialiser la longueur de la linge.

Le code produit le résultat voulu, seulement il arrête d'écrire dans le fichier b4 dès que celui-ci atteint 4.135 octets. Alors qu'il m'affiche en console que le pointeur se déplace tout le long des lignes du fichier.

[Sve@r]

Merci d'avoir jeté et excuse-moi pour avoir oublié d'indenté.
Si cela peut éclairer, il s'agite de recopier un fichier, non généré par moi, de la forme :
ligne1 \n
ligne2 \n
ligne3 \n
...
Et d'y ajouter </br> à la fin de chaque ligne. i représente en fait la longueur de la ligne à chaque étape. Le déplacement de -i-1 a pour but de revenir à la fin de la ligne une fois que j'en connais la longueur (je n'utilise pas la fonction strlen car il n'y a pas le caractère de fin de chaine à chaque ligne).

Ok ben alors ça sera plus simple de cette façon

Code c :

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
char *c;
char ligne[1000];
FILE *fichier1;
FILE *fichier2;
 
fichier1=fopen(...);
fichier2=fopen(...);
while (fgets(ligne, 1000, fichier1) != EOF)
{
    // Suppression '\n'
    if ((c=strchr(ligne, '\n') != NULL)
        *c='\0';
 
    // Ecriture nouvelle ligne (en y incluant le '\n' supprimé)
    fprintf(fichier2, "%s<br>\n", ligne);
}
 
// Fini
fclose(fichier1);
fclose(fichier2);

Donc le principe est là: tu lis une ligne, tu arranges un peu cette ligne et tu la réécris. Et bien que le fgets(), comme toute fonction génératrice de string, s'occupe de mettre lui-même le '\0', nul besoin d'utiliser strlen(). Seul danger: qu'une des lignes de ton fichier soit plus longue que 999 octets. Dans ce cas, tu en récupèreras 2 dans l'autre fichier...

Si je recherche '\0' dans le fichier et que je termine par ce même caractère c'est parce que dans mon fichier source je me suis aperçut qu'il s'agissait du dernier caractère du fichier.

Hum, là j'ai un très gros doute. Faudra que tu nous montres de quelle façon tu as fait cette recherche/découverte...

[diogene]

@lappin
Ta méthode n'est pas bonne :
- La fin de fichier n'est pas détectée de cette façon : fgetc() ne renvoie pas '\0' dans ce cas mais EOF
- Le comportement de fseek() utilisé de cette façon pour des fichiers texte est dépendante de l'implémentation.

Utilise plutôt la méthode préconisée par Sve@r

[lappin]

Merci pour les conseils sur le code, le recopiage fonctionne et la forme obtenue est bonne cependant, il reste quelques problèmes :

La ligne :

Code :

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

*c='\0';

, fait planter le programme. Si j'ai bien compris il s'agit de placer le caractère de fin de chaine pour la raccourcir (supprimer le saut de ligne). Comme le saut de ligne ne me dérange pas j'ai alors supprimé cette ligne. La chaine de caractères contient alors le '\n' mais s'arrête juste après.
La boucle ne s'arrête pas :
Le programme recopie la dernière ligne à l'infini.

Si je recherche '\0' dans le fichier et que je termine par ce même caractère c'est parce que dans mon fichier source je me suis aperçut qu'il s'agissait du dernier caractère du fichier.

Hum, là j'ai un très gros doute. Faudra que tu nous montres de quelle façon tu as fait cette recherche/découverte...

Je me suis juste empiriquement aperçut qu'avec cette condition la boucle "while" stoppait au bon moment ( à la dernière ligne du fichier).

Pour info 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
24
25
26
27
 
char *c;
char ligne[1000];
FILE *fichier1;
FILE *fichier2;
 
fichier1=fopen("b3.txt","r+");
fichier2=fopen("b4.txt","w+");
 
 
while (fgets(ligne, 1000, fichier1) != EOF)
{
     printf("%s ",ligne);
 
       if (c=strchr(ligne, '\n') != NULL)
       {
       /* *c ='\0';*/
 
         fprintf(fichier2, "%s</br>\n", ligne);
 
       }
}
 
fclose(fichier1);
fclose(fichier2);
 
return 0;

J'espère que l'indentation suffit, pratiquant la programmation en amateur je n'ai pas l'habitude que mon code sois lu.