Aide a la comphrehension d une fonction

Bonjour,

J ai trouvé une fonction sur internet qui permet de lire une ligne dans un fichier.
Quelqu un pourrait il m aider a comprendre le code car je ne le comprend pas trop !

Merci de votre aide .

Code:

---

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

# define MEM_SIZE 2048   char *get_next_line(const int fd) { char *str; static char buffer[MEM_SIZE]; static int i = 0; int j; static int k = 0;   j = 0;   str = malloc(MEM_SIZE * sizeof(char)); if (k == 0) k = read(fd, buffer, MEM_SIZE);   if (i == k) return (NULL);   while (i < k && buffer[i] != '\n') { str[j] = buffer[i]; i = i + 1; j = j + 1; }   if (buffer[i] == '\n') i = i + 1;   str[j] = '\0';   return (str); }

---

Ce code est très mauvais :langue2:

Code:

---

1 2 3 4 5 6 7 8 9

# define MEM_SIZE 2048   char *get_next_line(const int fd) { char *str; static char buffer[MEM_SIZE]; static int i = 0; int j; static int k = 0;

---

Début de fonction, définition et initialisation de variables

Code:

---

j = 0;

---

Ligne inutile puisque on peut la supprimer et initialiser j à la définition

Code:

---

str = malloc(MEM_SIZE * sizeof(char));

---

str est donc un tableau de MEM_SIZE caractères. Mais c'est complétement moche de faire cela :roll:: voir après

Code:

---

1 2	if (k == 0) k = read(fd, buffer, MEM_SIZE);

---

Le test ne sert à rien puisque k ne sera jamais modifié avant sa définition/ initialisation et ici :roll:
Et en plus il n'a aucune signification: qu'est ce qu'on s'en fiche de la valeur de k puisqu'elle va être écrasée? :koi: :koi:

Là on lit un fichier qui a été ouvert avec la fonction fopen
Du moins il faut l’espérer :roll:

De plus ce n'est pas la fonction read qui faut utiliser mais fread :aie:

Donc on a lu k caractères du fichier fd et ces k caractères sont mis dans buffer

Code:

---

1 2	if (i == k) return (NULL);

---

Là: code pourri :roll:

1. On teste si on a lu aucun caractère avec la variable i qui n'a rien à faire dans le test
2. Fuite mémoire parce que str n'a pas été libéré

Code:

---

1 2 3 4 5 6

while (i < k && buffer[i] != '\n') { str[j] = buffer[i]; i = i + 1; j = j + 1; }

---

Ici on va recopier une ligne (tous les caractères avant le fameux '\n') dans notre tableau str.

Au passage soit i soit j est inutile et on peut utiliser qu'une seule variable :roll:

Et c'est là que le malloc est mal fait. Parce qu'on va créer un tableau de MEM_SIZE caractères, mais une ligne peut avoir moins de caractères (au plus k).

Le mieux à faire c'est de chercher la taille de cette ligne et ensuite, de faire le malloc et un strncpy :mrgreen:

Code:

---

1 2	if (buffer[i] == '\n') i = i + 1;

---

Code inutile

Code:

---

str[j] = '\0';

---

Le caractère '\0' pour dire "fin de chaîne de caractères"

Code:

---

return (str);

---

On retourne notre tableau qu'il faudra libérer après utilisation :mrgreen:

Merci pour ta reponse, je comprends mieux, il est vrai que ce code n est pas vraiement optimisé.

Le plus gros probléme pour moi est que si par exemple ce que je veux lire fais 1500 caracteres et que j ai un buffer de 512, cette fonction ne va lire que 512 caracteres.

Comment je peux faire pour modifier le code et qu il fonctionne peu importe la taille du buffer ?

Merci

Il existe realloc() pour modifier la taille d'une zone mémoire allouée par malloc() (et donc l'agrandir sans perdre son contenu).

la fonction read existe :wow:

La différence avec la fonction fread:

Citation:

---

read() is a low level, unbuffered read. It makes a direct system call on UNIX.

fread() is part of the C library, and provides buffered reads. It is usually implemented by calling read() in order to fill its buffer.

---

Je ne comprends pas la question :koi:

Si tu veux lire X caractères, soit

• tu lis X caractères avec la fonction fread, ou une autre
• tu as un buffer plus petit que X, alors tu fais une boucle for ou while, en gérant bien la dernier lecture qui peut-être non complète

Et pour le malloc, soit tu réalloues ton buffer avec la fonction calloc au fur et à mesure, soit tu fais un test ou un parcours pour connaître la taille totale avec un fseek au début si nécessaire

merci de vos reponses, c est bien ce que je veux faire, une boucle while si mon buffer est plus petit que ce que je veux lire, mais je suis un peu perdu concernant la methode a suivre