Fuites mémoires avec Vector

**..alex..** · 09/08/2006, 10h43

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 :

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

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 ?

**Matthieu Brucher** · 09/08/2006, 11h08

Quand la structure est détruite -> delete
Autre solution, les pointeurs intelligents.

**superspag** · 09/08/2006, 11h21

Je te conseil de passer par des pointeurs intelligents:

si tu n'as pas boost :

Code :

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

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

**Médinoc** · 09/08/2006, 11h24

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.

**Matthieu Brucher** · 09/08/2006, 11h32

De toute manière, les auto_ptr ne passent pas avec les conteneurs standards, ça plante à la compilation, si le compilateur est standard.

**..alex..** · 09/08/2006, 14h08

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...

**Matthieu Brucher** · 09/08/2006, 14h22

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 -

**..alex..** · 09/08/2006, 14h29

Ok je vais tester ça merci

**zais_ethael** · 09/08/2006, 14h44

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.

**..alex..** · 09/08/2006, 15h50

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

Code :

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

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...

Merci à tout ceux qui pourront m'aider !

**Matthieu Brucher** · 09/08/2006, 15h53

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 :

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

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...

Merci à tout ceux qui pourront m'aider !

Si tu n'utilises pas ton objet, ça ne sert pas à grand chose

Code :

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

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

Code :

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

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);
}

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

Ok merci pour cette réponse rapide Miles !

**loufoque** · 09/08/2006, 18h43

Ou alors

Code :

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

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

Tout simplement.

**..alex..** · 10/08/2006, 09h42

Envoyé par loufoque

Ou alors

Code :

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

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 :

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

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

**Matthieu Brucher** · 10/08/2006, 10h01

Il faut ajouter un fichier d'en-tête apparemment

Tu as copié aussi les autres fichiers que l'en-tête shared_ptr demandait ?

Code :

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

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>

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

Ok merci pour tout !

Cela fonctionne

Fuites mémoires avec Vector

SL & STL C++

Discussions similaires

Partager

Partager