Discussion :

[Erakis]

Bonjour à tous,

Voici un appercu d'une template (liste chaînée double) que j'ai conçu hier :
CGenericDoubleLinkedList.h

Code :

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

1
2
3
4
5
6
7
 
template <class T> class CGenericDoubleLinkedList
{
public:
	CGenericDoubleLinkedList(void){...}
	virtual ~CGenericDoubleLinkedList(void){...}
	CGenericDoubleLinkedList(const CGenericDoubleLinkedList & copy){...}

Ensuite voici une classe que qui se sert de ma template :
SensorList.h

Code :

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

1
2
3
4
5
6
7
8
9
10
 
class CSensorList
{
public:
	CSensorList(void);
	CSensorList(const CSensorList & copy);
	virtual ~CSensorList(void);
 
private :
	CGenericDoubleLinkedList<CSensor>   m_ListSensor;

Et maintenant voici où est mon problème :
SensorList.cpp

Code :

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

1
2
3
4
5
6
CSensorList::CSensorList(const CSensorList & copy)
{
     m_ListSensor = CGenericDoubleLinkedList<CSensor>(copy);

}

J'essais de faire une copie de ma liste mais cela ne fonctionne pas
J'ai ce message d'erreur : error C2440: '<function-style-cast>' : impossible de convertir de 'const CSensorList' en 'CGenericDoubleLinkedList<T>'

Merci beaucoup pour votre aide

[coyotte507]

Salut,

Tu as m_ListSensor de type CGenericDoubleLinkedList<CSensor> et copy de type CSensorList.

Comment tu veux t'en sortir?
Il faudrait une fonction

Code :

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

CGenericDoubleLinkedList(const CSensorList & copy){...}

Mais cette fonction n'existe pas dans ton projet.

[Erakis]

Merde que je suis dans la lune !!!
Désoloé, j'avais oubilé quelque chose de très important

Code :

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

1
2
3
4
5
CSensorList::CSensorList(const CSensorList & copy)
{
    m_ListSensor = CGenericDoubleLinkedList<CSensor>(copy.m_ListSensor);
}

Mais merci quand même coyotte507, sans toi j'aurais peut-être continué à chercher un petit bout de temps. LoL

Mes vacances sont dues

[coyotte507]

Donc résolu?

[Médinoc]

Ou pour être précis:

Code C++ :

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

1
2
3
4
CSensorList::CSensorList(const CSensorList & copy)
 : m_ListSensor(CGenericDoubleLinkedList<CSensor>(copy.m_ListSensor))
{
}

Cele t'évite d'utiliser le constructeur par défaut de m_ListSensor pour appeler ensuite l'opérateur = ...

Au fait, as-tu implémenté ta propre fonction operator= ? C'est nécessaire pour des objets dont le constructeur de copie n'est pas trivial. Le plus simple pour ce genre d'opérateur, c'est faire une fonction de swap:

Code C++ :

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
class UneClasse
{
private:
	int * m_pVal;
public:
	UneClasse()
	 : m_pVal(new int[42])
	{
	}
	UneClasse(UneClasse const &src)
	 : m_pVal(new int[42])
	{
		std::copy(src.m_pVal, src.m_pVal+42, m_pVal);
	}
	~UneClasse()
	{
		delete[] m_pVal;
	}
 
	void Swap(UneClasse &other)
	{
		int *tmp = other.m_pVal;
		other.m_pVal = m_pVal;
		m_pVal = tmp;
	}
 
	UneClasse & operator= (UneClasse const &src)
	{
		UneClasse tmp(src);
		Swap(tmp);
	}
};

[Erakis]

J'ai effectivement overridé l'opérateur =.

Code :

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

1
2
3
4
5
6
7
8
9
inline CGenericDoubleLinkedList& operator = ( const CGenericDoubleLinkedList& right )
{
    if(this != &right)
    {
     // Même code que dans le constructeur de copie
    }
    return *this;
}

Donc si j'ai bien compris, de cette façon :

Code :

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

1
2
3
4
5
 
CSensorList::CSensorList(const CSensorList & copy)
 : m_ListSensor(CGenericDoubleLinkedList<CSensor>(copy.m_ListSensor))
{
}

Il y aura que le constructeur de copie qui sera appelé ?

Tandis que de mon ancienne façon, le constructeur de copie aurait été appelé pour créer l'instance et par la suite ensuite l'opérateur = aurait été appelé pour recopier (dans m_ListSensor) ce que le constructeur de copie venait de créer ?

[Médinoc]

En effet, c'est bien cela.

Tu devrais remplaçer le code de ton opérateur = par copie + swap. De cette façon, le code du constructeur de copie ne sera pas dupliqué, rendant ton programme plus maintenable.