Discussion :

[nunurs83]

Bonjour !!!
Donc j'ai une classe Horaire qui contient une donnée membre "minutes" et je veux pouvoir ajouter une durée en minutes à un Horaire en surchargeant l'opérateur + .
(pour le moment je n'ai fait aucun contrôle)

code de la surcharge :

Code :

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

1
2
3
4
5
 
void Horaire::operator+(const short uneDuree)
{
   minutes += uneDuree;     
}

et donc dans mon main j'aimerai pouvoir faire ceci :

Code :

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

1
2
3
4
5
6
 
short DureeMinute;
Horaire unHoraire;
unHoraire = new Horaire;
 
unHoraire += DureeMinute ;

Et bon apparement ca ne marche pas comme cela :-(

[Patriarch24]

Code :

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

1
2
3
 
Horaire unHoraire;
unHoraire = new Horaire;

Aie. N'aurais-tu pas oublié '*' dans ce code ?

Code :

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

1
2
3
4
5
 
void Horaire::operator+(const short uneDuree)
{
   minutes += uneDuree;     
}

Ceci n'est pas un opérateur +, mais un opérateur += mal écrit .
operator+ doit renvoyer une valeur (de type "const Horaire" par exemple), de son côté operator+= renvoie une référence sur this (return *this => Horaire& operator+=(...)).
Par ailleurs, operator+ ne doit pas être membre car il ne faut pas oublier que l'operateur + est commutatif, ce qui n'est plus vrai dans le cas d'un opérateur membre.

Ecrit comme ça, ton code ne risque pas de fonctionner. Un petit passage sur un tutoriel du site ne ferait pas de mal.
Au boulot.

[Mat007]

Salut,

+ et += sont sont deux opérateurs distincts...
Donc remplace operator+ par operator+= dans ton code déjà.

edit : ah oui en effet y'a un pb avec la signature aussi...

Code :

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

1
2
3
4
5
6
 
Horaire& Horaire::operator+=(const short uneDuree)
{
   minutes += uneDuree;
   return *this;
}

MAT.

[Médinoc]

+1 pour la confusion operator+ / operator+=

Quand je pense qu'il y a des profs qui nous apprennent des c****ries pareilles...

[Patriarch24]

Quand je pense qu'il y a des profs qui nous apprennent des c****ries pareilles...

Bah oui, c'est vrai, yen a qui croient savoir. Mais C++ est loin d'être un engin simplet .

[nunurs83]

Merci pour vos réponses je vais changer tout cela en conséquence

[Luc Hermitte]

Et pas de new. C'est pas hyper efficace, ni adapté, quand on manipule des valeurs.