Premier essai avec les classes : échec

**Glân von Brylân** · 07/04/2014, 15h04

Je reçois toute une floppée de "undefined reference to [...]" dans main() où j'essaie d'utiliser ma classe, qui est sensée représenter des fraction...Pouvez-vous me dire où est le problème ? Voici le code :
main.cpp :

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
#include <iostream>
#include "Fraction.h"
 
using namespace std;
 
int main()
{
    Fraction a(4,5);      //Déclare une fraction valant 4/5
    Fraction b(2);        //Déclare une fraction valant 2/1 (ce qui vaut 2)
    Fraction c,d;         //Déclare deux fractions valant 0
    c = a+b;               //Calcule 4/5 + 2/1 = 14/5
    d = a*b;               //Calcule 4/5 * 2/1 = 8/5
    cout << a << " + " << b << " = " << c << endl;
    cout << a << " * " << b << " = " << d << endl;
    if(a > b)
        cout << "a est plus grand que b." << endl;
    else if(a==b)
        cout << "a est egal a b." << endl;
    else
        cout << "a est plus petit que b." << endl;
    return 0;
}

Fraction.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
#ifndef FRACTION_H_INCLUDED
#define FRACTION_H_INCLUDED
 
#include <iostream>
 
class Fraction
{
private:
    int numerateur;
    int denominateur;
 
 
public:
    Fraction();
    Fraction(int a,int b);
    Fraction(int a);
    ~Fraction();
    int getnum() const; //renvoie la valeur du numerateur
    int getden() const; //renvoie la valeur du denominateur
    double getval() const; //renvoie la valeur numerique de la fraction
};
 
//puis les surcharges d'operateurs
bool operator==(Fraction const& a,Fraction const& b);
bool operator>(Fraction const& a,Fraction const& b);
int pgcd(int a,int b);
Fraction operator+(Fraction const& a,Fraction const& b);
Fraction operator*(Fraction const& a,Fraction const& b);
std::ostream& operator<<(std::ostream &flux,Fraction const& a);
 
#endif // FRACTION_H_INCLUDED

Fraction.cpp :

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
#include <iostream>
 
using namespace std;
 
int pgcd (int a,int b) //calcule le pgcd, pour pouvoir simplifier la fraction.
{
    if (a<0)
    {
        a=-a;
    }
    if (b<0)
    {
        b=-b;
    }
    int c;
    if (a<b)
    {
        c=a;
        a=b;
        b=c;
    }
    while (c!=0)
    {
        c=a%b;
        a=b;
        b=c;
    }
    return a;
}
class Fraction //la classe elle-meme
{
private:
    int numerateur;
    int denominateur;
 
 
public:
    Fraction::Fraction() //constructueur de Fraction par defaut
    {
        numerateur=0;
        denominateur=1;
    }
    Fraction::Fraction(int a,int b) constructeur de Fraction predeterminee
    {
        if (b==0)
        {
            cout<<"Erreur : impossible de diviser par 0. Denominateur initialise a 1."<<endl; //pas bete la mouette !
            b=1;
        }
        int c(pgcd(a,b));
        numerateur=a/c;
        denominateur=b/c;
    }
    Fraction::Fraction(int a) //constructeur de Fraction entiere
    {
        numerateur=a;
        denominateur=1;
    }
    ~Fraction() //destructueur ici pour la beaute de la chose...
    {
        //rien
    }
 
    int Fraction::getnum() const
    {
        return numerateur;
    }
 
    int Fraction::getden() const
    {
        return denominateur;
    }
    double Fraction::getval() const
    {
        return numerateur/denominateur;
    }
 
};
 
//puis les surcharges d'operateurs
bool operator==(Fraction const& a,Fraction const& b)
{
    double x(a.getval());
    double y(b.getval());
    if (x==y)
    {
        return true;
    }
    else
    {
        return false;
    }
}
bool operator>(Fraction const& a,Fraction const& b)
{
    double x(a.getval())
    double y(a.getval())
    if (x>y)
    {
        return true;
    }
    else
    {
        return false;
    }
}
 
Fraction operator+(Fraction const&a,Fraction const& b)
{
    int numa(a.getnum());
    int dena(a.getden());
    int numb(b.getnum());
    int denb(b.getden());
    int c(dena);
    numa*=denb;
    dena*=denb;
    numb*=c;
    denb*=c;
    Fraction ret(numb+numa,denb);
    return ret;
}
Fraction operator*(Fraction const& a,Fraction const& b)
{
    int numa(a.getnum());
    int dena(a.getden());
    int numb(b.getnum());
    int denb(b.getden());
    Fraction c(numa*numb,dena*denb);
    return Fraction;
}
ostream&*operator<<(ostream*&flux,*Fraction*const&*a) //surcharge de l'operateur de flux pour cout
{
    if(a.getden()==1)
    {
        cout<<a.getnum()<<endl;
    }
    else
    {
        cout<<a.getnum<<"/"<<a.getden<<endl;
    }
    return flux;
}

Alors bien sûr je n'ai pas surchargé TOUS les opérateurs, par exemple on ne peut pas faire de division...Bon, je ne me servirais probablement pas de ça, c'était juste pour m'entraîner à utiliser les classes et les surcharges d'opérateurs qui vont avec, mais ça ne fonctionne pas...Quelqu'un peut me dire quel est le problème ?
Merci d'avance.

**Bousk** · 07/04/2014, 15h08

Bonjour,

une classe définie dans un .h n'a pas à l'être dans le .cpp

**Glân von Brylân** · 07/04/2014, 18h17

Bon, voici le nouveau Fraction.cpp :

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
#include <iostream>
 
using namespace std;
 
int pgcd (int a,int b)
{
    if (a<0)
    {
        a=-a;
    }
    if (b<0)
    {
        b=-b;
    }
    int c;
    if (a<b)
    {
        c=a;
        a=b;
        b=c;
    }
    while (c!=0)
    {
        c=a%b;
        a=b;
        b=c;
    }
    return a;
}
 
 
    Fraction::Fraction()
    {
        numerateur=0;
        denominateur=1;
    }
    Fraction::Fraction(int a,int b)
    {
        if (b==0)
        {
            cout<<"Erreur : impossible de diviser par 0. Denominateur initialise a 1."<<endl;
            b=1;
        }
        int c(pgcd(a,b));
        numerateur=a/c;
        denominateur=b/c;
    }
    Fraction::Fraction(int a)
    {
        numerateur=a;
        denominateur=1;
    }
    Fraction::~Fraction()
    {
        //rien
    }
 
    int Fraction::getnum() const
    {
        return numerateur;
    }
 
    int Fraction::getden() const
    {
        return denominateur;
    }
    double Fraction::getval() const
    {
        return numerateur/denominateur;
    }
 
bool operator==(Fraction const& a,Fraction const& b)
{
    double x(a.getval());
    double y(b.getval());
    if (x==y)
    {
        return true;
    }
    else
    {
        return false;
    }
}
bool operator>(Fraction const& a,Fraction const& b)
{
    double x(a.getval())
    double y(a.getval())
    if (x>y)
    {
        return true;
    }
    else
    {
        return false;
    }
}
 
Fraction operator+(Fraction const&a,Fraction const& b)
{
    int numa(a.getnum());
    int dena(a.getden());
    int numb(b.getnum());
    int denb(b.getden());
    int c(dena);
    numa*=denb;
    dena*=denb;
    numb*=c;
    denb*=c;
    Fraction ret(numb+numa,denb);
    return ret;
}
Fraction operator*(Fraction const& a,Fraction const& b)
{
    int numa(a.getnum());
    int dena(a.getden());
    int numb(b.getnum());
    int denb(b.getden());
    Fraction c(numa*numb,dena*denb);
    return Fraction;
}
ostream&*operator<<(ostream*&flux,*Fraction*const&*a)
{
    if(a.getden()==1)
    {
        cout<<a.getnum()<<endl;
    }
    else
    {
        cout<<a.getnum<<"/"<<a.getden<<endl;
    }
****return*flux;
}

Ça ne fonctionne toujours pas...Qu'est-ce que j'ai encore fait de mal ?

**PilloBuenaGente** · 07/04/2014, 20h40

Bonsoir,

~~Les surcharges d'opérateur peuvent être mises dans la classe Fraction. (C'est même fortement conseillé).~~
Les membres privés pour celles ci pourront alors y être directement utilisés.

Ensuite certaines choses paraissent louche, "****return*flux;"
Le mieux serait de nous donner les erreurs.

**Glân von Brylân** · 07/04/2014, 21h49

Les *** sont des espaces dans mon code, ce doit être une erreur lors du copier-coller.
Pour les surcharges d'opérateurs, le tuto d'Open Classrooms (anciennement SDZ) disait qu'il ne fallait surtout pas les mettres dans la classe car ce n'étaient pas des méthodes...
Pour les erreurs, toujours le même "undefined reference to" :

**PilloBuenaGente** · 07/04/2014, 22h19

Inclure le header dans le fichier source :
Fraction.cpp

Code :

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

1
2
#include "Fraction.h"
...

#edit
Et en quoi les comparateurs ne pouraientt être des méthodes de cette classe ?
Fraction.h

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
#ifndef FRACTION_H_INCLUDED
#define FRACTION_H_INCLUDED
 
class Fraction
{
private:
    int numerateur;
    int denominateur;
public:
    bool operator==(Fraction const& f);
};
 
#endif//FRACTION_H_INCLUDED

Fraction.cpp

Code :

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

1
2
3
4
5
6
#include "Fraction.h"
 
bool Fraction::operator==(Fraction const& f)
{
    return((numerateur/denominateur)==(f.numerateur/f.denominateur));
}

On peut ainsi accèder directement aux membres privés.
Ainsi le .get() deviendrait obsolète, et on aura plus à se préocuper des valeurs contenue.

**Glân von Brylân** · 07/04/2014, 22h32

Ça ne change rien...Rah...Si je ne suis pas fichu de maîtriser les classes c'est pas la peine de me mettre à la POO !

**PilloBuenaGente** · 07/04/2014, 22h47

J'ai tout repris et enlevé les flux
main.cpp

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 "Fraction.h"
#include <iostream>
 
using namespace std;
 
int main()
{
    Fraction a(4,5);      //Déclare une fraction valant 4/5
    Fraction b(2);        //Déclare une fraction valant 2/1 (ce qui vaut 2)
    Fraction c,d;         //Déclare deux fractions valant 0
    c = a+b;               //Calcule 4/5 + 2/1 = 14/5
    d = a*b;               //Calcule 4/5 * 2/1 = 8/5
    return 0;
}

Fraction.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
#ifndef FRACTION_H_INCLUDED
#define FRACTION_H_INCLUDED
 
#include <iostream>
 
class Fraction
{
private:
    int numerateur;
    int denominateur;
 
 
public:
    Fraction();
    Fraction(int a,int b);
    Fraction(int a);
    ~Fraction();
    int getnum() const; //renvoie la valeur du numerateur
    int getden() const; //renvoie la valeur du denominateur
    double getval() const; //renvoie la valeur numerique de la fraction
};
 
//puis les surcharges d'operateurs
bool operator==(Fraction const& a,Fraction const& b);
bool operator>(Fraction const& a,Fraction const& b);
int pgcd(int a,int b);
Fraction operator+(Fraction const& a,Fraction const& b);
Fraction operator*(Fraction const& a,Fraction const& b);
 
#endif // FRACTION_H_INCLUDED

Fraction.cpp

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
#include "Fraction.h"
 
#include <iostream>
 
using namespace std;
 
int pgcd (int a,int b)
{
    if (a<0)
    {
        a=-a;
    }
    if (b<0)
    {
        b=-b;
    }
    int c;
    if (a<b)
    {
        c=a;
        a=b;
        b=c;
    }
    while (c!=0)
    {
        c=a%b;
        a=b;
        b=c;
    }
    return a;
}
 
 
    Fraction::Fraction()
    {
        numerateur=0;
        denominateur=1;
    }
    Fraction::Fraction(int a,int b)
    {
        if (b==0)
        {
            cout<<"Erreur : impossible de diviser par 0. Denominateur initialise a 1."<<endl;
            b=1;
        }
        int c(pgcd(a,b));
        numerateur=a/c;
        denominateur=b/c;
    }
    Fraction::Fraction(int a)
    {
        numerateur=a;
        denominateur=1;
    }
    Fraction::~Fraction()
    {
        //rien
    }
 
    int Fraction::getnum() const
    {
        return numerateur;
    }
 
    int Fraction::getden() const
    {
        return denominateur;
    }
    double Fraction::getval() const
    {
        return numerateur/denominateur;
    }
 
bool operator==(Fraction const& a,Fraction const& b)
{
    double x(a.getval());
    double y(b.getval());
    if (x==y)
    {
        return true;
    }
    else
    {
        return false;
    }
}
bool operator>(Fraction const& a,Fraction const& b)
{
    double x(a.getval());
    double y(a.getval());
    if (x>y)
    {
        return true;
    }
    else
    {
        return false;
    }
}
 
Fraction operator+(Fraction const&a,Fraction const& b)
{
    int numa(a.getnum());
    int dena(a.getden());
    int numb(b.getnum());
    int denb(b.getden());
    int c(dena);
    numa*=denb;
    dena*=denb;
    numb*=c;
    denb*=c;
    Fraction ret(numb+numa,denb);
    return ret;
}
Fraction operator*(Fraction const& a,Fraction const& b)
{
    int numa(a.getnum());
    int dena(a.getden());
    int numb(b.getnum());
    int denb(b.getden());
    Fraction c(numa*numb,dena*denb);
    return c;
}

Celui çi compile chez moi.

gl · 07/04/2014, 22h57

Envoyé par PilloBuenaGente

Les surcharges d'opérateur peuvent être mises dans la classe Fraction. (C'est même fortement conseillé).

Non, ce n'est pas fortement conseillé, bien au contraire. Pour un simple raison de symétrie (qui s'applique ici d'ailleurs étant donné qu'il y a un constructeur non explicit qui prends un int, ce qui est d'ailleurs assez logique) : la surcharge des opérateurs ne permettrait pas les mêmes conversions entre l'opérande de gauche et de droite (en gros b+2 avec b une fraction fonctionnerait, mais pas 2+b).

Généralement ce qui est conseillé est d'avoir des opérateurs "affectant" (i.e. de la forme op= comme +=, -=, etc.) membres de la classe et des opérateurs simple (+, -, etc.) non-membre et non-ami définis à partir des premiers.

Sinon concernant le problème initial : comme ce code est-il compilé ? Est-ce que Fraction.cpp est bien lié ?

**PilloBuenaGente** · 07/04/2014, 23h14

Envoyé par gl

la surcharge des opérateurs ne permettrait pas les mêmes conversions entre l'opérande de gauche et de droite (en gros b+2 avec b une fraction fonctionnerait, mais pas 2+b).

Pourquoi b+2 fonctionnerait ?

Ce serait b+c(2) avec b et c une fraction.

**Glân von Brylân** · 07/04/2014, 23h35

PilloBuenaGente : darn, pas chez moi !

Non le code ne compile pas ! J'ai essayé de redémarrer Code::Blocks (je possède la dernière version), et ça ne change rien : "undefined reference to" dans mein() sur tout ce qui vient de Fraction.h.

Comment ça "Est-ce que Fraction.cpp est bien lié ?" ? Je n'ai pas besoin de l'inclure où que ce soit, si ?

gl · 08/04/2014, 07h45

Envoyé par PilloBuenaGente

Pourquoi b+2 fonctionnerait ?

Parce qu'il y a un constructeur non explicit de fraction qui prends un entier en unique paramètre. Ce qui permet une conversion implicite (qui a du sens dans ce cas précis, un entier étant une fraction où le dénominateur vaut 1).

Envoyé par Neoflash Okashi

Comment ça "Est-ce que Fraction.cpp est bien lié ?" ? Je n'ai pas besoin de l'inclure où que ce soit, si ?

Il faut l'ajouter au projet code::block.
Peux-tu fournir le build log ?

**ekieki** · 08/04/2014, 07h47

Envoyé par Neoflash Okashi

PilloBuenaGente : darn, pas chez moi !

Non le code ne compile pas ! J'ai essayé de redémarrer Code::Blocks (je possède la dernière version), et ça ne change rien : "undefined reference to" dans mein() sur tout ce qui vient de Fraction.h.

Comment ça "Est-ce que Fraction.cpp est bien lié ?" ? Je n'ai pas besoin de l'inclure où que ce soit, si ?

Au niveau du projet CodeBlocks, il faut que tu indiques que le fichier Fraction.cc contribue à la fabrication de la cible.

**Glân von Brylân** · 08/04/2014, 12h18

J'ai trouvé, c'était effectivement ça ! J'avais rien coché dans "Belongs in tergets" lors de la création du fichier, pour voir ce que ça faisait...Ben ça fait que ça marche pas patate...
Là j'ai encore quelques erreurs, mais des trucs comme "expected ';' or ',' before 'double'", "stray \240 in program", des trucs que je connais et peux régler tout seul...Merci à tous !

**LittleWhite** · 08/04/2014, 12h45

Le stray 240, c'est un espace blanc (ou un caractère invisible), souvent se trouver autour des '[' ']' qui est, car on appuie trop longtemps sur Alt Gr. En effet, un Alt Gr + Espace provoque ce caractère invisible qui n'est pas un espace.