lire plusieurs lignes d'un fichier et les formater dans un tableau

Bonjour

j'ai une fonction qui m permet de lire une ligne dans un fichier,

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17

void readMem(gchar *ligne, int numLigne){   FILE *fp = NULL; gchar buf[81]; int cpt = 0;   fp = fopen("/proc/meminfo", "r"); while(fgets (buf, 80, fp)){ if (cpt == numLigne){ strcpy(ligne, strstr(buf, ":") +7); break; } cpt++; } fclose(fp); }

---

j'ai cree une structure

Code:

---

1 2 3 4 5

typedef struct memInfo{ gchar *memTotal; gchar *memFree; }memInfo;

---

et cette fonction

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14

void getMemInfo(gchar *label){   /* Recupere les informations de la memoire */ memInfo m; readMem(m.memTotal, 0); readMem(m.memFree, 1);   strcat(label, "Memoire Total: "); strcat(label, m.memTotal); strcat(label, "Memoire Libre: "); strcat(label, m.memFree); }

---

en gros dans ma fonction main():

Code:

---

1 2 3 4 5

gchar *pMemInfo;   getMemInfo(pMemInfo); contOnglet = g_strdup_printf(pMemInfo);

---

donc voila j'aimerai pourvoir lui passer mon tableau avec les info du fichier

j'ai pas compris la question ...
mais y me semble que t'as oublié ET la question ET d'allouer de l'espace pour pMemInfo.

que se passe t-il en ce moment? vous prenez tous des cours de rattrapage avec le même prof? :aie:

Code:

---

fp = fopen("/proc/meminfo", "r");

---

"fopen" peut échouer:

Code:

---

1 2 3

if(fp==NULL){ ... }

---

"fgets" lit n-1 caractères:

Code:

---

while(fgets (buf, sizeof(buf), fp))

---

A première vu

Code:

---

1 2 3

memInfo m; readMem(m.memTotal, 0); readMem(m.memFree, 1);

---

menTotal et memFree ne sont pas alloué.

et de meme pour pMemInfo

Code:

---

1 2 3 4

typedef struct memInfo{ gchar *memTotal; gchar *memFree; }memInfo;

---

"memTotal" et "memFree" ne sont que des pointeurs, pas des tableaux.
Donc soit:
-on les garde pointeur et on les fait pointer sur une zone mémoire valide

Code:

---

1 2 3 4

if (cpt == numLigne){ ligne=strstr(buf, ":") +7; break; }

---

-on en fait des tableaux:

Code:

---

1 2 3 4

typedef struct memInfo{ gchar memTotal[256]; gchar memFree[256]; }memInfo;

---

et:

Code:

---

1 2 3 4

if (cpt == numLigne){ strcpy(ligne, strstr(buf, ":") +7); break; }

---

Ceci est un bug vicieux :

Code:

---

1 2 3 4

if (cpt == numLigne){ ligne=strstr(buf, ":") +7; break; }

---

En effet on fournit à ligne l'adresse d'un buffer donc 2 possibilités:
Soit ce buffer est global, mais en cas d'utilisations multiples de ce buffer, ne sera conservée que la dernière information lue, donc on aura une plainte du type : "Je ne comprends pas, j'ai toujours la même information affichée".
Soit ce buffer est local à la fonction de lecture et au sortir de la fonction cette adresse ne sera plus valide et à l'affichage il pourra se passer n'importe quoi.
Cette méthode est donc à proscrire absoulment.
Le mieux dans ce cas est d'utiliser un strdup (ou d'écrire l'équivalent).
D'autre part, il faudrait peut-être tester le retour de strstr, on ne sait jamais.

Citation:

---

Ceci est un bug vicieux :

---

En fait j'ai déclaré "buf" ainsi:

Code:

---

char* buf=malloc(81);

---

Tu le réinitialises à chaque lecture ? De toute façon l'adresse de buf n'est pas mémorisé par cette action

Code:

---

ligne=strstr(buf, ":") +7;

---

Comment fais-tu pour libérer la mémoire ?

Citation:

---

Tu le réinitialises à chaque lecture ?

---

Ou est le problème?
A chaque fois la zone mémoire pointée est différente.
[EDIT]Ah! Pour la libération de la mémoire?[/EDIT]

Citation:

---

De toute façon l'adresse de buf n'est pas mémorisé par cette action

---

Pas celle de buf mais celle de l'élément à "buf"+7

Citation:

---

Comment fais-tu pour libérer la mémoire ?

---

Il faudrait sauvegarder dans un tableau les adresses des zones allouées, ce qui n'est pas pratique.

Ce n'est pas une solution optimale en effet.:aie:
Si tu as une idée.

strdup déjà mentionné ;)

Ca ne va pas poser les mêmes problèmes?

A la fin de l'utilisation des structures il les libères par des free, si j'ai bien compris le problème, il fait un free(m.memTotal) et un free(m.memFree), mais c'est vrai que je n'ai pas tout suivi.

Le plus simple est de faire une structure de tableaux statiques pour éviter les problèmes d'allocation dynamique.
Et à priori il n'est pas nécessaire de changer le reste du code.