Discussion :

[getud78]

Bonjour,

Après un cours élémentaire sur les structures et l'allocation de mémoire, j'en ai repris les éléments dans le fichier joints pour vérifier si j'avais bien compris. Ce programme compile mais l'exécution pose des problèmes.

Mon but est d'allouer la mémoire nécessaire à un tableau de structures de taille variable, d'en initialiser les champs de chaque élément du tableau et de vérifier le résultat en imprimant le tableau après initialisation. C'est un cas d'école qui n'a rien d'ambitieux.

J'ai réalisé cette tâche de deux façons différentes :
1 - allocation de mémoire dans le main() + initialisation du tableau dans le main() + impression dans le main().
Cette partie du programme s'exécute parfaitement.
2 - allocation de mémoire dans une fonction + initialisation du tableau dans la même fonction + impression dans le main().
Tout semble bien se passe dans la fonction où l'impression locale du tableau initialisé est correcte. Par contre, l'impression dans le main() ne fonctionne pas et le programme s'arrête.

Je pense que cela provient d'une gestion inadaptée des pointeurs mais je n'arrive pas à trouver la solution.

J'ai eu récemment un cours de base sur les pointeurs et je vois bien que je n'ai pas tout assimilé.

Si vous voyez où j'ai fait une erreur de débutant, je vous remercie par avance de l'aide que vous pourrez m'apporter,

Cordialement vôtre,

getud78

[Matt_Houston]

Peux-tu insérer le code du programme entre balises [code] ? Sa consultation en deviendra plus aisée pour tout un chacun.

[dalfab]

Bonjour,

La valeur du pointeur tab_pers_2 est transmise à la fonction créer_tab() qui met à jour sa copie qui s'appelle tab.
Ensuite main() utilise tab_pers_2 qui n'a pas changée (donc qui vaut toujours NULL) et l'utiliser va planter.

Une fonction ne peut pas modifier une données externe, transmettre l'adresse de tab_pers_2 est indispensable

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
/*...*/
void creer_tab( int dim , mapersonne** ptab ) {
/*...*/
    mapersonne *tab = malloc( dim * sizeof(mapersonne) ); // tab est locale
   *ptab = tab; // on met à jour via pointeur la variable de l'appelant
/*...*/
}
 
int main() {
/*...*/
   creer_tab( nb_pers , &tab_pers_2 );
/*...*/
}

Ou bien on fait l'inverse, c'est plus simple : la fonction créer_tab() calcule un tab, en retourne la valeur. Et cette valeur de retour est à copier dans tab_pers_2 :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
/*...*/
mapersonne* creer_tab( int dim ) {
    mapersonne *tab;
/*...*/
   return tab;
}
 
int main() {
/*...*/
   tab_pers_2 = creer_tab( nb_pers );
/*...*/
}

[Sve@r]

Je pense que cela provient d'une gestion inadaptée des pointeurs mais je n'arrive pas à trouver la solution.

J'ai eu récemment un cours de base sur les pointeurs et je vois bien que je n'ai pas tout assimilé.

Bonjour

Même réponse que dalfab mais exprimée de façon plus générale: tu as dû apprendre qu'une fonction qui doit modifier une variable ne peut le faire que si on lui passe l'adresse de cette variable. Tu as (j'espère) eu des exemple de scanf() où on écrit int i; scanf("%d", &i).

Il se trouve que du côté de la fonction, cette adresse qu'elle reçoit elle doit d'abord la stocker. Et on ne peut stocker une adresse que dans un pointeur.
De plus, le pointeur de stockage d'une adresse est alors du type de la variable à laquelle correspond cette adresse. Si par exemple je passe l'adresse d'un int, alors la fonction stockera cette adresse dans un "int étoile". Et si je passe l'adresse d'un double, alors la fonction stockera cette adresse dans un "double étoile". Bref quel que soit le type générique "ttt" de la variable initiale, son adresse est toujours un "ttt étoile" (on rajoute juste "étoile" derrière son type).

De plus, dans la fonction elle-même, toute référence à la variable se fera par le "pointé de". Et en C, l'opération "pointer sur" se fait via l'opérateur étoile.

Petit exemple: une fonction qui initialise un int et un double à 0

Code c :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
void init_int(int *x) {
	(*x)=0; 			// Je mets des parenthèses pour bien montrer comment lire cette expression => 0 va dans la zone pointée par x
}
 
void init_double(double *x) {
	(*x)=0; 			// Idem
}
 
int main() {
	int i=5;
	double dbl=10;
	init_int(&i);
	init_double(&dbl);
	printf("i=%d, dbl=%lf\n", i, dbl);
}

Accessoirement tu noteras ici une source de confusion: en effet, l'opérateur "étoile" quand il est écrit dans une déclaration de variable signifie "adresse de" et quand il est écrit dans une instruction signifie "pointe vers". Donc à toi de te souvenir de la nuance.

Si maintenant la variable initiale est elle-même un pointeur ; ben la règle ne change pas (là aussi on rajoute "étoile" derrière son type). La variable étant (par exemple) du type "maPersonne étoile", alors l'adresse de cette variable sera du type "maPersonne étoile étoile".
Et dans la fonction, là non plus la règle ne change pas. Si on veut faire référence à la variable d'origine, alors il faudra écrire "pointé de" en utilisant l'opérateur "étoile".

D'où l'exemple de dalfab

Code c :

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

1
2
3
4
void creer_tab( int dim , mapersonne** ptab /* La variable d'origine étant un pointeur, son adresse sera un "pointeur étoile" */) {
    mapersonne *tab = malloc( dim * sizeof(mapersonne) ); // tab est locale
   (*ptab) = tab; // on remplit le "pointé de ptab" donc le pointeur dont on a reçu l'adresse
}

[getud78]

bonsoir dalfab, bonsoir Sve@r,

Merci pour vos explications claires et structurées.
Je les ai mises en œuvre et maintenant, mon programme compile et s'exécute.

Encore merci pour votre aide.

Comme je suis autant débutant en "Forum" qu'en langage C, je peine un peu au niveau des conventions à mettre en oeuvre; mais je vais m'y mettre.

getud78

Peux-tu insérer le code du programme entre balises [code] ? Sa consultation en deviendra plus aisée pour tout un chacun.

Bonsoir Matt_Houston,

Merci de me le faire remarquer; comme c'est la première fois que je transmet du code dans un forum, je n'ai pas encore les bons réflexes.

Je vais d'ailleurs voir s'il est possible de transmettre le fichier c en P.J.

Cordialement,

getud 78

[Matt_Houston]

Tu as déjà employé la pièce jointe dans ton premier post. Je te proposais d'inclure le code directement au sein du message, comme l'ont fait les autres intervenants.

[getud78]

Bonjour,

La valeur du pointeur tab_pers_2 est transmise à la fonction créer_tab() qui met à jour sa copie qui s'appelle tab.
Ensuite main() utilise tab_pers_2 qui n'a pas changée (donc qui vaut toujours NULL) et l'utiliser va planter.

Une fonction ne peut pas modifier une données externe, transmettre l'adresse de tab_pers_2 est indispensable

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
/*...*/
void creer_tab( int dim , mapersonne** ptab ) {
/*...*/
    mapersonne *tab = malloc( dim * sizeof(mapersonne) ); // tab est locale
   *ptab = tab; // on met à jour via pointeur la variable de l'appelant
/*...*/
}
 
int main() {
/*...*/
   creer_tab( nb_pers , &tab_pers_2 );
/*...*/
}

Ou bien on fait l'inverse, c'est plus simple : la fonction créer_tab() calcule un tab, en retourne la valeur. Et cette valeur de retour est à copier dans tab_pers_2 :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
/*...*/
mapersonne* creer_tab( int dim ) {
    mapersonne *tab;
/*...*/
   return tab;
}
 
int main() {
/*...*/
   tab_pers_2 = creer_tab( nb_pers );
/*...*/
}

Bonsoir,

Merci pour ces explications qui m'ont fait comprendre les erreurs que j'avais faites.
Je intégré ces concepts dans mon code et tout fonctionne parfaitement.

Cela m'a été d'une grande aide,

Cordialement,

getud78