Constructeur de copie

Salut,
voilà j'utilise une classe CLayer qui dérive de std::list<CLED*>, à savoir qu'à chaque ajout d'un élément CLED* dans ma liste, un nouveau CLED est instancié avec allocation dynamique de mémoire.
J'ai eu besoin d'un constructeur de copie pour ma classe CLayer capable de réallouer les zones mémoire de chacun des CLED* de ma liste.
Voici comment j'ai procédé :

Code:

---

1 2 3 4 5 6 7

CLayer::CLayer(const CLayer &Source) { *this = Source; CLayer::iterator tmpIt; for (tmpIt=begin(); tmpIt!=end(); tmpIt++) *tmpIt=new CLED(**tmpIt); }

---

Cette méthode fonctionne sans problème, mais je me posais tout de même une question : admettons que je souhaite faire de l'itération sur Source et boucler des this->push_back(...), à savoir que le compilateur n'autorise pas à faire

Code:

---

tmpIt=Source.begin()

---

(apparement parce que Source est const et tmpIt ne l'est pas...), comment aurais-je dû m'y prendre ?

Merci de m'aider à satisfaire ma curiosité ;)

Rémy

Il suffit que tmpIt soit déclaré constant.

Code:

---

CLayer::const_iterator tmpIt;

---

A part ça, je ne vais pas pinailler sur le fait que tu dérives de std::list, mais c'est généralement une mauvaise idée, et mal fait aussi.

Merci bien, je ne connaissais pas const_iterator :)
Pour la classe qui dérive de std::list, on m'a déjà dit maintes fois que c'était une mauvaise idée...
A vrai dire le seul argument que j'ai pour justifier cet héritage c'est d'avoir définitivement besoin de surcharger les opérateurs de la STL pour mes propres besoins, et puis j'ai déjà codé plusieurs milliers de lignes comme ça sans m'embrouiller les pinceaux c'est le principal ;)

Merci encore !!

Rémy