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.

[dragonjoker59]

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.

Bien vu, je propose l'association enum + std::array, pour remplacer la std::map (par contre pour le fichier...)

Code :

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

1
2
3
4
5
6
7
8
9
10
enum class StatType
{
  eWisdom,
  eEarth,
  eFire,
  eZater,
  ...
 eCcount
};
std::array< short, size_t( StatType::eCount ) > m_values;

[Disixlis]

Je vous remercie pour vos réponses j'aurais appris d'autres manières de faire je pense néanmoins rester sur mon modèle actuel ^^"

Sujet résolue