Discussion :

[mazertys17]

Bonjour.

C'est un peu bateau, mais je ne parviens pas a copier un vector de string correctement :

j'ai un objet qui est censé en renvoyer un :

Code :

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

1
2
3
4
5
6
 
 
const std::vector < std::string >  Object::getVectorString()
{
    return _vectorString ;
}

ou encore dans le constructeur de copie :

Code :

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

1
2
3
 
 
_vectorString = other._vectorString ;

Mon code est-t-il mauvais ?

Merci si vous pouvez m'aider .

[jo_link_noir]

Bonsoir,

Je ne comprends pas la relation entre les 2 codes qui sont -hors contexte- bon. Quel est l'erreur ? Pourrais-tu fournir un exemple minimaliste ?

Il y a cependant 3 choses qui me choque:
- getVectorString() est visiblement un getter, il devrait être const.
- const dans le retour est de trop ou il manque une référence.
- x = y n'est pas un appel de constructeur mais une affectation. Le constructeur c'est x(y) dans la liste d'initialisation.

[mazertys17]

Merci pour ta réponse, jo.

getVectorString() est visiblement un getter, il devrait être const.

en effet.

Le constructeur c'est x(y) dans la liste d'initialisation.

ah ?
En fait c'est dans mon constructeur de copie :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
 
 
OBJECT::OBJECT ( OBJECT const& other )
{
    _name = other._name ;
    _type = other._type ;
 
    _variable = other._variable ;
 
    _MONVECTOR = other._MONVECTOR ; // c'est ici que ca a pas l'aire de marcher correctement
 
}

Le but étant simplement de copier l'objet en construisant un nouveau

Code :

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

1
2
3
4
5
6
 
 
OBJECT nouvelObject ( monObjectQuiSertDeModel ) ;
OBJECT *nouvelObjectPTR ( new OBJECT ( monObjectQuiSertDeModel ) ) ;
 
etc...

En tout cas, jusque là, tout fonctionne. Simplement, quand je tente de copier ( surement bêtement ) un std::vector, il n'a l'air d'en copier que le début ( sauf erreur )

[Invité]

Bonjour,

Le constructeur c'est x(y) dans la liste d'initialisation.

Mes constructeurs doivent-ils utiliser les listes d'initialisation ou l'affectation ?
Mais même comme fait actuellement, ton std::vector devrait être parfaitement copié. Tu dois avoir un autre soucis quelque part, et un ECM aiderait bien.

[mazertys17]

ton std::vector devrait être parfaitement copié

Autant pour moi, en effet. Cela fonctionne.

C'est bon pour le constructeur de copie. Maintenant j'aimerais pouvoir utiliser le vector contenue dans un class, depuis une autre class.
Je dois mal m'y prendre...je suppose que le mieux est la référence, mais comment faire avec un vector ?

car j'essaye ceci sans résultat : (objet qui envoi le vector )

Code :

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

1
2
3
4
5
6
 
 
std::vector < Object >& ObjetQuiEnvoiLaReferenceDuVectorPourLeFaireUtiliserParUnAutreObjet::renvoiLeVector ()
{
    return _vector_Object;
}

[jblecanard]

Il va falloir relire tes cours Simplement :

Code :

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

1
2
3
std::vector < Object >& ObjetQuiEnvoiLaReferenceDuVectorPourLeFaireUtiliserParUnAutreObjet::renvoiLeVector () {
    return _vector_Object;
}

[mazertys17]

est-ce la longueur de mon impressionnant nom d'objet qui vous fait sourir
ou pas..?

ah oui je vois...dsl, j'ai mis un "&" en return par mégarde ( et amateurisme je l'avoue )

En fait il compile, d'ailleurs, mais dès que le vector en vient à être updaté, il plante. ( le debugger confirme que c'est bien a l'utilisation du vector )..

Votre réponse peut elle me laisser penser que c'est censé marcher ?