amelioration du PG console qui plante

aidez moi a aider David

20 novembre 2007 à 18:12:29

bon alors j'ai demarre ya quelques semaines les tuto de Mateo, j'en suis au tuto c++ avec david et goliath qui se tapent dessus, et j'me suis dit que j'allais prendre mon temps et compliquer les choses. Donc j'ai voulu donner une valeur de precision aux armes (qui contrebalancera la puissance) qui influerait sur la reussite de l'attaque.
donc je cree ma valeur

int m_degats, m_precision;

, un accesseur pour recuperer sa valeur dans le main

int Arme::getPrecision() const
{
return m_precision;
}

et me voila qui ouvre un if dans la methode Personnage::attaquer. je cree une petite fonction simple pour avoir un nombre aleatoire, l'idée etant que si la precision est superieure au nombre, l'attaque reussi (j'me suis pas vraiment plongé dans la qualité de "l'algorithme", je veux deja que ca marche.)
mais ca marche pas

void Personnage::attaquer(Personnage &cible)
{
if(m_arme.getPrecision() > nb = RAND_NUM * m_arme.getPrecision)
{
cible.recevoirDegats(m_arme.getDegats());
cout<<m_nom<<" attaque et fait "<<m_arme.getDegats()<<" points de vie de degats."<<endl;
}
else
cout<<m_nom<<" a lamentablement rate son attaque."<<endl;

me renvoie une erreur :"error: invalid use of member (did you forget the `&' ?)" pour la ligne du if....
bref, help siouplait, mon david ne tiendra pas le coup sinon.....^^

chlablak

20 novembre 2007 à 18:20:36

- pas de signe '=' dans une condition
- d'ou vient ta variable 'nb', et a quoi sert-elle ?

C++ Références | Idiomes | FAQ

boiss007

20 novembre 2007 à 18:37:12

ah....c'etait aussi bete?....
J'ai crée une variable pour contenir le valeur aleatoire, et en fait... euh... elle sert à rien, je pense...
J'vais essayé sans^^
merci pour la reponse

bon, l'erreur est avant le signe =
mon nouveau code

void Personnage::attaquer(Personnage &cible)
{
if(m_arme.getPrecision() > RAND_NUM * m_arme.getPrecision)
{
cible.recevoirDegats(m_arme.getDegats());
cout<<m_nom<<" attaque et fait "<<m_arme.getDegats()<<" points de vie de degats."<<endl;
}
else
cout<<m_nom<<" a lamentablement rate son attaque."<<endl;
}

et toujours la meme reponse : "error: invalid use of member (did you forget the `&' ?)"
si j'ai bien suivi, member c'est un attribut, mais j'ai fait un accesseur expres pour pas violer la sacro sainte loi de l'encapsulation.
j'me goure quelque part mais z'ou donc....

euh ok, en fait j'avais pas mis les parentheses à la fin de m_arme.getPrecision. :-°

Maintenant ca compile, j'vais tester

chlablak

20 novembre 2007 à 18:43:51

montre Personnage.hpp

C++ Références | Idiomes | FAQ

boiss007

20 novembre 2007 à 18:56:04

bon, ca compile avec un resultat de 0 pour la valeur aleatoire, donc je vais faire ca propre, paske la ca marche pas

mon Personnage.h

#ifndef DEF_PERSONNAGE // Si la constante n'a pas été définie` le fichier n'a jamais été inclus
#define DEF_PERSONNAGE // On définit la constante pour que la prochaine fois le fichier ne soit plus inclus
/* Contenu de votre fichier .h (autres includes` prototypes de vos fonctions` defines...) */
#include "Arme.h"
class Personnage
{
public:
Personnage(std::string nom);
Personnage::Personnage( std::string nomArme, int degatsArme, int precisonArme, std::string nom);
~Personnage();
void recevoirDegats(int nbDegats);
void attaquer(Personnage &cible);
void lancerSort(Personnage &cible);
void changerArme(std::string nomNouvelleArme, int degatsNouvelleArme);
void boirePotion(int quantiteDePotion);
void recupererMana();
bool estVivant();
void afficherEtat();
private:
int m_vie, m_mana;
Arme m_arme;
std::string m_nom;
int m_degatsMana;
int nb;
};
#endif

Bon en fait je sais pas creer un nombre aleatoire sous c++, c'est ca mon vrai probleme.
donc qqun sais comment on fait???

chlablak

20 novembre 2007 à 19:05:52

- il manque des parenteses dans ton if
- avec rand() (il me smeble qu'il existe une include <random> mais prevue pour C++0x (std::tr1::randomise() peut-etre))

C++ Références | Idiomes | FAQ

boiss007

20 novembre 2007 à 19:09:25

j'ai vu, c'etait une bete faute d'inattention.... :honte:

tant qu'on y est, tu saurais pas creer une valeur aleatoire sous c++ (comme srand sous C....)???????

lol j'ai raté l'edit de quelques sec... j'vais me lancer a la recherche de rand^^

Merci encore^^

void Personnage::attaquer(Personnage &cible)
{
//int temp = m_arme.getPrecision();
//cout<<temp<<endl;
nb = rand()%10;
//cout<<nb<<endl;
if( m_arme.getPrecision() > nb*1.12 )
{
cible.recevoirDegats(m_arme.getDegats());
cout<<m_nom<<" attaque et fait "<<m_arme.getDegats()<<" points de vie de degats."<<endl;
}
else
cout<<m_nom<<" a lamentablement rate son attaque."<<endl;
}

TADAAAAAA!!!! tout marche, la vie est belle, et le challenger david a une petite chance!!!
bon, allez, on va mettre des pointeurs dans la POO maintenant....

merci encore, Chlab_lak