[C] fgets ne marche qu'une fois...

Recherche :

Mot : Pseudo : Filtrer
Bas de page
Auteur	Sujet : [C] fgets ne marche qu'une fois...

kaloskagatos

Bonjour,

J'ouvre un fichier texte avec FILE*streamFile = fopen(fileName,"r" ),

je lis ligne par ligne avec fgets != NULL

et quand c'est fini je ferme le fichier fclose(streamFile)

Tout se passe bien une fois.

Si je repasse le même nom de fichier, fgets renvoie NULL direct...

J'ai essayé de faire:

rewind(streamFile);
ou
fseek(streamFile,SEEK_SET,0);

après l'ouverture du fichier mais ça ne change rien. Tête dans le guidon je vois pas ce qui cloche... Merci d'avance.

Message cité 1 fois
Message édité par kaloskagatos le 16-10-2006 à 16:41:35

Publicité

kaloskagatos

J'ai codé le truc en c++ avec des ifstream et des std::getline et j'ai le même problème...

Chaos Intestinal

Pourrais tu nous mettre le code pertinent en entier (entre balises [cpp])

kaloskagatos

J'aurais dû faire ça tout de suite oui...

Code :

CHAINE ligne;
CHAINE word;
FILE *streamFile = NULL;
streamFile = fopen(fileName, "r" );
if ( streamFile > 0 )
{
//rewind(streamFile);
//fseek(streamFile,SEEK_SET,0);
while( fgets(ligne, MAXLEN, streamFile) != NULL )
{
//traitement de ligne
}
}
else
{
error = -1;
sprintf(errorMessage,"Can't open Chimera Mask file (%s)",fileName);
}
fclose(streamFile);

la première fois que j'ouvre le fichier je rentre dans le while, si j'ouvre à nouveau le fichier fgets renvoie NULL tout de suite, même avec rewind ou fseek.

je précise typedef char CHAINE[MAXLEN];

Message cité 1 fois
Message édité par kaloskagatos le 16-10-2006 à 16:58:49

Emmanuel Delahaye

C is a sharp tool

kaloskagatos a écrit :

J'ouvre un fichier texte avec FILE*streamFile = fopen(fileName,"r" ),
je lis ligne par ligne avec fgets != NULL
et quand c'est fini je ferme le fichier fclose(streamFile)
Tout se passe bien une fois.

OK.

Citation :

Si je repasse le même nom de fichier, fgets renvoie NULL direct...

Je ne connais pas de fonction 'repasse()'. Montre ce code là.

Message cité 1 fois

---------------
Des infos sur la programmation et le langage C: http://www.bien-programmer.fr Pas de Wi-Fi à la maison : http://www.cpl-france.org/

Emmanuel Delahaye

C is a sharp tool

kaloskagatos a écrit :

la première fois que j'ouvre le fichier je rentre dans le while, si j'ouvre à nouveau le fichier fgets renvoie NULL tout de suite, même avec rewind ou fseek.

Code :

typedef char CHAINE[MAXLEN];
/* -ed- c'est quoi MAXLEN ? */
CHAINE ligne;
CHAINE word;
/* -ed- c'est quoi CHAINE ? */
FILE *streamFile = NULL;
streamFile = fopen(fileName, "r" );
/* -ed-
if ( streamFile > 0 )
atypique, mais correct. Je prefere
*/
if ( streamFile != NULL )
{
while( fgets(ligne, MAXLEN, streamFile) != NULL )
{
//traitement de ligne
}
/* -ed- deplace */
fclose(streamFile);
}
else
{
error = -1;
sprintf(errorMessage,"Can't open Chimera Mask file (%s)",fileName);
}
/* -ed- mal place
fclose(streamFile);
*/

Essaye ça, c'est plus clair.

Si le défaut persiste, explique plus précisément ce qui se passe. Tu appelles 2 fois ce code ? Il est 2 fois dans ton programme ? Tu appelle deux fois le programme qui contient ce code ? J'ai du mal à saisir les circonstantes du problème signalé.

---------------
Des infos sur la programmation et le langage C: http://www.bien-programmer.fr Pas de Wi-Fi à la maison : http://www.cpl-france.org/

kaloskagatos

Emmanuel Delahaye a écrit :

Je ne connais pas de fonction 'repasse()'. Montre ce code là.

Salut,

J'appelle deux fois ce code avec le même nom de fichier (pendant la même exécution du programme). J'ai une fenêtre Motif dans laquelle je renseigne le "const char* fileName" qui est en paramètre de la fonction et ce code est appelé pour lire le fichier.

J'ai modifié le code comme tu me l'as proposé mais le comportement est le même, j'arrive à ouvrir/lire/fermer un fichier texte mais quand je rappelle cette fonction le fichier est bien ouvert (streamFile != NULL) mais fgets(ligne, MAXLEN, streamFile) == NULL

Le problème ne se produit que lorsque c'est le même fichier qu'on ouvre d'affilé, si ce sont deux fichier différents les deux sont correctement lus !

Par ailleurs : #define MAXLEN 1024

kaloskagatos

up !

Message cité 1 fois

---------------
« Le hasard, c’est différent de la chance. Parce que la chance, je n'en ai jamais. »

Emmanuel Delahaye

C is a sharp tool

kaloskagatos a écrit :

up !

Je n'ai aucun problème avec ce code : (appelé n fois, peu importe)

Code :

#include <stdio.h>
#define MAXLEN 1024
int main (void)
{
int error = 0;
static char const fileName[] = "xxx.txt";
FILE *streamFile = fopen (fileName, "r" );
if (streamFile != NULL)
{
char ligne[MAXLEN];
while (fgets (ligne, sizeof ligne, streamFile) != NULL)
{
//traitement de ligne
printf ("%s", ligne);
}
fclose (streamFile);
}
else
{
error = -1;
fprintf (stderr, "Can't open Chimera Mask file (%s)", fileName);
}
return 0;
}

---------------
Des infos sur la programmation et le langage C: http://www.bien-programmer.fr Pas de Wi-Fi à la maison : http://www.cpl-france.org/

kaloskagatos

Merci pour ton aide, j'aurais du faire un petit programme de test comme ça, c'est au niveau de la récupération du nom de fichier la deuxième fois que mon code avait un problème, je considérais le nom du répertoire et non le fichier lui-même...

Pardon de t'avoir fair perdre un peu de temps

Si tu repasses par là, peux-tu me dire pourquoi tu écris static char const fileName[] = "xxx.txt"; avec static et char const dans ce sens? Merci

Message cité 1 fois

Publicité

Emmanuel Delahaye

C is a sharp tool

kaloskagatos a écrit :

Si tu repasses par là, peux-tu me dire pourquoi tu écris static char const fileName[] = "xxx.txt"; avec static et char const dans ce sens ?

Euh, quel sens ? char et const sont commutatifs...

Sinon, le choix de ce codage est un vieux reflexe de codeur C pour l'embarqué. "Un byte est un byte".

static char const fileName[] = "xxx.txt"
-> Une zone de code contenant : {'x','x','x','.','t','x','t',0}

char const fileName[] = "xxx.txt"
static char fileName[] = "xxx.txt"
-> Une zone de code contenant : {'x','x','x','.','t','x','t',0}
-> Une zone de donnée : tableau de 8 char
-> Du code de recopie

static char const *fileName = "xxx.txt"
static char *fileName = "xxx.txt" (risque d'erreur)
char const *fileName = "xxx.txt"
char *fileName = "xxx.txt" (risque d'erreur)
-> Une zone de code contenant : {'x','x','x','.','t','x','t',0}
-> Une zone de donnée : un pointeur vers cette zone de code.

Message édité par Emmanuel Delahaye le 17-10-2006 à 10:54:39

---------------
Des infos sur la programmation et le langage C: http://www.bien-programmer.fr Pas de Wi-Fi à la maison : http://www.cpl-france.org/

kaloskagatos

Ok je vois un peu mieux, je vais creuser un peu plus.

A+

squall2023

Bon tout est bien qui fini bien, mais tout ça car tu t'étais planté dans le nom du fichier ... Sérieux, essaie de réfléchir avant de poster, merci.

Message cité 1 fois

Chaos Intestinal

squall2023 a écrit :

Bon tout est bien qui fini bien, mais tout ça car tu t'étais planté dans le nom du fichier ... Sérieux, essaie de réfléchir avant de poster, merci.

T'es qui toi ? [:mlc]

FORUM HardWare.fr

Programmation

[C] fgets ne marche qu'une fois...

Sujets relatifs
[] Afficher un msg que si le prog est executé pour la 1ére fois...	Avis aux flasheurs experts // comment ca marche PreviewCity
[C++] fgets et sscanf	[Résolu]Client/serveur qui marche qu'en local
[C++] "typage dynamique" marche pas	boost::bind, pourquoi this ne marche pas?
page appelée plusieurs fois au lieu d'une	Capturer 2 evenements utilisateurs à la fois?
Architecture pc : comment ca marche l'adressage des ports ISA ?	votez 2007 comment ça marche ?
Plus de sujets relatifs à : [C] fgets ne marche qu'une fois...

Page générée en 0.093 secondes