Template - Appelé le constructeur de copy

Bonjour à tous,

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

Code:

---

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:

---

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:

---

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

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:

---

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

---

Mais cette fonction n'existe pas dans ton projet.

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

Code:

---

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 :roll:

Donc résolu?
:D

Ou pour être précis:

Code:

---

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:

---

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); } };

---

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

Code:

---

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:

---

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 ?

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.