Discussion :

[killerjeff]

Bonjour,

Dans le logiciel que je developpe je dois utiliser un tableau de pointeur de Atom (etant un type créer par moi) et dont de je connais pas le nombre d'element dont il va etre composé.
j'aimerai savoir si la methode que je souhaite employé et bonne pour pouvoir faire une reallocation memoire chaque fois que mon tableau change de taille.

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
 
class Atom{
 
public :
...
   int NbBonds
  Atom **Bonds;
  Atom *suiv;
 
  void addOneBond()
  {
     ++NbBonds;
      Bonds = new (Atom*)[NbBonds];
   };

avec initialement NbBonds = 0

j'aiderai avoir votre avis la dessus.

merci d'avance a tous

[Laurent Gomila]

Ta méthode n'est pas correcte, car là tu vas avoir une fuite de mémoire (le tableau précédent n'est pas détruit), et tu vas en même temps perdre son contenu.

Mais heureusement pour toi, la bibliothèque standard du C++ dispose de tout ce qu'il faut pour ce genre de situation, notamment la classe std::vector<> qui fait office de tableau dynamique.

Tu peux trouver plein de topics traitant de ce sujet sur ce forum, et une documentation sur le site de SGI.

Que la force soit avec toi

[killerjeff]

Bon ben je vais me mettre a vector. Merci

En fait je suis le 4eme stagiaire a developper ce logiciel et avant Bonds était defini comme :

Code :

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

1
2
3
4
5
6
7
8
9
 
 
Atom ** Bonds;
 
  void addOneBond()
  {
     ++NbBonds;
      Bonds = (Atom **) realloc(Bonds,NbBonds*sizeof(Atom *));
   };

Le probleme est que le cas ou le nombre d'element dinimué nétait pas pris en compte et moi je dois le faire.
Donc je vais passer avec vector pour faire du vrai C++ (meme si il y a encore du C qui traine au milieu je suis en train d'en reduire la taille)

en plus pour je suis tjs pret

PS : je mets resolu mais je viendrai p-e le reouvrir si j'ai des problemes avec vector

[Luc Hermitte]

Remarque:
Les fonctions *alloc() ne savent pas traiter correctement les objets. Les structures simples oui, mais pas les objets. Soit, le realloc ici utilisé était faux -- en l'abscence du new de placement.

Bref, à ton niveau, ne t'embête pas et migre tout vers les collections de la bibliothèque standard.