Discussion :

[Treuze]

Bon nouveau probleme , j ai un personnes .cc qui ressemble a ca :

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
 
class personne { 
    private: 
        string nom ; 
        string prenom ;
        string etablissement ; 
    public:
        personne () : nom(""), prenom(""), etablissement("UTBM"){};
        personne (string nvnom, string nvprenom="", string nvetablis="UTBM") :
              nom(nvnom), prenom(nvprenom), etablissement(nvetablis){};
        virtual ~personne(){};
 
        virtual string get_nom() const;
        virtual string get_prenom() const;
        virtual string get_etablis() const;
        virtual string get_qualite() = 0;
 
        virtual personne& change_nom(string nvnom);
        virtual personne& change_prenom(string nvprenom);
        virtual personne& change_etablis(string nvetablis);
 
        virtual void affiche() const;
 
}; 
 
class etudiant : public personne { 
    private : 
        // donnees specifiques a etudiant 
        string branche; 
        string filiere; 
        int emploitemps;
 
    public : 
 
        etudiant() : branche(""),filiere(""),emploitemps(0){};
        etudiant(string nvnom, string nvprenom,string nvbranche="", string nvfiliere = "",                  int nvemploitemps = 0) : personne(nvnom, nvprenom),                  branche(nvbranche),filiere(nvfiliere),emploitemps(nvemploitemps){};
        ~etudiant(){};
 
        string get_branche() const;
        string get_filiere() const;
        int get_emploitemps() const;
 
        etudiant& change_branche ( string nvbranche);
        etudiant& change_filiere ( string nvfiliere);
        etudiant& change_emploitemps(int nvemploitemps);
        void affiche() const;
 
};

Mais au moment de la compilation , le compilo me dit qu il ne peut pas alouer un objet abstrait de type etudaint parce que cette fonction est pure ds la classe etudiant : personne :: get_qualite .

J ai pourtt redefini cette fonction ds etudiant comme ceci :

Code :

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

1
2
3
 
string etudiant :: get_qualite(){
return filiere ; }

Alors comprends pas ,encore ....

[Swoög]

la méthode surcharger doit apparaître dans la signature de ta classe etudiant, il faudrait que ton code ressemble à ceci :

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
class personne { 
    private: 
        string nom ; 
        string prenom ;
        string etablissement ; 
    public:
        personne () : nom(""), prenom(""), etablissement("UTBM"){};
        personne (string nvnom, string nvprenom="", string nvetablis="UTBM") :
              nom(nvnom), prenom(nvprenom), etablissement(nvetablis){};
        virtual ~personne(){};

        virtual string get_nom() const;
        virtual string get_prenom() const;
        virtual string get_etablis() const;
        virtual string get_qualite() = 0;

        virtual personne& change_nom(string nvnom);
        virtual personne& change_prenom(string nvprenom);
        virtual personne& change_etablis(string nvetablis);
         
        virtual void affiche() const;
         
}; 
  
class etudiant : public personne { 
    private : 
        // donnees specifiques a etudiant 
        string branche; 
        string filiere; 
        int emploitemps;
 
    public : 
             
        etudiant() : branche(""),filiere(""),emploitemps(0){};
        etudiant(string nvnom, string nvprenom,string nvbranche="", string nvfiliere = "",                  int nvemploitemps = 0) : personne(nvnom, nvprenom),                  branche(nvbranche),filiere(nvfiliere),emploitemps(nvemploitemps){};
        ~etudiant(){};

        string get_branche() const;
        string get_filiere() const;
        string get_qualite();
        int get_emploitemps() const;

        etudiant& change_branche ( string nvbranche);
        etudiant& change_filiere ( string nvfiliere);
        etudiant& change_emploitemps(int nvemploitemps);
        void affiche() const;
         
};

[Treuze]

oui j l avais fait , j ai poste le mauvais code ... mais ca ne marche pas

[Matthieu Brucher]

Et le code vrai code qui ne marche pas ?

[bigboomshakala]

peux-tu poster un code "minimal" qui permette de reproduire le problème ?

[Bakura]

J'ai réécrit le code et ça fonctionne, je ne sais pas si c'est ce que tu attends exactement mais bon (la présentation diffère très légèrement, désolé, j'ai appris comme ça ^^)...

class Personne
{
public:
Personne (string vNom = "", string vPrenom = "", string vEtablissement = "UTBM") :
nom (vNom), prenom (vPrenom), etablissement (vEtablissement) {};
~Personne () {};

virtual string get_nom () const {return nom;}
virtual string get_prenom () const {return prenom;}
virtual string get_etablis () const {return etablissement;}
virtual string get_qualite () = 0;

virtual void change_nom (string vNom) {nom = vNom;}
virtual void change_prenom (string vPrenom) {prenom = vPrenom;}
virtual void change_etablis (string vEtabli) {etablissement = vEtabli;}

private:
string nom;
string prenom;
string etablissement;
};

class Etudiant : public Personne
{
public:
Etudiant (string vNom = "", string vPrenom = "", string vBranche = "", string vFiliere = "") :
Personne (vNom, vPrenom), branche (vBranche), filiere (vFiliere), emploiTemps (0) {};
~Etudiant () {};

virtual string get_branche () const {return branche;}
virtual string get_filiere () const {return filiere;}
virtual string get_qualite () {};
virtual int get_emploiTemps () const {return emploiTemps;}

virtual void change_branche (string vBranche) {branche = vBranche;}
virtual void change_filiere (string vFiliere) {filiere = vFiliere;}
virtual void change_emploiTemps (int vEmploi) {emploiTemps = vEmploi;}

void afficher () const {};

private:
string branche;
string filiere;
int emploiTemps;
};

EDIT : avec ton code ça marche aussi, j'ai juste rajouter des {} sur les fonctions qui n'en avait pas, comme ça elles sont définies...

[Matthieu Brucher]

Lorsqu'une fonction est virtuelle, elle doit obligatoirement être définie dans le code, même si elle ne sera pas appelée, contrairement aux autres fonctions non virtuelles.
De même, il est important de renvoyer toujours le bon type, par exemple ta fonction get_qualite ne retourne pas un string pour un etudiant, tu as droit à un warning pour ça, et les warnings, c'est toujours à supprimer

[Treuze]

Code :

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

peux-tu poster un code "minimal" qui permette de reproduire le problème ?

Oui je peux :

personnes.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
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
 
#include <stdio.h>
#include <stdlib.h>
#include <iostream.h>
#include <string.h>
#include "liste.h"
 
using namespace std;
 
class personne { 
    private: 
        string nom ; 
        string prenom ;
        string etablissement ; 
    public:
        personne () : nom(""), prenom(""), etablissement("UTBM"){};
        personne (string nvnom, string nvprenom="", string nvetablis="UTBM") :
              nom(nvnom), prenom(nvprenom), etablissement(nvetablis){};
        virtual ~personne(){};
 
        virtual string get_nom() const;
        virtual string get_prenom() const;
        virtual string get_etablis() const;
        virtual string get_qualite() = 0;
 
        virtual personne& change_nom(string nvnom);
        virtual personne& change_prenom(string nvprenom);
        virtual personne& change_etablis(string nvetablis);
 
        virtual void affiche() const;
 
}; 
 
class etudiant : public personne { 
    private : 
        // donnees specifiques a etudiant 
        string branche; 
        string filiere; 
        int emploitemps;
 
    public : 
 
        etudiant() : branche(""),filiere(""),emploitemps(0){};
        etudiant(string nvnom, string nvprenom,string nvbranche="", string nvfiliere = "",                  int nvemploitemps = 0) : personne(nvnom, nvprenom),                  branche(nvbranche),filiere(nvfiliere),emploitemps(nvemploitemps){};
        ~etudiant(){};
 
        string get_branche() const;
        string get_filiere() const;
        string get_qualite() const{return "";};
        int get_emploitemps() const;
 
        etudiant& change_branche ( string nvbranche);
        etudiant& change_filiere ( string nvfiliere);
        etudiant& change_emploitemps(int nvemploitemps);
        void affiche() const;
 
};

personnes.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
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
 
#include <stdio.h>
#include <stdlib.h>
#include <iostream.h>
#include <string.h>
#include "personnes.h"
#include "liste.h"
 
using namespace std;
 
string personne :: get_nom() const{
 
return nom;
}
 
string personne :: get_prenom() const{
 
return prenom;
}
 
string personne :: get_etablis() const{
 
return etablissement;
}
 
personne& personne :: change_nom(string nvnom){
 
nom = nvnom;
return *this;
}
 
personne& personne :: change_prenom(string nvprenom){
 
prenom = nvprenom;
return *this;
}
 
personne& personne :: change_etablis(string nvetablis){
 
etablissement = nvetablis;
return *this;
}
 
void personne :: affiche() const {
cout << "nom :"<<nom << "prenom :" << prenom;
}
 
string etudiant :: get_branche() const{
return branche;
}
 
string etudiant :: get_filiere() const{
return filiere;
}
 
int etudiant :: get_emploitemps() const{
return emploitemps;
}
 
etudiant& etudiant :: change_branche ( string nvbranche){
branche = nvbranche;
return *this;
}
 
etudiant& etudiant :: change_filiere ( string nvfiliere){
filiere = nvfiliere;
return *this;
}
 
etudiant& etudiant :: change_emploitemps(int nvemploitemps){
emploitemps = nvemploitemps;
return *this;
}
 
void etudiant :: affiche() const{
personne :: affiche();
cout << "branche :" << branche << "filiere :" ;
cout << filiere << "charge hebdomadaire : " << emploitemps;
}

main.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
 
#include <stdio.h>
#include <stdlib.h>
#include <iostream.h>
#include <string.h>
#include "personnes.h"
#include "liste.h"
 
main()
{
    bddpersonnes b;
    personne *p = new etudiant;
    b.ajout(p);
}

J'ai réécrit le code et ça fonctionne

Oui cest depuis que j ai ecris le main/cc que ca ne va plus

[bigboomshakala]

normal que le compilo râle

classe personne

Code :

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

virtual string get_qualite() = 0;

classe etudiant

Code :

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

string get_qualite() const{return "";};

les 2 fonctions n'ont rien à voir puisqu'elles n'ont pas la même signature. donc effectivement la classe etudiant comporte une fonction virtuelle pure et ne peut être instanciée.

correction classe personne

Code :

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

virtual string get_qualite() const = 0;

plein d'autres corrections :
- enlever using namespace std; du .h (voir FAQ pour explication)
- enlever les en-têtes qui ne sont pas nécessaires
- et qui en plus ne sont pas toujours les bons, cf. iostream.h, string.h... (voir FAQ en-têtes C et C++)

[Treuze]

Arf c pas la premiere fois que ca m arrive en plus ...

Merci maintenant j ai plusieurs de references indefinies... est ce que qq un pourrait compiler ce programme pour voir si c est mon se qui deconne ?

Sinon merci pour m avoir signaler les autres erreurs c tjrs bon a prendre

[Treuze]

Volia j ai attache les fichiers si jamais ... j ai du change les extensions de personnes.cc et lsite .cc en cpp si jamais c est important

merci d avance