Discussion :

[Keyser]

Bonjour,

J'ai besoin pour mon programme de stocker une liste de points. Pour cela, j'utilise CList et CPoint comme suit :

typedef CList<CPoint,CPoint> Listepoints;

J'utilise ce type dans une classe pour une des variables membres. Seulement quand je crée une fonction qui permet de retourner une variable de type ListePoints, j'ai une erreur à la compilation : No copy constructor available.

Voilà la fonction en question :
Listepoints m_MaListe;
.....
.....

Listepoints CMyClass::GetMaListe()
{
return m_MaListe;
}

J'ai cherché dans l'aide de Visual C++ mais en vain, je ne m'en sors pas. Pourriez-vous m'aider ?

[bigquick]

Alors, si je ne me trompe pas ...

avec

Code :

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

1
2
3
4
Listepoints CMyClass::GetMaListe() 
{ 
return m_MaListe; 
}

tu comptes retourner une liste de points .... pas juste un pointeur... donc le compilateur doit savoir comment copier ta liste de points (m_MaListe) pour pouvoir en faire une copie et la retourner.
Il faudrait donc (je pense) creer une classe pour la liste de points, et redéfinir dedans l'opérateur de copie (=), afin qu'il sache comment s'y prendre.

Mais est-ce que c'est bien ce que tu veux faire (retourner une copie à chaque fois), sachant que si c'est juste pour y accéder c'est bien long...
Un pointeur ou une référence suffirait:

Code :

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

1
2
3
4
Listepoints* CMyClass::GetMaListe() 
{ 
return &m_MaListe; 
}

ou

Code :

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

1
2
3
4
Listepoints& CMyClass::GetMaListe() 
{ 
return m_MaListe; 
}

Voilà voilà ... mais attention dans ces cas là à ne pas modifier la liste dans la fonction qui la récupère ( *GetMaListe() pour le pointeur ou GetMaListe() pour la référence)... sinon l'original (m_MaListe) sera modifié aussi.

Bah euh dis moi si ça marche

A+


bigquick

[Luc Hermitte]

Pour pas que l'appelant modifie ce qui est retourné (genre une donnée privée), il y le spécifieur const qui permet justement d'empêcher cela. Retourner un pointeur n'est pas toujours une bonne idée, cela peut laisser croire qu'il faille appeller delete dessus.
=>

Code :

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

1
2
3
4
const Listepoints & CMyClass::GetMyList() const
{ 
    return m_MyList; 
}

Parenthèse, std::list (qui fait parti du C++ contrairement à CList) défini bien un constructeur de recopie. Par contre, il va y avoir une recopie de tous les chainons (contrairement à des solutions plus complexes essayant de mettre en oeuvre des move semantics -- je ne sais plus si VC++ supporte mojo).

[Keyser]

Merci pour ces réponses !

Après réflexion je me suis rendu compte que retourner un pointeur de la liste convenait plus avec ce que je voulais faire. Donc merci de m'avoir éclairé sur le sujet !

Et au passage je n'ai plus de message d'erreur !

@+

[tut]

et au passage, la classe CList est un peu une m*$*%ù comparée à celle de la librairie standard C++ fournit par Visual 6.0, d'ailleurs. Si tu as des problèmes de performance sur l'ajout/suppression d'éléments, essayes d'utiliser std::list à la place de CList.
Perso, ça a été le jour et la nuit en ce qui me concerne, chiffres (que je n'ai plus) à l'appuis.
Evidemment, c'était dans un contexte et pour des besoins particuliers, mais il y avait un facteur 100 sur le temps d'ajout d'élément entre CList et std::list, temps qui augmentait pour CList et qui restait constant pour std::list.
A bon entendeur...