Lecture de fichier

Recherche :

Mot : Pseudo : Filtrer
Bas de page
Auteur	Sujet : Lecture de fichier

Iscream

Je n'arrive pas lire dans un fichier texte avec cette fonction:

Code :

void Read_File(FILE* fd, char* buf)
{
char lettre;
int i=0;
while((lettre = (fgetc(fd))!= EOF))
{
buf[i] = lettre;
i++;
}
}

Puis dans le main ():

Code :

char buffer[1024];
fd = fopen("log.txt", "r+" );
Read_File(fd, buffer);

Cette fonction Read_File() fait planter le programme.
Quelqu'un peut-il m'aider ?

Publicité

Trap D

Il est possible que le "r+" te positionne à la fin du fichier, fait un fseek(fd, SEEK_SET, 0l);
D'autre part tu ne vérifies pas le retour de fopen.
Enfin fgetc renvoie un int et donc lettre doit être du type int, tu fais ensuite un cast de lettre ne char.

olivthill

La fonction fgetc() renvoie un entier, pas un caractère, donc il faudrait déclarer int lettre;

Par ailleurs, personnellement, je n'utilise pas EOF mais feof(fp), mais ça revient peut-être au même.

Il faut aussi faire attention à ne pas oublier de compléter le buffer avec un caractère nul après la boucle while, dans le cas courant où l'on désire avoir une chaine de caractères normale.

Message cité 1 fois

Trap D

olivthill a écrit :

Par ailleurs, personnellement, je n'utilise pas EOF mais feof(fp), mais ça revient peut-être au même.

Il faut aussi faire attention à ne pas oublier de compléter le buffer avec un caractère nul après la boucle while, dans le cas courant où l'on désire avoir une chaine de caractères normale.

Non, feof n'est à utiliser qu'en cas de problème de lecture pour savoir si on est en fin de fichier ou si un problème de lecture est effectivement intervenu.
Par contre tu as tout à fait raison pour le 0 terminal.

Message édité par Trap D le 12-04-2006 à 13:53:27

olivthill

Merci pour l'info Trap D, et désolé d'avoir répondu en même temps pour le int au lieu du char.

Message cité 1 fois

Trap D

olivthill a écrit :

Merci pour l'info Trap D, et désolé d'avoir répondu en même temps pour le int au lieu du char.

Je suis plus rapide du clavier

Iscream

Mince, impossible d'ouvrir mon fichier avec fopen.
Comme indiqué je verifie fopen et le probleme semble venir de là . (merci)

Code :

fd = fopen("log.txt", "r" );

Quand au '\0', la chaine en est remplie des le depart avec un memset.
Par ailleurs j'ai changé fgetc par un

Code :

fscanf(fd, "%s", buf[i])

Pourquoi je ne peux pas ouvrir ce Fichier ?

Message cité 1 fois
Message édité par Iscream le 12-04-2006 à 16:54:45

Taz

bisounours-codeur

#
char lettre;
#
int i=0;
#
#
while((lettre = (fgetc(fd))!= EOF))

lettre doit être de type int

Emmanuel Delahaye

C is a sharp tool

Iscream a écrit :

Mince, impossible d'ouvrir mon fichier avec fopen.
Pourquoi je ne peux pas ouvrir ce Fichier ?

Il n'existe pas
Il est déjà ouvert
...

Essaye ça :

Code :

#include <stdio.h>
#define fname "log.txt"
int main (void)
{
FILE *fp = fopen (fname, "r" );
if (fp != NULL)
{
/* ... */
fclose (fp), fp = NULL;
}
else
{
perror(fname);
}
return 0;
}

et dis nous quel est le message ?

---------------
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/

franceso

Code :

FILE *f = fopen( "log.txt", "r" );
if( f == NULL )
perror( "fopen" );

si ton fopen foire, avec ça tu sauras pourquoi ...

[EDIT] Emmanuel, tu as été plus rapide ...

Message édité par franceso le 12-04-2006 à 16:59:34

---------------
TriScale innov

Publicité

Iscream

merde quelle honte !

no such file or directory.
Mon fichier s'appelait log.txt.txt je suppose, renommé ca marche.

Merci, super pratique ce perror, faudra que je regarde ca plus souvent.

Iscream

Code :

void Read_File(FILE* fd, char* buf)
{
int lettre;
int i=0;
while((lettre = (fgetc(fd))!= EOF))
{
buf[i] = lettre;
i++;
MessageBoxA(NULL, buf, "MessageBoxA()", MB_OK);
}
}

Dsl me voila de retour plus tot que prevu.

Lorsque j'appelle cette fonction elle remplie buf[i] a chaque fois avec la premiere lettre du fichier sans passer a la suivante.
Au final la boucle est infinie car elle reste au debut du fichier et buf
ressemble a : buf[] = "rrrrrrrrrrrrrrrrrrrrrrrrrrrrrrrrrrrrrrrrrrrrrrrrr"
r etant la premiere lettre du fichier.

Comment puis faire une fonction qui remplisse une variable avec le contenue d'un fichier correctement ???

Message cité 3 fois

Trap D

En général on fait avec fgets qui lit une ligne du fichier.

Emmanuel Delahaye

C is a sharp tool

Iscream a écrit :

Code :

void Read_File(FILE* fd, char* buf)
{
int lettre;
int i=0;
while((lettre = (fgetc(fd))!= EOF))
{
buf[i] = lettre;
i++;
MessageBoxA(NULL, buf, "MessageBoxA()", MB_OK);
}
}

Comment est défini le buf de l'appelant ?
Pourquoi une MessageBox() ? Tu es sous GUI ? Pourquoi tu ne valides pas ton code en mode console, c'est nettement plus simple pour les traces...
Je suppose que MessageBoxA() attend une chaine de caractères valide. Tu penses que buf est une chaine valide ? Qui a mis le 0 final ?
Si tu veux lire un fichier txte, pourquoi tu ne le lis pas ligne à ligne avec fgets() ? C'est fait pour... GRILLAID

http://mapage.noos.fr/emdel/notes.htm#saisie
http://mapage.noos.fr/emdel/notes.htm#fichiers

Message édité par Emmanuel Delahaye le 12-04-2006 à 17:42:58

---------------
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/

Trap D

Emmanuel >> le P.O. nous a dit qu'il avait fait un memset avant l'appel, je suppose que c'est avec des 0.

skelter

Read_File dans le meme genre que gets, c'est une faille de sécurité

Sve@r

Iscream a écrit :

Code :

void Read_File(FILE* fd, char* buf)
{
int lettre;
int i=0;
while((lettre = (fgetc(fd))!= EOF))
{
buf[i] = lettre;
i++;
MessageBoxA(NULL, buf, "MessageBoxA()", MB_OK);
}
}

On peut se passer de "i" et de "buf[i]" qui est "relativement" long

Code :

void Read_File(FILE* fd, char* buf)
{
int lettre;
char *pt=buf;
while((lettre = (fgetc(fd))!= EOF))
{
*pt= lettre;
pt++;
}
*pt=0; // Ca évite même de remplir "buf" dans la fonction appelante
}

Maintenant, ptet que ma question va sembler idiote... mais pourquoi ne pas utiliser "getline()" qui est faite pour ça ???

Message édité par Sve@r le 13-04-2006 à 20:33:43

---------------
Vous ne pouvez pas apporter la prospérité au pauvre en la retirant au riche.

skelter

faudrais au moins passer la taille du tableau ! (surtout que le code appelant est lui meme assez vulnerable)
Sinon pour getline (celui de la glibc) c'est mieux mais ce n'est pas standard

Taz

bisounours-codeur

sinon t'as le droit de faire un fread hein ...

Message cité 1 fois

Sve@r

Iscream a écrit :

Code :

void Read_File(FILE* fd, char* buf)
{
int lettre;
int i=0;
while ((lettre = (fgetc(fd))!= EOF))
{
buf[i] = lettre;
i++;
MessageBoxA(NULL, buf, "MessageBoxA()", MB_OK);
}
}

Je suis désolé de répondre à une question par une autre question... mais as-tu réalisé que, ta fonction telle que tu la prépares, va stocker un fichier de taille inconnue dans une zone de 1024 octets ? T'es sûr que ça va tenir ???

Autre remarque que personne n'a vu jusqu'à présent (i'm the winner, désolé pour Emmanuel et Taz ), mais le parenthésage autour de "while ((lettre=...))" n'est pas correct. Bon, il est bluffant et on s'est tous fait avoir... mais si on regarde bien, lettre ne reçoit que le résultat de la comparaison de "fgetc()" avec EOF...
=> while ((lettre=fgetc(fd)) != EOF)

Bon, dernière remarque conceptuelle: Penses-tu pouvoir vraiment réaliser une fonction qui stocke dans un buffer un fichier complet ? Même avec des malloc et des realloc, si le fichier est plus gros que ta mémoire ça ne tiendra jamais...

Taz a écrit :

sinon t'as le droit de faire un fread hein ...

C'est vrai que cette fonction m'était sortie de la tête :jap:

Message édité par Sve@r le 14-04-2006 à 10:20:43

---------------
Vous ne pouvez pas apporter la prospérité au pauvre en la retirant au riche.

Iscream

merci et dsl pour les retard.

C'est bon la fonction marche... avec un scanf() et strcat() (sans les espaces ca m'arrange).

Sinon pour la tyaille du fichier ca n'a pas d'importance, il ne devrait pas contenir grand chose et je vais rajouter un controle a Read_File()

Merci, pour l'aide

Publicité

FORUM HardWare.fr

Programmation

Lecture de fichier

Sujets relatifs
Lecture dans un fichier texte avec separateur , ??	[Matlab] Boucle de lecture de fichier
ignorer d'un fichier txt lors de la lecture	lecture de fichier avec sscanf
Lecture d'un fichier binaire (fichier C3D)	Pb lecture fichier
perl - lecture d'un fichier par blocks en partant de la fin	Lecture fichier ASCII
lecture fichier
Plus de sujets relatifs à : Lecture de fichier

Page générée en 0.095 secondes