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.