problème au surcharge d'opérateur =

Version imprimable

Bonjour,
j'ai une structure comme suit

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
 
 
class P
public:
	char* DataFile;
	int nbter;
	int nbser;
	int nbf;
	int nbgp;
	vector<S *> ser;
	vector<F *> f;
	vector<T *> ter;
	vector<G *> grp;
	Cellule* cell;
public:
	P(char* df);
	P(const Pe& p);
	Properator=(const P &p);
	virtual ~P();
 
};

Et j'ai défini l'opérateur= comme ça

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
 
P P::operator=(const P &p)
{
	     nbter=p.nbter;
		nbsers=p.nbser;
		nbf=p.nbf;
		nbgp=p.nbgp;
		cell=p.cell;
	    ser.reserve(p.ser.size()); // optimize insertion
	    ter.reserve(p.ter.size());
	    grp.reserve(p.grp.size());
	    f.reserve(p.f.size());
 
	for (vector<S *>::const_iterator its = p.ser.begin();its != p.ser.end();its++)
	 ser.push_back(new S(**its));
	for (vector<F *>::const_iterator itf = p.f.begin();itf != p.f.end();itf++)
	    f.push_back(new F(**itf));
	.
        .
        .
    return *this;
}

Mais quand je declare 2 objet de type P: P a,b;
je modifie a, après a=b; l'affectation ca se produit pas a reste différente de b
Merci d'avance

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
 
P P::operator=(const P &p)
{
	     nbter=p.nbter;
		nbsers=p.nbser;
		nbf=p.nbf;
		nbgp=p.nbgp;
		//cell=p.cell;  //lol ,mark 
                         cell = new Cellule(*p.cell);
 
	    ser.reserve(p.ser.size()); // optimize insertion
	    ter.reserve(p.ter.size());
	    grp.reserve(p.grp.size());
	    f.reserve(p.f.size());
 
	for (vector<S *>::const_iterator its = p.ser.begin();its != p.ser.end();its++)
	 ser.push_back(new S(*its));//mark
	for (vector<F *>::const_iterator itf = p.f.begin();itf != p.f.end();itf++)
	    f.push_back(new F(*itf));//mark
	.
        .
        .
   ///aussi pour DataFile....
    return *this;
}

24/06/2010, 23h25
Luther13

Comment savez-vous que a et b sont différents ?
il faudrait aussi surcharger l'opérateur ==
25/06/2010, 07h56
jmv

Il faudrait d'abord vider les vecteurs de l'objet destination avant de faire les push_back() et du coup vérifier qu'on ne fait pas une autoaffectation (a=a) car dans ce cas on perd tout.
27/06/2010, 16h43
zougagh

j'affiche leur contenu et je vois qu'ils ont le même contenu
28/06/2010, 08h55
méphistopheles

et comment affiche-tu tes classes ?

ça n'a rien d'évident à priori.

D'autre part, vu ta métode de copie, les obets ne seront pas égaux puisque ter et grp ne sont pas alloué.

de plus, tu fais une copie avec des vecteurs de pointeurs donc les pointeurs finaux seront différents...

bref. Il faudrait nous dire qu'est-ce qui est différent et ne devrait pas l'être ainsi que comment tu constate cette différence .
28/06/2010, 13h07
zougagh
Au debut je charge ma structure de type P par un fichier dans le constructeur.
maintenant a de type P est rempli, quand je fais par exemple a=b. b est bien a le meme contenu que a.
J'affiche par exemple un element de 2 objet a et b
Code:

1 2 cout<<a.ser.at(1)->nbt<<"\n"; cout<<b.ser.at(1)->nbt<<"\n";
j'aurai :
2
2
Je modifie l'objet a par

Code:

a.ser.at(1)->nbt=5;

J'affiche les 2 objet jusqu'à maintenant c'est bien :
Code:

1 2 cout<<a.ser.at(1)->nbt<<"\n"; cout<<b.ser.at(1)->nbt<<"\n";
J'aurai :
5
2
Après je fais l'affectation a=b;
J'affiche a et b
Code:

1 2 cout<<a.ser.at(1)->nbt<<"\n"; cout<<b.ser.at(1)->nbt<<"\n";
j'obtiens
5
2
par contre moi je veux
2
2
car j'ai affecté le contenu de a à b
merci à vous tous et j'éspère que je suis claire cette fois
28/06/2010, 13h38
méphistopheles
je dois avouer que j'ai un peu de mal à comprendre comment ton compilo laisse passer ça, à moins que tu ne définisse un constructeur de S à partir d'un pointeur et dans ce cas, l'erreur vient du constructeur de S : en effet, si on regarde ton programme:

tu alloue la variable ser comme suit:
Code:

1 2 for (vector<S *>::const_iterator its = p.ser.begin();its != p.ser.end();its++) ser.push_back(new S(*its));//mark
ici, its est un itérateur de S* . On peut donc considérer ça comme un S** donc *its est un S* donc tu construit bien tes S à partir de S*. Deux solutions: ou tu n'a pas de constructeur à partir S* et là, je ne sais pas si ton compilo t'en fabrique un, ou , si tu en a un, alors, celui ci doit probablement se contenter de recopier le pointeur.

Jespère avoir été clair. Dans le cas contraire, il me faut la déclaration de S pour être plus clair.

Bonne chance.

voila la déclaration de S :

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
class S {
public:
	string id;
	vector<T*> ter;
	int nbt;
	vector<F *> fl;
	int nbfl;
	int nbgr;
	vector<G *> grp;
	int num;
	double RisqueInter;
public:
	S(char* i,int n);
	virtual ~S();
 
	void addTer(T*);
	void addFl(F *);
	void addGrp(G *);
	friend ostream& operator<<(ostream&, const S& );
};

Merci encore une fois de votre rèponse

Citation:
Envoyé par zougagh
voila la déclaration de S :
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 class S { public: string id; vector<T*> ter; int nbt; vector<F *> fl; int nbfl; int nbgr; vector<G *> grp; int num; double RisqueInter; public: S(char* i,int n); virtual ~S(); void addTer(T*); void addFl(F *); void addGrp(G *); friend ostream& operator<<(ostream&, const S& ); };
Merci encore une fois de votre rèponse
Merci d'ajouter les balises code (bouton #) pour plus de lisibilité.

Sinon, il semble que ton compilateur interprète:

Code:

ser.push_back(new S(*its));//mark

commme

Code:

ser.push_back(new S*(*its));//mark

il faut dire qu'il n'y a pas vraiment de méthode définie :?

tu a deux solutions:
ou tu définis un constructeur de copie pour S c'est à dire S(const S &)
ou tu laisse le contructeur par défaut, mais là, les copies de tableau pointeurs se feront sans réallocation.

Dans les deux cas, il faut remplacer

Code:

ser.push_back(new S(*its));//mark

par

Code:

ser.push_back(new S(**its));//mark

ce qui te permettra de construire S à partir d'un S et non d'un S*.

Bonne chance

28/06/2010, 15h59
zougagh

merci
Mais toujours même problème
28/06/2010, 16h05
méphistopheles

Citation:

Envoyé par zougagh

merci
Mais toujours même problème

tu a fais la modification ? Quel est ton code ? le compilateur t'affiche-t'il des warning ? Serait-il possible de VRAIMENT avoir le moceau de code qui te sert de test ?

Merci.
28/06/2010, 17h29
zougagh

ok ,j'ai réussi à résoudre le problème mais après une boucle d'affectation a=b et b=a j'obtiens nan sur la valeurs que je veux afficher sinon ya pas warning tous va bien .
Merci mille fois