Discussion :

[kamouminator]

bonjour a tous,

voila je suis confronté a un drole de probleme, mon code est compilable sous vc++ mais pas avec g++...

voila le code:

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
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
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
124
125
126
127
128
129
130
131
132
133
134
135
136
137
138
139
140
141
142
143
144
145
146
147
148
149
150
151
152
153
154
155
156
157
158
159
160
161
162
163
164
165
166
167
168
169
170
171
172
173
174
175
176
177
178
179
180
181
182
183
184
185
186
187
188
189
190
191
192
193
194
195
196
197
198
199
200
201
202
203
204
205
206
207
208
209
210
211
212
213
214
215
216
217
218
219
220
221
222
223
224
225
226
227
228
229
230
231
232
233
234
235
236
237
#include<iostream>
#include<fstream>
using namespace std;
 
 
 
 
 
 
 
 
 
 
 
class CFract{
 
 
 
public:
 
	int a,b;
 
	void Affiche(){
 
 
 
		//printf("CFract @=%08lX (%d,%d)\n",this,a,b); //l=long , X=hexa
		cout << "CFract @=" << "(" << a << "," << b << ")" << endl;
 
 
 
	}
 
 
 
	//Constructeur par copie
 
	CFract(CFract & f){		
 
 
 
		a=f.a;
 
		b=f.b;
 
		cout << "Ctor copie: ";Affiche();
 
 
 
	}
 
 
 
	//Constructeur 0 a 2 arguments
 
	CFract(int _a=1,int _b=1) : a(_a),b(_b)	{
 
 
 
		cout << "Ctor 0-2 param: "; Affiche();
 
 
 
	}
 
 
 
	//Surcharge "+"
 
	CFract operator+(const CFract & f) const /* pour que this ne soit pas modifié */{
 
 
		CFract res;
 
		res.a = f.a *b + a*f.b;
 
		res.b = f.b * b;
 
 
 
		return res; 
 
 
	}
 
 
 
	//Destructeur
 
	~CFract(){
 
 
 
		cout << "Dtor:";Affiche();
 
 
 
	}
 
 
 
	//ecriture sur fichier
 
	void Affiche(ofstream & pf) const {
 
 
 
		//fprintf(pf,"CFract @=%08lX (%d,%d)\n",this,a,b); //l=long , X=hexa
		pf << "CFract @=" << this << " (" << a << "," << b << ")" << endl;
 
 
 
 
	}
 
	//redirection de flux
 
	ofstream & operator>>(ofstream & pf)const{
 
 
 
		Affiche(pf);
 
 
 
		return pf;
 
 
 
	}
 
 
 
 
 
 
 
};
 
 
 
 
 
ofstream & operator<<(ofstream & pf, const CFract& f){
 
 
 
	f.Affiche(pf);
 
	return pf;
 
 
 
}
 
 
 
 
 
void Test1()
 
{
 
	//CFract f1={21,3};
 
 
 
 
 
	CFract f1;
 
	f1.a=21;
 
	f1.b=3;
 
	CFract f2(f1);
 
	CFract f3('z');
 
	f1.Affiche();
 
	f2.Affiche();
 
	f3.Affiche();
 
 
 
	//CFract f4(2.4);
 
	//f4.Affiche();
 
 
 
	CFract f5;
 
	f5=f1+f3;
 
	f5.Affiche();
 
	//FILE * pf=fopen("show.txt","wt");
	ofstream pf("show.txt");
 
 
 
 
 
	f5.Affiche(pf);
 
	f1>>(f2>>pf);
 
	pf<<f1<<f2;
 
 
 
	pf.close();
 
 
 
}
 
 
 
 
 
 
 
int main(){
 
 
 
 
 
	Test1();
	return 0;
 
 
}

l'erreure signalée est :

Code :

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

1
2
3
4
fract.cpp: In function «void Test1()»:
fract.cpp:102: erreur: no matching function for call to «CFract::CFract(CFract)»
fract.cpp:30: note: candidats sont: CFract::CFract(int, int)
fract.cpp:21: note:                 CFract::CFract(CFract&)

ligne 102: f5=f1+f3;

ce qui correspond a l'utilisation de la surcharge de l'operateur "+"
suivi d'une construction par copie...
sous vc++ la construction se fait correctement
avec g++ probleme, il veut un constructeur avec en argument la classe elle meme et pas la reference...

mon prof m'a donné comme réponse:
"gcc etant libre, il doit etre rempli de bugs" ....


merci d'avance

kamou

[ram-0000]

mon prof m'a donné comme réponse:
"gcc etant libre, il doit etre rempli de bugs" ....

Ouais, tu ne lui dira pas je suppose mais ca, c'est ce qu'on dit quand on sait pas quoi dire.

Je pense plutôt que le problème est dans ton constructeur de copie.

La forme normale est

Code :

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

1
2
3
4
 
	//Constructeur par copie
	CFract(const CFract & f){
...

Donc en l'occurence, c'est putôt gcc qui a raison mais c'est normal, c'est un logiciel libre (donc tout le monde peu regarder les sources, les corriger et en faire profiter la communauté)

[kamouminator]

ok merci ! ca marche !!

je le lui dirai peut etre a la fin de l'année ^^

[Obsidian]

Donc en l'occurence, c'est putôt gcc qui a raison mais c'est normal, c'est un logiciel libre (donc tout le monde peu regarder les sources, les corriger et en faire profiter la communauté)

D'autant que dans le logiciel propriétaire, reconnaître un bug auprès de ses clients peut avoir des conséquences économiques désastreuses.

[JolyLoic]

Désolé de vous contredire, mais :

Code :

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

CFract(CFract & f)

Est une définition légale pour un constructeur par recopie. Elle est moins utile que

Code :

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

CFract(CFract const & f)

mais peut quand même servir.

A non-template constructor for class X is a copy constructor if its first parameter is of type X&, const X&, volatile X& or const volatile X&, and either there are no other parameters or else all other parameters have default arguments

D'ailleurs, je ne vois pas où le constructeur par copie est supposé être appelé dans son code. Je verrai bien une erreur s'il avait écrit (comme on écrirait probablement dans du vrai code) :

Code :

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

CFract f5=f1+f3;

Dans ce cas, il y aurait bien tentative d'appel du constructeur de recopie avec un temporaire, ce qui n'est pas possible avec un argument qui est une référence non const.

Honnêtement, dans ce cas, même si le code initial n'est pas très bon, je pense qu'il devrait compiler, et que c'est gcc qui se trompe ici.

[loufoque]

Ça marche avec Microsoft Visual C++ parce qu'il n'est pas conforme à la norme.
Il permet la référence non-constante de temporaires. Ce qui n'est tout simplement pas autorisé.
Il n'est possible de référencer des temporaires qu'avec des références const.

[JolyLoic]

Peux-tu me dire dans le code en question où une référence non constante vers un temporaire est utilisée ?

Je rappelle la situation :
- Le code déclenchant l'erreur est f5=f1+f3;
- L'opérateur = n'est pas spécifié.
- Seul est spécifié le constructeur par recopie, qui prend une référence non constante en paramètre.
- Quand on modifie ce constructeur pour qu'il prenne en paramètre un référence constante, le code se met à marcher