Contructeur par défaut

Resalut,

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

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:

---

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:

---

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:

---

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

---

ou

Code:

---

Level = new CLevel // marche pas!!

---

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

Salut !

Code:

---

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

Citation:

---

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

---

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

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.

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

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

Ben merci bien!

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:

---

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

---