[C] pointeurs de pointeurs [RESOLU]

pointeurs de pointeurs [RESOLU] [C] - C - Programmation

Marsh Posté le 25-04-2011 à 09:36:49    

Bonjour,
 
J'étais en train d'écrire un petit programme de test pour bien comprendre les principes des pointeurs en C en i686 en gcc, tout se compile bien, les données sont lisibles dans les fonctions, mais plus à leur sortie, je ne comprend pas mon erreur.
 
Pouvez-vous me corriger ce code ?
Merci.
 
settings.h

Code :
  1. #ifndef __SETTINGS_H__
  2. #define __SETTINGS_H__
  4. typedef struct __test {
  5.     int  a;
  6.     char b;
  7.     char c[20];
  8.     char *d;
  9. } _TEST;
  11. void setinit(_TEST *);
  13. #endif


 
settings.c

Code :
  1. #include <stdio.h>
  2. #include <stdlib.h>
  3. #include <string.h>
  5. #include "settings.h"
  7. void setinit(_TEST *test)
  8. {
  9. test=malloc(sizeof(_TEST));
  10. if (test != NULL)
  11. {
  12.  test->a=34;
  13.  test->b='B';
  14.  sprintf(test->c, "%s", "test c ok" );
  15.  test->d=malloc((strlen("test d ok" ) + 1) * sizeof(char));
  16.  if (test->d != NULL)
  17.   test->d="test d ok";
  18.  printf("setinit [%d]\n", test->a);
  19.  printf("setinit [%c]\n", test->b);
  20.  printf("setinit [%s]\n", test->c);
  21.  printf("setinit [%s]\n", test->d);
  22. }
  23. }


 
main.h

Code :
  1. #ifndef __MAIN_H__
  2. #define __MAIN_H__
  4. typedef struct __essai {
  5.     int  a;
  6.     char b;
  7.     char c[20];
  8.     char *d;
  9. } _ESSAI;
  11. #endif


 
main.c

Code :
  1. #include <stdio.h>
  2. #include <stdlib.h>
  3. #include <string.h>
  6. #include "main.h"
  7. #include "settings.h"
  9. _TEST  *test;
  10. _ESSAI *essai;
  12. void maininit(_ESSAI *essai)
  13. {
  14. essai=malloc(sizeof(_ESSAI));
  15. if (essai != NULL)
  16. {
  17.  essai->a=12;
  18.  essai->b='b';
  19.  sprintf(essai->c, "%s", "essai c ok" );
  20.  essai->d=malloc((strlen("essai d ok" ) + 1) * sizeof(char));
  21.  if (essai->d != NULL)
  22.   essai->d="essai d ok";
  23.  printf("maininit [%d]\n", essai->a);
  24.  printf("maininit [%c]\n", essai->b);
  25.  printf("maininit [%s]\n", essai->c);
  26.  printf("maininit [%s]\n", essai->d);
  27. }
  28. }
  30. int main()
  31. {
  32. setinit(test);
  33. maininit(essai);
  36. if (test != NULL)
  37. {
  38.  printf("test  [%d]\n", test->a);
  39.  printf("test  [%c]\n", test->b);
  40.  printf("test  [%s]\n", test->c);
  41.  printf("test  [%s]\n", test->d);
  42. }
  44. if (essai != NULL)
  45. {
  46.  printf("essai [%d]\n", essai->a);
  47.  printf("essai [%c]\n", essai->b);
  48.  printf("essai [%s]\n", essai->c);
  49.  printf("essai [%s]\n", essai->d);
  50. }
  52. free(essai);
  53. essai=NULL;
  54. free(test);
  55. test=NULL;
  56.     return(0);
  57. }


 
Le résultat d'exécution donne ceci :

Code :
  1. setinit [34]
  2. setinit [B]
  3. setinit [test c ok]
  4. setinit [test d ok]
  5. maininit [12]
  6. maininit [b]
  7. maininit [essai c ok]
  8. maininit [essai d ok]


 
Il manque donc bien les lignes d'impression du main.
 
Merci pour votre aide


Message édité par Noobs69 le 25-04-2011 à 18:18:30
Reply

Marsh Posté le 25-04-2011 à 09:36:49   

Reply

Marsh Posté le 25-04-2011 à 10:04:49    

Si je comprends bien (tu aurais pu faire plus court comme test :o)
 
Tu espères pouvoir modifier tes pointeurs test et essai juste en les passant dans une fonction qui les assigne au résultat de malloc ?
 
Ca ne peut pas marcher pour une raison simple: Quand tu fais passer un paramètre à une fonction, cette fonction va en réalité travailler avec une copie de ce paramètre. A la fin de ta fonction, ce paramètre va donc être détruit et évidemment tu auras donc perdu le pointeur que t'a retourné malloc.
 
Si tu voulais pouvoir modifier ce pointeur "a distance", il faudrait que tu fasses passer en paramètre l'adresse du pointeur (donc setinit(&essai) et maininit(&test)), que tu modifies bien évidemment le prototype de ces fonctions en conséquence et une fois cela fait, tu aurais alors un pointeur sur pointeur.
 
Ce que ça change ? Tu pourras alors modifier l'adresse du pointeur essai et test en faisant une assignation toute simple comme *essai = NULL; ou *test = malloc() ou ce que tu veux :D
 
J'espère avoir répondu à ta question :jap:


---------------
"I can cry like Roger. It's just a shame I can't play like him" - Andy Murray, 2010
Reply

Marsh Posté le 25-04-2011 à 10:32:17    

Merci mais je n'arrive pas à compiler correctement à présent.
 
J'ai fait ça comme modifs :
 
 
settings.h

Code :
  1. ...
  2. void setinit(_TEST **);
  3. ...


 
settings.c

Code :
  1. ...
  2. void setinit(_TEST **test)
  3. {
  4. *test=malloc(sizeof(_TEST));
  5. if (test != NULL)
  6. {
  7.  test->a=34;
  8. ...


 
main.c

Code :
  1. ...
  2. void maininit(_ESSAI **essai)
  3. {
  4. *essai=malloc(sizeof(_ESSAI));
  5. if (essai != NULL)
  6. {
  7.  essai->a=12;
  8. ...
  10. int main()
  11. {
  12. setinit(&test);
  13. maininit(&essai);
  14. ...


 
Je vois bien que l'erreur est dans l'affectation à présent mais je ne vois pas comment écrire ça correctement.


Message édité par Noobs69 le 25-04-2011 à 10:35:37
Reply

Marsh Posté le 25-04-2011 à 10:41:58    

Pour te simplifier les choses (tu pourras comprendre après :D): Tu as bien modifié le prototype, par contre dans ta fonction à chaque fois que tu as "test" ou "essai" il faut qu tu remplaces par *test et par *essai (donc tu mets une étoile chaque fois, ce n'est pas l'adresse du pointeur de pointeur que tu veux modifier mais l'adresse du premier pointeur) :D

 

Donc par exemple, tu remplaces if (essai != NULL) par if (*essai != NULL) etc ...


Message édité par WiiDS le 25-04-2011 à 10:42:12

---------------
"I can cry like Roger. It's just a shame I can't play like him" - Andy Murray, 2010
Reply

Marsh Posté le 25-04-2011 à 10:45:37    

Je creuse pour comprendre pourtant !!
 
Ca fonctionne pour malloc et le test, mais je ne comprend pas la syntaxe à adopter pour l'affectation :
 

Code :
  1. essai->a=12;   //???
  2.  *essai->a=12;  //???
  3.  essai[0]->a=12; //CA, ça compile, mais j'y crois pas du tout !!
  4.  *essai->b='b';


 
Je me prend des :
 error: request for member `a' in something not a structure or union
 


Message édité par Noobs69 le 25-04-2011 à 10:47:11
Reply

Marsh Posté le 25-04-2011 à 11:04:07    

Tu as essayé (*essai)->a=12; ? Faut faire gaffe à l'endroit ou on met les *, faut pas déréférencer n'importe quoi :o

 

Parce que je pense qu'en faisant *essai->a = 12, ton compilateur doit comprendre essai->*a = 12


Message édité par WiiDS le 25-04-2011 à 11:04:45

---------------
"I can cry like Roger. It's just a shame I can't play like him" - Andy Murray, 2010
Reply

Marsh Posté le 25-04-2011 à 11:09:14    

J'essaye ça de suite !!
 
Moi, j'en étais là pour contourner :
 

Code :
  1. {
  2. _ESSAI *tmp;
  4. tmp=malloc(sizeof(_ESSAI));
  5. if (tmp != NULL)
  6. {
  7.  tmp->a=12;
  8.  tmp->b='b';
  9.  sprintf(tmp->c, "%s", "essai c ok" );
  10.  tmp->d=malloc((strlen("essai d ok" ) + 1) * sizeof(char));
  11.  if (tmp->d != NULL)
  12.   tmp->d="essai d ok";
  13.  printf("maininit [%d]\n", tmp->a);
  14.  printf("maininit [%c]\n", tmp->b);
  15.  printf("maininit [%s]\n", tmp->c);
  16.  printf("maininit [%s]\n", tmp->d);
  17. }
  18. *essai=tmp;


 
Merci de ton aide précieuse et claire, j'avance vite avec une aide qui explique et ne met pas des commentaires lapidaires et inutiles comme souvent sur les forums, j'apprécie.

Reply

Marsh Posté le 25-04-2011 à 11:17:05    

Ca fonctionne !!!
 
Et ton analyse du préprocesseur est correcte, il comprend l'inverse, belle blague celle-là, merci grandement de ta pédagogie.
 
Je mets tout au propre et je post ici mon code complet, si ça peut en aider d'autres à "naviguer" dans les pointeurs de pointeurs en C ...
 
settings.h

Code :
  1. #ifndef __SETTINGS_H__
  2. #define __SETTINGS_H__
  4. #include <stdbool.h>
  6. typedef struct __test {
  7.     int  a;
  8.     char b;
  9.     char c[20];
  10.     char *d;
  11. } _TEST;
  13. void setinit(_TEST **);
  15. #endif


 
settings.c

Code :
  1. #include <stdio.h>
  2. #include <stdlib.h>
  3. #include <string.h>
  5. #include "settings.h"
  7. void setinit(_TEST **test)
  8. {
  9. *test=malloc(sizeof(_TEST));
  10. if (*test != NULL)
  11. {
  12.  (*test)->a=34;
  13.  (*test)->b='B';
  14.  sprintf((*test)->c, "%s", "test c ok" );
  15.  (*test)->d=malloc((strlen("test d ok" ) + 1) * sizeof(char));
  16.  if ((*test)->d != NULL)
  17.   (*test)->d="test d ok";
  18.  printf("maininit [%d]\n", (*test)->a);
  19.  printf("maininit [%c]\n", (*test)->b);
  20.  printf("maininit [%s]\n", (*test)->c);
  21.  printf("maininit [%s]\n", (*test)->d);
  22. }
  23. }


 
main.h

Code :
  1. #ifndef __MAIN_H__
  2. #define __MAIN_H__
  4. typedef struct __essai {
  5.     int  a;
  6.     char b;
  7.     char c[20];
  8.     char *d;
  9. } _ESSAI;
  11. #endif


 
main.c

Code :
  1. #include <stdio.h>
  2. #include <stdlib.h>
  3. #include <string.h>
  6. #include "main.h"
  7. #include "settings.h"
  9. _TEST  *test;
  10. _ESSAI *essai;
  12. void maininit(_ESSAI **essai)
  13. {
  14. *essai=malloc(sizeof(_ESSAI));
  15. if (*essai != NULL)
  16. {
  17.  (*essai)->a=12;
  18.  (*essai)->b='b';
  19.  sprintf((*essai)->c, "%s", "essai c ok" );
  20.  (*essai)->d=malloc((strlen("essai d ok" ) + 1) * sizeof(char));
  21.  if ((*essai)->d != NULL)
  22.   (*essai)->d="essai d ok";
  23.  printf("maininit [%d]\n", (*essai)->a);
  24.  printf("maininit [%c]\n", (*essai)->b);
  25.  printf("maininit [%s]\n", (*essai)->c);
  26.  printf("maininit [%s]\n", (*essai)->d);
  27. }
  28. }
  30. int main()
  31. {
  32. setinit(&test);
  33. maininit(&essai);
  35. if (test != NULL)
  36. {
  37.  printf("test  [%d]\n", test->a);
  38.  printf("test  [%c]\n", test->b);
  39.  printf("test  [%s]\n", test->c);
  40.  printf("test  [%s]\n", test->d);
  41. }
  43. if (essai != NULL)
  44. {
  45.  printf("essai [%d]\n", essai->a);
  46.  printf("essai [%c]\n", essai->b);
  47.  printf("essai [%s]\n", essai->c);
  48.  printf("essai [%s]\n", essai->d);
  49. }
  51. free(essai);
  52. essai=NULL;
  53. free(test);
  54. test=NULL;
  55.     return(0);
  56. }


 
Résultat du code :

Code :
  1. maininit [34]
  2. maininit [B]
  3. maininit [test c ok]
  4. maininit [test d ok]
  5. maininit [12]
  6. maininit [b]
  7. maininit [essai c ok]
  8. maininit [essai d ok]
  9. test  [34]
  10. test  [B]
  11. test  [test c ok]
  12. test  [test d ok]
  13. essai [12]
  14. essai [b]
  15. essai [essai c ok]
  16. essai [essai d ok]


Message édité par Noobs69 le 25-04-2011 à 11:24:45
Reply

Marsh Posté le 25-04-2011 à 11:27:27    

Ca n'existe pas RESOLU sur ce forum ou je vois mal ?

Reply

Marsh Posté le 25-04-2011 à 11:42:27    

:jap:
 
Tu peux éditer le premier post et changer le titre si tu y tiens vraiment, mais effectivement il n'y a pas de politique concernant le titre des sujets :D
 
Bonne continuation :hello:


---------------
"I can cry like Roger. It's just a shame I can't play like him" - Andy Murray, 2010
Reply

Marsh Posté le 25-04-2011 à 11:42:27   

Reply

Marsh Posté le 25-04-2011 à 12:31:02    

Je vois pas trop ce que tu veux faire, mais si j'avais à utiliser ce type de structure, je procéderais ainsi:
(Bon, à reformater, la page web ayant pas la même notion de tabulations que mon emacs en mode electric-c)

Code :
  1. /* settings.h*/
  2. #ifndef __SETTINGS_H__
  3. #define __SETTINGS_H__
  4. typedef struct {
  5.     int  a;
  6.     char b;
  7.     char c[20];
  8.     char *d;
  9. } TEST;
  11. TEST *test_create_empty(void);
  12. TEST *test_create(int a, char b, char *c, char *d);
  13. bool test_set(TEST *test, int a, char b, char *c, char *d);
  14. void test_print(TEST *test);
  15. #endif


 

Code :
  1. /* settings.c*/
  2. #include <stdio.h>
  3. #include <stdlib.h>
  4. #include <string.h>
  5. #include <stdbool.h>
  6. #include "settings.h"
  8. TEST *test_create_empty(void)
  9. {
  10.     TEST *test = malloc(sizeof(TEST));
  11.     if (test) {
  12.         test->a = 0;
  13.         test->b = 0;
  14.         (test->c)[0] = 0;
  15.         test->d = NULL;
  16.     }
  17.     return test;
  18. }
  20. bool test_set(TEST *test, int a, char b, char *c, char *d)
  21. {
  22.     if (test) {
  23.         test->d = malloc((strlen(d) + 1));
  24.         if (test->d) {
  25.             test->a = a;
  26.             test->b = b;
  27.             strncpy(test->c, c, 20);
  28.             strcpy(test->d, d);
  29.             return true;
  30.         }
  31.     }
  32.     return false;
  33. }
  36. TEST *test_create(int a, char b, char *c, char *d)
  37. {
  38.     TEST *test = malloc(sizeof(TEST));
  39.     if (test) {
  40.         test->d = malloc((strlen(d) + 1));
  41.         if (test->d) {
  42.             test->a = a;
  43.             test->b = b;
  44.             strncpy(test->c, c, 20);
  45.             strcpy(test->d, d);
  46.         }
  47.         else {
  48.             free(test);
  49.             test = NULL;
  50.         }
  51.     }
  52.     return test;
  53. }
  55. void test_print(TEST *test)
  56. {
  57.     printf("a: [%d]\nb: [%c]\nc: [%s]\nd: [%s]\n", test->a, test->b, test->c, test->d);
  58. }


Code :
  1. /* main.c*/
  2. #include <stdlib.h>
  3. #include "settings.h"
  5. int main()
  6. {
  7.     TEST  *test = test_create_empty();
  8.     if (test) {
  9.         if (test_set(test, 34, 'B', "test c ok", "test d ok" )) {
  10.             test_print(test);
  11.         }
  12.         free(test);
  13.         return EXIT_SUCCESS;
  14.     }
  15.     return EXIT_FAILURE;
  16. }


In Hope It Helps,
A+,


Message édité par gilou le 25-04-2011 à 13:42:38

---------------
There's more than what can be linked! --    Iyashikei Anime Forever!    --  AngularJS c'est un framework d'engulé!  --
Reply

Sujets relatifs:

Leave a Replay

Make sure you enter the(*)required information where indicate.HTML code is not allowed