Using et std::vector

Version imprimable

Bonjour tout le monde :) J'ai un petit probl�me, dans mon programme j'ai une m�thode comme ceci ( venant de la class Attackable ) :
Code:

1 2 virtual AttackInfo attacksInfo() const = 0;
Dans un fichier hpp j'ai d�fini AttackInfo comme ceci :
Code:

1 2 using AttackInfo = std::vector<std::tuple<std::string, std::vector<DmgRoll>, std::vector<DmgRoll>>>;
Il faut comprendre le code comme ceci : Vecteur de plusieurs type de d�gat( nom du sort ou arme/ jet de d�g�ts de l'arme ou du sort, jet de d�g�ts critiques de l'arme ou du sort )

Ma classe Player h�ritent de Attackable et voici attacksInfo() :
Code:

1 2 3 4 5 6 7 8 9 10 11 AttackInfo Player::attacksInfo() const //à opti { ///using AttackInfo = std::vector<std::tuple<std::string, std::vector<DmgRoll>, std::vector<DmgRoll>>>; AttackInfo atkInfo; for( size_t i{0}; i < player_spells.size(); ++i ) { atkInfo[i].push_back( make_tuple(player_spells[i].name(), player_spells[i].spellDmg(), player_spells[i].spellDmgCrit()) ); } return atkInfo; }
J'ai l'erreur suivante : "push_back() ne fait pas partie de AttackInfo" Using ne garde pas les m�thodes des diff�rents objets ci dessus ? Cela me para�t tr�s bizarre x)

Merci d'�clairer ma lanterne :p

20/08/2016, 21h12
Pyramidev

atkInfo[i] n'est pas un AttackInfo, mais un std::tuple qui n'a pas de m�thode push_back, d'o� l'erreur de compilation.

A part �a, pour faire un code plus maintenable, pour d�finir ta classe AttackInfo, il me semble plus raisonnable de faire de l'encapsulation, en d�finissant une classe AttackInfo dont toutes les donn�es seront en visibilit� private.

Par exemple, peut-�tre que, un jour, tu ne voudras plus sauvegarder les donn�es sous la forme d'un std::vector<std::tuple<std::string, std::vector<DmgRoll>, std::vector<DmgRoll>>>, mais sous la forme d'un std::map<std::string, std::vector<std::pair<DmgRoll, DmgRoll>>>.
Ou alors, peut-�tre que, un jour, tu voudras ajouter un invariant � la classe AttackInfo, par exemple le fait que ses std::string soient non vides.
Dans les deux cas, le changement sera plus rapide � faire si, d�s le d�part, tu as fait de l'encapsulation.
20/08/2016, 22h36
Disixlis

Mais si je fait std::get<0>(atkInfo).push_back cela ne devrait pas fonctionner non plus non ? enfin bref je vois pas comment y acc�d� � chaque �l�ments de mon vecteur ^^". Et au d�but j'avais coder directement une classe AttackInfo, mais comme elle �tait vachement vide je me suis dit que en attendant je devrais faire un nouveau type via using :). Donc comment je dois r�soudre mon probl�me actuellement ?

atkInfo[i] n'est pas un AttackInfo mais un �l�ment de ton vecteur AttackInfo, alors

Code:

1
2
3
4
5
6
7
8
9
10
11
    AttackInfo atkInfo( player_spells.size() );
    for( size_t i{0}; i < player_spells.size(); ++i )
    {
        atkInfo[i] = make_tuple(player_spells[i].name(), player_spells[i].spellDmg(), player_spells[i].spellDmgCrit());
    }
ou
    AttackInfo atkInfo;
    for( size_t i{0}; i < player_spells.size(); ++i )
    {
        atkInfo.push_back( make_tuple(player_spells[i].name(), player_spells[i].spellDmg(), player_spells[i].spellDmgCrit()) );
    }

21/08/2016, 00h17
Bousk

Citation:

Envoy� par Disixlis

Mais si je fait std::get<0>(atkInfo).push_back cela ne devrait pas fonctionner non plus non ?

Non puisque atkInfo n'est pas un tuple mais un vector, comme on l'a d�j� montr� 2 fois :weird:

Citation:

Envoy� par Disixlis

enfin bref je vois pas comment y acc�d� � chaque �l�ments de mon vecteur ^^".

Ton truc est trop complexe et surement inutilement complexe. Tu as un vecteur de tuple qui contient lui-m�me 2 vecteurs... on sait m�me pas de quel vecteur tu parles. :calim2:
Dans AttackInfo atkInfo; atkInfo est un vector, atkInfo[i] est un tuple, fin de l'histoire. :weird:
21/08/2016, 21h36
Disixlis

Je viens de relire la connerie que j'ai �crit et c'est bon je sais quoi faire. Ma question n'a vraiment pas lieu d'�tre. Je crois que je devrais arr�t� de rush pendant plusieurs heures et je devrai r�fl�chir avant de poser une question c*n. Veuillez m'excuser mais merci quand m�me pour vos r�ponses :)
21/08/2016, 21h46
foetus

Et au passage, si tu r�fl�chis :mrgreen: pense � refaire ton code avec une classe contenant une cha�ne et 2 tableaux de dommages et ensuite d�finir AttackInfo comme un tableau de cette classe
22/08/2016, 09h59
Disixlis

Je comprend pas trop ce que tu veux dire l� ?
Enfaite j'ai une class Spell contenant un vector de jet de dommages, un autre vector de jet de dommades critiques et un nom.
J'ai plusieurs methodes dont une qui sert � r�cup�r� une attaque g�n�r� de Spell et l'autre methode est attacksInfo (ou un truc du genre, j'ai pas le porjet sous mes yeux ) qui elle va renvoyer toutes les caract�ristiques du sort pour par la suite afficher ses ces caract�ristiques via une IHM. attacksInfo h�rite de Attackable car certains monstre n'ont pas de sort mais une armes et donc pas le m�me fonctionnement. Je sais pas si je suis tr�s clair x)
22/08/2016, 22h23
foetus

Citation:

Envoy� par Disixlis

l'autre methode est attacksInfo [...] qui elle va renvoyer toutes les caract�ristiques du sort pour par la suite afficher ses ces caract�ristiques

Justement ton truc avec using correspond bien � cela: coder un truc pour relier plusieurs "objets" (un sort, une table de dommages et une table de dommages critiques)

Ce que je voulais dire :mrgreen: c'est que, de loin, cela semble foireux. D'accord cela se code vite, style "quick-and-dirty", mais cela ne semble pas terrible.

Donc je te proposais de continuer � r�fl�chir pour [commencer �] refaire cette partie avec une classe qui propose plus de coh�sion voire qui se rapproche �ventuellement d'une base de donn�es.

Mais, il n'y a que toi qui peux juger si ton using tient la route ou pas :mrgreen: :mrgreen:

Oui ce using est fait pour coder plus rapidemment, je veux bien le changer mais je ferai une classe de ce genre l� du coup ? :
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 class AttackInfo { public: AttackInfo(...); DmgRoll normal() const; DmgRoll crit() const; std::string nom() const; private: DmgRoll const normal; DmgRoll const crit; std::string const nom; };
Et il vaudrai mieux que je ne fasse pas de copie, donc je devrais utiliser des pointeur, mais je n'ai jamais su utiliser correctement un pointeur intelligent x)
Peut tu peux m'expliquer comment bien m'y prendre ?