Discussion :

[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 :

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

[cynique]

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 :

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
#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

[bretus]

Bonjour,

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
 
#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 :

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

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 :

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

1
2
3
4
5
6
7
//allocation avec pointeur
    a* pt = new a(10);
 
    //travail...
 
    //libération
    delete pt;

Code :

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

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

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.

[ram-0000]

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

[Edit] : grilled

[elmcherqui]

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

[white_tentacle]

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 ?

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[]

[ram-0000]

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

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

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é

[Médinoc]

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.