Discussion :

[Romvaillant]

Bonjour,

comme cela est dit dans le titre, j'ai un probleme avec la libération d'objet dynamique en C++. Soit je ne sais pas utiliser la fonction soit je ne sais pas d'où vient l'erreur. L'erreur est toujours la meme pour ceux qui code...
Debug Assertion Failed!

Program:...
File: dbgdel.cpp
Line 52

Expression:_BLOCK_TYPE_IS_VALID(pHead->nBlockUse)
(je vous met l'erreur des fois que ca veuille dire quelque chose pour quelqu'un!)
Je vous met un bout de code pour que cela soit plus compréhensible pour vous. je vous en remercie par avance!

Dans la bibliothèque:

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
....
student *ListStudent[100];    // un tableau de 100 pointeurs sur student
....
 
Ma fonction:
void CourseSection::RemoveAllStudent()
{
	for(int i=0;i<NumberStudent;i++)    // boucle pour passer par chacun des éléments
	{
		delete ListStudent[i];     // je fais la libération mémoire
	}
	NumberStudent=0;    // un compteur
}

dans mon main(pour tester)
je déclare un student avec la fonction new...cela marche bien car j'arrive à l'utiliser et je fais pointé ListStudent[NumberStudent] sur chaque student que je crée dynamiquement. Ce problème est assez embetant car j'utilise le delete aussi pour ma fontion destrutor, et dans une fonction OneRemove...

N'hesitez pas si vous avez une solution ou une question....

Je vous remercie beaucoup de votre aide!!

[Davidbrcz]

delete[] ListStudent;
http://cpp.developpez.com/faq/cpp/?p...S_delete_array

[Romvaillant]

Merci à toi Davidbrcz,
le "delete[] ListStudent; " s'est avéré bon pour mon constructor et mon RemoveAll().
mais mon problème principale reste toujours présent! La fonction "delete [] Tab[ i ];" qui semble tout à fait être ce dont j'ai besoin pour ma fonction OneRemove()... me retourne toujours la même erreur...
le code est toujours le meme que précedemment sauf que que j'ai déclaré mon tableau de de pointeur sur student en dynamique


student **ListStudent; //declaration de ma liste de student

ListStudent=new student * [100]; // allocation de 100 pointeurs sur student

student *NewStudent=new student; // allocation d'un student
Je declare chaque student dans un sous prog comme au dessus et je retour NewStudent pour l'enregistrer dans mon tableau de ListStudent[i].

Une question au passage:
Est ce que le "delete [] Tab;", pris dans le lien que Davidbrcz m'a donné, à lui tout seul supprimer toute mes allocations?pour moi non...et dans ce cas la, ca veut dire que ma fonction destructor et RemoveAll ne sont pas complète....j'en revient à mon problème où je doit libérer delete[] ListStudent[i].....

Merci pour votre lumière!

[Davidbrcz]

Code :

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

delete[] ListStudent;

te détruit toutes les objets de ton tableau un à un.
car
(FAQ):

delete [] se charge d'appeler le destructeur de chaque objet du tableau.

Ta méthodes devient alors :

Code :

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

1
2
3
4
void CourseSection::RemoveAllStudent()
{
delete[] ListStudent;
}

Et ctout.

[Médinoc]

Euh...

Tu me permettras d'avoir des doutes, là.
ListStudent est de type student * [100], c'est-à-dire un tableau de pointeurs bruts.
delete[] sur un tableau de pointeur bruts désalloue le tableau, mais ne fait en aucun cas un delete sur chaque pointeur!

En effet, le "destructeur" d'un pointeur brut ne fait rien.
S'il était question de pointeurs intelligents, là par contre il en serait autrement...

[Romvaillant]

J'ai modifié mon student *[100] par un student **
Sinon mon problème persiste toujours pour la supprimer d'un seul élément du tableau....
Le delete d'un pointeur ListStudent[i](qui est un student *) ne fonctionne pas.

[Médinoc]

Il faut voir comment tu alloues tes student, et comment tu remplis ton tableau.
De plus, je conseille d'initialiser tous les pointeurs du tableau à NULL lors de l'allocation, et de remettre immédiatement à NULL un pointeur après son delete.

[Romvaillant]

Oui j'initialise tout les pointeurs à null juste après l'allocation avec une boucle for!

code: (en version simplifié!)

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
 
student*   AddStudent()
{
     student *NewStudent=new student;  // allocation de mon student
     ...
     return NewStudent;
}
 
Main()
{
....
ListStudent[i]=AddStudent();
....
}

Pour moi il n'y a aucune erreur....

[millie]

Déjà une question.

Pourquoi n'utilises tu pas plutôt std::vector<student*> au lieu de te farcir un tableau à la main ?

[Romvaillant]

Parceque je ne sais même pas ce que c'est mais c'est une piste à étudier!!Le problème c'est que cela m'est imposé d'utiliser un tableau...

[CedricMocquillon]

Ta fonction AddStudent() est extrèmement dangereuse codée comme cela en effet comme elle renvoie un pointeur sur un objet alloué localement, si tu ne le récupère pas (le pointeur) tu auras forcément une fuite mémoire. Le seul moyen de renvoyer un "pointeur" de manière propre est de renvoyer un pointeur intelligent. Je ne comprend pas pourquoi dans ton main tu ne fais tout simplement

Code :

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

ListStudent[i]=new student();

[Romvaillant]

parcequ'en fait j'ai plein de truc à faire pendant la création d'un student....mettre nom, prenom, adresse, mail....etc...si je le fais dans mon main c'est moins propre...

[Médinoc]

Ta fonction AddStudent() est extrèmement dangereuse codée comme cela en effet comme elle renvoie un pointeur sur un objet alloué localement, si tu ne le récupère pas (le pointeur) tu auras forcément une fuite mémoire. Le seul moyen de renvoyer un "pointeur" de manière propre est de renvoyer un pointeur intelligent. Je ne comprend pas pourquoi dans ton main tu ne fais tout simplement

Code :

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

ListStudent[i]=new student();

N'importe quoi...

Donc pour toi, toute fonction retournant directement le résultat d'une allocation dynamique est "dangereuse" ?
On est en C++ ici, pas en java: Les programmeurs sont censés savoir qu'ils utilisent une fonction qui retourne un pointeur alloué dynamiquement...

[CedricMocquillon]

"dangereuse" est effectivement peut être un peu excessif (d'autant que j'ajoutais "extrèmement"), il n'empêche que l'utilisation de "AddStudent();" tel quel ne provoque pas d'erreur de compilation ni même un warning et qu'il faut donc l'utiliser avec attention et s'assurer de récupérer à chaque fois l'objet créé dynamiquement en local. L'utilisation d'un smart pointeur permet de pallier aux erreurs d'inatention puisque dans ce cas l'objet renvoyé sera détruit (compteur de référence à zéro).
Effectivement on peut dire que le programmeur est en C++ et qu'il est sensé savoir ce qu'il fait, il n'empêche que je préfère voir quelqu'un utiliser un vecteur plutôt qu'un tableau.

PS: je ne suis pas non plus complètement obtu sur l'utilisation des tableaux ou des fonctions renvoyant un objet alloué localement, juste prudent quand j'en voie.

[Romvaillant]

Il n'y a aucun probleme avec mon return dans ma fonction de AddStudent(), car j'utilise par la suite tout les student que j'ai déclaré(dynamiquement) pour modifier ou afficher des valeurs!!!

Je ne voit vraiment pas d'où vient mon problème....et apparement je ne suis pas le seul...Je vous remercie quand de vous pencher sur mon problème!

Mon problème vient du fait que je n'arrive pas à supprimer la deuxième dimension de mon tableau...je n'ai pas d'erreur pour "delete[] ListStudent" donc pour libérer les pointeurs de student mais j'en ai pour "delete[] ListeStudent[i]" pour libérer les student(ce qui est plus important que la libération des pointeurs à 2 octets!)

[CedricMocquillon]

Salut, effectivement, désolé, on s'était un peu éloigné de ton problème, voici une solution qui fonctionne avec les vecteurs

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
41
42
43
44
45
46
 
#include <iostream>
#include <vector>
 
using namespace std;
 
struct student
{
	int val;
	student()
	{
		val=rand()%100;
	}
};
 
 
vector<student*> ListStudent;
 
void RemoveAllStudent()
{
	int NumberStudent = 100;
	for(int i=0;i<NumberStudent;i++)    // boucle pour passer par chacun des éléments
		delete ListStudent[i];     // je fais la libération mémoire
	ListStudent.clear();
	NumberStudent=0;    // un compteur
}
 
student* AddStudent()
{
	student* NewStudent = new student;
	return NewStudent;
}
 
int main (void)
{
	ListStudent.resize(100);
	for (int i=0; i<100; ++i)
		ListStudent[i] = AddStudent();
 
	for (int i=0; i<100; ++i)
		cout << ListStudent[i]->val << ' ';
 
	RemoveAllStudent();
	cout << endl << ListStudent.capacity() << endl;
	return 0;
}

Par contre, je n'ai pas trouvé comment obliger le vecteur à désallouer complètement la mémoire qu'il a réservé.

[yan]

Bonjour,

Par contre, je n'ai pas trouvé comment obliger le vecteur à désallouer complètement la mémoire qu'il a réservé.

Il le fera tout seule lors de sa destruction. Il garde la memoire pour une utilisation futur. Histoire d'evité une nouvelle allocation memoire.
Sinon tu devrait pourvoir faire
ListStudent.reserve(0);

[Romvaillant]

desole pour le retard j'étais au chute du Niagara et à toronto pendant quelques jours....
Je vous remercie beaucoup pour votre aide...l'échéance de mon rapport étant tombée je n'ai pas utilisé les vectors mais j'ai laissé mon problème avec les delete en le spécifiant!En revanche ayant déjà un autre projet je vais directement commencer avec les vectors(un outils que je ne connaissait pas avant...)

Merci beaucoup!