Discussion :

[Sachiel31]

Bonjour,

J'ai créé une classe Bordereau qui contient plusieurs objets de types Destinataire, donc un vectior<Destinataire>.

Mon problème est que j'arrive à remplir le 1er élément du vecteur avec un destinataire, mais pour les suivants cela ne marche pas.

Déclaration de mon vecteur :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
class Bordereau
{
private:
	std::string date;
	std::string devise;
	std::string montantTotal;
 
public:
	std::vector<Destinataire> conteneurDest;
	std::vector<Destinataire>::iterator itDest;
 
	Destinataire theDestinataire;
 
	Bordereau();
 
	void afficheB();
...
};

Utilisation :

Code :

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

1
2
3
4
5
	monBordereau->theDestinataire.afficheD(n);
	monBordereau->conteneurDest.insert(monBordereau->conteneurDest.end(), monBordereau->theDestinataire);
	monBordereau->conteneurDest.push_back(monBordereau->theDestinataire);
	monBordereau->conteneurDest.insert(monBordereau->itDest, monBordereau->theDestinataire);
	monBordereau->itDest = monBordereau->conteneurDest.end();

Je pense avoir essayer plein de solutions mais je bloque

Merci de votre aide

[Matthieu Brucher]

Et c'est quoi l'erreur ?

[SnakemaN]

Et avec une boucle bien propre avec un iterator ? ca serait pas mieux ?

[Sachiel31]

Comme je l'ai dit le premier destinataire est stocké dans le vecteur mais pas les suivants.

Il me garde le contenu du premier élément mais change son adresse. C'est comme si le premier élément était supprimé (en copiant le contenu) et qu'un nouveau était créé avec le contenu de l'ancien.

Voila le contenu de mon vecteur au premier passage :
- monBordereau
-> conteneurDest
=> _FIRST 0x003259e8 (avec tous les champs rensignés)
=> _LAST 0x003225a68 (vide)
=> _END 0x003225a68 (vide)
-> itDest 0x003225a68

Voila le contenu de mon vecteur au deuxieme passage :
- monBordereau
-> conteneurDest
=> _FIRST 0x00326448 (avec le contenu du 1er element)
=> _LAST 0x00326548 (vide)
=> _END 0x00326548 (vide)
-> itDest 0x00326548


Voila donc mon problème

[Matthieu Brucher]

Bon, déjà le troisième est faux, ça vaut quoi après une insertion monBordereau->itDest ? ce n'est plus valide, donc à jeter !
Combien t'affiche un size() du vecteur ?

[SnakemaN]

Et pourquoi il t'affiche l'adresse du contenu et non pas le contenu ? c'est voulu ?

et puis pourquoi tu n'incremente pas l'iterateur ? tu réecris sur le meme la

[Eusebe]

Si tu stoques directement l'objet, c'est normal que l'adresse change parce qu'il en fait une copie. Si tu ne le veux pas, je crois qu'il faut que tu stoques des pointeurs et non des objets.

Sinon, tu fais comment pour parcourir ton vector ?

[SnakemaN]

Si tu stoques directement l'objet, c'est normal que l'adresse change parce qu'il en fait une copie. Si tu ne le veux pas, je crois qu'il faut que tu stoques des pointeurs et non des objets.

Sinon, tu fais comment pour parcourir ton vector ?

Arf grilled

je plussoie

[Sachiel31]

Quand je fais un size() du vecteur il m'affiche 2.

Le contenu est affiché à l'interieur du _FIRST.

Mon iterateur est incrémenté puisqu'après l'ajout le lui donne l'adresse du dernier élément (monBordereau->itDest = monBordereau->conteneurDest.end(); )

[Matthieu Brucher]

Donc il insère bien 2 éléments, le troisième ne peut pas être inséré puisque l'itérateur n'est plus valide.

[SnakemaN]

et ben voila ta réponse tu lui donne l'adresse du dernier et non pas du dernier +1

d'ailleur c'est louche qu'il t'en crée un deuxieme car il devrait ecraser le premier directement, non ?

[Sachiel31]

J'ai modifié l'incrémentation de l'itérateur : je lui passe end() +1
Mais du coup quand je fais :
monBordereau->conteneurDest.insert(monBordereau->itDest, monBordereau->theDestinataire);

il me met un access violation


Mais normalement end() renvoi un itérateur qui pointe apres le dernier élément, non????

[Matthieu Brucher]

end() + 1 ne peut pas exister puisque end() pointe déjà après la fin du vecteur.
Et quand fais-tu itDest = ... ?

[Sachiel31]

J'incrément mon itérateur apres avoir insérer mon élément (logique non???)

[Eusebe]

push_back ne te suffit pas pour l'insertion ?

[Sachiel31]

J'ai essayé la méthode push_back, mais cela ne marche pas.

[SnakemaN]

oui bien sur un end() + 1 ne marchera pas car il n'existe pas encore...

et ben c vraiment chelou tout ca ...

[Matthieu Brucher]

J'incrément mon itérateur apres avoir insérer mon élément (logique non???)

Tu veux dire après avoir fait ça ?

Code :

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

1
2
3
4
5
	monBordereau->theDestinataire.afficheD(n);
	monBordereau->conteneurDest.insert(monBordereau->conteneurDest.end(), monBordereau->theDestinataire);
	monBordereau->conteneurDest.push_back(monBordereau->theDestinataire);
	monBordereau->conteneurDest.insert(monBordereau->itDest, monBordereau->theDestinataire);
	monBordereau->itDest = monBordereau->conteneurDest.end();

J'ai indiqué que le troisième est faux car l'itérateur n'est plus valide après une insertion.

[Sachiel31]

Bon en oubliant que la 3eme solution est fausse, pourquoi un simple pushback ou le insert(monBordereau->conteneurDest.end()) ne marche pas???


Ca fait 2jours que je suis dessus je commence à buguer

[SnakemaN]

et en faisant un truc du genre :

Code :

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

1
2
3
4
5
6
for ( itDest=conteneurDest. begin ( ) ;
itDest !=conteneurDest. end ( ) ;
++itDest ) {
monBordereau->theDestinataire.afficheD(n); // tu affiches
monBordereau->conteneurDest.insert(itDest, monBordereau->theDestinataire); //tu inseres
}

bon apres tu adaptes car je ne sais pas ou ce situe ce code, dans une methode ta classe peut etre ?

sinon va voir la :

http://www.cppreference.com/

et surtout la :

http://www.cppreference.com/cppvector/insert.html

car je crois que tu n'utilise pas l'adresse (&) de ton destinataire et tu n'utilise pas de "const" on plus pour ton iterator