surcharge de >>

Bonsoir à tous.

Voilà, je vous écrit ce petit post car je rencontre un problème lorsque je souhaite utiliser l'operator ">>" afin d'aller relire des données dans un fichier.

Donc, pour ce faire, je passe par une méthode que j'appel load. Je la définit comme suit:

Code:

---

1 2	void load(char* n, char* p);

---

Cette méthode ce situe dans une classe "Navire". Et dans le corps de la méthodes, voilà ce que je fais:

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21

void Navire::load(char* n, char* p) { cout<<"Chargement du navire"<<endl; (*this)._flag = 1;   char *tempo = new char[strlen(n) + strlen(p) + 4]; strcpy(tempo, n); strcpy(&tempo[strlen(n)], p); strcpy(&tempo[strlen(n)+ strlen(p)], ".ser"); cout<<"Nom du fichier: "<<tempo<<endl;   fstream foc(tempo, ios::in); cout<<"foc cree"<<endl; foc>>(*this); cout<<"foc lu..."<<endl; foc.close(); cout<<"foc close"<<endl;   (*this)._flag = 0; }

---

La variable _flag est un integer qui me permet de dire si je me trouve dans la partie de code de la surcharge qui me permettra de relire dans le fichier, ou alors si c'est dans la partie ou je saisis les valeurs de mon bateau.

Et voici pour finir, la surcharge de l'operator>>:

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

istream &operator>>(istream &s, Navire& B) { cout<<"surcharge de cin dans Navire"<<B._flag<<endl; if(B._flag == 1) { cout<<"relecture dans le fichier"; s>>B._Nom; s>>B._PortAttache; s>>B._Numero; s>>B._NombreEquipage; s>>B._IdAssurance; s>>B._NomCapitaine; s>>B._Tonnage; s>>B._Longueur; s>>B._Tirant; return s; } else { char CR; char N[20];   cout<<"\tVeuillez saisir les valeurs suivantes:"<<endl; cout<<"- nom: "; s>>N; B.setNom(N); cout<<"- port d'attache: "; s>>N; B.setPort(N); cout<<"- le numero du bateau: "; s>>N; B.setID(N); cout<<"- le nombre de membre d'equipage: "; s>>B._NombreEquipage; cout<<"- l'id de l'assurance: "; s>>N; B.setAssurance(N); cout<<"- le nom du capataine: "; s>>N; B.setCapitaine(N); do{ try{ cout<<"- tonnage: "; s>>B._Tonnage; if( B._Tonnage<= 0) { InvalidMeasureException IME("Valeur ne peut etre negative ou nulle"); throw (IME); } cout<<"- longueur du vaisseau: "; s>>B._Longueur; if( B._Longueur<= 0) { InvalidMeasureException IME("Valeur ne peut etre negative ou nulle"); throw (IME); } cout<<"- tirant d'eau du vaisseau: "; s>>B._Tirant; if( B._Tirant<= 0) { InvalidMeasureException IME("Valeur ne peut etre negative ou nulle"); throw (IME); } } catch(InvalidMeasureException& e) { cout<<e.what(); } }while(B._Tonnage<= 0 || B._Longueur<= 0 || B._Tirant<= 0); return s; } }

---

Et en fait, lorsque je lance mon applic, je demande de recharger les fichiers que j'ai saisi précédemment, et alors il me fait la chose suivante:

Code:

---

1 2 3 4 5 6 7 8

Veuillez saisir le nom du bateau: lulu veuillez saisir le port d'attache: popol Chargement du navire Nom du fichier: lulupopol.ser foc cree surcharge de cin dans Navire1 Segmentation Fault (core dumped)

---

Alors que mon fichier lulupopol.ser existe bien. Apparement il butte sur le fait que je fasse un if dans la surcharge ou autre problème du genre, car j'affiche bien la valeur de _flag, et elle vaut 1 (juste après surcharge de cin dans Navire).

Voilà, je pense avoir tout dit, comme toujours, j'en fait surement trop, mais au moins comme ça, je suis sûr que vous pourrez m'aider.

Merci d'avance pour l'aider que vous pourrez m'apporter, et dans l'attente de réponses, je vous souhaite une bonne fin de soirée, de week-end, ...

p.s.: je compile sur une machine virtuelle de SUN Solaris

Salut,

Code:

---

cout<<"relecture dans le fichier";

---

Je crois que ca n'affiche pas avant qu'il y ait un endl (ca dépend). Essaie donc:

Code:

---

cout<<"relecture dans le fichier"<<endl;

---

Si ca marche, ca veut dire que le problème est après. En plaçant des cout un peu partout tu pourras sûrement trouver la ligne fautive ;)

Salut,

Ca ne serait pas par hasard :

Code:

---

char *tempo = new char[strlen(n) + strlen(p) + 4 + 1];

---

J'avoue que je suis un peu rouillé en manipulations de chaînes C...
Tu devrais de toute façon utiliser des std::string à la place, les chaînes C déjà c'est du C et ensuite c'est vraiment pénible et dangereux à manipuler.

Ah et puis :

Code:

---

(*this)._flag = 1;

---

Ca s'écrit, si tu tiens vraiment à mettre le this :

Code:

---

this->_flag = 1;

---

Mais aussi plus simplement :

Code:

---

_flag = 1;

---

Au passage c'est pas génial cette magouille avec _flag et en plus ça a l'air totalement inutile. Déjà tu devrais utiliser un booléen mais bon...
Et surtout pourquoi tu ne fais pas une méthode dans laquelle tu mets tout le code compris dans le if( _flag == 1 ) plutôt que de vouloir à tout prix appeler l'opérateur de désérialisation ?

MAT.

Ben voilà.

- j'ai rajouter un champ (enfin un +1) dans ma chaine tempo.
- j'ai rajouter les endl à la fin de mes lignes :

Code:

---

cout<<"relecture dans le fichier";

---

Et je constate maintenant, que j'ai une erreur lorsque je fais appel aux lignes suivantes:

Code:

---

1 2 3 4 5 6 7 8 9 10

s>>B._Nom; s>>B._PortAttache; s>>B._Numero; s>>B._NombreEquipage; s>>B._IdAssurance; s>>B._NomCapitaine; s>>B._Tonnage; s>>B._Longueur; s>>B._Tirant;

---

Aucune d'entres elles ne passent. J'ai vérifier dans les notes de mon cours théorique, et le prof procède bien de la même manière.

Et je suis d'accord avec toi MAT, mais notre prof souhaite que l'on procède de la sorte, je ne peux rien y faire.

Dans l'attente d'une solution possible au traitement de mon erreur, je vous remercie d'avance, et vous souhaite une bonne fin d'après midi.

Richard_sraing

Salut,

Code:

---

fstream foc(tempo, ios::in);

---

Tu as essayé:

Code:

---

1 2 3 4 5

fstream foc(tempo, ios::in); if (!foc) { cout << "Erreur dans l'ouverture du foc?!" << endl; }

---

Sinon, si ça marche toujours pas, tu peux montrer la déclaration de la classe et le contenu du fichier?

Coyote507

Bon, alors je ne suis pas sûr de ce que veux dire:

Citation:

---

tu peux montrer la déclaration de la classe et le contenu du fichier?

---

Je suppose que c'est tout mon fichier, donc le voilà:

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 66 67 68 69 70 71 72 73 74 75 76 77 78 79 80 81 82 83 84 85 86 87

#ifndef _NAVIRE #define _NAVIRE   #ifdef VISUAL #include<iostream.h> #elif BORLAND #include<iostream.h> #elif DEC #include<stream.h> #elif SUN #include<iostream> using namespace std; #endif   #include<fstream>   #include"Exception.h"   class Navire { protected: char* _Nom; char* _PortAttache; char* _Numero; int _NombreEquipage; char* _IdAssurance; char* _NomCapitaine; int _Tonnage; int _Longueur; int _Tirant; int _flag;   public:   Navire(); Navire(char* N,char* PA, char* Num, int NE, char* IA, char* NC, int T, int L, int Ti); Navire(Navire& N); ~Navire();   /* methodes de get */ char* getNom(){ return _Nom; } char* getPort(){ return _PortAttache; } char* getID(){ return _Numero; } int getEquipage(){ return _NombreEquipage; } char* getAssurance(){ return _IdAssurance; } char* getCapitaine(){ return _NomCapitaine; } int getTonnage(){ return _Tonnage; } int getLongueur(){ return _Longueur; } int getTirant(){ return _Tirant; } virtual void affiche();   /* methodes de set */ void setNom(char* value); void setPort(char* value); void setID(char* value); void setEquipage(int value); void setAssurance(char* value); void setCapitaine(char* value); void setTonnage(int value); void setLongueur(int value); void setTirant(int value); virtual void insert(); void setAll(char* N, char* PA, char* Num, int NE, char* IA, char* NC, int T, int L, int Ti);   /* methode de bateau definis */ void VEDETTE(); void AMBULANCE();   /* surcharge des operateurs */ //Navire operator+(const Equipage &Equ); friend istream& operator>>(istream& s, Navire& B); friend ostream& operator<<(ostream& s, const Navire& B);   int compN(Navire& N1, Navire& N2); int operator==(Navire& N); int operator!=( Navire& N); int operator<(Navire& N); int operator>(Navire& N);   /* FLUX FICHIERS */ void save(); void load(char* n, char* p);   };   #endif

---

Et pour le fichier .cxx, le voici:

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 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 238 239 240 241 242 243 244 245 246 247 248 249 250 251 252 253 254 255 256 257 258 259 260 261 262 263 264 265 266 267 268 269 270 271 272 273 274 275 276 277 278 279 280 281 282 283 284 285 286 287 288 289 290 291 292 293 294 295 296 297 298 299 300 301 302 303 304 305 306 307 308 309 310 311 312 313 314 315 316 317 318 319 320 321 322 323 324 325 326 327 328 329 330 331 332 333 334 335 336 337 338 339 340 341 342 343 344 345 346 347 348 349 350 351 352 353 354 355 356 357 358 359 360 361 362 363 364 365 366 367 368 369 370 371 372 373

#include "Navire.h"   Navire::Navire(void) { _Nom = NULL; _PortAttache = NULL; _Numero = NULL; _NombreEquipage = 0; _IdAssurance = NULL; _NomCapitaine = NULL; _Tonnage = 0; _Longueur = 0; _Tirant = 0; }   Navire::Navire(char* N, char* PA, char* Num, int NE, char* IA, char* NC, int T, int L, int Ti) { setNom(N); setPort(PA); setID(Num); setEquipage(NE); setAssurance(IA); setCapitaine(NC); setTonnage(T); setLongueur(L); setTirant(Ti); }   Navire::Navire(Navire& N) { strcpy(_Nom, N._Nom); strcpy(_PortAttache, N._PortAttache); strcpy(_Numero, N._Numero); strcpy(_IdAssurance, N._IdAssurance); strcpy(_NomCapitaine, N._NomCapitaine); _NombreEquipage = N._NombreEquipage; _Tonnage = N._Tonnage; _Longueur = N._Longueur; _Tirant = N._Tirant; }   Navire::~Navire() { delete _Nom; delete _PortAttache; delete _Numero; delete _IdAssurance; delete _NomCapitaine; }   void Navire::setNom(char* value) { if( _Nom != NULL) delete _Nom; _Nom = new char[strlen(value)+1]; strcpy(_Nom, value); } void Navire::setPort(char* value) { if( _PortAttache != NULL) delete _PortAttache; _PortAttache = new char[strlen(value)+1]; strcpy(_PortAttache, value); } void Navire::setID(char* value) { if( _Numero != NULL) delete _Numero; _Numero = new char[strlen(value)+1]; strcpy(_Numero, value); } void Navire::setEquipage(int value) { _NombreEquipage = value; } void Navire::setAssurance(char* value) { if(_IdAssurance != NULL) delete _IdAssurance; _IdAssurance = new char[strlen(value)+1]; strcpy(_IdAssurance, value); } void Navire::setCapitaine(char* value) { if( _NomCapitaine!= NULL) delete _NomCapitaine; _NomCapitaine = new char[strlen(value)+1]; strcpy(_NomCapitaine, value); } void Navire::setTonnage(int value) { _Tonnage = value; } void Navire::setLongueur(int value) { _Longueur = value; } void Navire::setTirant(int value) { _Tirant = value; }   void Navire::insert() { char CR; char N[20]; char P[20]; char Num[20]; char IDA[20]; char NC[20];   cout<<"Veuillez saisir les elements suivants: "<<endl; cout<<"- Nom: "; cin.get(CR); cin.getline(N,20); setNom(N); cout<<"- le port d'attache: "; cin.getline(P,20); setPort(P); cout<<"- le numero du bateau: "; cin.getline(Num,20); setID(Num); cout<<"- le nombre de membre d'equipage: "; cin>>_NombreEquipage; cout<<"- l'id de l'assurance: "; cin.get(CR); cin.getline(IDA,20); setAssurance(IDA); cout<<"- le nom du capataine: "; cin.getline(NC,20); setCapitaine(NC); /* verifier les valeurs et lancer l'exception InvalidMeasureException */ do{ try{ cout<<"- tonnage: "; cin>>_Tonnage; if( _Tonnage<= 0) { InvalidMeasureException IME("Valeur ne peut etre negative ou nulle"); throw (IME); } cout<<"- longueur du vaisseau: "; cin>>_Longueur; if( _Longueur<= 0) { InvalidMeasureException IME("Valeur ne peut etre negative ou nulle"); throw (IME); } cout<<"- tirant d'eau du vaisseau: "; cin>>_Tirant; if( _Tirant<= 0) { InvalidMeasureException IME("Valeur ne peut etre negative ou nulle"); throw (IME); } } catch(InvalidMeasureException& e) { cout<<e.what(); } }while(_Tonnage<= 0 || _Longueur<= 0 || _Tirant<= 0); }   void Navire::setAll(char* N, char* PA, char* Num, int NE, char* IA, char* NC, int T, int L, int Ti) { setNom(N); setPort(PA); setID(Num); setEquipage(NE); setAssurance(IA); setCapitaine(NC); setTonnage(T); setLongueur(L); setTirant(Ti); }   void Navire::affiche() { cout<<"Voici le bateau:"<<endl;   cout<<"- Nom: "<<getNom()<<endl; cout<<"- Port d'attahce: "<<getPort()<<endl; cout<<"- Identifiant du bateau: "<<getID()<<endl; cout<<"- Nombre de membres d'equipage: "<<getEquipage()<<endl; cout<<"- Assurance du navire: "<<getAssurance()<<endl; cout<<"- Nom du capitaine: "<<getCapitaine()<<endl; cout<<"- Tonnage du vaisseau: "<<getTonnage()<<endl; cout<<"- Longueur du vaisseau: "<<getLongueur()<<endl; cout<<"- Tirant d'eau du vaisseau: "<<getTirant()<<endl; }   void Navire::VEDETTE(void) { setNom("OFFICIER DE PORT"); setPort("Port-Madrovil"); setID("BEL0001"); setEquipage(5); setAssurance("PORT"); setTonnage(20); setLongueur(10); setTirant(3); setCapitaine("Officier de port Commandant"); }   void Navire::AMBULANCE(void) { setNom("AMBULANCE"); setPort("Port-Madrovil"); setID("BEL0000"); setEquipage(4); setAssurance("PORT"); setTonnage(10); setLongueur(8); setTirant(2); setCapitaine("Officier de port Medecin"); } /* Navire Navire::operator+(const Equipage &Equ) {   }*/   istream &operator>>(istream &s, Navire& B) { cout<<"surcharge de cin dans Navire valeur de B._flag: "<<B._flag<<endl; if(B._flag == 1) { cout<<"relecture dans le fichier"<<endl; s>>B._Nom; cout<<"relecture dans le fichier1"<<endl; s>>B._PortAttache; cout<<"relecture dans le fichier2"<<endl; s>>B._Numero; cout<<"relecture dans le fichier3"<<endl; s>>B._NombreEquipage; cout<<"relecture dans le fichier4"<<endl; s>>B._IdAssurance; cout<<"relecture dans le fichier5"<<endl; s>>B._NomCapitaine; cout<<"relecture dans le fichier6"<<endl; s>>B._Tonnage; cout<<"relecture dans le fichier7"<<endl; s>>B._Longueur; cout<<"relecture dans le fichier8"<<endl; s>>B._Tirant; cout<<"relecture dans le fichier9"<<endl; return s; } else { char CR; char N[20];   cout<<"\tVeuillez saisir les valeurs suivantes:"<<endl; cout<<"- nom: "; s>>N; B.setNom(N); cout<<"- port d'attache: "; s>>N; B.setPort(N); cout<<"- le numero du bateau: "; s>>N; B.setID(N); cout<<"- le nombre de membre d'equipage: "; s>>B._NombreEquipage; cout<<"- l'id de l'assurance: "; s>>N; B.setAssurance(N); cout<<"- le nom du capataine: "; s>>N; B.setCapitaine(N); do{ try{ cout<<"- tonnage: "; s>>B._Tonnage; if( B._Tonnage<= 0) { InvalidMeasureException IME("Valeur ne peut etre negative ou nulle"); throw (IME); } cout<<"- longueur du vaisseau: "; s>>B._Longueur; if( B._Longueur<= 0) { InvalidMeasureException IME("Valeur ne peut etre negative ou nulle"); throw (IME); } cout<<"- tirant d'eau du vaisseau: "; s>>B._Tirant; if( B._Tirant<= 0) { InvalidMeasureException IME("Valeur ne peut etre negative ou nulle"); throw (IME); } } catch(InvalidMeasureException& e) { cout<<e.what(); } }while(B._Tonnage<= 0 || B._Longueur<= 0 || B._Tirant<= 0); return s; } }   ostream& operator<<(ostream& s, Navire& B) { s<<"Voici les valeurs du Navire:"<<endl; s<<"- Nom: "<<B.getNom()<<endl; s<<"- Port d'attahce: "<<B.getPort()<<endl; s<<"- Identifiant du bateau: "<<B.getID()<<endl; s<<"- Nombre de membres d'equipage: "<<B.getEquipage()<<endl; s<<"- Assurance du navire: "<<B.getAssurance()<<endl; s<<"- Nom du capitaine: "<<B.getCapitaine()<<endl; s<<"- Tonnage du vaisseau: "<<B.getTonnage()<<endl; s<<"- Longueur du vaisseau: "<<B.getLongueur()<<endl; s<<"- Tirant d'eau du vaisseau: "<<B.getTirant()<<endl; return s; }   int Navire::operator==(Navire& N) { return compN((*this), N)==0; }   int Navire::operator!=(Navire &N) { int temp;   temp=compN((*this), N); if (temp!=0) return 1; else return 0; }   int Navire::operator<(Navire& N) { return compN((*this), N)==-1; }   int Navire::operator>(Navire& N) { return compN((*this), N)==1; }   int Navire::compN(Navire& N1, Navire& N2) { char *ID1, *ID2;   ID1 = new char[strlen(N1.getNom())+1]; ID2 = new char[strlen(N2.getNom())+1];   strcpy(ID1,N1.getNom()); strcpy(ID2,N2.getNom());   return strcmp(ID1,ID2); }   void Navire::save() { cout<<"Sauvegarde du navire en cours...";   char *tempo = new char[strlen(_Nom) + strlen(_PortAttache) + 4]; strcpy(tempo, _Nom); strcpy(&tempo[strlen(_Nom)], _PortAttache); strcpy(&tempo[strlen(_Nom)+ strlen(_PortAttache)], ".ser"); cout<<"Nom du fichier: "<<tempo<<endl;   /* il faut creer le fichier avec le nom de tempo et ecrire les valeurs dedans */ fstream foc(tempo, ios::out); foc<<_Nom<<endl<<_PortAttache<<endl<<_Numero<<endl<<_NombreEquipage<<endl<<_IdAssurance\ <<endl<<_NomCapitaine<<endl<<_Tonnage<<endl<<_Longueur<<endl<<_Tirant; foc.close();   }   void Navire::load(char* n, char* p) { (*this)._flag = 1;   char *tempo = new char[strlen(n) + strlen(p) + 5]; strcpy(tempo, n); strcpy(&tempo[strlen(n)], p); strcpy(&tempo[strlen(n)+ strlen(p)], ".ser"); cout<<"Nom du fichier: "<<tempo<<endl;   fstream fic(tempo, ios::in); if (!fic) { cout << "Erreur dans l'ouverture du foc?!" << endl; } else { cout<<"fic cree"<<endl; fic>>*this; cout<<"fic lu..."<<endl; fic.close(); cout<<"fic close"<<endl; } (*this)._flag = 0; }

---

Voilà, désolé si c'est assez brouillon, mais les profs ne nous apprennent pas vraiment a faire une belle indentation, et celle que j'ai adopter est très proche de celle des profs.

Pour information, dans mon fichier qui fera appel à ma classe, je procède comme suit:

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14

int main() { Navire *nav = new Navire(); // pour saisir mon navire cin>>*nav; // pour sauver mon navire sur fichier nav->save(); //pour relire mon navire du fichier // avec n et p préalablement saisi par le user nav->load(char* n, char* p); cout<<*nav; return 0; }

---

J'espère ne rien avoir oublier. Merci pour le coup de clavier :mrgreen:, et oui, il faudra que je commence à me démener un plus tout seul :roll:

Richard_sraing

p.s.: je sais aussi qu'il y a des méthodes qui font la même choses, mais c'est au niveau de l'énoncé que l'on nous l'impose comme ça...

Au moment où tu fais :

Code:

---

s>>B._Nom;

---

Le pointeur vaut 0.

Pourquoi tu ne fais pas exactement pareil que dans l'autre branche du if, à savoir avec le N intermédiaire puis en appelant setNom ?

Cela dit (je me répête je sais) utiliser des chaînes C en C++ est la porte ouverte à toutes les fenêtres, et tu peux le dire de ma part à ton prof ! 8-)
Tu es vraiment sûr de ne pas avoir le droit d'utiliser std::string ?
Ou peut-être que le but est justement au prochain cours de vous montrer que c'est super simple avec des std::string, qui sait...

En tous cas actuellement il y a plein d'autres problèmes potentiels de dépassement de tableaux, notamment à chaque fois que tu fais un strcpy et que tu lis depuis un stream dans un tableau de caractères...

MAT.

Je suis super content.
Merci du coup de clavier, ça marche bien.
Sauf que j'ai un petit problème avec le NomCapitaine orsque je relis.
Comme je saisis un nom et un prénom pour ce champ, il y a un espace entre, et il termine la chaine quand il tombe sur l'espace.

Un dernier petit coup de souris pour ça, et ce sera nickel :lol:

Merci d'avance....

Il faut utiliser getline (la fonction membre de std::cin si tu veux rester avec des char*, voir par ex là).

MAT.

OK.
Merci de votre aide.

Au moins comme ça, j'aurais pu atteindre un des derniers points dans mon applic.

Encore merci.

Richard_sraing