Discussion :

[yves2]

Salut,

J'ai dans une classe les lignes suivantes:

Code :

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

1
2
3
4
class MyClass{
...
   std::list<String> tags;
};

Lors de la destruction d'une instance de MyClass j'ai une erreur dans le nettoyage du heap (visual C++ 7.1 en debug) sans avoir jamais touché à cette liste. La vérification automatique lors du nettoyage du heap me dit qu'un pointeur est incorrect. Dois-je explicitement détruire cet object (tags) dans le destructeur de MyClass? Et si oui comment (delete tags; ???).

Merci.
Yves

[yan]

Salut.
qu'es ce que String ?

[yves2]

J'utilise une librairie speciale qui s'appelle juce. C'est une classe de string (chaine de caractere).

[yves2]

J'ai essaye en remplacant String par int et j'ai le meme probleme.

[yan]

J'ai essaye en remplacant String par int et j'ai le meme probleme.

c'est donc ailleur.
Utilise tu des pointeur? et tu sur de la manière dont tu les utilisent? vérifie les accès memoire.

[yves2]

Je n'utilise pas tags, je me contente de le creer en faisant une instance de MyClass. Par contre le fait de l'inclure dans la definition de MyClass cree le probleme. Mais je ne sais pas exactement ce qui se passe lorsque l'on instancie une list. Y a t'il quelque chose de creer qu'il faut detruire ou nettoye dans le destructeur de MyClass?

[yan]

Y a t'il quelque chose de creer qu'il faut detruire ou nettoye dans le destructeur de MyClass?

non.
Ca doit être une erreur mémoire
Vérifie l'utilisation tes pointeurs dans ton code.

[yves2]

La seule chose que je fasse est d'ajouter des objets graphiques de la librairie juce dans le créateur. Tous ces objets sont censés être correctement détruit par la fonction juce deleteAllChildren(). Il doit y avoir un problème dans la librairie. Ce qui est curieux c'est que le fait d'ajouter une liste dans la définition de la classe fasse ressortir le problème.

[yan]

Il doit y avoir un problème dans la librairie

Ca j'y croit pas trop...

Ce qui est curieux c'est que le fait d'ajouter une liste dans la définition de la classe fasse ressortir le problème.

Ben c'est le principe d'une erreur mémoire, ça fait n'importe quoi. Parfois ça tombe même en marche (comme quand tu enleve std::list<String> tags
. Et le problème est que cela peut être n'importe où dans ton code.

Peut tu mettre le code de ta class?? histoire de voir si déjà si l'erreur n'est pas là

[yves2]

Je n'ai pas reussi a mettre le fichier en piece jointe; seul le constructeur est appelé:

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
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
 
#include "juce.h"
 
class CTab3: public Component,
             public ButtonListener
{
public:
 
	CTab3();
	~CTab3();
private:
    TextButton* LoadButton;		// bouton
	Label *LPath;
	TextEditor* TextPath;
	TextEditor* TextError;		// liste des erreurs
 
	std::list<String> tags;		// liste des tag en attente de fermeture /tag
};
 
 
CTab3::CTab3()
{
	int X00, X0, DX, Y0, DY;
 
	// bouton
	addAndMakeVisible (LoadButton = new TextButton (T("LOAD")));
    LoadButton->setBounds (24, 16, 40, 20);
    LoadButton->addButtonListener (this);
	LoadButton->setTooltip("charge data");
 
	// path
	X00 = X0 = 5;
	Y0 = 40;
	DX = 40;
	DY = 20;
    addAndMakeVisible (LPath = new Label (T("bof"), T("Path")));
    LPath->setFont (Font (15.0000f, Font::plain));
    LPath->setJustificationType (Justification::centredLeft);
    LPath->setEditable (false, false, false);
    LPath->setColour (TextEditor::textColourId, Colours::black);
    LPath->setColour (TextEditor::backgroundColourId, Colour (0x0));
    LPath->setBounds (X0, Y0, DX, DY);
	X0 += DX; 
	// TextPath
	DX = 100;
    TextPath = new TextEditor();
    addAndMakeVisible (TextPath);
    TextPath->setBounds (X0, Y0, DX, DY);
    TextPath->setText (T("..\\..\\out\\"));
	Y0 += DY;
 
	// TextError
	X0 = X00;
	DX = 350;
	DY = 80;
    TextError = new TextEditor();
    addAndMakeVisible (TextError);
    TextError->setBounds (X0, Y0, DX, DY);
    TextError->setText (T(""));
	TextError->setMultiLine(true);
	Y0 += DY;
};

[yves2]

Et voici le destructeur que j'avais oublié; apparement le fichier joint a fonctionné dans le post précédent:

Code :

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

1
2
3
4
5
 
CTab3::~CTab3()
{
	deleteAllChildren();
};

[yan]

Y as rien de choquant...
Ca ne me semble pas être ici.
Regarde dans le reste du code. Surtout la manipulation des pointeurs.

Sinon tu devrais essayer cette class seul pour voir si tu as le même problème.

[loufoque]

Vérifie que ton programme fait pas n'importe quoi avec valgrind sous linux. (bien sûr, pour ça il faut que ton code puisse tourner sous linux)