Pointeurs et structures: erreur de segmentation

Bonjour à tous,

débutant en C depuis quelques mois, j'essaie d'attaquer les pointeurs de structure.

Voici la déclaration:

Code:

---

1 2 3 4 5 6 7

#ifndef __VECTEUR_DYNAMIQUE__ #define __VECTEUR_DYNAMIQUE__   typedef struct donnees_vecteur { int taille; double *donnees; } *vecteur;

---

et dans le point c:

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

#include "vecteur_dynamique.h" #include <stdlib.h> #include <stdio.h>   vecteur allouer_vecteur(int taille) { vecteur v; v->taille = taille; v->donnees = (double*)malloc(taille * sizeof(double));   if ( v->donnees == NULL ){ printf("Allocation Impossible"); exit(1); }   return v; }   /* [...] */   int main(){   vecteur v = allouer_vecteur(8);   v->donnees[2]=5; printf("\n%lf\n",*(acces_vecteur(v,2)));   printf("\nTaille: %d\n",taille_vecteur(v));   liberer_vecteur(v); printf("\n%d\n", est_vecteur_invalide(v));   return 0; }

---

j'ai passé les passages contenant les déclarations/définition des autres foncions car c'est la fonction allouer_vecteur qui pose problème.

En effet, je n'ai aucune erreur de compilation mais une erreur de segmentation qui provient de cette fonction.

Ca fait des heures que je cherche et je trouve toujours pas ! 8-) : j'aime l'informatique :aie:

Please: HELP :lol:

Citation:

---

Envoyé par Skize

Voici la déclaration:

Code:

---

1 2 3 4 5 6 7

#ifndef __VECTEUR_DYNAMIQUE__ #define __VECTEUR_DYNAMIQUE__   typedef struct donnees_vecteur { int taille; double *donnees; } *vecteur;

---

---

Cacher un pointeur dans un typedef, c'est MAL. C'est vraiment risquer des ennuis pour rien. Ça se fait beaucoup dans les écoles mais c'est une bidouille pour faire ressembler le langage à autre chose.

Citation:

---

Code:

---

1 2 3 4 5 6 7 8

#include "vecteur_dynamique.h" #include <stdlib.h> #include <stdio.h>   vecteur allouer_vecteur(int taille) { vecteur v; v->taille = taille;

---

---

Il est là, ton problème : tu déclares un pointeur vers une structure donnee_vecteur, mais tu n'alloues pas la structure proprement dite. Ton pointeur « v » pointe dans le vide. Mais comme tu l'as appelé « vecteur », tu as fait abstraction de ce détail.

Citation:

---

Envoyé par Obsidian

mais c'est une bidouille pour faire ressembler le langage à autre chose.

---

:mouarf: :mouarf: :mouarf:

Citation:

---

Envoyé par Skize

Code:

---

1 2 3 4 5

if ( v->donnees == NULL ){ printf("Allocation Impossible"); exit(1); }

---

---

Pas bien. Un des principes de base est que si on appelle une fonction, la fonction doit nous rendre la main quoi qu'il arrive (et pas quitter le programme avec exit()).
S'il y a un problème, la fonction renvoie alors une valeur spéciale et c'est à l'appelant de gérer le problème. La fonction ne doit pas quitter le programme.

Imagine ce qui se passerait si malloc() quittait par exit() dès qu'une allocation échoue ? Tu ne pourrais jamais savoir que ça a échoué... Ben c'est le même principe pour les fonctions que tu crées...

Merci à vous, j'ai rajouté

Code:

---

v=malloc(sizeof(vecteur));

---

dans le code de ma fonction et enlevé exit(1). Tout marche nikel.

Je contine mon apprentissage ,à + tard :ccool:

Citation:

---

Tout marche nickel.

---

C'est un coup de bol !
N'oublis pas que vecteur est un type pointeur
sizeof(vecteur) te retourneras la taille d'un pointeur.

Citation:

---

Envoyé par Skize

Merci à vous, j'ai rajouté

Code:

---

v=malloc(sizeof(vecteur));

---

dans le code de ma fonction et enlevé exit(1). Tout marche nikel.

---

Citation:

---

Envoyé par Lucien63

C'est un coup de bol !
N'oublis pas que vecteur est un type pointeur
sizeof(vecteur) te retourneras la taille d'un pointeur.

---

+1

Skize : retire l'étoile « * » dans un premier temps et essaie de faire un programme qui compile comme ça. Comme on te l'a dit, c'est le meilleur moyen de rendre confus quelque chose qui, à la base, ne l'est pas du tout.

en fait cet exercice est réalisé dans le cadre de mes études (L3 Informatique)). Nous avons en effet dans un premier temps la classe vecteur comme telle:

Code:

---

1 2 3 4

typedef struct donnees_vecteur { int taille; int *donnees; } vecteur;

---

puis défini quelques fonctions la concernant. Mon programme s'est exécuté correctement. L'exercice suivant était de redéfinir les fonctions avec la classe *vecteur, le sujet de ce topic.

Cependant, je n'ai pas compris mon erreur:

Code:

---

v=malloc(sizeof(vecteur));

---

dois-je plutôt mettre:

Code:

---

v=malloc(sizeof(int));

---

ou encore

Code:

---

v=malloc(sizeof(int*));

---

J'ai testé les deux options et ça fonctionne. Pourquoi? Est-ce parce que l'attribut donnees (de la classe *vecteur) pointe sur un int?

Ca c'est bon :

Code:

---

1 2 3 4 5 6

typedef struct donnees_vecteur { int taille; int *donnees; } vecteur;   v=malloc(sizeof(vecteur));

---

Le problème plus haut c'était que tu cachais un pointeur dans un typedef, ce qui n'est pas une erreur en soi, mais une source de confusions. En effet, tu allouais non pas la taille de la structure mais la taille d'un pointeur.

Code:

---

1 2 3 4

typedef struct donnees_vecteur { int taille; double *donnees; } *vecteur;

---

Ici, vecteur n'est pas une structure mais un pointeur sur structure. Donc lorsque tu fais

Code:

---

v=malloc(sizeof(vecteur));

---

tu réserves non pas la taille d'une structure mais celle d'un pointeur de structure. Or, la taille n'est pas censée être la même, ce qui a toutes les chances de provoquer un crash.
C'est pour cette raison qu'il est préférable d'éviter de cacher un pointeur dans un nom de structure car même si le langage le permet ça reste quand même une bonne manière de créer la confusion et donc un programme buggué.

Citation:

---

dois-je plutôt mettre:

---

La règle est simple : toujours mettre la taille de l'objet pointé ou un multiple de cette taille si on veut un tableau.

Et pour simplifier davantage les choses tu peux même écrire :

Code:

---

1 2	type * pobjet ; pobjet = malloc( sizeof *pobjet ); /* sizeof *pobjet == sizeof( type ) */

---