Lecture de la premiere ligne d'un fichier

**ze veritable farf** · 31/10/2006, 12h19

Bonjour, j'ai un petit soucis, j'utilise read, pour lire dans un fichier (ce qui fonctionne tres bien) mais je voudrais ne lire que la premiere ligne de ce dernier, mais je n'arrive pas, votre aide me serait d'un grand secours. Merci d'avance.

Voici ma fonction :

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
char    *get_my_line(int fd)
{
  int   nb_read;
  char  *buffer;
  int i = 0;
 
  buffer = (char *) malloc(sizeof(char) * 512);
  while ((nb_read = read(fd, buffer, 512)) > 0 && buffer[i] != '*')
        write(1, buffer, nb_read);
  if (nb_read == (-1))
    {
      return ("file is empty !");
    }
  return (buffer);
  free(buffer);
}

et voici mon main :

Code :

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

1
2
3
4
5
6
7
8
9
10
int     main(int ac, char **argv)
{
  char  *line;
  int   fd;
 
  fd = open(argv[1], O_RDONLY);
  get_next_line(fd);
  close(fd);
  return (0);
}

**hegros** · 31/10/2006, 12h22

fgets est parfait pour la lecture d'une ligne....

Ton code est relativement compliqué pour rien tu alloues une taille connue à l'avance dans ce cas ne devrais-tu pas passer par un tableau automatique char t[512] ?

D'autant plus que le free n'est jamais executé...Tu as eu aucun warning aucun message d'avertissement de ton compilateur ?
Quel compilateur utilises-tu ?

**ze veritable farf** · 31/10/2006, 12h31

Pour ce qui est du compilateur, j'utilise cc, et en fait, je n'utilise pas fgets car je suis actuellement en formation C/Unix et mon patron m'interdit d'utiliser autre chose que open, read et write pour cet exercice.

Mais j'ai du mal a m'en sortir ...

**ze veritable farf** · 31/10/2006, 12h32

par contre, aucun message du compilateur et affichage du fichier entier OK.

**Emmanuel Delahaye** · 31/10/2006, 12h41

Envoyé par ze veritable farf

par contre, aucun message du compilateur et affichage du fichier entier OK.

Supaïr ton compilo. Tu fais un free() après un return et il ne dit rien ? Manque pas des options de compilation pour le rendre plus bavard ?

**ze veritable farf** · 31/10/2006, 12h45

Je vais reprendre, car je crois que je n'ai pas ete tres clair, ma fonction prends pour unique parametre un file descriptor, et doit retourner la premiere ligne et uniquement la premiere ligne de ce fichier, pour le faire, je n'ai a disposition que malloc, free, open, read et write.

Pour l'instant, le code suivant affiche (et non pas retourne) l'integralite du fichier. Je voudrais donc savoir comment recuperer uniquement la premiere ligne et comment la stoker dans un char* afin de la retourner directement en vue d'une expoitation ulterieure.

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
char    *get_my_line(int fd)
{
  int   nb_read;
  char  *buffer;
  int i = 0;
 
  buffer = (char *) malloc(sizeof(char) * 512);
  while ((nb_read = read(fd, buffer, 512)) > 0)
    write(1, buffer, nb_read);
  if (nb_read == (-1))
    {
      return ("file is empty !");
    }
  return (buffer);
  free(buffer);
}

Merci.

**Stevie Wonder** · 31/10/2006, 12h45

Oh tien un ptit gars d'EPITECH

Tek1? Tek2? Moi Tek2... Bref...

Je tien a signaler, pour ceux qui repondront a ta question dans le futur, que ceci est un exercice donné par l'ecole EPITECH et que les eleves sont tres restrincts. En effet, ils n'ont pas le droit d'utiliser les libs std, ils n'ont droit que aux fonctions qu'ils ont recodé d'eux meme (pour l'instant).

Une methode simple, compte tenu de ce dont vous avez le droit:

Deja, pas la peine de malloc ton buffer (et jamais de free() apres ton retour de valeur, c'est useless au possible), declare le simplement en lui accordant un nombre de case du genre : char buffer[BUFF_SIZE]; admettons.
Tu peux utiliser une macro pour la taille de ton buffer (#define BUFF_SIZE 512).Dans ta boucle ensuite, tu peux copier ton buffer dans une chaine de char prealablement mallocé avec la taille du buffer + 1, caractere par caractere tout en checkant si il n'y a pas de \n ou de \0. Si c'est le cas tu retournes la chaine. Sinon tu realloc ta chaine de char avec (BUFF_SIZE + 1) x le nombre de fois que tu read.

C'est une methode. Mais de mémoire, l'exo etait plus compliqué que ca...

**ze veritable farf** · 31/10/2006, 12h47

Merci de vos reponses mais au lieu de me dire que mon compilateur c'est de la merde, pourriez vous m'indiquer ou dois-je mettre mon free ?

Je suis desole mais je debute en C et je ne maitrise pas encore tres bien. Je viens du Java et la gestion de la memoire est quand meme nettement plus aisee.

**ze veritable farf** · 31/10/2006, 13h05

Merci de cette derniere reponse mais j'ai encore une derniere question, je ne peux pas faire le malloc apres le return, mais si je le fais avant, je ne retournerais pas la bonne valeur etant donne que j'aurais libere l'espace memoire vers lequel le pointeur que je retourne pointe ? non ?

**Stevie Wonder** · 31/10/2006, 13h10

Je suppose que tu voulais parler du free(). Inutile de "freer" si tu retournes une valeur contenue dans une variable mallocé.
Tu as une definition des pointeurs un peu floue, revois vite ca.

**hegros** · 31/10/2006, 13h10

Merci de cette derniere reponse mais j'ai encore une derniere question, je ne peux pas faire le malloc apres le return, mais si je le fais avant, je ne retournerais pas la bonne valeur etant donne que j'aurais libere l'espace memoire vers lequel le pointeur que je retourne pointe ? non ?

T'as t-on contraint d'utiliser malloc ?

Sinon il y a une régle qui dis que tout code aprés un return n'est pas executé ce qui fais que ton free la ou il se situe et soit la pour décorer ou pour autre chose mais dans tout les cas ici inutile.

**Emmanuel Delahaye** · 31/10/2006, 13h18

Envoyé par hegros

T'as t-on contraint d'utiliser malloc ?

Tu ne vas quand même pas retourner l'adresse d'un tableau local...

**Emmanuel Delahaye** · 31/10/2006, 13h28

Envoyé par Stevie Wonder

Une methode simple, compte tenu de ce dont vous avez le droit:

Deja, pas la peine de malloc ton buffer <...> declare le simplement en lui accordant un nombre de case du genre : char buffer[BUFF_SIZE]; admettons.
Tu peux utiliser une macro pour la taille de ton buffer (#define BUFF_SIZE 512).Dans ta boucle ensuite, tu peux copier ton buffer dans une chaine de char prealablement mallocé avec la taille du buffer + 1, caractere par caractere tout en checkant si il n'y a pas de \n ou de \0. Si c'est le cas tu retournes la chaine. Sinon tu realloc ta chaine de char avec (BUFF_SIZE + 1) x le nombre de fois que tu read.

Il n'a pas le droit à realloc(). Mais on peut le simuler avec malloc(), free() et une boucle de copie... C'est très crade mais c'est sans doute fait exprès...

**Emmanuel Delahaye** · 31/10/2006, 13h30

Envoyé par ze veritable farf

Merci de vos reponses mais au lieu de me dire que mon compilateur c'est de la merde, pourriez vous m'indiquer ou dois-je mettre mon free ?

Après usage, c'est à dire, ici, dans le main()...

**hegros** · 31/10/2006, 13h30

Envoyé par Emmanuel Delahaye

Tu ne vas quand même pas retourner l'adresse d'un tableau local...

Je pensais plus à le déclarer static si vraiment il fallait utiliser malloc mais bon...

**Stevie Wonder** · 31/10/2006, 13h33

Envoyé par Emmanuel Delahaye

Il n'a pas le droit à realloc(). Mais on peut le simuler avec malloc(), free() et une boucle de copie... C'est très crade mais c'est sans doute fait exprès...

Le mot "realloc" que j'ai employé ne signifiait pas utiliser LE realloc() std, nuance, comme je l'ai dit plus haut, il n'a pas le droit d'utiliser les fonctions qu'il n'a pas recodé, il est tres facile de se faire un realloc de fortune comme tu viens de le preciser.

**Stevie Wonder** · 31/10/2006, 13h36

Envoyé par Emmanuel Delahaye

Après usage, c'est à dire, ici, dans le main()...

Le main ne sera pas compris dans le code en fait, ils doivent juste fournir la fonction (en vue de se faire une librairie perso qui palliera les libs std). Je suis egalement dans cette ecole, je suis passé par la.

**ze veritable farf** · 31/10/2006, 13h46

J'ai une petite question et j'en aurais fini :

Envoyé par Emmanuel Delahaye

Après usage, c'est à dire, ici, dans le main()...

Mes pinteur etant declares dans une autre fonction que mon main, je ne comprends pas comment je peux faire un free dans le main ?

A moins que je le fasse sur l'adresse que j'ai retournee et que j'ai recu dans le main ?

**Emmanuel Delahaye** · 31/10/2006, 13h58

Envoyé par ze veritable farf

Mes pinteur etant declares dans une autre fonction que mon main, je ne comprends pas comment je peux faire un free dans le main ?

A moins que je le fasse sur l'adresse que j'ai retournee et que j'ai recu dans le main ?

**ze veritable farf** · 31/10/2006, 14h17

ok merci pour toutes vos reponses

Lecture de la premiere ligne d'un fichier

C

Discussions similaires

Partager

Partager