Discussion :

[speedy_souris]

Bonjour à toutes et tous,

j'ai simplement une petite question, comment peut-on surcharger un constructeur de cette façon

exemple

Code :

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

1
2
3
Personnage::Personnage(string epee, int degatsEpee) : m_Vie(....), m_arme(epee, degatsEpee){}
 
Personnage::Personnage(string bouclier, int protectionBouclier) : m_vie(....), m_arme(bouclier, protectionBouclier){}

[foetus]

Simple

Code :

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

Personnage::Personnage(string value_str, int value, bool is_damage) : m_Vie(....), m_arme(value_str, value, is_damage){}

Édit: Simple

Code :

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

Personnage::Personnage(string value_str, int value, bool is_damage) : (is_damage? m_Vie(....): m_vie(....)), m_arme(value_str, value){}

[white_tentacle]

Tu as conscience que tes deux surcharges sont strictement identiques ?

Si tu as besoin de différencier bouclier / épée, peut-être qu’il serait intéressant de les typer plus fortement que simplement avec un std::string.

[foetus]

Tu as conscience que tes deux surcharges sont strictement identiques ?

Il y a une majuscule m_Vie au lieu de m_vie

Après faute de frappe ou pas

[speedy_souris]

Il y a une majuscule m_Vie au lieu de m_vie

Après faute de frappe ou pas

c'est bien une faute de frappe...

speedy_souris a encore frapper

aussi peut etre creuser la piste sur le fait que le bouclier n'est pas une arme mais une protection (a voir autre classe )

[Médinoc]

Franchement, pour ce genre de jeu, je verrais plutôt une gestion de l'équippement de ce genre:

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
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
enum PlaceEquipement //Note: utiliser "enum class" si supporté
{
	Tete,
	Torse,
	MainGauche,
	MainDroite,
	MAXp1_PLACE_EQUIPEMENT
};
 
struct ObjetEquipable
{
	//Ici, on supporte TOUTES les propriétés des objets,
	//comme ça certains peuvent servir aux deux (bouclier à pointe, épée avec garde, etc.)
	string nom;
	int protection;
	int degats;
};
 
class Personnage
{
private:
	vector<ObjetEquipable*> equipement;
	bool estGaucher;
public:
	Personnage(bool estGaucher) : equipement(MAXp1_PLACE_EQUIPEMENT), estGaucher(estGaucher) {  }
 
	void equipe(PlaceEquipement place, ObjetEquipable* objet) { equipement.at((int)place) = objet; }
 
	ObjetEquipable const * getArme() const
	{
		return estGaucher ? equipement.at((int)MainGauche) : equipement.at((int)MainDroite);
	}
	//Ensuite, pour la protection, un truc de ce genre, ou de l'autre:
	int getSommeProtections() const
	{
		//Note: En fait, il doit y avoir un truc dans <algorithm> qui fait ça pour toi
		int total = 0;
		for(vector<ObjetEquipable*>::const_iterator it = equipement.begin() ; it != equipement.end() ; ++it)		
		{
			if((*it) != null)
				total += (*it)->protection;
		}
		return total;
	}
	ObjetEquipable const * getMeilleureProtection() const
	{
		//Note: En fait, il doit y avoir un truc dans <algorithm> qui fait ça pour toi
		ObjetEquipable const* meilleur = null;
		for(vector<ObjetEquipable*>::const_iterator it = equipement.begin() ; it != equipement.end() ; ++it)		
		{
			if((*it) != null)
			{
				if(meilleur==null || (*it)->protection > meilleur->protection) 
					meilleur = *it;
			}
		}
		return meilleur;
	}
};