Discussion :

[djaws]

Bonjour à tous,

Je suis actuellement en train d'apprendre le C++ et après de nombreuses heures à chercher, je vous demande de l'aide...

Pour résumer, je chercher à comparer deux objets appartenant à la même classe "Duree" afin de verifier si les deux durées identiques.

Contenu du fichier Duree.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
#ifndef DEF_DUREE
#define DEF_DUREE
 
class Duree
{
public:
	// Constructeur de la classe
	Duree(int heures = 0, int minutes = 0, int secondes = 0);
 
	// Opérateur de comparaison pour vérifier que les deux objets de la classe Duree sont égaux avec le symbole "=="
	bool operator==(Duree const& a, Duree const& b);
        bool Duree::estEgal(Duree const& b) const;
 
 
	private:
 
	int m_heures;
	int m_minutes;
	int m_secondes;
};
 
#endif

Contenu du fichier Duree.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
#include "Duree.h"
 
using namespace std;
 
// Constructeur de la classe Duree
Duree :: Duree(int heures, int minutes, int secondes) : m_heures(heures), m_minutes(minutes), m_secondes(secondes) {
}
 
// Opérateur de comparaison pour vérifier que les deux objets de la classe Duree sont égaux avec le symbole "=="
bool operator==(Duree const& a, Duree const& b) {
	return a.estEgal(b);
}
 
bool Duree::estEgal(Duree const& b) const {
	//Teste si a.m_heure == b.m_heure etc.  
	return (m_heures == b.m_heures && m_minutes == b.m_minutes && m_secondes == b.m_secondes);
}

Contenu main.cpp :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
#include "Duree.h"
 
int main() {
	Duree duree1(0, 10, 28), duree2(0, 10, 28);
 
	if (duree1 == duree2)
		cout << "Les durees sont identiques";
	else
		cout << "Les durees sont differentes";
 
	return 0;
}

Et quand je compile mon projet, j'obtiens cette erreur :
/usr/bin/g++ -c "/home/djaws/Projects/OpenClassRooms/OpenClassRooms/OpenClassRooms/Duree.cpp" -g -O0 -Wall -o ./Debug/Duree.cpp.o -I. -I.
/usr/bin/g++ -c "/home/djaws/Projects/OpenClassRooms/OpenClassRooms/OpenClassRooms/main.cpp" -g -O0 -Wall -o ./Debug/main.cpp.o -I. -I.
In file included from /home/djaws/Projects/OpenClassRooms/OpenClassRooms/OpenClassRooms/main.cpp:1:
/home/djaws/Projects/OpenClassRooms/OpenClassRooms/OpenClassRooms/Duree.h:11:48: error: 'bool Duree::operator==(const Duree&, const Duree&)' must have exactly one argument
bool operator==(Duree const& a, Duree const& b);

In file included from /home/djaws/Projects/OpenClassRooms/OpenClassRooms/OpenClassRooms/Duree.cpp:1:
/home/djaws/Projects/OpenClassRooms/OpenClassRooms/OpenClassRooms/Duree.h:11:48: error: 'bool Duree::operator==(const Duree&, const Duree&)' must have exactly one argument
bool operator==(Duree const& a, Duree const& b);

J'espère que vous verrez s'il y a problème dans mon code, si vous avez besoin de plus d'infos je reste à votre disposition

Djaws

"I like being weird. Weird's all i've got.
That, and my sweet style" - Maurice Moss.

[Guesset]

Bonjour,

Comme ce n'est pas une déclaration globale de l'opérateur, le compilateur préfère :

Code C++ :

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

1
2
3
4
bool operator==(const Duree & d);          // dans la déclaration de la classe.hpp
 
bool Duree::operator==(const Duree & d) {  // dans l'implémentation de la class .cpp
}

Le premier argument est de facto l'instance de l'objet appelant, il est à comparer au second (ici d). Comme le compilateur l'indique, la fonction n'a qu'un seul argument.

Par ailleurs le code ne fonctionnera pas correctement : par exemple Duree(1, 0, 0) sera considéré comme != Duree(0, 60, 0) qui est autorisée par l'initialisation mais non renormalisé. Il pourrait être préférable de travailler avec un stockage int32_t ou int64_t, par exemple int32_t dt = ((heures * 60) + minutes)*60 + secondes et une fonction d'affichage.

Salutations

[dalfab]

Bonjour,

Pour définir l'opérateur == entre 2 Duree, tu as plusieurs possibilités:
- ça peut être une fonction membre à un seul paramètre (l'autre c'est l'objet courant)

Code :

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

     bool  operator==( Duree const& b )const;

- ça peut être une fonction amie avec 2 paramètres

Code :

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

     friend bool  operator==( Duree const& a, Duree const& b );

- ça peut être une fonction libre, donc à déclarer à l'extérieur de la classe Duree

Code :

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

bool  operator==( Duree const& a, Duree const& b );

- ça peut aussi utiliser l'opérateur "spaceship" <=> (c'est dans le C++20, mais ton cours s'est arrêté avant le C++98.), ça définit en une seule fois == != < <= > >=, je ne m'y arrête pas ici.

Ensuite tu peux définir le corps de la fonction dans un fichier source par:

Code :

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

1
2
bool  Duree::operator==( Duree const& b )const { ...... }     // 1er cas
bool  operator==( Duree const& a, Duree const& b ) { ...... } // 2nd ou 3ième cas

Tu peux aussi définir les cas comme étant inline. Dans ce cas le corp de la fonction est dans le fichier entête.

Code :

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

1
2
3
     bool  operator==( Duree const& b )const { ...... }
     friend bool  operator==( Duree const& a, Duree const& b ) { ...... }
inline bool  operator==( Duree const& a, Duree const& b ) { ...... }

S'il fallait en préférer une, ça serait la fonction amie et inline, c'est celle qui te donnera le meilleur "Argument Dependant Lookup", concept à voir plus tard.

Mais qu'as-tu écris dans ta class Duree? Ça ne correspond à aucun des 3 cas possibles.