Discussion :

[Skize]

Bonjour à tous,

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

Voici la déclaration:

Code :

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

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 :

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
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 ! : j'aime l'informatique

Please: HELP

[Obsidian]

Voici la déclaration:

Code :

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

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.

Code :

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

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.

[Sve@r]

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

Code C :

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

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...

[Skize]

Merci à vous, j'ai rajouté

Code :

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

v=malloc(sizeof(vecteur));

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

Je contine mon apprentissage ,à + tard

[Lucien63]

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.

[Obsidian]

Merci à vous, j'ai rajouté

Code :

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

v=malloc(sizeof(vecteur));

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

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.

[Skize]

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 :

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

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 :

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

v=malloc(sizeof(vecteur));

dois-je plutôt mettre:

Code :

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

v=malloc(sizeof(int));

ou encore

Code :

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

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?

[jeroman]

Ca c'est bon :

Code :

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

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 :

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

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 :

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

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é.

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 :

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

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