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...