Vector et classe

Bonjour,
Alors voila mon probleme j'ai une classe B:

Code:

---

1 2 3 4 5 6 7

class B: { String id; public: B(String _id) : id(_id) {}; };

---

et une classe A

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21

class A: { std:vector<B> vB; public: A(); A(const A& a); std::vector<B> & getVector(); };   A::A () : vb(0) {} A::A(const A & a) { vb(a.getVector().size()); vb = a.getVector(); }   std::vector<B> & getVector() { return vb; }

---

Mon soucis est pour l'initialisation des vecteur dans le constructeur par defaut j'ai un message :

Citation:

---

no matching function for call to ‘B::B()’

---

Et mon constructeur par recopie a aussi des problèmes

Citation:

---

- passing ‘const A’ as ‘this’ argument of ‘std::vector<B>& A::getVector()’
discards qualifiers [-fpermissive]

---

Voila, toute aide est la bienvenue, j'espère avoir été clair.
Merci

Oui, il y a plusieurs problèmes.

Saches déjà que quand tu codes

Code:

---

1 2	vb(a.getVector().size());

---

tu demandes à créer un vecteur d'une certaine taille N qui contiendra N éléments B initialisés par défaut. Ce n'est donc possible que si ta classe B a un constructeur par défaut. Or la tienne n'en a pas. Tu as 2 solutions :
1. Ajouter un constructeur vide :

Code:

---

1 2

B() {};

---

2. Mettre un argument par défaut à ton constructeur qui initialise avec la chaine vide par exemple.

Code:

---

1 2	B(String _id = "") : id(_id) {};

---

Ensuite il me semble que pour ton constructeur par copie, tu t'embêtes pour rien. Utilises assign Pas besoin de passer la taille etc..
Si tu as 2 vecteurs std::vector<int> vi1 et std::vector<int> vi2 déjà remplis, vi1 de taille 100 et vi2 de taille 50 par exemple, alors tu peux faire

Code:

---

1 2	vi1.assign(vi2.begin(),vi2.end());

---

Tu copieras le contenu de vi2 dans vi1, pas besoin de se soucier de la taille.

Merci pour ta réponse,
J'ai toujours un problème avec mon constructeur par recopie, j'ai l'impression que mon getVector ne me renvoie pas le vector

j'ai donc :

Code:

---

1 2 3 4 5

A::A(const A & a) { vb.assign(a.getVector().begin(), a.getVector().end() ); }

---

le message d'erreur commence par : passing ‘const

J'ai donc modifier ma fonction getVector par :

Code:

---

1 2	std::Vector<B> &A::getVector() const;

---

Je pensais que en appelant getVector sur un objet const dans le constructeur par recopie aller poser des problèmes donc j'ai mis la fonction en const pour signaler qu'elle ne modifiait rien.
Seulement maintenant j'ai un message d'erreur dans la fonction getVector :

Citation:

---

invalid initialization of reference of type ‘std::vector<B>&’ from expression of type ‘const
std::vector<B>’

---

Mon raisonnement est sûrement faux, ou j'ai fais une faute d’inattention.
En tout cas merci pour ta précédente réponse

Tu dois renvoyer une référence constante sur ton vecteur dans ta méthode getVector() . (et en passant y a t'il une raison pour qu'une telle fonction existe?) Pour ton constructeur, tu peux directement utiliser la liste d'initialisation + le fait que la visibilité private concerne les classes et non pas les instances:

Code:

---

A(A const& other):vB(other.vB) {}

---

Salut,
Plusieurs remarques :

1. B(String _id) : id(_id) {} : Attention au nom commençant par _ => à éviter. Cf F.A.Q. : Quels sont les identificateurs interdits par la norme ?
   
2. Comme dit ci-dessus, préfère les listes d'initialisation : F.A.Q. Mes constructeurs doivent-ils utiliser les listes d'initialisation ou l'affectation ?
   
   Code:

   ---

   A::A () : vb {}

   ---

3. Inutile de définir une constructeur de copie. Le compilateur va en générer un automatiquement et celui-ci devrait être suffisant au regard de ta classe. F.A.Q. : Quand dois-je définir un constructeur par copie ?
   
4. Certes getVector doit être constant, mais il devrait aussi retourner une référence constante : std::Vector<B> const & getVector() const;
5. Mais en fait, il est fortement déconseillé de pourvoir sa classe d'accesseur/mutateur 'brut'. Ils brisent l'encapsulation et exposent des détails d'implémentation.
   
6. question de style donc complètement discutable : je préfère d'abord mettre les membres publiques dans une classe puis terminer par les parties privées. Cela permet d'avoir dès le début de lecture de la classe son interface publique.

Au final, ta classe peut être drastiquement réduite :

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 23 24 25 26 27 28 29 30 31

#include <string> class B { public: B(std::string id_) : id(id_) {}   private: std::string id;   };     #include <vector> class A { public: A(); private: std::vector<B> vb; };   A::A ():vb() {}   int main() { A a1; A a2;   a2 = a1; return 0; }

---