R�duction de code

Version imprimable

R�duction de code

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:

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 :)

07/10/2016, 11h04
dragonjoker59

Salut tu peux peut-�tre d�couper ta structure en sous structures? (BaseStats, DamageStats, ResistanceStats, ...) et avoir une structure qui les chapeauterait?
07/10/2016, 11h51
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... :calim2:
07/10/2016, 14h19
nikko34
Tu peux faire une std::map< std::string, int > o� tu rajoutes tes stats:
Code:

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:

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.
07/10/2016, 14h42
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...
07/10/2016, 15h07
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.
07/10/2016, 15h41
dragonjoker59
Citation:

Envoy� par 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.

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

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;
10/10/2016, 13h54
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 :)