Méthode qui retourne une référence à un conteneur

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

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

Citation:

---

Envoyé par Laurent Gomila

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

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

tu peux le renvoyer en référence const

Code:

---

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

---

Citation:

---

Envoyé par nikko34

tu peux le renvoyer en référence const

Code:

---

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* ???

Citation:

---

Envoyé par tnarol

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.

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.