Discussion :

[cosmoff]

bonjour,
voila j'ai un petit soucis, voici mon code :
personnage.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
20
#ifndef DEF_PERSONNAGE
#define DEF_PERSONNAGE
 
#include <iostream>
 
class Personnage{
 
    public:
        Personnage();
        void attaquer(Personnage *P1);
        void degat();
        //void switch_arm();
        int getLifePoint();
 
    private:
        int m_lifePoint;
        int m_degatArme;
 
};
#endif

personnage.cpp:

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
20
21
22
23
24
25
26
#include "personnage.h"
using namespace std;
 
Personnage::Personnage(){
    int m_lifePoint = 100;
    int m_degatArme = 10;
}
void Personnage::attaquer(Personnage *P1){
    (*P1).degat();
}
void Personnage::degat(){
    m_lifePoint -= m_degatArme;
}
int Personnage::getLifePoint(){
    return m_lifePoint;
}
 
int main(int argc, char const *argv[])
{
    Personnage V;
    Personnage K;
    cout<<"la vie de K est de : "<<K.getLifePoint()<<endl;
    V.attaquer(&K);
    cout<<"la vie de K est de : "<<K.getLifePoint()<<endl;
    return 0;
}

mon probleme est que la vie de K est égale a 408000 soit une valeur aléatoire, et je ne comprend pas pourquoi
pouvez vous m'aider,
merci d'avance

[Médinoc]

Bonjour,
Dans ton constructeur, tu re-déclares tes variables membres en tant que variables locales, au lieu d'assigner les variables membres.

Par ailleurs:

• Tu n'est pas dans la bonne section du forum; ici c'est pour le langage C++/CLI du Framework .Net.
• Pour l'instant dans ta méthode degat(), tu blesses le personnage avec sa propre arme. La méthode doit prendre la valeur des dégâts en paramètre pour corriger ce problème.
• La méthode attaquer() ne modifie pas le personnage attaquant (seulement le personnage attaqué, passé en paramètre), elle devrait donc être déclarée comme méthode const.
• getLifePoint() devrait être une méthode const également.
• (*P1).degat(); peut être réécrit plus simplement P1->degat();

[cosmoff]

ok merci médinoc, le probleme est résolue.
Oui il y a pas mal d'incohérence dans mon code mais je voulais avoir une premiere approche des classes.
merci pour l'astuce de la fleche ->