Problème de vector

Version imprimable

Voir 40 message(s) de cette discussion en une page

Problème de vector

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:

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:

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

10/07/2006, 11h35
Matthieu Brucher

Et c'est quoi l'erreur ?
10/07/2006, 11h47
SnakemaN

Et avec une boucle bien propre avec un iterator ? ca serait pas mieux ?
10/07/2006, 11h52
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 :mouarf:
10/07/2006, 11h55
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 ?
10/07/2006, 11h58
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
10/07/2006, 11h59
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 ?
10/07/2006, 12h03
SnakemaN

Citation:

Envoyé par 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 ?

Arf grilled

je plussoie :D
10/07/2006, 12h09
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(); )
10/07/2006, 12h11
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.
10/07/2006, 12h12
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 ?
10/07/2006, 12h19
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????
10/07/2006, 12h21
Matthieu Brucher

end() + 1 ne peut pas exister puisque end() pointe déjà après la fin du vecteur.
Et quand fais-tu itDest = ... ?
10/07/2006, 12h23
Sachiel31

J'incrément mon itérateur apres avoir insérer mon élément (logique non???)
10/07/2006, 12h25
Eusebe

push_back ne te suffit pas pour l'insertion ?
10/07/2006, 12h28
Sachiel31

J'ai essayé la méthode push_back, mais cela ne marche pas.
10/07/2006, 12h29
SnakemaN

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

et ben c vraiment chelou tout ca ...

Citation:

Envoyé par Sachiel31

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

Tu veux dire après avoir fait ça ?
Code:

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.

10/07/2006, 12h36
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 :mur:

et en faisant un truc du genre :
Code:

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

10/07/2006, 12h39
Matthieu Brucher

Ben écoute, tu me montres 2 insertions et à la fin le tableau est de taille 2, je ne vois pas où est le problème !
10/07/2006, 12h44
SnakemaN

Citation:

Envoyé par Miles

Ben écoute, tu me montres 2 insertions et à la fin le tableau est de taille 2, je ne vois pas où est le problème !

+1
vu que le troisieme (celui la -> monBordereau->conteneurDest.insert(monBordereau->itDest, monBordereau->theDestinataire); ) ne marche pas

Citation:

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

ben la il manque un argument à ton insert()... cf mon deuxieme lien

Citation:
Envoyé par SnakemaN
et en faisant un truc du genre :
Code:

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 }
(Tu peux utiliser la balise code stp ?)
Je ne pense pas que l'on puisse faire ça : l'itérateur ne doit plus être valide après l'insertion.
Et en plus, je n'en vois pas l'intérêt : pourquoi insérer le même élément à tous les emplacements d'un vector ?

Sachiel31, tu pourrais nous mettre ton code qui permet de regarder ce que contiens le vector ?

10/07/2006, 13h14
Sachiel31

J'ai refais un test, apparement il prend bien en compte l'insertion d'un nouvel élément puisque la taille de mon vector s'incrémente à chaque fois.

Le problème c'est que je n'arrive pas à afficher le contenu de mon vecteur. J'ai une erreur me disant que l'operateur "<<" n'est pas défini pour afficher des objets de types Destinataires.

Comment on fait déja la surcharge de l'opérateur???
10/07/2006, 13h16
Eusebe

Va voir la :faq: :
http://c.developpez.com/faq/cpp/?page=surcharge
10/07/2006, 13h24
SnakemaN

Citation:

Envoyé par Eusebe

(Tu peux utiliser la balise code stp ?)
Je ne pense pas que l'on puisse faire ça : l'itérateur ne doit plus être valide après l'insertion.
Et en plus, je n'en vois pas l'intérêt : pourquoi insérer le même élément à tous les emplacements d'un vector ?

Sachiel31, tu pourrais nous mettre ton code qui permet de regarder ce que contiens le vector ?

a non je ne voulais pas dire d'inserer le meme, c'etait une idée

et puis la balise CODE je trouve qu'elle n'est pas pratique a cause des scrolling verticaux (surtout) ils sont un peu chiant on voit pas bien le code quand il y a peu de ligne (sous IE 6.0.2 en tout cas):roll:
10/07/2006, 13h26
Eusebe

Citation:

Envoyé par SnakemaN

(sous IE 6.0.2 en tout cas):roll:

[Mode Troll]Ben passes sous FF alors ! :P [/Mode Troll]

Citation:

Envoyé par Eusebe

Va voir la :faq: :
http://c.developpez.com/faq/cpp/?page=surcharge

le mieux c'est de faire un fichier operator.cpp par ex
avec en include ton destinataire.cpp

et des surcharges comme :
Code:

1 2 3 4 5 6 7 8 9 10 11 ostream& operator << (ostream & os, destinataire &myDest) { os << ...... return os; } istream& operator >> (ostream & is, destinataire &myDest) { //tu peux utiliser ds getline() c'est bien pratique return is; }

10/07/2006, 14h25
Matthieu Brucher

un const & serait quand même mieux...
10/07/2006, 14h46
SnakemaN

Citation:

Envoyé par Miles

un const & serait quand même mieux...

aussi aussi

Citation:

Envoyé par Eusebe

[Mode Troll]Ben passes sous FF alors ! [/Mode Troll]

J'aimerai bien figures-toi mais c'est un PC d'entreprise (pas du tout d'informatique) alors....:arf:

Alors là je sais pas pourquoi ca ne marche pas.
J'ai déclaré ma fonction de surcharge de l'opérateur << comme suit :
Code:

1 2 3 4 5 6 7 8 9 10 11 12 ostream& operator<< (ostream &fluxSortie, const Destinataire &dest){ fluxSortie << dest.matricule << endl; fluxSortie << dest.libelleCptD << std::endl; fluxSortie << dest.nomBanqueD << std::endl; fluxSortie << dest.codeGD << std::endl; fluxSortie << dest.numCptD << std::endl; fluxSortie << dest.paieNet << std::endl; fluxSortie << dest.libelle << std::endl; fluxSortie << dest.codeBD << std::endl; return fluxSortie; };
et j'obtiens cette erreur :
error C2804: binary 'operator <<' has too many parameters

J'aimerais bien comprendre ce qui ne lui plaît pas :question: :!:

10/07/2006, 15h07
Matthieu Brucher

C'est une fonction libre, pas une fonction membre, operator<<, puisque le premier argument n'est pas ta classe, mais un flux. Tu as bien regardé la FAQ ?
10/07/2006, 15h11
Sachiel31

Oui j'ai bien regardé. J'ai également trouvé d'autres exemples de cette surcharge et c'est écris de la meme façon.

Le seul poitn qui diffère par moment, c'est que certains déclarent :
friend ostream& opertor<<(ostream &fluxSortie, const Destinataire &dest) {...}
10/07/2006, 15h19
Matthieu Brucher

"friend", c'est dans la déclaration de la classe au cas où cette fonction non membre a besoin d'accéder aux variables privées de la classe. Dans le cas général, il n'y aucune allusion à cette fonction libre dans la classe.
10/07/2006, 15h28
Sachiel31

Je sais bien mais pourquoi Visual me met une erreur sur la fonction opertor<< :piou:

*ca craint si je me défoule sur le PC, c'est pas le mien :?*
10/07/2006, 15h42
Matthieu Brucher

Elle est déclarée où ta fonction ?
10/07/2006, 15h45
Sachiel31

Je la déclare dans le .h de ma classe Destinataire
10/07/2006, 15h47
Matthieu Brucher

Et elle est où par rapport à ta classe ? De plus, si le prototype que tu as donné est celuiq ue tu utilises, enlève le using namespace std dans ton fichier d'en-tête ;)

Voila la déclaration de ma classe avec la surcharge :

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
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
#ifndef _DESTINATAIRE_H_
#define _DESTINATAIRE_H_
 
#include <string>
#include <iostream>
#include <ostream>
 
 
class Destinataire
{
private:
	std::string matricule;
	std::string libelleCptD;
	std::string nomBanqueD;
	std::string codeGD;
	std::string numCptD;
	std::string paieNet;
	std::string libelle;
	std::string codeBD;
 
public:
	Destinataire();
 
	void afficheD(int n);
 
	const std::string get_matricule() const;
	void set_matricule(std::string value);
 
	const std::string get_libelleCptD() const;
	void set_libelleCptD(std::string value);
 
	const std::string get_nomBanqueD() const;
	void set_nomBanqueD(std::string value);
 
	const std::string get_codeGD() const;
	void set_codeGD(std::string value);
 
	const std::string get_numCptD() const;
	void set_numCptD(std::string value);
 
	const std::string get_paieNet() const;
	void set_paieNet(std::string value);
 
	const std::string get_libelle() const;
	void set_libelle(std::string value);
 
	const std::string get_codeBD() const;
	void set_codeBD(std::string value);
 
	// surcharge de l'operateur <<
	ostream& operator<< (ostream &os, const Destinataire &dest){
		os << dest.matricule << endl;
		os << dest.libelleCptD << std::endl;
		os << dest.nomBanqueD << std::endl;
		os << dest.codeGD << std::endl;
		os << dest.numCptD << std::endl;
		os << dest.paieNet << std::endl;
		os << dest.libelle << std::endl;
		os << dest.codeBD << std::endl;
 
		return os;
	};
};
 
#endif // _DESTINATAIRE_H_

et les erreurs générées :

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
--------------------Configuration: Bordereau - Win32 Debug--------------------
Compiling...
Bordereau.cpp
c:\...\bordereau\destinataire.h(63) : error C2804: binary 'operator <<' has too many parameters
c:\...\bordereau\destinataire.h(63) : error C2333: '<<' : error in function declaration; skipping function body
Destinataire.cpp
c:\...\bordereau\destinataire.h(63) : error C2061: syntax error : identifier 'ostream'
c:\...\bordereau\destinataire.h(63) : error C2805: binary 'operator <<' has too few parameters
c:\...\bordereau\destinataire.h(63) : error C2333: '<<' : error in function declaration; skipping function body
main.cpp
c:\...\bordereau\destinataire.h(63) : error C2804: binary 'operator <<' has too many parameters
c:\...\bordereau\destinataire.h(63) : error C2333: '<<' : error in function declaration; skipping function body
Error executing cl.exe.
 
Bordereau.exe - 7 error(s), 0 warning(s)

10/07/2006, 15h52
Médinoc

L'opérateur doit être HORS de la classe.

Voir 40 message(s) de cette discussion en une page