petit probleme rapide

bonsoir , je reprend la programmation en C++ apres une petite absence et je suis entrain de tester les cas limites et je bloque sur ce petit bout de code facile a comprendre qui bug juste avant de finir , merci de m'aider :)

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

#include<iostream> #include<cstdlib> using namespace std;   class a{ double *tab; public: a(int max){ tab = new double [max]; } ~a(){ delete tab; } };       int main(){ a t(40000000); system("pause"); t.~a(); //si j'enleve cette ligne tous va bien printf("\ndedans\n"); system("pause"); //coince ici   cout<<endl; return 0; }

---

Citation:

---

Envoyé par elmcherqui

bonsoir , je reprend la programmation en C++ apres une petite absence et je suis entrain de tester les cas limites et je bloque sur ce petit bout de code facile a comprendre qui bug juste avant de finir , merci de m'aider :)

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

#include<iostream> #include<cstdlib> using namespace std;   class a{ double *tab; public: a(int max){ tab = new double [max]; } ~a(){ delete tab; } };       int main(){ a t(40000000); system("pause"); t.~a(); //si j'enleve cette ligne tous va bien printf("\ndedans\n"); system("pause"); //coince ici   cout<<endl; return 0; }

---

---

C++ garantie d'appeler les constructeurs et déstructeurs automatiquement. Si tu appeles le déstructeur toi-même, le compilateur va l'appeler une deuxième fois, et :bug:

Bonjour,

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

#include<iostream> #include<cstdlib> using namespace std;   class a{ double *tab; public: a(int max){ tab = new double [max]; } ~a(){ delete tab; } };       int main(){ a t(40000000); system("pause"); t.~a(); //si j'enleve cette ligne tous va bien printf("\ndedans\n"); system("pause"); //coince ici   cout<<endl; return 0; }

---

Bon, je vois quelques petits problème là dedans. Dans ton destructeur, tu ne supprimes que le premier élément du tableau. Pour supprimer tous les éléments, utilise :

Code:

---

1 2	delete [] tab; // indique au compilo qu'il s'agit d'un tableau

---

Tu n'as pas a appeler ton destructeur toi même ici, car tu n'as pas créé un pointeur sur un objet de la classe "a". Tu appelles delete [] tab dans le destructeur, car tab est un pointeur.

De plus, le destructeur s'appelle via le delete. Si tu veux faire :

Code:

---

1 2 3 4 5 6 7

//allocation avec pointeur a* pt = new a(10);   //travail...   //libération delete pt;

---

Code:

---

printf("\ndedans\n");

---

C'est pas très joli, ça fait un mix C++ / C. Utilise std::cout plutôt, c'est plus cohérent.

Je te conseille de lire un coup la faq C++ ici. Elle est vraiment pas mal, ça va vite à lire en diagonale, et alors que je croyais avoir "tout" compris dans le C++, j'ai appris des trucs... :aie:

Aussi, il est souvent indispensable de conserver la taille d'un tableau alloué.

Bon courage, ça va revenir
++

ps :
- même pour les tests, surtout pour des tests, a et t, c'est pas très parlant ;)
- Je vois venir le sizeof de loin. Dans la mesure où le sizeof est périlleux (il ne fonctionne que dans certaines parties du code pour faire bref), il faut l'éviter et conserver la taille des tableaux alloués.

et puis quand on fait un tab = new double [max]; c'est pas un delete [] tab; qu'il faut faire.

[Edit] : grilled

ah ! moi qui croyais qu'on utilisais les crochet dans delete pour un tableau d'objet :) 8O
merci pour vos reponse en tous cas , sujet resolu

Citation:

---

et puis quand on fait un tab = new double [max]; c'est pas un delete [] tab; qu'il faut faire.

---

Tu as mis un pas de trop dans ta phrase, non ?

Citation:

---

ah ! moi qui croyais qu'on utilisais les crochet dans delete pour un tableau d'objet

---

J'ai l'impression qu'on t'a plus embrouillé qu'autre chose.

La règle est simple :
- alloué avec new, libéré avec delete
- alloué avec new[], libéré avec delete[]

Citation:

---

Envoyé par white_tentacle

Tu as mis un pas de trop dans ta phrase, non ?

---

Non, mais il manquait un ? à la fin de la phrase :oops:

Citation:

---

Envoyé par white_tentacle

J'ai l'impression qu'on t'a plus embrouillé qu'autre chose.

La règle est simple :
- alloué avec new, libéré avec delete
- alloué avec new[], libéré avec delete[]

---

Dit comme cela, c'est vrai que c'est plus simple et sans ambiguïté

Aussi, ton type doit soit posséder un constructeur de copie et un opérateur d'affectation gérant correctement ton tableau, soit être non-copiable.