Discussion :

[insomniak]

J'ai un petit problème, je voudrais faire un vector de classe, mais je vois pas comment faire avec le constructeur...


C_Terrain.hpp :

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
58
59
60
61
62
63
64
65
66
 
#include "SDL_Event.hpp"
#include "C_Case.hpp"
#include "C_FileData.hpp"
 
 
using namespace std;
 
/*
 
*/
 
 
 
class C_Terrain
{
private:
 
    //Taille de la map
    int LONGUEUR;
    int LARGEUR;
    int Pos_x;
    int Pos_y;
 
 
    vector<SDL_Surface *> m_ImgTexturesSol;
    vector<SDL_Surface *> m_ImgTexturesBord;
    vector<SDL_Surface *> m_ImgDecors;
    vector<SDL_Surface *> m_ImgCoffre;
    vector<SDL_Surface *> m_ImgCharset;
    SDL_Surface *Terrain;
    SDL_Events *m_pEvents;
    C_FileData *m_pFileData;
    TTF_Font *font;
 
    void Affichage_Decors(int const x, int const y);
    void Mise_a_jour();
 
    vector<C_Case> m_Case;
 
    int clicked;
 
public:
 
    // Constructeur.
    C_Terrain(SDL_Events &Events, C_FileData &FileData);
 
    // Destructeur.
    ~C_Terrain();
 
    //Chargement des set
    int Init();
 
    //Chargement, Sauvegarde de maps
    int LoadMap(string fichier);
    int NewMap(int const largeur, int const longueur);
    int SaveMap(string Name);
 
    //Affichage
    void Affichage(SDL_Surface *cible);
 
    //Clic sur une case
    int Clic(int const numSritex, int const numSpritey, int const xsouris, int const ysouris, int const clic);
};
 
#endif

Le constructeur de C_Case

Code :

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

C_Case(SDL_Events &Event, C_FileData &FileData);

J'avais pensé àa ça :

Code :

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

vector<C_Case> m_Case(m_pEvents, m_pFileData);

Mais Code::Blocks me dit que m_pEvents et m_pFileData ne sont pas des types... o_O

Merci d'avance de votre aide, vous devez etre probablement très patient avec toutes les questions -betes- que je pose

[Laurent Gomila]

Il ne faut pas confondre construction du vecteur, et construction des éléments que tu vas y insérer.

Lorsque tu ajoutes une case à ton tableau :

Code :

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

m_Case.push_back(C_Case(m_pEvents, m_pFileData));

[Swoög]

Salut !

normalement les vector ont un constructeur par défaut...

qui crée un vector vide...

ensuite, tu peux le remplir avec insert ou push_back...

Si c'est pour faire un tableau de taille fixe, il vaut encore mieux utiliser un C_Case[], dans ce cas, il faut s'assurer que C_Case ait un constructeur par défaut.

Si tu veux un tableau de taille fixe mais que tu ne peux pas mettre de constructeur par défaut à C_Case, il y a aussi un constructeur qui prend en paramètre la taille et une instance de C_Case qui servira à le remplir...

ensuite, m_pEvents et m_pFileData sont des pointeurs, or ton constructeur attend des références, il faut donc extraire les références pour les passer (ajoute juste * devant)

cependant, si comme je le soupsone, tu stockes les pointeurs vers les objets qui sont passés par référence au constructeur, il vaut mieux demander directement des pointeurs, parce que ce que tu stockes, c'est l'adresse, pas l'instance...

cependant, si tu es sûr que tu n'auras jamais besoin de changer ce sur quoi pointe ces deux pointeurs, tu peux également les transformer en références... (et à ce moment là, demander des références dans le constructeur) ça permet d'éviter la syntaxe pénible liée aux pointeurs...

[insomniak]

Bon, alors, merci de vos réponses.

Swoog : Case est évolutif, car ça dépend de la taille du terrain, c'est pour ça que j'ai choisi d'utiliser les vectors.
Pour les pointeurs, ça fonctionne très bien comme j'ai fait, sans syntaxe trop compliquée.


Par contre, pushback ajoute une case, hors, je préfererais qu'il se place sur une case fixe... List ne serait-il pas plus approprié ?

Merci d'avance.

[Swoög]

il y a insert si tu veux insérer à un endroit fixe...

cependant, si ta case existe déjà (position < size - 1) il te suffit de faire :

Code :

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

Case[position] = ...

[insomniak]

Donc... Case[2]=C_Case Case(//constructeur ??

Bon, après réflexion, je vais peut-etre utiliser push_back... Je vais voir.

[Swoög]

si tu veux simplement construire ton vector avec un nombre de cases prédéfinies dans le constructeur de C_Terrain, il te suffit de le mettre ainsi :

Code :

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

1
2
3
4
5
C_Terrain::C_Terrain(SDL_Events &Events, C_FileData &FileData)
   :
[liste d'initialisation], Case([size], [instance de C_Case qui servira à le remplir]), [fin liste d'initialisation]
{
<div style="margin-left:40px">[corps du constructeur]</div>}

c'est à dire, si j'ai bien compris ce que tu souhaites faire, ton constructeur devrait ressembler à ceci :

Code :

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

1
2
3
4
5
C_Terrain::C_Terrain(SDL_Events &Events, C_FileData &FileData)
   :
[liste d'initialisation], Case([size], C_Case(Events, FileData)), [fin liste d'initialisation]
{
<div style="margin-left:40px">[corps du constructeur]</div>}