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