Discussion :

[NainTernaute]

Bonjour à tous,

Dans le cadre d'un projet de jeu 2D, je pense considérer les différentes entités du jeu sur des maps "parallèles" ; on a, dans le cas de ce bomberman, une map globale (qui affiche les différents blocs), une map avec les players, une map avec les bonus, etc...

Chaque map hérite d'une classe mère abstraite, où je définis notamment le tableau permettant de la stocker. J'aimerais que chaque case contienne soit la valeur d'une enum (sous forme de int, FREE=0, etc...), soit un pointeur vers l'objet présent sur cette case. Ainsi, il me serait très simple de parcourir les différentes maps et d'afficher correctement la scène.
Il me serait du coup également très simple de vérifier les morts de joueurs, etc...

Savez-vous s'il est possible de faire cela, et comment ?
Pensez-vous autrement qu'il s'agit d'une erreur de conception, et pourriez-vous m'aiguiller vers la meilleure façon de faire ?

Merci d'avance

[bruno_pages]

Bonjour,

quand vous parlez de map, parlez vous vraiment d'une table d'association clef-valeur et donc pas exemple d'une std::map, ou 'map' signifie 'tableau 'pour vous comme le titre le laisse penser?

Le fait d'avoir des 'maps' parallèles laisse à penser qu'il y a peut être un problème d'archi. Par exemple est-ce que le parallélisme des 'map' de joueurs et de bonus signifie-t-elle que cela permet 'indiquer le bonus de chaque joueur ?

pour revenir à Tableau stockant enum(int) ou pointeur, si le seul cas ou il n'y a pas de pointeur est le cas 'libre', alors il existe le pointeur nul

[NainTernaute]

Ah non, quand je parle de map ce sont des maps de jeux.
Le cas "FREE" n'est pas le seul, ça dépend en réalité des maps ; la map globale affiche chaque blog, et donc peut prendre 3 états (mais pas de pointeur). En revanche, la map joueur possède soit un pointeur soit NULL.

Par ailleurs, chaque map possèdera un type de pointeur différent (Player* pour la map des joueurs, Bonus* pour la map des bonus, etc...). Comme j'ai déjà une classe abstraite, j'aimerais bien que l'attribut représentant le tableau soit lui aussi générique et présent dans cette classe (les méthodes retournant l'état d'une case l'étant aussi).

Est-ce possible ?

[bruno_pages]

il suffit d'utiliser une classe template

[Médinoc]

Ce genre d'"optimisation en taille" est possible avec les unions, mais ça fait partie du genre de choses qu'on évite de faire à moins d'en avoir absolument besoin.

S'il le faut vraiment, tu peux utiliser boost::variant, qui encapsule ça.

[NainTernaute]

Et bien, ça semble marcher parfaitement avec une classe template ; merci !

[NainTernaute]

Hum en fait, j'ai encore un petit problème ; chaque élément du jeu (joueur, bonus, bombe, ...) dérive d'une classe abstraite "Entity". Celle-ci contient un pointeur vers une CMap contenant celle-ci des pointeurs vers les objets instanciés.

Plus concrêtement, un joueur aura un attribut contenant un pointeur vers sa propre map (comprendre map au sens graphique) ; cette map contiendra quand à elle des pointeurs vers chaque joueur, à l'endroit de leurs coordonnées.

J'ai essayé quelques trucs mais j'ai toujours une erreur à la compilation :

.\Player.cpp(33) : error C2027: utilisation du type non défini 'CMap<T>'
1> with
1> [
1> T=CEntity *
1> ]

La classe CEntity est pourtant bien déclarée !

Voici les différentes codes :

Entity.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
 
#ifndef _CENTITY_
#define _CENTITY_
 
template<typename T> class CMap;
 
/*
	Abstract class CEntity
	Defines elements that can be herited by all entities (players, bombs, bonus)
*/
 
class CEntity {
 
	protected:
		int m_posX;
		int m_posY;
		CMap<CEntity*>* m_pMap;
 
	public:
		virtual const int getPosX();
		virtual const int getPosY();
		virtual void setPosX(const int x);
		virtual void setPosY(const int y);
		virtual void setMap(CMap<CEntity*>* map);
 
};
 
#endif

Map.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
 
#ifndef _CMAP_
#define _CMAP_
 
#include "config.h"
 
/*
	Abstract class CMap
	Defines elements that can be herited for each map : 
		- a map table (MAP_SIZE x MAP_SIZE)
		- getters & setters methods
		- init method
*/
 
template <typename TableType> 
class CMap {
 
	protected: 
		TableType m_mapTable[MAP_SIZE][MAP_SIZE];
 
	public:
		const TableType getCaseState(int x, int y);
		void setCaseState(int x, int y, TableType caseState);
		virtual void init() = 0;
 
};
 
template<typename TableType> const TableType CMap<TableType>::getCaseState(int x, int y) {
	return m_mapTable[y][x];
}
 
template<typename TableType> void CMap<TableType>::setCaseState(int x, int y, TableType state) {
	m_mapTable[y][x] = state;
}
 
#endif

Enfin, j'essaie d'utiliser mon tableau dans la classe CPlayer comme suit :

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
 
void CPlayer::move(Direction direction) {
	switch(direction) {
		case UP:
			m_posY--;
			break;
		case DOWN:
			m_posY++;
			break;
		case LEFT:
			m_posX--;
			break;
		case RIGHT:
			m_posX++;
			break;
	}
 
	m_pMap->setCaseState(this);
}

Auriez-vous une idée ? Merci d'avance

[bruno_pages]

[QUOTE=NainTernaute;5110999].\Player.cpp(33) : error C2027: utilisation du type non défini 'CMap<T>' difficile de vous aidez puisque vous ne donnez pas la ligne de code en question

ceci dit CMap<T> n'est pas utilisable tel quel (en dehors de la def de CMap bien-sûr) car T n'est pas connu

m_pMap->setCaseState(this) n'est pas valide tout simplement parce que cette opération prends 3 arguments et non un seul

[NainTernaute]

La ligne 33 est justement :

Code :

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

1
2
 
m_pMap->setCaseState(m_posX, m_posY, this);

Autant pour moi pour les 3 arguments, mais ça ne marche pas mieux comme cela en fait. :/ Et je n'ai pas utilisé le type "T" mais bien "CEntity*" (voir dans ma classe CENtity) :

Code :

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

1
2
 
CMap<CEntity*>* m_pMap;

D'où mon incompréhension lors de cette erreur, qui complète est plus explicite :

1>.\Player.cpp(33) : error C2027: utilisation du type non défini 'CMap<T>'
1> with
1> [
1> T=CEntity *
1> ]

EDIT : Par ailleurs, lorsque j'essaie de faire un setMap j'ai cette erreur :

1>.\Game.cpp(11) : error C2664: 'CEntity::setMap'*: impossible de convertir le paramètre 1 de 'CPlayersMap *' en 'CMap<TableType> *'
1> with
1> [
1> TableType=CEntity *
1> ]
1> Les types pointés n'ont aucun rapport entre eux*; conversion nécessitant reinterpret_cast, cast de style C ou cast de style fonction

Et les lignes concernées :

Code :

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

1
2
 
m_player.setMap(&m_maps.getPlayersMap());

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
 
#ifndef _CPLAYERSMAP_
#define _CPLAYERSMAP_
 
#include "Map.h"
#include "Player.h"
 
class CPlayersMap : public CMap<CPlayer*> {
 
	protected:
 
	public: 
		void init();
 
};
 
#endif

Voici un lien vers mon code complet si ça peut aider à la résolution du problème : http://www.jeremy-dutheil.fr/projects/Bomber.rar

[bruno_pages]

c'est simple : Map.h n'est pas inclue directement ou indirectement dans Player.cpp

[edit]dans ma première réponse je n'avais pas osé demandé si CMap était bien inclue car cela paraissait tellement énorme, finalement j'aurais du [/edit]

[NainTernaute]

Bouarf, j'ai honte

En revanche, quelque chose m'étonne toujours ; dans ma classe CEntity je définis cette fonction :

Code :

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

1
2
 
virtual void setMap(CMap<CEntity*>* map);

Et ma classe PlayersMap comme ceci :

Code :

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

1
2
 
class CPlayersMap : public CMap<CPlayers*>

Quand je fais cela, je ne peux pas appeler ma méthode setMap avec un pointeur vers mon objet de CPlayersMap ; en revanche, en définissant CPlayersMap comme suit ça marche :

Code :

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

1
2
 
class CPlayersMap : public CMap<CEntity*>

Je pensais que le fait que la fonction soit virtuelle lui permettrait de s'adapter aux classes dérivées de CEntity automatiquement, ai-je loupé quelque chose ?

En tout cas, merci pour l'aide apportée.