Discussion :

[Invité]

Bonjour,

Je veux faire un petit jeux de strategie, un espece d'echec amélioré, sur un damier, avec des unités etc...
J'ai une classe Grille, qui contient un vector de Case ainsi qu'une vector d'Unité déclarés en attribut dans la classe Grille :

Code :

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

1
2
3
4
5
6
7
8
9
 
include <vector>
class Grille{
private :
std::vector<Case> v_case;
std::vector<Unite> c_unite;
public :
.....
};

J'ai des méthodes de la classe Grille qui permettent d'accéder à mes cases et mes unités :

Code :

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

1
2
3
4
5
6
7
getCase(int i){
return v_case[i];
}
 
getUnite(int i){
return v_unite[i];
}

A partir de la j'ai des méthodes de ma classe Unite qui permettent logiquement de modifier certains attributs :

Une unité a des HP, j'ai donc un attribut :

Code :

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

int hp;

Avec une méthode Attack() de la classe unité, je veux pouvoir attaquer une autre unité, ce qui va modifier ses HP.

Code :

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

1
2
3
void Attack(Unite &u){
u.changeHP(-(this->getAttack()));
}

Je vous passe les détails sur mes fonctions changeHP et getAttack() elles sont assez explicites.

A partir de là, voilà mon problème :

Tant que je suis dans ma méthode Attack(), si je fais des tests en affichant les HP de la cible, il y a bien modification de ses HP. Mais en dehors de la méthode, si je veux afficher ses HP, ils n'ont pas changé.

Pour régler ce problème faut-il qu'à la place de mes vectors d'Unite et de Case je fasse des vectors de pointeur vers Unite et vers Case ?

[white_tentacle]

Bonjour,

Code :

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

1
2
3
getUnite(int i){
return v_unite[i];
}

Il manque le type de retour.

Je pense qu'il faut que tu renvoies des références, et que ce n'est pas ce que tu fais .

[Invité]

Oui je me suis trompé effectivement sur mon message, mais dans mon code je spécifie le type de retour qui est dans le cas de cette méthode :

Code :

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

1
2
3
Unite Grille::getUnite(int i){
return v_unite[i];
}

[3DArchi]

Comme dit précédemment, si tu dois modifier l'élément en retour, il te faut renvoyer une référence:

Code :

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

1
2
3
4
 
Unite& Grille::getUnite(int i){
return v_unite[i];
}

[r0d]

Effectivement, il te faut renvoyer une référence. Pourquoi?
Explication:

Prenons une classe Dum qui possède une variable membre X de type entier:

Code :

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

1
2
3
4
5
6
7
8
class Dum
{
public:
   Dum( int x ) : X(x) {} //constructeur
   int GetX() { return X;} //GetX() renvoie une copie de X!! 
private:
   int X;
};

Maintenant un exemple de code qui utilise un objet de type Dum:

Code :

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

1
2
3
Dum d(1); //crée un objet Dum avec X=1
int x=d.GetX(); // x est une copie de d.X
x++; //incrémente x, mais pas d.X

C'est pour cela que si GetX() renvoie non plus une copie de X, mais une référence sur X, alors x=d.GetX() ne sera plus une copie de X, mais une référence sur d.X et alors tu pourras le modifier.

Cela dit, ton problème met en évidence une erreur de conception, ou au moins de sémantique. En effet, un accesseur (GetX dans mon exemple) est fait pour récupérer une variable membre, et ne doit pas permettre de le modifier, sinon ton encapsulation devient un peu bancale, d'autant plus lorsque la variable en question est dans un conteneur (ce qui est ton cas ici).

Hope it helps.

[Invité]

D'accord tout s'explique, c'est peut être une erreur de conception mais au moins j'ai compris !
Merci beaucoup pour l'explication en tous les cas !