Discussion :

[Sixclopes]

Bonjour à tous
- Pour parcourir mon fichier j'utilise
Mes déclarations

#define MAX_REC_LEN 2044
char bufline[MAX_REC_LEN+2];
FILE* curr_file;

while ((code = read_line()) != EOF)
{
...

-La fonction read_line est la suivante:

Code :

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

1
2
3
4
5
6
7
8
9
10
11
 
int read_line()
{
	memset(bufline,0,MAX_REC_LEN+2);
	if (fgets(bufline, MAX_REC_LEN+2, curr_file)==NULL)
	{
		return EOF;
	}
	if (bufline[0]==0) return EOF;
	return 1;
}

curr_file: Mon fichier
bufline : est censé contenir la ligne du fichier.

Mon problème c'est que ma fonction read_line me retourne parfois un EOF (le premier) directement, mais ce problème est aléatoire, je remets le même fichier dans le répertoire de traitement et il est traité sans problème.

Est ce que c'est possible que ça soit un problème de mémoire ? ou c'est autre chose ?

Merci pour votre aide

[Invité]

Bonjour,
C'est quoi EOF?
Ca veut dire "End Of File", mais je ne pense pas que ce soit une constante définie.
Habituellement, on écrit cela

Code :

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

1
2
3
4
5
6
7
8
FILE *in=fopen("fictier","rt"); // ou autre-chose
while (!feof(in))
{
...
//lecture
...
} 
fclose(in);

Si vous replacez EOF par 0 (par exemple) vous y verrez déjà plus clair.

[Sixclopes]

Oui c'est bien END OF FILE, elle est définit dans un autre fichier, mais j'ai oublié de mentionner ça.

Ce que je n'ai pas compris, c'est que c'est un problème aléatoire, je n'ai pas toujours ce problème.

[Invité]

Avez regardé le bout de code que j'ai écrit et ce qu'a écrit jeroman.?

[diogene]

Bonjour,
C'est quoi EOF?
Ca veut dire "End Of File", mais je ne pense pas que ce soit une constante définie.
....
Si vous replacez EOF par 0 (par exemple) vous y verrez déjà plus clair.

EOF est une constante définie comme étant un int de valeur négative. Elle est définie dans <stdio.h>

[Invité]

Exact, pardon.

Code :

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

#define EOF (-1)                /* End of file indicator */

[jeroman]

Je soupçonne l'utilisation de globales (car ta fonction de lecture n'a aucun argument), c'est pas propre et source de bugs (on peut modifier la valeur n'importe où dans le code).

Ce code (une ébauche), qui sert d'exemple, fonctionne chez moi. A adapter.

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
#include <stdio.h>
 
#define MAX_REC_LEN 2044
 
int read_line(char * bufline , int taille , FILE * curr_file)
{
    if ( fgets(bufline , taille , curr_file) == NULL )
    {
        return EOF;
    }
 
    /* A FAIRE : rechercher '\n' ou "\r\n" et remplacer par '\0' */
 
    if (bufline[0] == 0)
        return EOF;
 
    return 1;
}
 
int main(void)
{
    int code;
    int numero_ligne = 0;
    char bufline[MAX_REC_LEN+2];
    FILE* curr_file;
 
    if ((curr_file = fopen("Fichier.txt","rb")) == NULL)
        return 0;
 
    while ( (code = read_line(bufline , MAX_REC_LEN+2 , curr_file)) != EOF )
    {
        numero_ligne++;
        printf( "ligne %d : %s" , numero_ligne , bufline );
    }
 
    return 0;
}

J'ai viré le memset, qui ne sert à rien. Il y a des choses à rajouter (voir commentaire).

Je ne comprend pas pourquoi créer un tableau de dimension REC_LEN+2. Pourquoi +2 ? Et pourquoi précisément 2044 ? J'ai laissé tel quel, mais je trouve ça bizarre.

[Jean-Marc.Bourguet]

Est ce que c'est possible que ça soit un problème de mémoire ?

Le problème ne vient pas de ce qui est montré, même si le code n'est pas d'un style particulièrement à conseiller.

Habituellement, on écrit cela

Code :

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

1
2
3
4
5
6
7
8
FILE *in=fopen("fictier","rt"); // ou autre-chose
while (!feof(in))
{
...
//lecture
...
} 
fclose(in);

J'espère bien que non. Habituellement l'utilisation de feof pour contrôler une boucle dénote plutôt une mauvaise compréhension du comportement des IO en C. Si le code de l'OP est stylistiquement pas à conseiller, une telle utilisation de feof est généralement erronée.

La boucle de lecture par ligne canonique en C est

Code :

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

1
2
3
4
5
6
7
8
9
10
 
while (!fgets()) {
   /* vérification de tampon trop court */
   /* utilisation */
}
if (ferror(...)) {
   /* erreur d'IO */
} else {
   /* fin de fichier */
}

[Invité]

Bonjour,

Habituellement l'utilisation de feof pour contrôler une boucle dénote plutôt une mauvaise compréhension du comportement des IO en C. Si le code de l'OP est stylistiquement pas à conseiller, une telle utilisation de feof est généralement erronée.

Il serait intéressant de préciser une utilisation de feof() "à conseiller".
Dans votre exemple, vous faites une lecture du fichier avec fgets(). En réalité, et dans le cas général, la lecture peut être faite avec plusieurs fonctions, et un bonne gestion prévoira la présence d'une donnée marquent la fin du fichier, de façon à sortir de la bouche while.
En quelque-sorte, l'instruction !feof(in) est là par sécurité et dans la pratique, un programme doit prévoir cette instruction et les instructions de lecture seront à l'intérieur de la boucle.
D'ailleurs, à titre personnel, je n'utilise fgets() que dans les cas où je suis sûr de l'organisation en lignes, par exemple un fichier CSV. C'est à dire que j'utilise fgetc(), naturellement par l'intermédiaire d'une fonction de lecture.

[gl]

Dans votre exemple, vous faites une lecture du fichier avec fgets(). En réalité, et dans le cas général, la lecture peut être faite avec plusieurs fonctions

Ce qui ne change pas fondamentalement les choses, toutes les fonctions de lecture ont un mécanisme similaire permettant d'indiquer qu'une erreur ou que la fin de fichier a été rencontré. Il suffit d'utiliser le test correspondant à la fonction de lecture utilisée.

En quelque-sorte, l'instruction !feof(in) est là par sécurité et dans la pratique, un programme doit prévoir cette instruction et les instructions de lecture seront à l'intérieur de la boucle.

Le problème c'est que simplement boucler sur feof() ne fonctionne pas. Il faut bel et bien tester le retour des fonctions de lecture. feof() permet simplement, si c'est nécessaire, de distinguer la fin de fichier d'une autre erreur de lecture.

[Invité]

OK,
Mais je ne sais toujours pas quand la fonction feof() peut être utilisée correctement.

[jeroman]

Comment ta fonction connait le pointeur sur le fichier si tu ne lui passes pas en argument ? Même chose pour bufline... Ce sont des globales ?

Dans l'ordre :
Il faut ouvrir le fichier.
Vérifier s'il est bien ouvert.
Si tout est bon : ensuite, tu peux lire chaque ligne via ta fonction en lui passant le pointeur sur le fichier et l'adresse où écrire la ligne (bufline), en argument.

Je ne comprends pas l'utilité du memset. S'il s'agit d'un fichier texte, la ligne est censée se terminer par un '\n' (qui pourra être supprimé par la suite en le remplaçant par un '\0'). Et s'il n'y en a pas, c'est que le buffer est trop petit. Remplir toute la ligne avec le caractère '\0' avant chaque lecture consomme des ressources inutilement.

[Sixclopes]

Comment ta fonction connait le pointeur sur le fichier si tu ne lui passes pas en argument ? Même chose pour bufline... Ce sont des globales ?

Dans l'ordre :
Il faut ouvrir le fichier.
Vérifier s'il est bien ouvert.
Si tout est bon : ensuite, tu peux lire chaque ligne via ta fonction en lui passant le pointeur sur le fichier et l'adresse où écrire la ligne (bufline), en argument.

Je ne comprends pas l'utilité du memset. S'il s'agit d'un fichier texte, la ligne est censée se terminer par un '\n' (qui pourra être supprimé par la suite en le remplaçant par un '\0'). Et s'il n'y en a pas, c'est que le buffer est trop petit. Remplir toute la ligne avec le caractère '\0' avant chaque lecture consomme des ressources inutilement.

salut,
C'est ce que je fais.
j'ouvre le fichier
if ((curr_file = fopen(filename,"r")) == NULL) return 0

sinon la ligne se termine par un 0D 0A (Hex)
un exemple de mon fichier en pièce jointe.

Désolé si je l'ai répété, mais je ne comprends pas pourquoi la problème est aléatoire.

Pierre Dolez
Oui je viens de voir, merci, je vais essayer de modifier mon code. mais j'ai toujours utilisé cette fonction et je n'avais jamais eu un tel problème avant.

Merci

[Invité]

Votre fichier contient une seule ligne très longue.
Il faut le lire autrement.