Discussion :

[nsarras]

Bonjour � tous,

J'ai un comportement bizarre lorsque j'essaie de lib�rer un double *, membre de ma classe.

Voici les d�tails :

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
35
36
37
38
39
40
//header
class CPolygon  
{
public:
	CPolygon();
	virtual ~CPolygon();
 
	CPolygon( const CPolygon &);
 
	std::vector<double> pts;
 
	double* bounds;
};
 
//implem
CPolygon::CPolygon()
{
	bounds=0;
}
 
CPolygon::~CPolygon()
{
	if(bounds)
	{
		delete []bounds;
		bounds=0;
	}
}
 
CPolygon::CPolygon( const CPolygon &poly)
{
	if(!bounds)
		bounds = new double[4];
 
	if(poly.bounds)
	{
		for(int i=0; i<4; i++)
			this->bounds[i] = poly.bounds[i];
	}
}

Dans une autre classe, je manipule un std::vector<CPolygon> et au moment du resize, �a plante dans le constructeur de copie dans le for() car this->bounds n'est pas correctement initialis� (0xcdcdcd).

Code :

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

1
2
3
4
5
6
7
8
//std::vector<CPolygon> klines
if(klines.size()==0)
{
	CPolygon nullpoly;
 
	klines.resize(2,nullpoly);
	...
}

Je ne comprends pas ce comportement vu que je pensais avoir tout fait pour allouer correctement mon bounds dans tous les cas.

Quelqu'un peut-il m'expliquer le soucis ?

Merci par avance.

[dragonjoker59]

Dans un constructeur, les variables membres ne sont pas initialis�es avant que tu ne le fasses (ou que chaque variable ait un constructeur par d�faut).
Donc ton test en entr�e de ton constructeur par copie est invalide :

Code :

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

if (!bounds)

car 'bounds' n'est pas initialis�e. Tu dois donc virer ce test et mettre directement ton initialisation :

Code :

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

bounds = new double[4];

Je suppose vu ce bout de code que tu viens du Java, o� le constructeur par d�faut est appel� au d�but des autres constructeurs. Sache qu'en C++ ce n'est pas le cas (du moins pas par d�faut, et m�me ainsi pas encore sur tous les compilateurs), tu DOIS initialiser tes variables membres dans tous les constructeurs.

[kessoufi]

salut,

� mon avis tu devras d�finir un copy constructor pour ta classe Polygon, o� tu fais une duplication m�moire (deep copy) de la variable de type pointer

celui g�n�r� par le compilateur se contente de faire une affectation de variable, ce qui fait qu'elle est doublement d�truite

[dragonjoker59]

Euh ... Il l'a d�j� �crit son copy constructor ... (avec une erreur, certes, mais quand m�me) ...

[nsarras]

Ok tout b�tement.

Je ne viens pas du Java mais j'ai pas forc�ment l'habitude de coder des constructeurs par copie du coup j'avais pas forc�ment bien assimil� le principe.

Merci bien