Discussion :

[virtuadrack]

bonjour,

j'essaye de comprendre un peu le concept de malloc, je ne sais pas comment le faire pour une array de struct.

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
17
18
19
20
21
22
23
24
25
26
27
28
29
30
#include <stdio.h>
#include <stdlib.h>
 
typedef struct {
   int id;
   char name[30];
   int rooms;
} Hall_info;
 
main (){
 
FILE *fp;
char line[256];
int nh, f=0,j;
 
Hall_info *h;
h = (Hall_info*) malloc(sizeof(Hall_info));
 
if (h == NULL) {
      fprintf(stderr, "out of memory\n");
      exit(1);
}
 
 fp = fopen("residencehalls.txt","r");
   while( fgets(line, 256, fp) ){
      if (line[0] != '#'){
         sscanf(line, "%s %i %i", h[f].name, &h[f].id, &h[f].rooms, );
         nh++;
       } f++;
      } fclose(fp);

compile, mais ne me donne une seg fault... et meme pas le message d'erreur "out of memory". Je pense avoir loupe un episode.
Des idees sur comment resoudre le probleme?

merci

[diogene]

Code :

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

h = (Hall_info*) malloc(sizeof(Hall_info));

n'alloue de la place que pour UNE seule structure Hall_info. Pour stocker n structures :

Code :

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

h = malloc(n*sizeof(Hall_info));

Le cast est inutile.

[virtuadrack]

doit-je attribuer une valeure intiale a 'n'?

parce que je connais pas encore combien de structure le programme contiendra.

meme apres avoir essayer de donner un valeur

Code :

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

h = malloc(n * sizeof(Hall_info));

il ya toujours la meme seg fault.
est ce que la programmation est correcte ?

[orfix]

Supposons que le n est égale à 10, il ne faut pas boucler plus de 10 fois sinon tu débordes en écrivant sur un espace mémoire non alloué ( d'où le SegFault ), à toi de faire le nécessaire.
Il faut aussi vérifier que le fichier a bien été ouverts, en testant le retour de fopen .

[aimad41]

si tu ne sais pas combien de structures tu auras tu peut utiliser les listes chainées qui seront allouées en mémoire à chaque nouvelle déclaration...