Discussion :

[Disixlis]

Bonjour � tous , je code un comparateur de caract�ristiques pour un jeu nomm� Dofus.
Je compte charg� tout les items contenant les caract�ristiques et donc je ne les �crit pas en dure dans le code pour �viter de devoir recompiler � chaque ajouts.
Voici mon code :

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
 
#ifndef Stat_included
#define Stat_included
 
class Stat
{
public:
 
    Stat( int v, int i, short w, short e, short f, short wa, short a, short p, short ac, short m, short pr, short r, short in, short d, short dn, short de, short df, short dw, short da, short dc,
          short dp, short rn, short re, short rf, short rw, short ra, short rfn, short rfe, short rff, short rfw, short rfa, short apr, short mpr, short apd, short mdp, short h, short ch,
          short t, short l );
 
private:
 
    int const stat_vitality;
    int const stat_initiative;
    short const stat_wisdom;
    short const stat_earth;
    short const stat_fire;
    short const stat_water;
    short const stat_air;
    short const stat_power;
    short const stat_actions;
    short const stat_movements;
    short const stat_prospecting;
    short const stat_reach;
    short const stat_invocation;
    short const stat_damage;
    short const stat_damage_neutral;
    short const stat_damage_earth;
    short const stat_damage_fire;
    short const stat_damage_water;
    short const stat_damage_air;
    short const stat_damage_critical;
    short const stat_damage_pushback;
    short const stat_resistance_neutral;
    short const stat_resistance_earth;
    short const stat_resistance_fire;
    short const stat_resistance_water;
    short const stat_resistance_air;
    short const stat_resistance_fixed_neutral;
    short const stat_resistance_fixed_earth;
    short const stat_resistance_fixed_fire;
    short const stat_resistance_fixed_water;
    short const stat_resistance_fixed_air;
    short const stat_actions_point_reduction;
    short const stat_movements_point_reduction;
    short const stat_actions_point_dodge;
    short const stat_movements_point_dodge;
    short const stat_heal;
    short const stat_critical_hit;
    short const stat_tackle;
    short const stat_leak;
 
};
 
#endif //Stat_included

Ce n'est pas une erreur en soient mais le probl�me est que le constructeur est beaucoup trop long. Y'a-t-il une possibilit� de r�duire ce code ? Je vous remercie

[dragonjoker59]

Salut tu peux peut-�tre d�couper ta structure en sous structures? (BaseStats, DamageStats, ResistanceStats, ...) et avoir une structure qui les chapeauterait?

[Bousk]

Etant donn� que toutes ces stats sont particuli�res et n'ont rien � voir entres elles, tu finiras fatalement par toutes les �crire.
Tu peux les d�couper comme dit ci-dessus, mais �a ne changera strictement rien au fait que tu dois toutes les �crire...

[nikko34]

Tu peux faire une std::map< std::string, int > o� tu rajoutes tes stats:

Code :

S�lectionner tout - Visualiser dans une fen�tre � part

1
2
3
4
5
 
std::map< std::string, int > stats;
 
stats[ "v" ] = 1;
stats[ "i" ] = 50;

etc..

tu peux la rajouter dans ta classe stats. Et faire des fonctions genre

Code :

S�lectionner tout - Visualiser dans une fen�tre � part

1
2
3
4
5
 
void Stat::addStat( const std::string  & statName, int value )
{
    this->statlist_[ statName ] = value;
}

Tu peux les charger dynamiquement depuis un fichier du coup, et effectivement, t'auras rien � compiler si tu en rajoutes une.

[Matt_Houston]

En g�n�ral on a tout de m�me besoin de sp�cifier une valeur par d�faut pour le param�tre. On peut certes choisir une valeur arbitraire et l'appliquer � tout le monde, mais pas s�r qu'elle convienne � tous les param�tres...

[Bousk]

L'utilisation d'une map, ou n'importe quelle structure, coupl�e � un fichier peut �tre int�ressante, mais il faut s'assurer d'avoir des protections en cas de donn�es corrompues (assert, ...), parce que le compilateur ne pourra plus t'indiquer que la stat � laquelle tu veux acc�der n'existe pas/plus.