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 ?

[mazertys17]

ok...desolé, je suis vraiment a côté de la plaque parfois...Faut que je see ++...Merci en tout cas pour votre réponse. Le bug vient d'autrepart.

Toujours à propose de std::vector :

Je suis bien souvent agacé avec les vectors, lors de la suppression d'elements :
lorsque je supprime des éléments au fur et a mesure, lorsqu’il arrive au dernier, il plante.

Ma technique, (pour l'instant ) , est sans doute branquignole, mais fonctionne...Elle consiste a rajouter un élément en begin du vector a la construction/initialisation de l'objet.

Code :

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

1
2
3
4
5
6
7
 
 
    Objet modelBidon ;
    modelBidon.setPosition ( -10000, -10000 ) ; //là je suis sur qu'il ne sera pas génant, par ex
    modelBidon.setLife ( "invulnerable" ) ; // ou mieux, je le rend carrément indestructible...au moins je suis tranquille.
 
    _monVector.push_back( modelBidon ) ;

et je détruits les objets ainsi :

Code :

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

1
2
3
4
5
6
7
8
9
10
 
 
     for ( vector < Objet > ::iterator  it = _monVector.being() ; it != _monVector.end()  ; it ++ ) //il me semble qu'auto ne marche pas dans ce cas.
    {
             if ( it -> conditionX() )
             {
                        _monVector.erase( it ) ;
                        it = monVector.begin() ;
             }
    }

Ca marche, donc...mais c'est pénible, inutile, et prend un peu de ressources pour rien...Surtout que ça rajoute parfois des complications dans le code...

Mon problème vous paraît-il normal ?
Connaissez-vous un methode pour y remédier, à tout hasard ?

Merci si vous pouvez m'aider

[jo_link_noir]

[FAQ] Comment supprimer correctement des éléments d'un conteneur ?

[mazertys17]

Merci !!