Discussion :

[ashesClaw]

Bonjours à tous !

J'ai un petit soucis qui me prend la tête depuis un moment et je n'arrive pas à comprendre d'où vient l'erreure

Alors voilà :
J'ai une classe Game qui va déclarer un vecteur qui contient une lise de pointeurs sur une autre classe Character

Dans le fichier game.hpp

Code :

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

1
2
 
vector<Character*> listePersos;

Dans le fichier game .cpp

Code :

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

1
2
3
4
5
6
7
8
 
Game::Game(){
    // Ici je crée une entrée  dans la liste
    this->listePersos.push_back(new Character());
 
    //Et la je l'envois à un constructeur d'une autre class
    Menu* menu = new Menu(&this->listePersos);
}

Dans le fichier menu.hpp

Code :

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

1
2
 
 vector<Character*>* listePersos;

Dans le fichier menu.cpp

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
 
Menu::Menu(vector<Character*>* listePersos){
    this->listePersos = listePersos;
}
 
// Plus loin dans une methode de cette classe
int i;
 
if (listePersos->size() > 0){
    for (i = 0; i < listePersos->size(); ++i){
        if (listePersos[i]->estControlable()){
             //Du code
        }
    }
}

A la compilation j'obtiens les l'erreur suivante

Code :

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

1
2
3
4
 
error: base operand of ‘->’ has non-pointer type ‘std::vector<Character*>’
    if (listePersos[i]->estControlable())
                      ^

j'ai aussi essayé en remplaçant

Code :

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

1
2
 
    if (listePersos[i]->estControlable())

par

Code :

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

1
2
 
    if (listePersos[i].estControlable())

et l'a j'obtiens ceci

Code :

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

1
2
3
4
 
error: ‘class std::vector<Character*>’ has no member named ‘estControlable’
    if (listePersos[i].estControlable())
                      ^

J'ai simplifier le code pour vous éviter les paramètres inutiles, si vous avez besoin de plus d'informations n'hésitez pas !

Merci d'avance et bonne journée

En fait je ne comprend pas pourquoi il me dit que ‘estControlable’ n'est pas une methode de vector<Character*> alors que je suis sensé faire appel au pointeur Character contenu dedans
Pire encore sachant que l'attribut listePersos de la classe Menu est un pointeur sur vecteur, de ce fait je comprend encore moins la première erreur de compilations

[Flob90]

Remplaces tout tes :

Code :

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

listePersos[i]->

Par :

Code :

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

(*listePersos)[i]->

listePersos est un pointeur, donc *listePersos pour accéder à l'objet qui est un vector, donc (*listePersos)[i] pour accéder à un élément, qui est un pointeur donc (*listePersos)[i]-> pour accéder à un membre.

[bacelar]

Et pensez à utiliser des unique_ptr à la place des pointeurs nus.
http://fr.cppreference.com/w/cpp/memory/unique_ptr

[koala01]

Salut,

La première question que je voudrais poser c'est : pourquoi transmettre ton std::vector<Character *> par pointeur Aurais tu peur qu'il puisse ne pas exister le transmettre par référence (constante pour le fonctions qui n'ont pas vocation à modifier la liste des objets) t'éviterait bien des soucis

Ensuite, Bacela n'a absolument pas tort : l'utilisation de pointeur intelligents (std::unique_ptr en tête) t'évitera surement quelques cheveux gris avant l'heure

Enfin, j'ose espérer que tu as pris toutes les précautions nécessaire pour éviter la copie et l'affectation des objets de type Character (et de ses types dérivés)

[ashesClaw]

Merci à vous tous pour vos réponse.

Flob90:listePersos est un pointeur, donc *listePersos pour accéder à l'objet qui est un vector, donc (*listePersos)[i] pour accéder à un élément, qui est un pointeur donc (*listePersos)[i]-> pour accéder à un membre.

En effet ça fonctionne, mais entre temps j'ai trouvé une solution plus lisible à mes yeux

Code :

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

1
2
 
listePersos->at(i)->estControlable();

qui fonctionne tout aussi bien.

bacelar:
Et pensez à utiliser des unique_ptr à la place des pointeurs nus.
http://fr.cppreference.com/w/cpp/memory/unique_ptr

Je n'avais encore jamais entendu parlé de ce type pointeurs. Mais je me demande si ce là fonctionnerais dans mon cas. (voir la réponse en dessous)

Enfait pour te répondre koala01.
J'ai une classe Game qui instancie d'autres classe tels que la fenêtre, le gestionnaire de rendu graphique ect..
Le problème c'est que pour chaque classe instanciée je dois lui donner un pointeur sur une autre contenue dans Game.
Par exemple le gestionnaire de rendu à besoin d'un pointeur sur la fenêtre.

Ou pour cet exemple plus concret,
la classe Menu (qui gère le menu de sélection ex: la liste des personnages du joueur) à besoin d'un pointeur sur le vecteur listePersos, mais d'autres classes auront aussi besoin de ce vecteur.

Donc ma question est la suivante, est-il possible qu'une classe B instanciée dans la classe A, puisse accéder aux attributs et méthodes de la classe A ?

Par contre je n'ai pas bien compris ta dernière question

[PilloBuenaGente]

Bonjour,

Donc ma question est la suivante, est-il possible qu'une classe B instanciée dans la classe A, puisse accéder aux attributs et méthodes de la classe A ?

Voici comment faire :

B.h

Code :

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

1
2
3
4
5
6
struct A;//Declaration anticipée
struct B
{
    B(const A& a);//B prend comme parametre A
    const A& a;//On garde une reference ou un pointeur, const ou non, de la classe A
};

B.cpp

Code :

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

1
2
3
4
5
#include "A.h"//On peut inclure A sans problème dans le cpp
B::B(const A& a) : a(a) 
{ 
    a.foo();
}

A.h

Code :

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

1
2
3
4
5
6
7
#include "B.h"
struct A
{
    A() : b(*this) {}//On initialise B avec ce même A
    const B b;
    void foo() const {}
};

Par contre je n'ai pas bien compris ta dernière question

Tes classes vont avoir des operateurs de copie et d'affectation mis par défaut :

Code :

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

1
2
3
A a; 
A a2(a);//affectation
a = a2;//copie

Le problème est quand ces classes (entitée) contiennent des pointeurs, les copies ou affectation vont copier les pointeurs mais pas leurs contenus.
On peut s'assurer que ça n'arrive en ajoutant ces deux lignes à la classe :

Code :

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

1
2
EntityClassA(EntityClassAconst &) = delete;
EntityClassA& operator=(const EntityClassA&) = delete;

[koala01]

Enfait pour te répondre koala01.
J'ai une classe Game qui instancie d'autres classe tels que la fenêtre, le gestionnaire de rendu graphique ect..
Le problème c'est que pour chaque classe instanciée je dois lui donner un pointeur sur une autre contenue dans Game.
Par exemple le gestionnaire de rendu à besoin d'un pointeur sur la fenêtre.

Mais, la question demeure :pourquoi travailler avec des pointeurs et non avec des réféences, si tu sais que, quoi qu'il arrive, l'objet transmis doit forcément exister

Ou pour cet exemple plus concret,
la classe Menu (qui gère le menu de sélection ex: la liste des personnages du joueur) à besoin d'un pointeur sur le vecteur listePersos, mais d'autres classes auront aussi besoin de ce vecteur.

Encore une fois, pourquoi par pointeur et non par référence

Bien sur, cela signifie que tu devra initialiser les différents éléments dans un ordre bien précis (il faut que le tableau de Character* existe (sans forcément être remplis, ceci dit) avant de créer ton menu), mais les références sont là pour te simplifier la vie... profites-en

Donc ma question est la suivante, est-il possible qu'une classe B instanciée dans la classe A, puisse accéder aux attributs et méthodes de la classe A ?

Lorsque tu appelles le constructeur de ta classe B, il n'y a absolument rien qui t'empêche de lui fournir un des membres de la classe A comme paramètre

Notes juste que, si tu décides de maintenir la référence vers le membre de la classe A dans la classe B sous la forme d'une référence, tu devras (obligatoirement) passer par les listes d'initialisation car une référence doit être définie lorsqu'elle est créée / déclarée. Mais comme les listes d'initialisation sont la bonne manière de travailler, tu fais d'une pierre deux coups

Par contre je n'ai pas bien compris ta dernière question

Typiquement, les classes qui ont sémantique d'entité devraient être non copiable et non affectables afin de garantir l'"unicité référentielle" de chaque objet (comprend: tu ne peux jamais, à un instant T de ton programme, plusieurs instances d'une classe présentant exactement les mêmes valeurs car tu dois pouvoir identifier chacune des instances de manière unique et non ambigüe). Ta classe Character entre très clairement dans cette catégorie et devrait donc en respecter les règles (voir l'entrée suivant le dernier lien donné pour savoir comment faire si tu ne peux pas utiliser C++11 ou supérieur )

[Bousk]

En effet ça fonctionne, mais entre temps j'ai trouvé une solution plus lisible à mes yeux

Code :

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

1
2
 
listePersos->at(i)->estControlable();

qui fonctionne tout aussi bien.

C'est pas tout à fait identique à l'opérateur[], y'a des tests faits en plus et une exception générée en cas d'out-of-bound.
La syntaxe identique serait listePersos->operator[](i)->estControlable(); ou, plus claire : (*listePersos)[i]->estControlable(); comme donné par Koala.

J'ai une classe Game qui instancie d'autres classe tels que la fenêtre, le gestionnaire de rendu graphique ect..
Le problème c'est que pour chaque classe instanciée je dois lui donner un pointeur sur une autre contenue dans Game.
Par exemple le gestionnaire de rendu à besoin d'un pointeur sur la fenêtre.

Ta conception n'est pas terrible, tu prends la chose à l'envers amha.
Généralement c'est l'appli qui ouvre la fenêtre et initialise un Game avec celle-ci. Game instancie tous les éléments nécessaires (soundmanager, network manager, renderer, ....). Ou bien initialisé à côté (le renderer par exemple, ne dépend pas de Game, pas plus que la fenêtre).
Et tu as un accesseur global typiquement GetGame() accessible.