Discussion :

[Chii-san]

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 :

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

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 :

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

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

[Laurent Gomila]

Il suffit que tmpIt soit déclaré constant.

Code :

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

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.

[Chii-san]

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