Discussion :

[Fiquet]

Resalut,

Désolé de reposter mais là je sèche sur une connerie...

Dans ma classe CLevel j'ai un constructeur par défaut en plus du constructeur. Quand j'appelle le constructeur avec les arguments depuis mon application, il n'y a aucun problème, mais si j'initialise avec le constructeur par défaut, mes briques ne s'affichent pas. Pourtant c'est exatctement le même code! En fait dans la console je vois que le destructeur 2 fois dont une directement après...

Voici le constructeur:

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
 
CLevel::CLevel(unsigned int nBricks, unsigned int nCols, unsigned int Offset, unsigned int widthBrick, unsigned int heightBrick)
{
	// Initialise la liste des briques
	for(unsigned int i=0; i<nBricks; i++)
	{
		// Pour placer les briques de façon homogène
		unsigned int xOrg = ( (widthBrick+5) * (i % nCols) ) + Offset;
		unsigned int yOrg = ( (heightBrick+5) * (i / nCols) ) + Offset;
 
		// Ajout des briques dans la liste chaînée
		m_brickList.push_back(new CBrick(255, 0, 0, xOrg, yOrg, widthBrick, heightBrick) );
	}
 
	cout <<"Constructeur de CLevel\n";
}

et le constructeur par défaut:

Code :

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

1
2
3
4
5
6
7
 
// Constructeur par défaut
CLevel::CLevel()
{
	// Crée un niveau par défaut
	CLevel(240, 15, 10, 40, 30);
}

Dans le constructeur de l'application:

Code :

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

Level = new CLevel(240, 15, 15, 40, 30); // marche

ou

Code :

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

Level = new CLevel // marche pas!!

[Luc Hermitte]

Tu ne peux pas (encore) appeler un constructeur depuis un autre.
Ce que tu as fait, c'est déclarer et construire une variable temporaire non nommée.

En attendant, tu peux :
- faire que ton constructeur initialisant prennent des paramètres par défaut -- histoire de le transformer en constructeur par défaut
- laisser tomber le constructeur par défaut qui de toutes façons contient des constantes magiques ce qui est rarement une bonne chose
- definir une fonction init() qui sera appelée par les deux constructeurs
- faire des trucs tordus en introduisant un niveau d'héritage factice

[bigquick]

Salut !

Code :

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

1
2
3
4
5
6
// Constructeur par défaut
CLevel::CLevel()
{
   // Crée un niveau par défaut
   CLevel(240, 15, 10, 40, 30);
}

Tu ne peux pas chainer les appels aux constructeurs. En faisant ça, tu contruits un nouvel objet dans ton constructeur, et il est simplement détruit à la fin de celui-ci (puisque c'est un objet local). Ce qui explique que tu puisse voir un appel au destructeur immédiatement.

La solution est de crééer une méthode d'init, qui sera appelée dans les 2 constructeurs

cf. la FAQ : http://c.developpez.com/faq/cpp/?pag...R_appel_chaine

[bigquick]

Tu ne peux pas (encore) appeler un constructeur depuis un autre.

Luc, tu veux dire qu'on pourra bientôt ?

[Luc Hermitte]

J'ai vu une proposition en ce sens. De Gaby, je crois. A vérifier.
Après, je n'ai pas suivi où cela en était.

[Médinoc]

Et en appelant un autre constructeur comme un initialiseur de membre, ça ne marche pas non-plus?

Sinon, il reste peut-être la possibilité de donner une valeur par défaut à chaque paramètre du constructeur, mais dans ce cas, il peut être appelé avec N nombres de paramètres différents, au lieu de 2 (0 ou N)...

[Fiquet]

Ah bon??

Ben vous m'en apprenez une là. Je suis quasiment sûr que dans mon bouquin il y avait plusieurs exemples comme ça..

Enfin je regarderai et je vous confirmerai mais il me semble bien.

Et j'ai plusieurs fois du employer ça moi aïe ..

Ben merci bien!

[Fry]

si tu veu vraiment initialiser le constructeur par default en construisant l'object tu peu faire comme ca:

- creer un object comme tu a fait
- assigner l'object creer a l object

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
 
#include <iostream>
 
class test
{
private:
	int a;
	int b;
 
public:
	test(int a, int b)	{
		this->a = a;
		this->b = b;
	}
 
	// constructeur par default
	test()	{
		*this = test(5, 10);
	}
 
	void print()	{
		std::cout << " [ " << a << " | " << b << " ] " << std::endl;
	}
};
int main()
{
	std::cout << "Start" << std::endl;
 
	test *a = new test();
	a->print();
 
	std::cout << "End" << std::endl;
	std::cin.ignore();
	return 0;
}