Discussion :

[mikebranque]

j ai une classe voiture :
voiture.h

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
 
#ifndef VOITURE_H
#define VOITURE_H
#include "cdate.h"
#include "cloc.h"
#include <string>
#include <list>
using namespace std;
 
 
class Voiture {
 public:
  Voiture(); 
  Voiture(const string, const int, const string,const list<CLoc>);
  ~Voiture();
  int GetKm() const;
  void SetKm(int);
  string GetIm();
  void SetIm(string);
  string GetMarque();
  void SetMarque(string);
  void Affiche() ;
  // list<CLoc> loc;
  void ajouterLoc (CLoc);
  void test (int i);
 
 private:
  string immatriculation;
  int kilometrage;
  string marque;
  list<CLoc> loc; //liste contenant les location
  list<int> ls; //juste pour test
};
 
#endif

voici le voiture.cc

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
 
#include "cvoiture.h"
#include "cloc.h"
#include "cdate.h"
#include <list>
#include <string>
#include <iostream>
using namespace std;
 
Voiture::Voiture() {}
 
Voiture::Voiture(const string im, const int km, const string marq,const list<CLoc> location)
 
 {
  immatriculation=im;
  kilometrage=km;
  marque=marq;
 
  }
 
Voiture::~Voiture() {
 
}
 
void Voiture::ajouterLoc (CLoc c) {
 
  loc.push_front(c);//

COMPILE errueur:
/usr/lib/gcc/i686-pc-cygwin/3.4.4/include/c++/bits/stl_construct.h: In function `void std::_Construct(_T1*, const _T2&) [with _T1 = CLoc, _T2 = CLoc]':
/usr/lib/gcc/i686-pc-cygwin/3.4.4/include/c++/bits/stl_list.h:438: instantiated from `std::_List_node<_Tp>* std::list<_Tp, _Alloc>::_M_create_node(const _Tp&) [with _Tp = CLoc, _Alloc = std::allocator<CLoc>]'
/usr/lib/gcc/i686-pc-cygwin/3.4.4/include/c++/bits/stl_list.h:1163: instantiated from `void std::list<_Tp, _Alloc>::_M_insert(std::_List_iterator<_Tp>, const _Tp&) [with _Tp = CLoc, _Alloc = std::allocator<CLoc>]'

Code :

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

1
2
3
4
5
6
7
 
 
}
 
void Voiture::test (int i) {
 
  ls.push_front(i);

COMPILE OK

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
 
}
 
int Voiture::GetKm() const
{
  return kilometrage;
}
void Voiture::SetKm(int km) {
  kilometrage=km;
}
string Voiture::GetIm() {
  return immatriculation;
}
void Voiture::SetIm(string im) {
  immatriculation=im;
}
string Voiture::GetMarque() {
  return marque;
}
void  Voiture::SetMarque(string m) {
  marque=m;
}
void Voiture::Affiche()  {
   cout<<"________Voiture________"<<endl;
   cout<<"immatriculation : "<<GetIm()<<endl;
   cout<<"Kilométrage     : "<<GetKm()<<endl;
   cout<<"Marque          : "<<GetMarque()<<endl;
 
}

Alors pourquoi alors que je peux manipuler ma liste d'entier, je ne peux pas manipuler ma liste de CLoc, il s agit d'une classe contenant (deux date s et un booléens). Il semble peut etre qu'il y ait un probleme de constructeur??? Je suis perdu.[/code]

[BigNic]

Cloc a-t-il un constructeur par défaut et un constructeur par copie ?
Sont-ils tous les deux public ?
Peux-tu nous donner Cloc.h ?
ok je t'aide pas trop là, mais c'est histoire de faire avancer le shmilimmilimblic

[julien.sagnard]

Une chose qui m'agresse les yeux quand je lit ton code, c'est que tu passe tout par valeur :

Code :

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

1
2
 
Voiture::Voiture(const string im, const int km, const string marq,const list<CLoc> location)

Signifie que tu vas faire une copie inutile de im, marq et location. Je te dit pas le gachi mémoire et cpu si ce sont de gros objets. Utilise les références pour éviter ça.
Tu trouveras une explication sur ce que c'est ici :
http://cpp.developpez.com/cours/cpp/

Ensuite, pourquoi passes-tu location dans tes arguments si tu ne t'en sert pas ?
D'ailleur, j'ai l'impression que ton compilateur gueule par ce qu'il n'arrive justement pas à trouver comment recopier cette liste.

Et pour finir, tu devrais lire ça :
http://c.developpez.com/faq/cpp/?pag...EUR_liste_init

Reviens nous voir si tu as toujours ton problème après ça.

[mikebranque]

cloc.h

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
 
#ifndef LOC_H
#define LOC_H
#include <iostream>
#include "cdate.h"
#include <list>
using namespace std;
 
class CLoc {
 public:
  CLoc();
  CLoc(bool,CDate,CDate);
  //constructeur de copie
  CLoc(CLoc &o);
  ~CLoc();
  bool GetLoue ();
  void SetLoue (bool);
  CDate GetDeb();
  void SetDeb(CDate);
   CDate GetFin();
   void SetFin(CDate);
 private:
  bool loue;
  CDate debut;
  CDate fin;
 
};
 
#endif

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
 
#include <cassert>
#include <string>
#include <iostream>
 
using namespace std;
#include "cloc.h"
#include "cdate.h"
CLoc::CLoc() {}
CLoc::CLoc(bool l,CDate d,CDate f) {
  loue=l;
  debut=debut;
  fin=f;
}
CLoc::CLoc(CLoc &o) {
  loue=o.GetLoue();
  debut=o.GetDeb();
  fin=o.GetFin();
}
CLoc::~CLoc () {}
bool CLoc::GetLoue () {
  return loue;
}
void CLoc::SetLoue ( bool b ) {
  loue=b;
}
 
CDate CLoc::GetDeb () {
  return debut;
}
void CLoc::SetDeb ( CDate deb ) {
  debut=deb;
}
CDate CLoc::CLoc::GetFin () {
  return fin;
}
void CLoc::SetFin ( CDate f ) {
  fin=f;
}

[BigNic]

à part les intialisateurs dans les constructeur comme te l'a fais remarquer Julien, je ne voit rien de qui cloche.
Quand j'ai 5 misn je test au compilo chez moi.

[BigNic]

bon alors je devais pas avoir les yeux en face des trous car ce sont des erreurs classique que tu as fait.

1) le constructeur par copie utilisé par la liste (et beacoup d'autres conteneur s STL) est:

Code :

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

 CLoc(const CLoc &o);

et non pas

Code :

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

 CLoc(CLoc &o);

comme tu l'avais mis. un petit const ça change tout !

2) si tu fais ça tu doit changer le proto des fonctions appellées dedans. en effet ta référence étant const, tu n'a le droit d'appeler dessus que des méthodes const. Donc il faut que tu modifes tes accesseurs en lectures de ta classe Cloc.

Sinon tu devrait fair un peu le ménage dans ton code. Il ya des #includes sur des header non utilisé et les using namespace il vaut mieux eviter dans les
header.

Si cela fonctionne pense à cliquer sur résolu.

[Pragmateek]

Code :

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

1
2
3
Voiture::~Voiture() {
 
}

Ca sert a quoi?

A respecter la forme canonique de Coplien?

[Aurelien.Regat-Barrel]

Une petite remarque : il faut éviter les "using namespace" dans les .h:
http://c.developpez.com/faq/cpp/?pag...sing_namespace

[mikebranque]

Merci .
effectivement il faut :

Code :

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

1
2
 
CLoc (const CLock ..)