récuperer un caractère dans un fichier avec read

Recherche :

Mot : Pseudo : Filtrer
Bas de page
Auteur	Sujet : récuperer un caractère dans un fichier avec read

Benh_31

Bonjour à tous,

alors voila j'ai fais une fonction permettant de renvoyer la première ligne d'un fichier.
Le problème c'est que dès la ligne 10, quand j'essaie d'afficher le caractère lu dans la variable courant, ben ça marche pas ^^
quelqu'un saurait m'expliquer ?

Code :

char* read_line(int fd)
{
int i=0;
char *line;
char courant;
line=malloc(sizeof(char*));
read(fd,&courant,1);
printf("%c\n",courant);
while(courant!='\n')
{
line[i]=courant;
i++;
realloc(line,(1+i)*sizeof(char*));
read(fd,&courant,1);
}
line[i]='\0';
return line;
}

avec son main :

Code :

int main()
{
int f=open("ff",O_WRONLY | O_CREAT, 0644);
printf("%s",read_line(f));
return 0;
}

le fichier ff contient bien des caractères, des retours à la ligne...

merci d'avance

Publicité

Farian

Bonjour !
Vous ouvrez le fichier en mode "write only" (O_WRONLY), en demandant au système de le créer si nécessaire, ce qui n'est pas des plus astucieux pour lire le contenu du fichier !

Vous devriez tenter :

int f = open("ff", O_RDONLY);

Puis tester (éventuellement dans la fonction "read_line" qui pourrait dans ce cas-là renvoyer NULL), que f n'est pas égal à -1 (que le fichier a donc bien été ouvert).

Par ailleurs, votre gestion de la mémoire comporte plusieurs erreurs :
* ligne 6 : line = (char *) malloc (sizeof(char)); [Note : la norme impose sizeof(char) = 1, donc on peut ne pas le préciser, mais ce n'est pas une faute de le mettre], car vous voulez stocker dans line 1 caractère, pas un pointeur sur un caractère.
* ligne 16 : line = (char *)realloc(line, (1+i)*sizeof(char));
* Au vu des faibles tailles de mémoire qui sont demandées, les fonctions malloc et realloc se déroulement correctement, mais, en théorie, vous devriez tester qu'elles ne renvoient pas NULL, ce qui indiquerait que la mémoire demandée ne peut être allouée.

De plus, vous ne vérifiez pas que la fonction read se déroule correctement (code de retour = nombre d'octets lus, soit 1 dans le cas présent).

Pour terminer, n'oubliez pas de libérer la mémoire allouée par la fonction "read_line"

Bonne continuation à vous !

Message cité 1 fois
Message édité par Farian le 24-11-2012 à 18:47:38

Benh_31

Farian a écrit :

Bonjour !
Vous ouvrez le fichier en mode "write only" (O_WRONLY), en demandant au système de le créer si nécessaire, ce qui n'est pas des plus astucieux pour lire le contenu du fichier !

Vous devriez tenter :

int f = open("ff", O_RDONLY);

Puis tester (éventuellement dans la fonction "read_line" qui pourrait dans ce cas-là renvoyer NULL), que f n'est pas égal à -1 (que le fichier a donc bien été ouvert).

Par ailleurs, votre gestion de la mémoire comporte plusieurs erreurs :
* ligne 6 : line = (char *) malloc (sizeof(char)); [Note : la norme impose sizeof(char) = 1, donc on peut ne pas le préciser, mais ce n'est pas une faute de le mettre], car vous voulez stocker dans line 1 caractère, pas un pointeur sur un caractère,
* ligne 16 : line = (char *)realloc(line, (1+i)*sizeof(char)); (Idem)

De plus, vous ne vérifiez pas que la fonction read se déroule correctement (code de retour = nombre d'octets lus, soit 1 dans le cas présent).

Bonne continuation à vous !

Merci pour votre réponse, par contre, à l’exécution :

Code :

printf("%c\n",courant);

n'affiche pas le bon caractère qui est sensé se trouver dans mon fichier mais un autre

et puis juste après l'avoir affiché, l’exécution plante même après vos modifications

Farian

Re-bonjour !

Comme je le dis souvent : "CMCM" ! (Chez Moi, Ça Marche ), et valgrind ne trouve rien à redire lors de l'exécution ...

Il doit y avoir une autre petite erreur dans votre code, postez-le, cela ne doit pas être grand chose.

Message édité par Farian le 24-11-2012 à 18:56:52

Benh_31

Je suis trop bête !!!!! j'avais oublié de mettre mon fichier dans le même dossier que mon main ^^

problème résolu merci encore

Message édité par Benh_31 le 24-11-2012 à 19:14:21

Farian

Je ne vois pas de faute, si ce n'est que j'aurais remplacé le code "if (errno)" par "if (f < 0)", qui est quand même plus propre, mais qui doit revenir au même ...

Chez moi, votre code s'exécute sans problème (valgrind ok)

Que ce soit sous windows ne devrait, en théorie, pas poser de problème ... la seule chose que je vois est que si votre fichier ne contient qu'une seule ligne et donc pas de retour chariot, le comportement sera sans doute étrange ...)

Ma version du code, qui ajoute simplement la gestion du code de retour de la fonction "read" :

Code :

#include <stdio.h>
#include <stdlib.h>
#include <unistd.h>
#include <sys/types.h>
#include <sys/stat.h>
#include <fcntl.h>
char* read_line(int fd)
{
if (fd < 0)
{
return NULL;
}
int i=0;
char *line;
char courant;
line=(char *)malloc(sizeof(char*));
int res = read(fd,&courant,1);
if (res == 1)
{
// printf("%c\n",courant);
while(courant!='\n' && res == 1)
{
line[i]=courant;
i++;
line = (char *)realloc(line,(1+i)*sizeof(char*));
res = read(fd,&courant,1);
}
}
line[i]='\0';
return line;
}
int main()
{
int f=open("ff",O_RDONLY, 0644);
char *retour = read_line(f);
if (retour != NULL)
{
printf("%s\n",retour);
free(retour);
}
return 0;
}

Farian

Hé hé ! J'étais trop occupé à mettre en forme ma version du code et j'avais raté votre dernier message

Tout est bien qui finit bien, donc !

Bonne continuation à vous !

Benh_31

j'ai toujours un problème j'arrive bien à récupérer ma ligne, mais après mon return line; je récupère NULL je comprends pas...

Farian

Précisez votre problème ...

Le code que j'ai posté, moyennant quelques petites adaptations, cf. ci-dessous, fonctionne correctement, pour plusieurs lignes :

Code :

#include <stdio.h>
#include <stdlib.h>
#include <unistd.h>
#include <sys/types.h>
#include <sys/stat.h>
#include <fcntl.h>
char* read_line(int fd)
{
if (fd < 0)
{
return NULL;
}
int i=0;
char *line = NULL;
char courant;
int res = read(fd,&courant,1);
if (res == 1)
{
line=(char *)malloc(sizeof(char*));
while(courant!='\n' && res == 1)
{
line[i]=courant;
i++;
line = (char *)realloc(line,(1+i)*sizeof(char*));
res = read(fd,&courant,1);
}
line[i]='\0';
}
return line;
}
int main()
{
int f=open("ff",O_RDONLY, 0644);
char *retour;
int numeroLigne = 1;
while ((retour = read_line(f)) != NULL)
{
printf("Ligne %d : %s\n", numeroLigne, retour);
free(retour);
numeroLigne++;
}
return 0;
}

Message cité 2 fois
Message édité par Farian le 25-11-2012 à 09:22:21

Benh_31

je viens de reregarder mon code et j'avais oublié de faire un free, maintenant ça marche ( pour de bon cette fois ^^ )

merci encore

Publicité

Sve@r

Farian a écrit :

Le code que j'ai posté, moyennant quelques petites adaptations, cf. ci-dessous, fonctionne correctement, pour plusieurs lignes

Bonjour
Oui il fonctionne. Mais faire des realloc de 1 en 1 c'est pas super super optimisé vitesse. De plus, mettre le mode 0644 dans open() ne se justifie que si le fichier est créé donc si on passe le flag "O_CREATE" (ce qui s'adapte mal avec le flag O_RDONLY)
Et tant qu'à mettre un mode, autant utiliser les constantes appropriées S_IRUSR | S_IWUSR | S_IRGRP | S_IROTH...

Message cité 1 fois
Message édité par Sve@r le 25-11-2012 à 12:43:07

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

Benh_31

Sve@r a écrit :

Pour le realloc de 1 en 1 c'est ce qui m'est demandé : ça permet de pas avoir de problèmes si la chaine dépasse le buffer et aussi de pas gaspiller de l'espace pour rien.

Message cité 1 fois

Sve@r

Benh_31 a écrit :

Pour le realloc de 1 en 1 c'est ce qui m'est demandé : ça permet de pas avoir de problèmes si la chaine dépasse le buffer et aussi de pas gaspiller de l'espace pour rien.

Il est bien évident qu'il faut réallouer pour éviter un dépassement. Mais de 1 en 1...
En fait, le gaspillage d'espace est bien évidemment un problème à prendre en compte mais c'est là qu'on peut faire des compromis entre ce problème et les autres. Par exemple allouer de 100 en 100 permettra une certaine souplesse (surtout qu'on lit une ligne et que les lignes de plus de 100 caractères ne sont pas courantes) et le gaspillage n'est pas énorme.

Bien entendu s'il est demandé spécifiquement une allocation de 1 en 1 tu le fais mais rien ne t'empêche d'aller au-delà de ce qui est demandé. Par exemple la taille d'allocation pourrait être une macro ce qui permettrait à ton code d'évoluer facilement si besoin était. C'est ce qu'on nomme "évolutivité"...

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

Benh_31

c'est sur je suis tout à fait d'accord avec toi, dans mon cas c'est ce qui m'est demandé

gilou

Modérateur
Modzilla

Farian a écrit :

Précisez votre problème ...

Le code que j'ai posté, moyennant quelques petites adaptations, cf. ci-dessous, fonctionne correctement, pour plusieurs lignes :

Il marche, mais il n'est néanmoins pas parfait: c'est malloc(sizeof(char)) que l'on devrait avoir, et non malloc(sizeof(char*)) et idem pour le réalloc.

A+,

Message édité par gilou le 26-11-2012 à 22:51:01

---------------
There's more than what can be linked! -- Iyashikei Anime Forever! -- AngularJS c'est un framework d'engulé! --

FORUM HardWare.fr

Programmation

récuperer un caractère dans un fichier avec read

Sujets relatifs
Déclenchement Macro à l'ouverture du fichier	lecture d'un istringstream , recuperer l'erreur
Récupérer une URL en PHP contenue dans une page	envoi direct de fichier postcript a une imprimante reseau sans drivers
Accès fichier sur réseau local netbios	Récupérer index des éléments différents de 0
executer un JDK avec un fichier .bat	Remplacer une partie de texte par une autre dans un fichier texte
Uploader un fichier dans tous les dossiers de mon ftp ?	Recupérer le code lang locale iso 639 (fr_FR) avec std::locale
Plus de sujets relatifs à : récuperer un caractère dans un fichier avec read

Page générée en 0.072 secondes