probleme de surcharge de l'opérateur<<

Version imprimable

qulequ'un peut me dire c'est quoi le probleme avec ce code i!!!
parce que ce code génére des erreurs
PS: je travaille sous Microsoft visual C++ 6

voila mes classes :

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
 
#include<iostream>
#include<string>
using namespace std;
 
class vehicule  
{
protected:
	string imatriculation;
	string marque;
	double kilometre;
	string date_mise_circulation;
public:
	vehicule();
	vehicule(string ,string, double, string);
	void affiche();
	 friend ostream & operator<<(ostream &sortie, vehicule & v );
};

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
// vehicule.cpp: implementation of the vehicule class.
#include<iostream>
#include "vehicule.h"
using namespace std;
//////////////////////////////////////////////////////////////////////
// Construction/Destruction
//////////////////////////////////////////////////////////////////////
 
vehicule::vehicule()
{
	 imatriculation="";
	 marque="";
	 kilometre=0;
	 date_mise_circulation="";
}
 
vehicule::vehicule(string matr,string marq, double km, string date)
{
	imatriculation=matr;
	marque=marq;
	kilometre=km;
	date_mise_circulation =date;
}
 
 
void vehicule::affiche()
{
	cout<<"la vehicule de matrucule: "<<imatriculation<<"\t de marque: "<<marque<<endl;
}
 
ostream& operator<<(ostream & sortie,vehicule & veh)
{
	sortie<<"la vehicule de matrucule: "<<veh.imatriculation<<"\t de marque: "<<veh.marque<<"\tkm: "<<veh.kilometre<<endl;
	return sortie;
}

07/01/2012, 19h57
JolyLoic

Quel message d'erreur ? (un rapide coup d’œil ne m'a rien montré)
07/01/2012, 20h04
renalte16

voila l'erreurs:

vehicule.cpp
vehicule.cpp(50) : error C2248: 'imatriculation' : cannot access protected member declared in class 'vehicule'
vehicule.h(18) : see declaration of 'imatriculation'
vehicule.cpp(50) : error C2248: 'marque' : cannot access protected member declared in class 'vehicule'
c:\users\boub\workspacec++\gestion agence assurence\vehicule.h(19) : see declaration of 'marque'
vehicule.cpp(50) : error C2248: 'kilometre' : cannot access protected member declared in class 'vehicule'
vehicule.h(20) : see declaration of 'kilometre'
Error executing cl.exe.

Gestion agence assurence.exe - 3 error(s), 0 warning(s)
j'ai utiliser operator<< comme amie pour éviter cette situation
aussi lorsque j' ai exécuté ce code en devC++ sa marche b1 !!
08/01/2012, 02h13
koala01
Salut,

Avant de répondre :
1. Tu devrais utiliser les listes d'initialisation pour tes constructeur... C'est "simplement" préférable ;)
2. Tu ne devrait pas utiliser la directive using namespace std; dans un fichier d'en-tête: Avec le jeu des inclusions en cascade, cela propagera la directive de manière plus ou moins anarchique, et cela ne peut que finir par poser problème ;)
3. Pourquoi placer les membres dans l'accessibilité protégée :question: De prime abord, ils seraient bien mieux dans l'accessibilité privée ;)
4. VS6 est... anti déluvien, et il me semblerait bien plus intéressant de passer à un compilateur un peu plus récent (imagine: il VC6 date d'avant la première standardisation de C++, ou de juste après)
5. Tu devrais veiller à déclarer toute fonction n'ayant pas vocation à modifier ton objet (je pense à la fonction affiche) comme étant constante. Cela te mettra en excellente situation pour mettre en place le respect de ce qui est connu sous le nom de const correctness ;)
6. Plutot que d'utiliser explicitement cout à l'intérieur de la fonction affiche(), étant donné que le résultat sera le meme quel que soit le flux envisagé, pourquoi ne pas rajouter un argument de type ostream & à la fonction :question
Ceci dit, l'erreur est relativement simple: il faut passer ton véhicule sous la forme d'une référence constante à l'opérateur << ;)

Les conseils 5 et 6 feraient que ta fonction affiche ressemblerait au final à
Code:

1 2 3 4 void vehicule::affiche(std::ofstream & os) const { os<<"la vehicule de matrucule: "<<imatriculation<<"\t de marque: "<<marque<<endl; }
ce qui te permettrait alors de supprimer l'amitié avec l'opérateur << et d'avoir un code pour celui-ci ressemblant à
Code:

1 2 3 4 5 6 // remarque que je passe le véhicule sous la forme d'une référence constante ;) ostream& operator<<(ostream & sortie,vehicule const & veh) { veh.affich(sortie); return sortie; }
Cela aurait, de plus, l'avantage de te faire respecter un conseil souvent cité en eXtreme Programming: DRY (Don't repeat Yourself, ou, si tu préfères en français: ne vous répétez pas) ;)
08/01/2012, 09h34
3DArchi

Citation:

Envoyé par renalte16

qulequ'un peut me dire c'est quoi le probleme avec ce code i!!!
parce que ce code génére des erreurs
PS: je travaille sous Microsoft visual C++ 6

L'erreur est peut être là : Microsoft visual C++ 6.
Ton code compile sans problème avec les compilateurs récents.
09/01/2012, 10h20
oodini

Je crois qu'il ne se conforme même pas à la norme (je en parle évidemment pas d C++11).