Fuites mémoires avec Vector

Version imprimable

BOnjour à tous!

Déjà merci de prendre le temps de lire mon post !

Ensuite je pense que c'est un cas déjà traité, j'ai fait une recherche sur tous les problémes de fuite mémoire mais je ne vois pas comment faire dans mon cas.

En gros, j'ai deux structure de données :
Code:

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 struct Toto { std::vector<Etat*> listTrip; } struct Etat { int a; int b; int c; int d; int e; } Et dans mon programme je fais : for (condition) { Etat* course = new Etat; course->a = (données); course->b = (données); course->c = (données); course->d = (données); course->e = (données); toto->listTrip.push_back(course); }
Ensuite je me sers plus loin dans mon programme de la liste de courses que j'ai faite...

Mais on voit bien que je fais un new dans une boucle et que fatalement cela va découler sur des fuites mémoires si aucun delete n'est fait. Ce que purify me dit...

Je reçois des informations de l'exterieur, et donc cette boucle for j'y retourne plusieurs fois...

J'aimerai savoir comment faire pour effacer cette liste lorsque je n'en ai plus besoin ?

09/08/2006, 11h08
Matthieu Brucher

Quand la structure est détruite -> delete
Autre solution, les pointeurs intelligents.
09/08/2006, 11h21
superspag
Je te conseil de passer par des pointeurs intelligents:

si tu n'as pas boost :
Code:

1 2 3 4 5 6 7 #include <memory> ... struct Toto { std::vector< std::auto_ptr<Etat> > listTrip; } ...
si tu as boost : http://miles.developpez.com/tutoriel...ost/smartptrs/

En passant par des pointeurs intelligents, lorsque tu fais un listTrip.clear(), les data sont detruites automatiquement :D
09/08/2006, 11h24
Médinoc

SURTOUT PAS des auto_ptr !!
Il y a beaucoup d'affectations dans les conteneurs de la STL et la mémoire serait perdue!

Il faut utiliser un boost::shared_ptr() ou un pointeur tout bète.
09/08/2006, 11h32
Matthieu Brucher

De toute manière, les auto_ptr ne passent pas avec les conteneurs standards, ça plante à la compilation, si le compilateur est standard.
09/08/2006, 14h08
..alex..

wai les pointeurs intelligents...Le soucis c'est que je n'ai pas la possibilité d'ajouter des "packs" au projet...

Sinon, à un moment donné, je fais :

toto->oldListTrip = toto->listTrip;

Pour en fait pouvoir comparer ces 2 listes dans le but d'effectuer des changements sur mon IHM si jamais il y a des valeurs qui ont bougé.

Et si jamais j'efface tous les pointeurs de toto->listTrip, est ce que les pointeurs de toto->oldListTrip seront effacés ?
Si j'ai bien compris ? Oui...:roll:
09/08/2006, 14h22
Matthieu Brucher

Eh oui... D'où l'intérêt des pointeurs intelligents !
Une piste : copie simplement les headers dans ton projet, ça passera sans problème et personne n'y verra rien - tout est templaté, il n'y a pas de .cpp, comme c'est indiqué dans mon tuto d'ailleurs -
09/08/2006, 14h29
..alex..

Ok je vais tester ça merci
09/08/2006, 14h44
zais_ethael

Heu... sinon dans ce cas précis tu n'as que des structures, tu pourrais aussi bien utiliser un vector<Etat>, ca règlerait le problème :) (et puis c'est un peu comme ca qu'on est sensés utiliser la stl). Sinon, dans boost il y a beaucoup mieux que les smart pointer pour gérer les collections: les pointer containers.
Ce sont des classes calquant le fonctionnement des containers de la stl mais où tout est utilisé par pointeur et libéré automatiquement. C'est plus joli niveau syntaxe et probablement plus optimisé à grande échelle.

Ok et donc pour appliquer ces smart pointers à mon probléme, j'ai un peu de mal donc si je reprends mon code :
Code:

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 struct Etat { int a; int b; int c; int d; int e; } typedef boost::shared_ptr<Etat>EtatPtr; // ça c'est bon ? ENSUITE JE VOIS PAS TROP OU FAIRE LES MODIFS ? struct Toto { std::vector<Etat*> listTrip; // ici y a t-il des modifs à faire ? } Et dans mon programme je fais : for (condition) { Etat* course = new Etat; // Et là, modifs à faire ?? course->a = (données); course->b = (données); course->c = (données); course->d = (données); course->e = (données); toto->listTrip.push_back(course); }
J'ai essayé pas mal de choses mais je n'arrive pas à compiler ou alors lorsque cela compile, mes fuites sont toujours présentes...:roll:

Merci à tout ceux qui pourront m'aider !

Citation:
Envoyé par ..alex..
Ok et donc pour appliquer ces smart pointers à mon probléme, j'ai un peu de mal donc si je reprends mon code :
Code:

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 struct Etat { int a; int b; int c; int d; int e; } typedef boost::shared_ptr<Etat> EtatPtr; // ça c'est bon ? ENSUITE JE VOIS PAS TROP OU FAIRE LES MODIFS ? struct Toto { std::vector<Etat*> listTrip; // ici y a t-il des modifs à faire ? } Et dans mon programme je fais : for (condition) { Etat* course = new Etat; // Et là, modifs à faire ?? course->a = (données); course->b = (données); course->c = (données); course->d = (données); course->e = (données); toto->listTrip.push_back(course); }
J'ai essayé pas mal de choses mais je n'arrive pas à compiler ou alors lorsque cela compile, mes fuites sont toujours présentes...:roll:

Merci à tout ceux qui pourront m'aider !
Si tu n'utilises pas ton objet, ça ne sert pas à grand chose ;)
Code:

1 2 3 4 struct Toto { std::vector<EtatPtr> listTrip; Là, on l'utilise }
Code:

1 2 3 4 5 6 7 8 9 10 11 for (condition) { EtatPtr course(new Etat); // Et là, modifs à faire ?? OUI !! course->a = (données); course->b = (données); course->c = (données); course->d = (données); course->e = (données); toto->listTrip.push_back(course); }

09/08/2006, 15h55
..alex..

Ok merci pour cette réponse rapide Miles ! :D
09/08/2006, 18h43
loufoque
Ou alors
Code:

1 2 3 4 struct Toto { std::vector<Etat> listTrip; }
Tout simplement.

Citation:
Envoyé par loufoque
Ou alors
Code:

1 2 3 4 struct Toto { std::vector<Etat> listTrip; }
Tout simplement.
Non cela ne fonctionne pas pour mon application...

ok j'ai mis les header seulement de Boost comme tu me l'as suggéré miles, et maintenant à la compilation, il me manque quelque chose saurais tu quoi exactement ?

J'ai ce message d'erreur :
Code:

1 2 3 4 error LNK2001: unresolved external symbol "void __cdecl boost::throw_excep tion(class exception const &)" (?throw_exception@boost@@YAXABVexception@@@Z) build\appli.exe : fatal error LNK1120: 1 unresolved externals scons: *** [build\appli.exe] Error 1120
Merci

Il faut ajouter un fichier d'en-tête apparemment :koi:
Tu as copié aussi les autres fichiers que l'en-tête shared_ptr demandait ?
Code:

1 2 3 4 5 #include <boost/assert.hpp> #include <boost/checked_delete.hpp> #include <boost/throw_exception.hpp> #include <boost/detail/shared_count.hpp> #include <boost/detail/workaround.hpp>

10/08/2006, 11h35
..alex..

Ok merci pour tout !

Cela fonctionne ;)