Mon "jeu" David VS Goliath

span> Personnage::afficherEtat()

{

cout << m_vie << "/ 10 d'energie." << endl;

}

Voila, le jeu marche très bien, je voulais juste avoir vos avis sur ce code, je compte faire une version plus améliorée (avec mana, différentes armes, un mode 2 joueurs,des combats plus longs...etc...)

Voila, tout est dit, Merci d'avance

de, mais plus forte.
Vous avez 1/2 chance de toucher Goliath, et vous le ferez perdre 2/10 d'energie.

L'attaque puissante est tres lente mais tres puissante
Vous avez 1/3 chance de toucher Goliath, et vous le ferez perdre 3/10 d'energie.

En plus des attaques, vous pouvez regenerez votre energie en buvant une potion.
Il y a 1/2 chance que la potion agisse et vous redonne 2/10 d'energie.

Soyez le meilleur, bonne chance !
Voici la source :

Main.cpp :

#include <iostream>
#include <string>
#include<time.h>
#include "Personnage.h" // Ne pas oublier
using namespace std;
int main()
{
/*Pour commencer, declaration des variables, ,
ia c'est l'adverssaire qui choisit son attaque (nombre au hasard)
choixAttaque c'est le joueur qui choisit son attaque
chance determine si une attaque est reussie ou non (nombre au hasard) */
int chance = 0, ia = 0, choixMenu = 0, choixAttaque = 0, choixReco = 0;
bool recommencer = true;
srand(time(NULL));
// Création des personnages
Personnage david, goliath;
//Presentation
cout << " ***************************************************" << endl;
cout << " ***************************************************" << endl;
cout << " *** Bienvenue dans le combat David VS Goliath ! ***" << endl;
cout << " ***************************************************" << endl;
cout << " ***************************************************" << endl << endl;
do //Boucle de recommencer le jeu
{
do // Boucle si Aide a été choisi (on revient au menu
{
do // Boucle en cas d'erreur
{
cout << endl << endl << "1.Lancer le jeu" << endl << "2.Aide" << endl << "3.Quitter" << endl << endl << "Votre choix : ";
cin >> choixMenu;
if(choixMenu < 1 || choixMenu > 3)
{
cout << endl << "Choix incorrect, vous avez le choix entre : ";
}
}while(choixMenu < 1 || choixMenu > 3 );
if (choixMenu == 2)
{
cout << endl << endl << endl << endl << "Les regles sont simples, vous etes David et votre but est de tuer Goliath." << endl;
cout << "Pour cela, vous disposez de 3 attaques :" << endl << endl;
cout << endl << "1.Attaque simple" << endl << "2.Attaque double" << endl << "3.Attaque puissante" << endl;
cout << endl << "L'attaque simple est tres rapide, mais aussi tres faible." <<endl << "Vous avez 2/3 chance de toucher Goliath, et vous le ferez perdre 1/10 d'energie." << endl;
cout << endl << "L'attaque double est moins rapide, mais plus forte." << endl << "Vous avez 1/2 chance de toucher Goliath, et vous le ferez perdre 2/10 d'energie." << endl;
cout << endl << "L'attaque puissante est tres lente mais tres puissante" << endl << "Vous avez 1/3 chance de toucher Goliath, et vous le ferez perdre 3/10 d'energie." << endl;
cout << endl << endl << "En plus des attaques, vous pouvez regenerez votre energie en buvant une potion." << endl << "Il y a 1/2 chance que la potion agisse et vous redonne 2/10 d'energie." << endl;
cout << endl << endl << "Soyez le meilleur, bonne chance !" << endl << endl;
}
}while(choixMenu == 2);
if(choixMenu == 3)
{
cout << endl << "Au revoir !" << endl;
recommencer = false;
}
else if(choixMenu == 1)
{
while(david.estVivant() == true && goliath.estVivant() == true)
{
cout << endl << endl << " *** COMBAT ! ***" << endl;
do
{
cout << endl << "1.Attaque simple" << endl << "2.Attaque double" << endl << "3.Attaque puissante" << endl << "4.Boire Potion" << endl << endl << "Votre choix : ";
cin >> choixAttaque;
if(choixAttaque < 1 || choixAttaque > 3)
{
cout << endl << "Choix incorrect, vous avez le choix entre : " << endl;
}
}while(choixAttaque < 1 || choixAttaque > 4);
cout << endl;
if(choixAttaque == 1 || choixAttaque == 3)
{
chance = (rand() % (3 - 1 + 1)) + 1;
if(choixAttaque == 1)
{
if(chance == 1 || chance == 2)
{
david.attaquer(goliath, 1);
cout << "David utilise Attaque simple sur Goliath" << endl;
}
else if(chance == 3)
{
cout << "Attaque simple echoue" << endl;
}
}
if(choixAttaque == 3)
{
if(chance == 1)
{
david.attaquer(goliath, 3);
cout << "David utilise Attaque puissante sur Goliath" << endl;
}
else if(chance == 2 || chance == 3)
{
cout << "Attaque puissante echoue" << endl;
}
}
}
else if(choixAttaque == 2 || choixAttaque == 4)
{
chance = (rand() % (2 - 1 + 1)) + 1;
if(choixAttaque == 2)
{
if(chance == 1)
{
david.attaquer(goliath, 2);
cout << "David utilise Attaque double sur Goliath" << endl;
}
else if(chance == 2)
{
cout << "Attaque double echoue" << endl;
}
}
else if(choixAttaque == 4)
{
if(chance == 1)
{
david.boirePotionDeVie(2);
cout << "David boit une potion et regagne 2/10 d'energie." << endl;
}
else if(chance == 2)
{
cout << "Potion inneficace" << endl;
}
}
}
ia = (rand() % (4 - 1 + 1)) + 1;
if(ia == 1 || ia == 3)
{
chance = (rand() % (3 - 1 + 1)) + 1;
if(ia == 1)
{
if(chance == 1 || chance == 2)
{
goliath.attaquer(david, 1);
cout << "Goliath utilise Attaque simple sur David" << endl;
}
else if(chance == 3)
{
cout << "Attaque simple echoue" << endl;
}
}
if(ia == 3)
{
if(chance == 1)
{
goliath.attaquer(david, 3);
cout << "Goliath utilise Attaque puissante sur David" << endl;
}
else if(chance == 2 || chance == 3)
{
cout << "Attaque puissante echoue" << endl;
}
}
}
else if(ia == 2 || ia == 4)
{
chance = (rand() % (2 - 1 + 1)) + 1;
if(ia == 2)
{
if(chance == 1)
{
goliath.attaquer(david, 2);
cout << "Goliath utilise Attaque double sur David" << endl;
}
else if(chance == 2)
{
cout << "Attaque double echoue" << endl;
}
}
else if(ia == 4)
{
if(chance == 1)
{
goliath.boirePotionDeVie(2);
cout << "Goliath boit une potion et regagne 2/10 d'energie." << endl;
}
else if(chance == 2)
{
cout << "Potion inneficace" << endl;
}
}
}
cout << endl << "Vie de David : ";
david.afficherEtat();
cout << "Vie de Goliath : ";
goliath.afficherEtat();
cout << endl;
david.estVivant();
goliath.estVivant();
}
if(david.estVivant() == false && goliath.estVivant() == false)
{
cout << endl << "Match nul !" << endl;
}
else if(goliath.estVivant() == false && david.estVivant() == true)
{
cout << endl << "Vous avez gagne !" << endl;
}
else if(david.estVivant() == false && goliath.estVivant() == true)
{
cout << endl << "Vous avez perdu !" << endl;
}
do
{
cout << "Voulez vous refaire une partie ?" << endl << endl << "1.Oui" << endl << "2.Non" << endl << endl << "Votre choix : ";
cin >> choixReco;
if(choixReco < 1 || choixReco > 2)
{
cout << endl << "Choix incorrect, vous avez le choix entre : " << endl;
}
}while(choixReco < 1 || choixReco > 2);
if(choixReco == 1)
{
david.boirePotionDeVie(10);
goliath.boirePotionDeVie(10);
david.estVivant();
goliath.estVivant();
recommencer = true;
}
else
{
recommencer = false;
cout << endl << "Au revoir !" << endl;
}
}
else
{
cout << "Une erreur s'est produite, j'ai le regret de devoir fermer le programme." << endl;
recommencer = false;
}
}while(recommencer);
return 0;
}

Personnage.h

#ifndef DEF_PERSONNAGE
#define DEF_PERSONNAGE
class Personnage
{
public:
Personnage(); // Constructeur
void recevoirDegats(int nbDegats);
void attaquer(Personnage &cible, int nbDegats);
void boirePotionDeVie(int quantitePotion);
bool estVivant();
void afficherEtat();
private:
int m_vie;
};
#endif

Personnage.cpp

#include <iostream>
#include <string>
#include "Personnage.h"
using namespace std;
Personnage::Personnage() : m_vie(10)
{
// Rien à mettre dans le corps du constructeur, tout a déjà été fait !
}
void Personnage::recevoirDegats(int nbDegats)
{
m_vie -= nbDegats; // On enlève le nombre de dégâts reçus à la vie du personnage
if (m_vie < 0) // Pour éviter d'avoir une vie négative
{
m_vie = 0; // On met la vie à 0 (ça veut dire mort)
}
}
void Personnage::attaquer(Personnage &cible, int nbDegats)
{
cible.recevoirDegats(nbDegats); // On inflige à la cible les dégâts que causent notre arme
}
void Personnage::boirePotionDeVie(int quantitePotion)
{
m_vie += quantitePotion;
if (m_vie > 10) // Interdiction de dépasser 100 de vie
{
m_vie = 10;
}
}
bool Personnage::estVivant()
{
if (m_vie > 0) // Plus de 0 de vie ?
{
return true; // VRAI, il est vivant !
}
else
{
return false; // FAUX, il n'est plus vivant !
}
}
void

Je viens de survoler le code en diagonale. Il y a quelques trucs, parmis lesquels :
- mettre un enchaînement de endl est très laid. Ça n'est pas juste un retour à la ligne, endl flush le stdout ;
- commenter c'est bien. Commenter du code trivial, c'est carrément débile : ton code va être susceptible d'évoluer, le commentaire en lui-même deviendra obsolète, il rend plus difficile la lecture d'un code simple, bref c'est à éviter complètement. Mieux vaut avoir des noms de fonctions corrects et compréhensibles, avec du commentaire sur les parties difficiles plutôt que l'inverse.
- certaines fonctions sont relativement lourdes et peuvent être réécrites selon des règles plus simples :

bool Personnage::estVivant()
{
if (m_vie > 0) // Plus de 0 de vie ?
{
return true; // VRAI, il est vivant !
}
else
{
return false; // FAUX, il n'est plus vivant !
}
}

C'est moche. Ça peut être réécrit comme ceci :

bool Personnage::estVivant(void)
{
return (m_vie > 0);
}

C'est plus court, c'est plus lisible.

Bon, et comme je suis pas du tout un expert C++ :-°

je te souhaite bonne chance !

Anonyme

19 septembre 2007 à 16:37:48

Kevin Leonhart

19 septembre 2007 à 17:01:06

Salut.

Si je puis me permettre, je ne vois pas trop le coté "orienté objet" de ton code...
Avec un main de 250 lignes, tu doit avoir un problème de conception.

19 septembre 2007 à 17:05:25

Lutinbleu

19 septembre 2007 à 17:19:07

Salut

Tu aurais pu au lieu de tout mettre dans le main créer divers fonctions : une fonction pour l'aide, une autre pour le choix de l'attaque, une autre pour le test de réussite... séparées dans plusieurs fichiers. Le main ne devrait contenir que l'"introduction" et la "conclusion" de ton programme, le corps du programme étant placé dans d'autres fonctions. On voit mieux ainsi comment elles s'enchaînent et quelle est la trame du code. Le main ne constitue qu'un "plan". (j'ai l'impression de parler comme un prof de français tout à coup

)

Autrement, bien joué. Ca a l'air très intéressant.

19 septembre 2007 à 17:47:58

Salut à tous.

Déjà, merci de vos réponses.

Kevin > Merci, je savais déjà que mon code avait un proble niveau POO, c'est la première version du code, je changerais ça en améliorant le code.

Metzgermeister > Merci pour tes remarques, mais je ne connais pas d'autre moyen de sauter une ligne, y en a un ?

Lutinbleu > Merci, mais c'est pas mélanger 2 styles de programmation ça ? On peut mélanger POO et Prog' impérative ?

19 septembre 2007 à 18:14:23

Déjà, oui : on peut mélanger différents paradigmes (par exemple, en Caml on peut mélanger le fonctionnel et l'impératif, et en OCaml on peut mélanger la POO, le fonctionnel et l'impératif).

Ensuite, le caractère « \n » a toujours une valeur de saut de ligne en C++.

Nanoc

19 septembre 2007 à 18:16:43

Hello,

Ton code fonctionne c'est bien ! Par contre on peut faire beaucoup mieux niveau programmation.

Essaye de ne pas faire des fonction (main y compris) de plus de 1 écran de long.
Évite les successions de endl, préfère les \n .
Mélanger POO et programmation impérative est la marque d'une mauvaise conception du jeu en général.
Par exemple l'affichage des textes d'action n'ont rien à faire dans le main mais plutot dans la classe.
Ne jamais faire de copier-coller de quelque partie de code que ce soit.

Co-auteur du cours de C++. ||| Posez vos questions sur le forum ||| Me contacter.

19 septembre 2007 à 18:18:30

Puisque le copier/coller pur marque en général une factorisation de code oubliée.

19 septembre 2007 à 19:13:16

Merci pour vos réponses

Nanoc > Je me disais bien que le fait de mélanger posait problème, je vais plutôt me consacrer à la POO (sinon pourquoi faire du c++).

Metzgermeister > Je pense mettre tout mon code en méthode découpé minutieusement, le main ne contiendra que des appels de méthodes. Ainsi au moins, mon code sera aussi automatiquement factorisé.

Je donnerais des nouvelles quand ça sera fait.

Vos avis sur le code

Mon "jeu" David VS Goliath

Supprimer

Modérer

Signaler le message aux modérateurs