Discussion :

[mouss31]

Bonjour,
je souhaite surcharger l'op�rateur "= " pour pouvoir faire une affectation entre deux objets de ma classe CParam.

voici le code de ma classe :

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
 
class CParam  
{
public:
	CParam();
	CParam(CString Name,CString Value,CString* Validation);
	virtual ~CParam();
	// Redéfinition de l'opérateur d'affectation
	CParam & operator=(const CParam & e);
 
public:
	CString m_strValue;
	CString* m_strValidation;
	CString m_strName;
};

voici le code de ma surcharge

Code :

S�lectionner tout - Visualiser dans une fen�tre � part

1
2
3
4
5
6
7
8
9
10
 
// Définition de l'opérateur d'affectation
CParam & CParam::operator=(const CParam & e) 
{
	m_strName = e.m_strName;
	m_strValidation = e.m_strValidation;
	m_strValue = e.m_strValue;
 
  return *this; // On retourne une référence sur l'objet courant
}

� l'ex�, je me retrouve avec cette erreur : Unhandled exception , access violation au moment de l'affectation de

Code :

S�lectionner tout - Visualiser dans une fen�tre � part

m_strName = e.m_strName;

je ne comprends pas du tout pourquoi...
quelqu'un pourrait il m'aider?
merci...

[M�dinoc]

Je ne sais pas non plus, mais cette ligne-ci me fait peur:

Code :

S�lectionner tout - Visualiser dans une fen�tre � part

m_strValidation = e.m_strValidation;

Ici, tu dupliques un pointeur, et non l'objet CString... Je ne connais pas le code de ton destructeur, mais j'esp�re que tu ne fais pas de delete dessus...


PS: Pour ton erreur, l'access violation peut aussi venir d'autre chose qui se serait pass� avant la copie...
V�rifie aussi que tu ne passes pas une r�f�rence nulle (oui, �a peut arriver) :

Code :

S�lectionner tout - Visualiser dans une fen�tre � part

1
2
3
CParam *pParam = NULL;
CParam par;
par = *pParam; //Crash!

[farscape]

salut,
je pense que ton probleme est plutot sur l'affectation de
m_strValidation qui est declar�e en tant que pointeur dans ta classe.
sachant que tu vas copier l'adresse et non le contenu, lors de la destruction d'un de tes objets CParam un de tes objets m_strValidation pointera sur une adresse invalide (si tu l'a lib�r� dans le destructeur) provoquant ainsi une assertion d'erreur � l'utilisation...
ton affectation dans l'operateur pour cette variable est incorecte...
de plus tu devrais rajouter:

Code :

S�lectionner tout - Visualiser dans une fen�tre � part

1
2
 
if(this==&e) return *this;

pour finir je ne vois l'interet de d�clarer une CString sous forme de pointeur dans une classe ....

[mouss31]

non mon destructeur est vide... je ne le lib�re nulle part.
et puis, c'est vraiment lors de cette affectation que ca plante (m_strName = e.m_strName; ), si j'en crois mon debbuger...
De plus, si j'enl�ve le pointeur m_strValidation (qui ne me sert pas dans l'imm�diat), j'ai toujours le m�me soucis :

Param.h

Code :

S�lectionner tout - Visualiser dans une fen�tre � part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
 
class CParam  
{
public:
	CParam();
	CParam(CString Name,CString Value);
	virtual ~CParam();
	// Redéfinition de l'opérateur d'affectation
	CParam & operator=(const CParam & e);
 
public:
	CString m_strValue;
	CString m_strName;
};

Param.cpp

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
 
CParam::CParam()
{
	m_strName = "Name";
	m_strValue="Value";
}
 
CParam::CParam(CString Name,CString Value)
{
	m_strName = Name;
	m_strValue=Value;
}
 
CParam::~CParam()
{
}
 
// Définition de l'opérateur d'affectation
CParam & CParam::operator=(const CParam & e) 
{
	m_strName = e.m_strName;
	m_strValue = e.m_strValue;
 
	return *this;
}

De plus, Farscape, si j'ajoute ce que tu me recommande :

Code :

S�lectionner tout - Visualiser dans une fen�tre � part

if(this==&e) return *this;

j'ai l'erreur en compil qui me dit qu'il est poussible que la fonction ne retourne rien. et je ne sais pas quoi mettre dans le else...

[M�dinoc]

Tu ne dois pas toucher au return actuel: Il fallait juste que tu ajoutes cette ligne en t�te de fonction.

Mais je r�it�re mon conseil, sous une autre forme: Rajoute en plus:

Code :

S�lectionner tout - Visualiser dans une fen�tre � part

assert(&e != NULL);

Si tu es en MFC, tu peux essayer ATLASSERT(), puis ASSERT() si <assert.h> ou <cassert> n'est pas inclus de base.
Si tu compiles sans MFC, ajoute le header standard <cassert>.