Discussion :

[killwin]

Bonjour, je reviens avec un vieux probl�me.

J'ai une fonction qui cr�e des objets pour les placer dans un vecteur. Comme tout le monde sait, lorsqu'une fonction se termine elle lib�re tout ce qui a �t� d�clar�, instanci� ainsi que les pointeurs (directs).

J'aimerais �crire donc ceci de fa�on moins compliqu�e et avec un style c++ :

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
private :
vector<CDraw*> images;
 
...
 
void fonction() {
 
CDraw **draw;
draw = (CDraw**)malloc(sizeof(CDraw*));
*draw = new CDraw(true, complet_file.c_str(), coord_, coef_resol);
 
// We reverse picture on Y axis
if(orientation_)
(*draw)->reverse_H_SDL();
 
images.push_back(*draw);
 
}

[superZozo]

salut !

c'est le bazard ton truc ?!

Comme tout le monde sait, lorsqu'une fonction se termine elle lib�re tout ce qui a �t� d�clar�, instanci� ainsi que les pointeurs (directs).

c'est quoi un pointeur direct. Si tu instancie avec new il faut lib�rer avec delete dans ta fonction ou ailleurs (destructeur, autre fonction...).

je ne comprends pas trop l'int�ret du double pointeur ni du malloc...

enfin si tu fait du c++, tu fait du c++, pas du C style c++

[killwin]

Justement, si je fais au milieu de la fonction un

Code :

S�lectionner tout - Visualiser dans une fen�tre � part

1
2
3
void fonction() {
   int * p1 = new int;
}

Lorsque la fonction prendra fin, le pointeur p1 dispara�tra. J'aurais des probl�mes de r�f�rencement delete ou pas delete.

L'int�ret du double pointeur en C, c'est de permettre de conserver une allocation dynamique faite dans une fonction.

PS (la fin de'une fonction ne lib�re pas tout l'objet en m�moire me suis mal exprim� plus haut)

[superZozo]

si tuveux r�cup�rer ton pointeur immediatement, tu peux faire

Code :

S�lectionner tout - Visualiser dans une fen�tre � part

1
2
3
4
5
 
obj* fonction() {
obj* p1 = new obj; 
return p1;
}

sinon c'est le dernier �l�ment de ton vector, si tu a fait ton push_back.

Classiquement, tu lib�re le contenu de ce vector avec le destructeur de ta classe. Il y a diff�rentes m�thodes +/- sophistiqu�es/�l�gantes/...

new est une allocation dynamique.

[Bousk]

Justement, si je fais au milieu de la fonction un

void fonction() {
int * p1 = new int;
}

Lorsque la fonction prendra fin, le pointeur p1 dispara�tra. J'aurais des probl�mes de r�f�rencement delete ou pas delete.

L'int�ret du double pointeur en C, c'est de permettre de conserver une allocation dynamique faite dans une fonction.

PS (la fin de'une fonction ne lib�re pas tout l'objet en m�moire me suis mal exprim� plus haut)

Bonjour,

il faudrait (re)voir un cours sur les pointeurs.
Oui p1 sera supprim�e, mais scoop : on s'en moque.
La zone m�moire allou�e et sur laquelle pointe p1 subsiste jusqu'� son delete.

Tu n'as aucun int�r�t � allouer un pointeur de pointeur, � part se faire un noeud au cerveau et cr�er une possible fuite m�moire suppl�mentaire.
Et d'ailleurs, ton code en pr�sente une belle de fuite m�moire.

Code :

S�lectionner tout - Visualiser dans une fen�tre � part

1
2
3
4
5
6
7
8
9
10
11
void fonction() {
 
CDraw *draw = new CDraw(true, complet_file.c_str(), coord_, coef_resol);
 
// We reverse picture on Y axis
if(orientation_)
draw->reverse_H_SDL();
 
images.push_back(draw);
 
}

[camboui]

Code :

S�lectionner tout - Visualiser dans une fen�tre � part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
vector< shared_ptr<CDraw> > images;
 
...
 
void fonction()
{
shared_ptr<CDraw> draw = shared_ptr<CDraw>::make(true, complet_file.c_str(), coord_, coef_resol);
 
// We reverse picture on Y axis
if(orientation_)
 draw->reverse_H_SDL();
 
images.push_back(draw);
}

C'est tout je crois.

[gbdivers]

Code :

S�lectionner tout - Visualiser dans une fen�tre � part

shared_ptr<CDraw>::make(...) -> make_shared<CDraw>(...)

Ou unique_ptr si on veut que le tableau soit l'unique responsable de la dur�e de vie des objets point�s (ie tous les objets sont dans le tableau et si le tableau est d�truit, aucun objet n'est cens� encore �tre utilis�)

Mais sinon, c'est effectivement l'approche la plus sur (pas de pointeurs nus) donc celle qui faut recommander pour les d�butants et les personnes plus exp�riment�s (en premi�re approche ; si ensuite, une personne plus exp�riment�e sait ce qu'elle fait et qu'elle a fait des tests de performances et vu que les smart pointeurs sont un frein, elle pourra changer dans un second temps)

[killwin]

Merci, je vais tester shared_ptr qui est la r�ponse � ma question et que je ne connaissais pas.

[piemur2000]

Code :

S�lectionner tout - Visualiser dans une fen�tre � part

1
2
3
4
5
6
7
8
9
10
11
void fonction() {
 
CDraw *draw = new CDraw(true, complet_file.c_str(), coord_, coef_resol);
 
// We reverse picture on Y axis
if(orientation_)
draw->reverse_H_SDL();
 
images.push_back(draw);
 
}

Il faut noter que ce code contient aussi une fuite m�moire si la fonction reverse_H_SDL peut lancer une exception. Dans ce cas il convient de faire:

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
void fonction() {
 
    CDraw *draw = new CDraw(true, complet_file.c_str(), coord_, coef_resol);
 
    // We reverse picture on Y axis
    try{
        if(orientation_)
            draw->reverse_H_SDL();
    }
    catch (...) {      // Catch all exceptions.
        delete draw;   // Delete the draw object.
        throw;         // Rethrow the exception so it can be handled outside the function.
    }
 
    images.push_back(draw);
 
}

et bien sur il faut d�truire les objets cr��s dans le destructeur:

Code :

S�lectionner tout - Visualiser dans une fen�tre � part

1
2
3
4
5
~ClassName() {
    for (std::vector<CDraw*>::iterator it = images.begin(); it != images.end(); ++it) {
        delete *it;
    }
}

Tout cela est bien sur beaucoup moins lisible et safe que l'utilisation de shared_ptr (ou unique_ptr).

[3DArchi]

Salut,

Merci, je vais tester shared_ptr qui est la r�ponse � ma question et que je ne connaissais pas.

Un peu de lecture tr�s utile en ce cas : Pr�sentation des pointeurs intelligents en C++ par Lo�c Joly

[killwin]

C'est r�solu et merci pour la doc sur les pointeurs intelligents.