Discussion :

[italiasky]

Bonjour,

J'aimerais avoir vos avis, vos conseils concernant les bonnes habitudes à adopter pour la programmation en C++...

Voila la situation, dans une classe, j'ai une liste (un vector) d'objets, pour ne pas mettre les objets eux même dans le vecteur ce qui serait plus lourd, je mets les pointeurs...

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
 
class monobjet
{
private:
  vector<monobjet2*> mesobjets;
 
public:
// La si je ne me trompe pas, un nouvel objet vector est créé, rempli et renvoyé ?
vector<monobjet2*> getMesobjets(void) const;
// Référence ? accès direct à mon objet vector de ma classe ?
vector<monobjet2*>& getMesobjets(void) const;
 
// Dangereux de mettre un setter si on perd l'autre vector du coup... ?
void setMesobjets(monbobjet2*);
}

Après, quelles méthodes pensez-vous qu'il soit judicieux d'avoir dans cette classe sans craindre à l'intégrité du vector, des fuites de mémoires, etc...

Un setter et getter qui renvoie mon attribut vector, en référence ?
un accès direct au objets du vector ?

Enfin bref, comment vous codez vous d'habitude dès que vous avec comme ca des conteneurs, vous faites bien des conteneurs de pointeurs et non directement d'objets ?
Mais après, quels accès faites vous ?

Merci
++

[dtrosset]

Bonjour,

Eh bien non, justement. Pour éviter les problèmes de fuite de mémoire, ou d'utilisation de pointeurs sur des objets qui ont été supprimés, je mets directement les objets dans le conteneur.

Je ne sois pas bien sûr que ta justification "ce qui serait plus lourd" soit vraiment pertinente ...

Didier

[3DArchi]

Effectivement, en général, j'utilise les conteneurs directement avec l'objet et non un pointeur sur l'objet. Ensuite, les accesseurs si besoin retourne une référence (const au besoin).
Si d'aventure, une bibliothèque m'oblige à l'utilisation de pointeurs, alors je les encapsule dans des choses comme boost::shared_ptr, qui m'évite de me poser des questions (et de faire de bugs)!

[Mat007]

Salut,

Mais après, quels accès faites vous ?

Aucun.
Le vecteur est un détail d'implémentation et seul 'monobjet' (qui devrait s'appeler 'maclasse' au passage) doit être au courant de son existence.

MAT.

[Alp]

Effectivement, en général, j'utilise les conteneurs directement avec l'objet et non un pointeur sur l'objet. Ensuite, les accesseurs si besoin retourne une référence (const au besoin).
Si d'aventure, une bibliothèque m'oblige à l'utilisation de pointeurs, alors je les encapsule dans des choses comme boost::shared_ptr, qui m'évite de me poser des questions (et de faire de bugs)!

Ou alors ptr_container, de Boost toujours.

[italiasky]

Ok merci

Bonne journée