Discussion :

[Philippe320]

Bonjour,

toujours aussi peu doué pour les titres....
dans une CDialog, j'ai des tableaux déclarés par

Code :

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

1
2
 
X=new double[NombrePoints];

je dois bien sûr faire un

Code :

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

1
2
 
delete[] X;

en quittant la boîte de dialogue, mais où ?

dans le destructeur (mais comment le déclarer ? sachant que le constructeur est )

Code :

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

1
2
3
4
5
6
7
8
 
CAcquisition::CAcquisition(CWnd* pParent /*=NULL*/)
	: CDialog(CAcquisition::IDD, pParent)
{
	//{{AFX_DATA_INIT(CAcquisition)
		// NOTE: the ClassWizard will add member initialization here
	//}}AFX_DATA_INIT
}

ou dans OnDestroy()

[farscape]

Dans le destructeur de la dialogue.

[Philippe320]

Merci de ta réponse, mais ...

ça plante :
j'ai les constructeurs et destructeurs suivants :

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
 
CAcquisition::CAcquisition(CWnd* pParent /*=NULL*/)
	: CDialog(CAcquisition::IDD, pParent)
{
	//{{AFX_DATA_INIT(CAcquisition)
		// NOTE: the ClassWizard will add member initialization here
 
NombrePoints=200;
Xc=new double[NombrePoints];
Yc=new double[NombrePoints];
Zc=new double[NombrePoints];
T=new double[NombrePoints];	
donneesIRS=new IRS[NombrePoints];
rho=new double[NombrePoints];
theta=new double[NombrePoints];
phi=new double[NombrePoints];
Cm=new double[NombrePoints];
T=new double[NombrePoints];	
 
	//}}AFX_DATA_INIT
} //constructeur
 
CAcquisition::~CAcquisition()
{
	//{{AFX_DATA_INIT(CAcquisition)
		// NOTE: the ClassWizard will add member initialization here
 
 
delete[] Xc;
delete[] Yc;
delete[] Zc;
delete[] T;	
delete[] donneesIRS;
delete[] rho;
delete[] theta;
delete[] phi;
delete[] Cm;
delete[] T;	
 
 
	//}}AFX_DATA_INIT
} //destructeur

mais lors de la destruction de CAcquisition (par l'application principale qui a fait un new CAcquisition, puis delete CAcquisition),
j'ai les erreurs suivantes :

[3532] memory check error at 0x0033D0CC = 0xDD, should be 0xFD.
[3532] memory check error at 0x0033D0CD = 0xDD, should be 0xFD.
[3532] memory check error at 0x0033D0CE = 0xDD, should be 0xFD.
[3532] memory check error at 0x0033D0CF = 0xDD, should be 0xFD.

Pourquoi ?
Merci d'avance
Philippe

[farscape]

Xc et compagnie sont des variables membres de la classe dialogue ?
si ça plante dans le destructeur c'est que tu pertubes la memoire quelque part ...
par exemple en affectant une valeur hors indice du tableau..

[r0d]

Bonjour,

je me permet d'intervenir car j'estime que c'est vraiment très important: il faut utiliser les conteneurs. Si tu utilise la mfc, il y a la classe template CArray.
- ça t'évitera ce genre de problème (new et delete)
- ça t'évitera les "sortie de tableau"
- ces classes sont optimisées
- ces classes sont stables
- ces classes sont très simples à utiliser.

Hope it helps.

[Philippe320]

Bonjour r0d,

merci de ton intervention qui pourrait me sauver la mise...
car j'ai ce problème à de multiples endroits, toujours avec des tableaux dynamiques, avec des fuites de mémoire impressionnantes

je vais tout modifier avec CArray (ça rejoint le conseil le Farscape un peu plus tôt ce matin)

je repasserai en cas de problème
merci à vous deux
Philippe

[Philippe320]

C'est toujours moi ...
j'ai remplacé mes new/delete par des CArray, c'est parfait ça compile, mais ....

Soit un tableau Xc définit par :

Code :

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

1
2
3
 
public:
	CArray<double,double&> Xc;

et initialisé par

Code :

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

1
2
 
Xc.SetSize(n);

Comment passer ce tableau à une procédure SetDatas (pour affichage des valeurs),

Code :

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

1
2
3
4
 
void SetDatas(int nombrePoints, double* tableau) {
....
}

sachant qu'elle n'accepte que double* comme 2ème argument
(elle appartient à une bibliothèque, donc je ne peux rien y faire !)

en l'appelant par SetDatas(n,Xc);
j'ai l'erreur :
error C2664: 'void __thiscall CPGLLine2D::SetDatas(int,double *)' : cannot convert parameter 2 from 'class CArray<double,double
&>' to 'double *'
No user-defined-conversion operator available that can perform this conversion, or the operator cannot be called

Quel opérateur de conversion faut-il écrire pour passer tout simplement de CArray<double,double &> à double* ?

Merci d'avance de vos réponses
Philippe

[farscape]

ce matin c'est la serie des CArray et compagnie ,

alors:

Code :

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

1
2
 
typedef CArray<double,double&> DoubleArray;

Code :

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

1
2
3
4
5
6
7
 
 
void SetDatas(int nombrePoints, double * ptableau)
 {
}
DoubleArray dArray;
SetData(..., dArray.GetData( ));

[Philippe320]

Farscape, j'ai cherché un smiley qui se prosterne, mais y'a pô !

alors j'ai pris celui là


Bluffé, ça marche !!!
Et puis quand je vois ta réponse de ce matin concernant la CList ....
alors re

A bientôt
Philippe

[nico-pyright(c)]

maitre farscape