Discussion :

[tnarol]

Salut,

J'aimerais avoir votre avis sur la validité d'une méthode de classe qui retournerait une référence de conteneur (qui serait un de ses attributs), par exemple :

vector<Child*>& getChildren();

J'ai le sentiment que ce n'est pas très correct mais je ne sais pas dire pourquoi en dehors du fait que ça viole l'encapsulation.

Merci

[Laurent Gomila]

Pour quelle raison as-tu besoin de donner un accès en écriture à ce tableau ?

[tnarol]

Pour quelle raison as-tu besoin de donner un accès en écriture à ce tableau ?

Le but serait d'éviter d'avoir un accès plus immédiat (et donc probablement plus rapide) aux données que si je devais appeler la classe en lui demandant à chaque fois combien elle a d'éléments dans son vector (GetChildrenCount()) puis en lui demandant plein d'éléments de son vecteur par le biais d'indices (GetChild(i)).

[Sylvain Togni]

Ça brise pas mal l'encapsulation, en effet.

Une solution parfois c'est d'avoir une interface à base d'itérateurs, avec des fonctions begin(), end(), ... Comme ça au moins tu peut changer de conteneur de façon transparente.

[nikko34]

tu peux le renvoyer en référence const

Code :

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

1
2
 
const vector<Child*> & getChildren();

[tnarol]

tu peux le renvoyer en référence const

Code :

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

1
2
 
const vector<Child*> & getChildren();

Ca veut die que je ne pourrai pas modifier le vector ? Mais quand même les instances référencées par Child* ???

[nikko34]

Ca veut die que je ne pourrai pas modifier le vector ? Mais quand même les instances référencées par Child* ???

c'est peut être pas une bonne idée finalement

Tu es sûr que ton traitement tu ne peux pas le faire dans la classe qui contient le vector? Sinon il y a des fois où on a pas le choix, même si ça brise l'encapsulation.

[Laurent Gomila]

GetChild et GetChildrenCount, si vraiment ça te bouffe tes perfs (question que je ne me poserais pas personnellement avant d'avoir utilisé un profiler...), tu peux les mettre inline, ainsi tu n'auras aucun surcoût.