probleme de dereferencement

probleme de dereferencement - C - Programmation

Marsh Posté le 15-06-2005 à 11:43:34    

Bonjour a vous,
 
alors voila j'ai cree cette procedure :
 

Code :
  1. //mode = mode d ouverture
  2. //prod = identifiant fichier
  3. //nom  = nom du fichier
  4. //type = nom de la structure
  5. //enr  = structure a lu ou a copier
  10. void ** lire(char *nom,char *mode,size_t type,void *enr)
  11. {
  12. FILE *prod=NULL;
  13. size_t retour1=0;
  15. void **tab=NULL;
  16. int taille=10;
  18. prod= load(prod,nom,mode);
  19. tab =(void **) malloc (type*taille);// faire une boucle de double allocation
  22. for (int i=0;i<taille;i++)
  23. {
  25.  retour1=fread(enr,type,1,prod);
  27.  if (retour1==0)
  28.  {
  29.   printf("%s","MessageErreur(Ecriture impossible,Appuyer sur une touche pour continuer)" );getch();
  30.  }
  31.  else
  32.  {
  33.   tab[i]=enr;
  34.  }
  35. }
  37. fclose(prod);
  39. return(tab);
  40. }


 
 
je voulais en faire une fonction polymorphe :)  car j en ai besoin de nombreuse fois mais avec des structures differentes mais lorsque je recupere tab dans le prog appelant et que je l affiche il me met l'adresse du pointeur  :pt1cable: donc données inexploitables  je ne sais pas comment utiliser tab  :heink: pour recuperer les données merci d'avance  :ange:

Reply

Marsh Posté le 15-06-2005 à 11:43:34   

Reply

Marsh Posté le 15-06-2005 à 11:49:42    

Code :
  1. tab =(void **) malloc (type*taille);// faire une boucle de double allocation


 
Et elle est où, ta "boucle de double allocation" ?  :o  
Qui t'a mis ce commentaire ?
 
En plus le cast est inutile ici. Les noms de tes variables sont ambigus, je trouve.
 
EDIT : et puis je viens de voir que tu mets l'adresse de enr dans tab[i], mais enr ne change jamais... donc tous tes tab[i] vont pointer sur la même zone mémoire.


Message édité par Elmoricq le 15-06-2005 à 11:53:44
Reply

Marsh Posté le 15-06-2005 à 12:06:30    

Citation :

Et elle est où, ta "boucle de double allocation" ?  :o  
Qui t'a mis ce commentaire ?


 
ne pas en tenir compte c'est moi qui me le suis mis pour moi c'est rien juste une idée qu'il fallait que je note au cas ou  
 

Citation :

et puis je viens de voir que tu mets l'adresse de enr dans tab[i], mais enr ne change jamais... donc tous tes tab[i] vont pointer sur la même zone mémoire


 
ben non mon fichier est lu 10 fois j avance donc dans mon fichier a chaque tour de boucle  
 
 

Citation :

En plus le cast est inutile ici. Les noms de tes variables sont ambigus, je trouve.
 


 
pourquoi ca fait pas longtemps que je programme alors je comprends pas bien pourquoi c'est ambigu??

Reply

Marsh Posté le 15-06-2005 à 13:33:26    

Je vais essayer de mieux expliquer, en commentant ta fonction :
 

Code :
  1. /* ces commentaires ne sont pas en phase avec la realite, concernant
  2.    le parametre type.
  3.    Et puis il vaut mieux ne pas melanger les commentaires utiles aux
  4.    parametres, et ceux de variables locales. Histoire de ne pas jeter
  5.    le trouble. */
  6. //mode = mode d ouverture
  7. //prod = identifiant fichier
  8. //nom  = nom du fichier
  9. //type = nom de la structure
  10. //enr  = structure a lu ou a copier
  13. void ** lire(char *nom,char *mode,size_t type,void *enr)
  14. {
  15.    FILE *prod=NULL;
  16.    size_t retour1=0;
  17.    void **tab=NULL;
  19.    /* pourquoi "10", surtout dans une fonction se voulant generique ? */
  20.    int taille=10;
  22.    prod = load(prod,nom,mode);
  24.    /* Ici, tu alloues le tableau de pointeurs. Pourquoi pas, mais  
  25.       n'oublie pas que chaque pointeur qui sera stocke dans tab devra  
  26.       etre alloue separement.
  27.       De plus, "type * taille" est ambigu, surtout avec 'type' de  
  28.       type size_t.
  29.       En effet, ta variable ne contient pas de quoi identifier un  
  30.       type, mais plutot la taille d'un element unique de ton  
  31.       tableau.  
  32.       De plus, il n'est pas utile d'effectuer un cast au retour de
  33.       malloc().
  34.       Encore une chose : et si malloc() echoue et que tab contient
  35.       NULL ? Beau plantage en perspective si cela arrive, tu devrais
  36.       _toujours_ tester le retour d'une (re)allocation memoire.
  37.       Evite egalement les commentaires en fin de ligne de code, ca
  38.       embrouille et surcharge inutilement la ligne. Normalement, ce
  39.       commentaire aurait du se situer juste au-dessus de la ligne.*/
  40.    tab =(void **) malloc (type*taille);// faire une boucle de double allocation
  43.    /* Et si dans ton fichier il y a moins de 10 elements a lire ? */
  44.    for (int i=0;i<taille;i++)
  45.    {
  46.       retour1=fread(enr,type,1,prod);
  48.       if (retour1 == 0)
  49.       {
  50.          /* Pour une erreur, il vaut mieux ecrire le message sur stderr.
  51.             Tu peux pour cela utiliser "fprintf(stderr, "...)"
  52.             De plus, ce serait pas mal de savoir pourquoi il y a eu une
  53.             erreur.
  54.             fread() peut en effet echouer a cause d'une vraie erreur,  
  55.             ou tout simplement parce qu'il a atteint... la fin du  
  56.             fichier !
  57.             Voir feof() et ferror() pour l'identification de l'echec */
  58.          printf("%s","MessageErreur(Ecriture impossible,Appuyer sur une touche pour continuer)" );
  59.          getch();
  60.       }
  61.       else
  62.       {
  63.          /* Oui mais non. En admettant que enr ait sa propre zone  
  64.             memoire allouee (ce que j'espere vraiment), tu ne te  
  65.             contentes ici que de placer dans tab[i] l'adresse de  
  66.             enr. Qui ne change jamais dans ta boucle.  
  67.             Et a chaque fread(), le contenu precedent de enr est  
  68.             ecrase.
  69.             Si bien qu'a la fin de ta boucle, toutes les lignes de  
  70.             tab ne contiendront que l'adresse de enr, dans lequel  
  71.             se trouvera simplement le tout dernier resultat de  
  72.             fread().
  73.             Pour faire une copie, il faut dupliquer enr (avec  
  74.             l'allocation memoire ad hoc), et copier l'adresse
  75.             obtenue dans tab[i]. Les free() correspondant devront
  76.             egalement se faire sur chaque element de tab. */
  77.          tab[i]=enr;
  78.       }
  79.    }
  81.    fclose(prod);
  82.    return(tab);
  83. }


Message édité par Elmoricq le 15-06-2005 à 13:47:23
Reply

Marsh Posté le 15-06-2005 à 14:23:10    

Merci pour tes conseils sage qui m'ont bien eclairé je suis finalement arrivé a mes fins grace a tes conseils  
 

Reply

Sujets relatifs:

Leave a Replay

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