Discussion :

[Ice Berg]

Salut, je suis debutant et j ai un petit soucis avec cet petit exemple d'implementation de surcharge d'operateur+. le code est le suivant:

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
 
#ifndef DUREE_H
#define	DUREE_H
 
class Duree {
 
public:
    Duree(int heures , int minutes, int secondes);
    int GetSecondes() const;
    int GetMinutes() const;
    int GetHeures() const;
 
 
private:
    int m_heures;
    int m_minutes;
    int m_secondes;
};
 
Duree operator+(Duree const& a, Duree const& b);
 
#endif	/* DUREE_H */
 
 
#include "Duree.h"
using namespace std;
 
Duree::Duree(int heures , int minutes, int secondes) {
 
    m_heures = heures;
    m_minutes = minutes;
    m_secondes = secondes;
 
}
 
int Duree::GetSecondes() const {
    return m_secondes;
}
 
int Duree::GetMinutes() const {
    return m_minutes;
}
 
int Duree::GetHeures() const {
    return m_heures;
}
 
Duree operator+(Duree const& a, Duree const& b)
{
    Duree resultat; 
 
     resultat.GetHeures() = a.GetHeures() + b.GetHeures();
     resultat.GetMinutes() = a.GetMinutes() + b.GetMinutes();
     resultat.GetSecondes() = a.GetSecondes() + b.GetSecondes();
 
    return resultat;  
}
 
 
 
#include <iostream>
 
#include "Duree.h"
 
using namespace std;
 
int main() {
 
    Duree duree1(0, 10, 28), duree2(0, 10, 28);
 
    Duree somme = duree1 + duree2;
 
     cout<< "la somme est:"<< somme.GetHeures()<<"h "<< somme.GetMinutes()<< "min "<< somme.GetSecondes()<< "sec"<< endl;
 
 
    return 0;
}

Mais le compilateur signale une erreur au niveau du fichier.cpp. Que faire?

Merci d avance pour votre aide

[Bousk]

Bonjour,

indiquer l'erreur serait plus simple pour les intervenants.

M'enfin,

Code :

S�lectionner tout - Visualiser dans une fen�tre � part

1
2
3
resultat.GetHeures() = a.GetHeures() + b.GetHeures();
     resultat.GetMinutes() = a.GetMinutes() + b.GetMinutes();
     resultat.GetSecondes() = a.GetSecondes() + b.GetSecondes();

Comprends-tu ce que tu �cris ?
Ces 3 lignes sont surement ton erreur et n'ont pas grand sens.
GetXX comme son nom l'indique, retourne un entier, par copie.
A quel moment crois-tu que ceci affecte une heure, minute ou seconde � ton objet dur�e ?

Code :

S�lectionner tout - Visualiser dans une fen�tre � part

1
2
3
resultat.m_heures = a.GetHeures() + b.GetHeures();
     resultat.m_minutes = a.GetMinutes() + b.GetMinutes();
     resultat.m_secondes = a.GetSecondes() + b.GetSecondes();

[Ice Berg]

Bonjour, j'ai essay� de modifier le code tel que Bousk me l'a conseill�, mais j obtiens toujours des erreurs de compilation entre la ligne 48 et 57:

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
 
#ifndef DUREE_H
#define	DUREE_H
 
class Duree {
 
public:
    Duree(int heures , int minutes, int secondes);
    int GetSecondes() const;
    int GetMinutes() const;
    int GetHeures() const;
 
 
private:
    int m_heures;
    int m_minutes;
    int m_secondes;
};
 
Duree operator+(Duree const& a, Duree const& b);
 
#endif	/* DUREE_H */
 
 
#include "Duree.h"
using namespace std;
 
Duree::Duree(int heures , int minutes, int secondes) {
 
    m_heures = heures;
    m_minutes = minutes;
    m_secondes = secondes;
 
}
 
int Duree::GetSecondes() const {
    return m_secondes;
}
 
int Duree::GetMinutes() const {
    return m_minutes;
}
 
int Duree::GetHeures() const {
    return m_heures;
}
 
Duree operator+(Duree const& a, Duree const& b)
{
    Duree resultat; 
 
     resultat.m_heures = a.GetHeures() + b.GetHeures();
     resultat.m_minutes = a.GetMinutes() + b.GetMinutes();
     resultat.m_secondes = a.GetSecondes() + b.GetSecondes();
 
    return resultat;  
}
 
 
 
#include <iostream>
 
#include "Duree.h"
 
using namespace std;
 
int main() {
 
    Duree duree1(0, 10, 28), duree2(0, 10, 28);
 
    Duree somme = duree1 + duree2;
 
     cout<< "la somme est:"<< somme.GetHeures()<<"h "<< somme.GetMinutes()<< "min "<< somme.GetSecondes()<< "sec"<< endl;
 
 
    return 0;
}

[PilloBuenaGente]

Bonjour,

Code :

S�lectionner tout - Visualiser dans une fen�tre � part

int GetSecondes() const;

Ceci renvoit une copie de l'entier seconde.
Lui affecter un nouveau nombre ne sera donc que temporaire. (Comme l'a expliqu� Bousk).

Retour par copie = const int getSecondes() const { return m_secondes; } (Bien penser a mettre le retour constant, cela peut eviter ce genre d'erreur.
Retour par reference = int& getSecondes() { return m_secondes; } (La valeur retourn�e sera donc modifiable, faire attention avec son utilisation)

Pour retourner l'objet lui m�me il faut donc soit passer le retour par reference non constante, soit utiliser un setter, ou encore le mieux a faire serait ainsi :

Code :

S�lectionner tout - Visualiser dans une fen�tre � part

1
2
3
4
5
6
7
Duree operator+(Duree const& a, Duree const& b)
{
     int heures = a.GetHeures() + b.GetHeures();
     int minutes = a.GetMinutes() + b.GetMinutes();
     int secondes = a.GetSecondes() + b.GetSecondes();
     return Duree(heures, minutes, secondes);  
}

[Bousk]

Faudrait r�fl�chir 2s avant de copier/coller du code.
Tes m_XX sont private, et ton operator+ n'est pas friend.

[PilloBuenaGente]

Je me suis fait tapper sur les doigts sur ce retour.
Dans ce cas pr�sent, il serait donc envisageable de mettre les operateurs directement dans la classe Dur�e non ?
Car faire "1 + duree" n'aurait pas de sens !?