Discussion :

[Ade14]

Bonjour !

Je suis en train de suivre un tutoriel (SdZ) pour apprendre à programmer en C++. Nous avons créer une classe Personnage et une autre Arme (sans importance ici).
Je voudrais qu'au début du programme, le nombre de personnages/de joueurs soit saisi et qu'ensuite ceux-ci soient créés.
Je pensais faire une boucle for mais ne voit pas comment créer les objets dedans (d'autant plus qu'ils seront détruits à la fin de la boucle). J'ai pensé aux vectors mais ne vois pas comment faire. J'ai essayé ceci mais ai 1 erreur et 3 notes :

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
#include <iostream>
#include <string>
#include <vector>
#include "Personnage.h"
 
using namespace std;
 
int main()
{
    int i,nbJoueurs;
    string nomPersonnage;
 
    cout<<"<--------------------- Jeu de combats magiques --------------------->"<<endl<<endl;
 
    do
    {
        cout<<"Nombre de joueurs : ";
        cin>>nbJoueurs;
    }while(nbJoueurs<2);
 
    vector<Personnage> tabJoueurs;
 
    for(i=0;i<nbJoueurs;i++)
    {
        cout<<"Nom du joueur : ";
        cin>>nomPersonnage;
        Personnage* joueur = new Personnage(nomPersonnage);
        tabJoueurs.push_back(joueur);
    }
 
    return 0;
}

avec :

De plus, j'ai trouvé cela sur internet. Je ne connais pas encore new ni l'astérique après le type Personnage...

Je vous remercie par avance de votre aide !

AdelineJ

[ternel]

Cela désigne les pointeurs.
Un pointeur stocke l'adresse mémoire d'une autre variable, et new alloue une variable dans une partie extérieure de la mémoire (le tas).
Pour l'instant, tu ne sembles pas en avoir besoin, donc je te laisse le temps d'y arriver.
Tu as un peu de temps avant d'en avoir besoin.
Sache que le pointeur à base de new et d'étoile n'est pas la meilleure solution. En général, on préfèrera les "smart pointers" tels que std::unique_ptr.



Par contre, c'est une erreur du tuto, enlève les, simplement.

Code :

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

1
2
Personnage *joueur = new Personnage(nomPersonnage);
        tabJoueurs.push_back(joueur);

devient alors

Code :

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

1
2
Personnage joueur = Personnage(nomPersonnage);
        tabJoueurs.push_back(joueur);

en utilisant le constructeur immédiatement:

Code :

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

1
2
Personnage joueur(nomPersonnage);
        tabJoueurs.push_back(joueur);

et en ne créant pas de variable locale pour rien

Code :

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

tabJoueurs.push_back(Personnage(nomPersonnage));

[Ehonn]

Et en utilisant la fonction membre .emplace_back(...) (disponible dès C++11) qui prend directement les paramètres du constructeur :

Code :

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

tabJoueurs.emplace_back(nomPersonnage);

[ternel]

Je l'avais oublié, celui-là

[skeud]

[HS]un peu étrange quand même de faire un tuto sur SDZ et poster sur DVP.[/HS]

[Astraya]

Salut,

Tu as déclaré un std::vector contenant des Personnages et pas des pointeurs sur des personnages. Or tu push_back un pointeur sur un personnage.
Donc soit tu fais:

Code :

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

1
2
3
 
std::vector<Personnage> tabJoueurs;
tabJoueurs.push_back(Personnage(nomPersonnage));

Soit tu fais (note le std::vector de pointeur de personnage):

Code :

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

1
2
3
 
std::vector<Personnage*> tabJoueurs;
tabJoueurs.push_back(new Personnage(nomPersonnage));

Attention pour la deuxième solution, tu ne dois pas oublier de delete avec l'opérateur du même nom chaque instance que tu alloues avec new.