Bonjour à tous cher zéros !

Alors voilà, je débute en C++, et je suis en train de faire l'exercice du RPG. J'ai fait tout ce que proposait M@teo21, et quand j'ai compilé tout marchait très bien

Maintenant, j'ai voulu rendre le programme plus intéressant, en permettant à l'utilisateur de pouvoir intéragir avec le programme via cin, jusqu'à ce que l'un des 2 combattants n'ait plus de points de vie. Et me voilà avec un gros problème... voilà tout d'abord le code en question :

main.cpp




Le problème vient des lignes 4 et 5 .. Effectivement, chaque tour, l'attaquant change de personnage.. donc je dois faire une référence ou un pointeur vers le personnage qui attaque, de même pour le personnage qui défend.
La référence attaquant pointera à tour de rôle sur personnage1 et personnage2 à chaque passage dans la boucle while..

J'imagine que c'est vraiment simple, mais le noob que je suis n'arrive pas à saisir la subtilité

Si vous pouviez me corriger, en m'expliquant rapidement le pourquoi du comment, je vous en serais extrèmement reconnaissant

Merci à l'avance !!!

Non les références ne peuvent pas être utilisées ainsi... ce que tu pourrais faire c'est une fonction qui prendre en paramètres par référence l'attaquant et l'attaqué et faire les action de "l'attaque"... Avec un simble booléen ou un modulo sur le nombre de tour tu décide quel personnage attaquera.


en passant :

C'est supposé faire quoi? On est pas en PHP ou en JavaScript!

effectivement, j'ai souvent tendance à essayer des méthodes que je connais d'autres langages pour parvenir à mes fins X_X

Merci pour l'explication, je devrais m'en sortir

Et pourquoi pas coupler les deux techniques ? Ca pourrait faire joli

Je reconnais que c'est peut-être pas très performant, mais ça minimise les changements de code !
Ou alors, autrement plus brutal


J'avoue que c'est vraiment juste histoire de dire quoi

Citation : groscask

Et pourquoi pas coupler les deux techniques ? Ca pourrait faire joli

1. unsigned tour = 0;
2. while ( ... ) {
3.     Personnage & attaquant = (tour % 2 ? perso1 : perso2);
4.     Personnage & defenseur = (tour % 2 ? perso2 : perso1);
5.     //... Action !
6.     tour++;
7. }

Je reconnais que c'est peut-être pas très performant, mais ça minimise les changements de code !
Ou alors, autrement plus brutal

1. Personnage & attaquant = perso1;
2. Personnage & defenseur = perso2;
3. while ( ... ) {
4.     // ... Action !
5.     Personnage & ancien_attaquant = attaquant;
6.     attaquant = defenseur;
7.     defenseur = ancien_attaquant;
8. }

La deuxième technique nécessite un tas de copie de données inutiles et en plus attaquant et defenseur seront 100% égaux après le premier tour de boucles.

la première technique donne la même chose qu'avec une fonction mais sans fonction, code non modularisé.


Exemple d'une ébauche de code modularisé... (Les saisies n'étant pas validées)

Copie de données inutiles ? Alors que je ne fais que copier des références ? C'est un bête swap dans le fond... Donc des références ou des pointeurs, je ne vois pas où sont les copies inutiles...

Si tu peux m'éclairer, je suis preneur !
Quoiqu'il en soit : j'ai déjà mentionné que c'était du code "histoire de"

ng", avec des espaces, auquel cas il y a une boucle infinie..

J'imagine qu'il considère que "Glaive" est le nom, et cherche jusqu'à la fin des temps ce que peux être " long"...
Comment corriger ce problème ?

Merci beaucoup de vos interventions ss="code2 cpp">

1. #include <iostream>
2. #include <string>
3. #include "Personnage.h"
4. using namespace std;
5. void tourDeJeu(Personnage &attaquant, Personnage &defenseur);
6. int main()
7. {
8.     cout << "##########################################" << endl;
9.     cout << "# Bienvenue dans l'arène des gladiateurs #" << endl;
10.     cout << "##########################################" << endl << endl;
11.     string nomPersonnage1, nomPersonnage2;
12.     cout << "Premier combattant :"<< endl << "Quel est le nom de votre personnage ? ";
13.     cin >> nomPersonnage1;
14.     cout << "Deuxième combattant :"<< endl << "Quel est le nom de votre personnage ? ";
15.     cin >> nomPersonnage2;
16.     cout << endl << endl;
17.     Personnage personnage1(nomPersonnage1), personnage2(nomPersonnage2,"Epée aiguisée",20);
18.     int nombreDeTours = 0;
19.     while (personnage1.estVivant() && personnage2.estVivant())
20.     {
21.         if (nombreDeTours % 2 == 0)
22.         {
23.                 tourDeJeu(personnage1, personnage2);
24.         }
25.         else
26.         {
27.                 tourDeJeu(personnage2, personnage1);
28.         }
29.         nombreDeTours++;
30.     }
31.     if (personnage1.estVivant())
32.     {
33.         cout << personnage1.getNom() << " est le vainqueur du duel !!!!!!";
34.         cout << " Félicitations à toi, noble guerrier !" << endl << endl;
35.     }
36.     else if (personnage2.estVivant())
37.     {
38.         cout << personnage2.getNom() << " est le vainqueur du duel !!!!!!";
39.         cout << " Félicitations à toi, noble guerrier !" << endl << endl;
40.     }
41.     cout << "Etat final des 2 combattants :" << endl;
42.     cout << "------------------------------" << endl << endl;
43.     personnage1.afficherEtat();
44.     personnage2.afficherEtat();
45.     cout << endl;
46.     return 0;
47. }
48. void tourDeJeu(Personnage &attaquant, Personnage &defenseur)
49. {
50.         attaquant.regenererMana();
51.         cout << attaquant.getNom() << ", A toi de jouer !" << endl;
52.         cout << "Voilà ton profil :" << endl;
53.         cout << "------------------" << endl;
54.         attaquant.afficherEtat();
55.         bool erreur;
56.         do
57.         {
58.                 cout << "Que veux-tu faire ? (choix : attaquer - boirePotionDeVie - changerArme)" << endl;
59.                 string choixTour;
60.                 cin >> choixTour;
61.                 if (choixTour == "attaquer")
62.                 {
63.                         string typeAttaque;
64.                         cout << "Que veux-tu faire comme attaque ? (choix : magique - tranchante)" << endl;
65.                         cin >> typeAttaque;
66.                         attaquant.attaquer(defenseur, typeAttaque);
67.                         cout << "Vlam ! " << attaquant.getNom() << " a porté une attaque " << typeAttaque;
68.                         cout << " mortelle, et " << defenseur.getNom() << " se voit infliger des dégats.";
69.                         cout << endl << endl;
70.                         defenseur.afficherEtat();
71.                         erreur = false;
72.                 }
73.                 else if (choixTour == "boirePotionDeVie")
74.                 {
75.                         attaquant.boirePotionDeVie(20);
76.                         attaquant.afficherEtat();
77.                         erreur = false;
78.                 }
79.                 else if (choixTour == "changerArme")
80.                 {
81.                         cout << "Quelle arme voulez-vous prendre ? ";
82.                         string nomArme;
83.                         cin >> nomArme;
84.                         cout << endl << "Combien de dégats infligés par l'arme ? ";
85.                         int nbDegats;
86.                         cin >> nbDegats;
87.                         attaquant.changerArme(nomArme,nbDegats);
88.                         attaquant.afficherEtat();
89.                         erreur = false;
90.                 }
91.                 else
92.                 {
93.                         cout << "Vous avez rentré un mauvais choix, reessayez !" << endl << endl;
94.                         erreur = true;
95.                 }
96.                 cout << endl << endl;
97.         } while (erreur);
98. }

Mais effectivement, suite à vos interventions, je me rend compte que je pourrais faire une autre fonction avec la boucle while pour rendre le code plus lisible (d'ailleurs j'ai 0 commentaires, faudra que j'en fasse quelques uns histoire de )

Merci beaucoup en tout cas


PAR CONTRE :

J'ai un autre problème, et ne sais pas vraiment comment corriger cela...
Lorsque je veux changer d'arme, le programme me demande le nom de la nouvelle arme. Ce nom est récupéré avec cin, et placé dans la variable nomArme. Le problème surgit lorsque j'écris un nom tel que "Glaive lo

Citation : groscask

Copie de données inutiles ? Alors que je ne fais que copier des références ? C'est un bête swap dans le fond... Donc des références ou des pointeurs, je ne vois pas où sont les copies inutiles...

Si tu peux m'éclairer, je suis preneur !
Quoiqu'il en soit : j'ai déjà mentionné que c'était du code "histoire de"

Les références sont initialisé lors de leur déclaration et ne peuvent pas "référer" un autre zone mémoire lors du déroulement... Ce que tu fais ici c'est appeler l'opérateur d'affectation de la classe.

Exemple :


sortie :



Pourtant, selon toi, a devrait valoir 10. En fait à la ligne 12 rb qui réfère à b copie la valeur de ra (référence de a, soit 0). À ligne suivante rt toujours référé à rb qui réfère toujours à b qui vaut maintenant 0 et a affecte cette valeur. a et b ont la même valeur et celle que contenait b est perdue.

Citation : Fandekasp

J'ai un autre problème, et ne sais pas vraiment comment corriger cela...
Lorsque je veux changer d'arme, le programme me demande le nom de la nouvelle arme. Ce nom est récupéré avec cin, et placé dans la variable nomArme. Le problème surgit lorsque j'écris un nom tel que "Glaive long", avec des espaces, auquel cas il y a une boucle infinie..

J'imagine qu'il considère que "Glaive" est le nom, et cherche jusqu'à la fin des temps ce que peux être " long"...
Comment corriger ce problème ?

Je viens d'en parler dans un autre sujet... utilise getline()!

Pour les espaces j'ai trouvé dans la doc ça :

Tu fais un getline(cin, variableString) et c'est tout !

Pour récupérer un nombre et que vous ne voulez pas que quelqu'un, au lieu de mettre un nombre entier, met quelque chose d'autre qui causera une boucle inconditionelle, tu fais :



Ainsi, tu n'auras plus de boucle interminable !

A plus !

Ah super !!!

Merci beaucoup

Va falloir que j'apprenne à chercher plus efficacement dans la doc lol


A bientôt

J'ai oublié de te dire ça : il faut inclure <sstream> pour que ça marche !
@+

oué j'avais une erreur, mais j'ai vite trouvé le problème


Parcontre, j'ai un problème étrange concernant le getline().

Je l'ai fait 2 fois :
- 1 fois pour demander le nom de l'arme
- 1 fois pour demander le nombre de dégats infligés par l'arme (j'y ajoute un stringstream après pour convertir en int)

Le problème est qu'il ne s'arrête pas pour me demander le nom de l'arme, mais me demande le nombre de dégats




Voilà le code :




Qu'en pense-tu ? n'est-ce pas vraiment étrange ?

Merci beaucoup MatteX pour tes précisions...
Dans ma tête, les références n'étaient que des pointeurs améliorés, là pour la syntaxe... Je prendrai garde à l'avenir de ne pas les considérer comme tel

Est-ce que tu fais d'autres saisies avec cin avant le getline. Car si ton buffer contient un '\n' c'est normal qu'il ne demande pas le nom de l'arme

Ah effectivement c'est le cas, je fais plusieurs cin avant... mais comment faire pour vider le buffer alors ?


edit:
Bon j'ai remplacé tous les cin pour des getline(), donc je n'ai plus de problèmes.. mais si vous avez d'autres suggestions pour "vider" le buffer, je suis preneur

réponse @ FAQ C++ de developpez.com
(Lien dans ma signature)