Discussion :

[Avatar36]

Bonjour,

J'essaye en vain de faire un accesseur pour un vecteur d'une classe. J'ai fait des accesseurs pour tous les attributs de ma classe mais je n'arrive pas à faire celui du vecteur. J'utilise Qt mais à prioris ça n'a pas d'incidence sur mon problème. Je précise que j'ai cherché déjà plein d'exemples sur le net et qu'ils ne marchent pas chez moi, inutile de m'accabler de reproches.

voici la déclaration dans le .h:

Code :

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

1
2
3
std::vector<QString> genres[5];
 
const std::vector<QString> &getGenres();

et dans le .cpp:

Code :

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

1
2
3
4
const std::vector<QString>& Film::getGenres()
{
    return genres;
}

C'est la copie conforme d'un des exemples que j'ai trouvé mais il me renvoie l'erreur: invalid initialization of reference of type 'const std::vector<QString>&' from expression of type 'const std::vector<QString>*'
Au passage, est-ce que vous pouvez me donner le code pour faire un Set pour modifier les valeurs du vecteurs?
Merci d'avance.

Cordialement

[Ehonn]

Bonjour

Remplace

Code :

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

std::vector<QString> genres[5];

par

Code :

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

std::vector<QString> genres;

Le deuxième est un tableau (taille dynamique) de QString.
Le premier est un tableau (taille fixe à 5) de tableau (taille dynamique) de QString.

Si ta méthode ne modifie pas l'objet, tu "dois" l'indiquer :
Remplace

Code :

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

const std::vector<QString> &getGenres()

par

Code :

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

std::vector<QString> const & getGenres() const

et

Code :

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

const std::vector<QString> &getGenres()

par

Code :

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

std::vector<QString> const & Film::getGenres() const

PS: Utilise la balise [CODE]

[koala01]

Salut,

Maintenant que l'on t'a donné la solution, je tiens quand meme à insister sur le fait que, à mon sens, tu ne devrais pas prévoir un accesseur sur ton vecteur de Qstring, pas plus que tu ne devrais prévoir un mutateur (que je subodore que tu as aussi prévu) d'ailleurs!

Ce que tu aurais très largement intérêt à faire, c'est de réfléchir aux services que tu es en droit d'attendre de la part des ta classes qui sont en rapport avec la présence de ce tableau.

Car il existe une règle érigée en loi qui s'appelle la loi de Déméter qui nous dit que, si un objet de type A (ta classe Film) utilise en interne un objet de type B(le std::vector<QString>), l'utilisateur de la classe A (la classe Film) ne devrait pas avoir à connaître la classe B pour pouvoir utiliser son objet de type A.

C'est encore plus vrai pour tout ce qui a trait aux conteneurs : Typiquement, il ne devraient servir qu'au maintient en interne des données utilisées par la ta classes

[ternel]

Rien n'empêche, par contre, de prévoir un itérateur... si c'est dans le sens réel de la classe. (en ce cas, prévoir a la fois begin() et end(), et un ++)

L'itérateur masque le vecteur, sans cacher la sémantique de collection de genres.

[PilloBuenaGente]

Bonsoir,

L'itérateur masque le vecteur, sans cacher la sémantique de collection de genres.

Excusez moi, mais en quoi l'iterateur masquerait-il le vecteur ? Pour appeler un iterateur, on est obliger de specifier son conteneur non ? (à part peut être pour le c++11 avec auto ?)

[Bousk]

Non parce que tu itères sur les objets contenus dans la collection, quelle que soit la colletion.