[C] Allocation memoire structures

Recherche :

Mot : Pseudo : Filtrer
Bas de page
Auteur	Sujet : [C] Allocation memoire structures

elbarto347

Bonjour a tous . Je debute en C et j'ai quelques soucis avec mon code , ça compile mais ça s’exécute pas tout le temps comme il le faut

Voici le code :

Code :

struct date
{
int jj,mm,aa;
};
struct releveTemp
{
struct date *D;
int temp;
};
void affiche(struct releveTemp **s,int taille)
{
int i;
for(i=0;i<taille;i++)
{
printf("Le %d/%d/%d il faisait %d degres",s[i]->D[0].jj,s[i]->D[0].mm,s[i]->D[0].aa,s[i]->temp); //erreur segmentation détectée ici
}
}
struct releveTemp saisieTemps()
{
struct releveTemp a;
a.D=malloc(1*sizeof(struct date));
printf("Jour : " );
scanf("%d",&a.D[0].jj);
printf("Mois : " );
scanf("%d",&a.D[0].mm);
printf("Année : " );
scanf("%d",&a.D[0].aa);
printf("Temperature :" );
scanf("%d",&a.temp);
return a;
}
int main()
{
int i,taille; taille=2;
struct releveTemp *test;
test=malloc(taille*sizeof(struct releveTemp));
for(i=0;i<taille;i++)
{
test[i].D=malloc(1*sizeof(struct date));
test[i]=saisieTemps();
}
affiche(&test,taille);
return 0;
}

Tant que la variable taille est égale a 1 ça fonctionne mais des qu'elle est supérieure le programme s’arrête. Un erreur mémoire ? D'ou peut elle venir ?

Merci d'avance.

Edit : d'apres code blocks ça viendrai de la ligne 19

Message édité par elbarto347 le 17-05-2013 à 10:39:56

Publicité

Farian

Bonjour !

C'est normal, vous gérez mal votre tableau "test" entre l'allocation et l'affichage :

"test" pointe vers N structures, accessibles par test[i], qui est une structure.

Lors de l'affichage, vous passez l'adresse de test et vous vous en servez en faisant un "(&test)[i]" et en considérant que c'est un pointeur sur une structure, alors que c'est &(test[i]) que vous auriez dû utiliser.

Soit vous modifiez le prototypage de la fonction affiche en lui passant un "struct releveTemps*", soit vous utilisez "(*s)[i]." au lieu de "s[i]->" dans la fonction d'affichage.

Note : La première solution paraît tellement plus logique que je ne peux que vous encourager à l'utiliser.

Par ailleurs, vous créez une fuite mémoire en allouant test[i].D puis en faisant test[i] = ..., la mémoire allouée est perdue aussi vite car plus référencée.

(Je ne parle pas du fait de ne pas du tout désallouer les champs "D" du tableau test, car j'imagine que cela sera fait dans la version finale, mais pas dans le programme de test fait pour illustrer le problème).

D'ailleurs, pourquoi utiliser un pointeur et ne pas mettre inclure directement une structure "date" dans la structure "releveTemp" ? Cela n'a de sens que si vous voulez pointer sur une mesure externe.

elbarto347

Merci de votre aide . J'ai modifié le type de la fonction affiche() et ça fonctionne !

En effet ça serai plus simple de fusionner les 2 structure mais c'est demandé comme ça , pour faire bosser les tableaux de struct.

Farian

Sans parler de fusionner les deux structures, vous déclarez simplement :

Code :

struct date
{
int jj,mm,aa;
};
struct releveTemp
{
struct date D;
int temp;
};

Bonne continuation !

elbarto347

Ah oui j'y avais pas pensé merci

FORUM HardWare.fr

Programmation

[C] Allocation memoire structures

Sujets relatifs
C++ : Certification - laquelle ?	Aide programmation C++ (débutant)
Peut on garder une variable php en mémoire?	un exercice en C
probleme fuite de memoire	[C] [Résolu] Tetris -> SDL ? Réseaux ?
Implimenter Un Menu En C Avec Allegro	mysql C++ code blocks
[C] Pointer de pointer de pointer	[C#] Utilisation de Google Calendar
Plus de sujets relatifs à : [C] Allocation memoire structures

Page générée en 0.046 secondes