Discussion :

[Kimy_Ire]

Ami du soir, bonsoir !

C'est re-l'enquiquineur de service !
J'espère que je ne flood pas trop... mais je ne sais pas si c'était une bonne idée de mettre un problème complètement différent à la suite de mon précédent message. J'ouvre donc ce nouveau topic...

Voila, je voudrais overloader l'operator "+" d'une classe.

Mon main :

Code :

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

1
2
3
4
    player play1 = player(1 , "Kimy_Ire"); //créer un joueur
    card card1 = card(); //créer une carte
    play1 = play1 + card1; //serait sencé attribuer une carte à un joueur
    play1.info(); //vérification

Vous l'aurez compris, je souhaiterai pouvoir ajouter un élément "card" à un joueur avec le signe "+".

Mon player.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
class player
{
      private:
              int id;
              std::string name;
              bool GM;
              bool mayor;
              int t;
 
              card player_card;
 
      public:        
             player(int id_player, std::string name_player);
             //Liste de toutes les méthodes...
 
             player operator + (card c);
 
 
};

Mon player.ccp :

Code :

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

1
2
3
4
5
6
7
8
9
10
void player::give_card(card c)
{
     player_card = c;
}
 
 
player player::operator + (card c)
{
       give_card(c);
}

J'ai retiré toutes les mentions inutiles. Bref, j'ai un soucis de compréhension, je pense. Je ne sais pas pourquoi cela plante. En effet, je n'ai pas de message d'erreur mais le compilateur plante complètement...

J'ai essayé de remplacer toutes les "card c" par des "card *c" ainsi qu'aux endroits appropriés lors des définitions mais cela m'a juste conduit à avoir des erreurs de compilation du type :

no match for 'operator+' in 'play1 + *card1'

J'ai essayé l'ajout de "return this" à la fin de "player player::operator + (card c)" sans réel succès...

J'espère que vous pourrez m'aider !

Merci infiniment du temps que vous prendrez pour moi par avance !


Kimy from Ireland.

[Steph_ng8]

Bonsoir,
Je pense que tu devrais commencer par t'intéresser aux références.
Parce que là, tu fais beaucoup de copies d'objets pour rien.


D'autre part, la sémantique de l'opérateur « + » est généralement : « À partir de deux instances qui ne seront pas modifiées, créer une nouvelle instance ».
Cela ne correspond pas vraiment à ce que tu veux faire.

En effet, l'instruction play1 = play1 + card1 peut se décomposer ainsi :

1. Utiliser play1 et card1 sans les modifier pour créer une nouvelle instance de player (operator+) ;
2. Remplacer le contenu de play1 par l'instance nouvellement créée (operator=).

Encore une fois, beaucoup de création/copie/affectation pour rien.


Ici, c'est plutôt l'opérateur « += » que tu devrais utiliser.
« À partir d'une instance qui ne sera pas modifiée, modifier la valeur d'une autre instance, et retourner cette instance ».

Ce n'est pas une obligation, mais l'opérateur « + » est souvent implémenté à partir de l'opérateur « += » :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
TYPE operator + (TYPE const& lhs, TYPE const& rhs)
{
    TYPE ret = lhs;
    return ret += rhs;
}
 
/***********************/
 
TYPE1 TYPE1::operator + (TYPE2 const& rhs) const
{
    TYPE1 ret = *this;
    return ret += rhs;
}

Voilà, maintenant à toi de jouer.
Il ne reste plus qu'à écrire un opérateur « += ».

[gbdivers]

Bonjour

Perso, sémantiquement, ça me gène d'additionner des pommes et des poires. Il se semblerait plus logique d'utiliser l'opérateur <<

[Steph_ng8]

Pas faux...

+1

[Kimy_Ire]

Merci pour votre aide ! Je vous tiens au courant pour la suite !

[3DArchi]

En effet, je n'ai pas de message d'erreur mais le compilateur plante complètement...

Ton compilateur ou ton programme plante ??

S'il s'agit d'un plantage à l'exécution, il faudrait voir card car c'est probablement un problème de copie de cette classe.

Sinon, +1000 avec gbdivers et ses pommes et ses poires.