Discussion :

[zougagh]

Bonjour,
j'ai une structure comme suit

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
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 :

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
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

[Mimini]

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
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;
}

[Luther13]

Comment savez-vous que a et b sont différents ?
il faudrait aussi surcharger l'opérateur ==

[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.

[zougagh]

j'affiche leur contenu et je vois qu'ils ont le même contenu

[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 .

[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 :

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

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 :

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

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

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

Code :

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

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 :

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

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

[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 :

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

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.

[zougagh]

voila la déclaration de S :

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
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

[méphistopheles]

voila la déclaration de S :

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
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 :

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

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

commme

Code :

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

        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 :

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

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

par

Code :

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

        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

[zougagh]

merci
Mais toujours même problème

[méphistopheles]

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.

[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