Relation entre deux classes ?

Version imprimable

17/05/2010, 19h15
kanaziwok

Relation entre deux classes ?

Bonjour à tous,

J'explique la situation :
J'ai une classe GAME qui utilise une classe PLATEAU , JOUEUR, CARTES , REGLES, ...
Le jeux est lancé via la classe GAME, c'est elle qui gère tout.
Elle crée les joueurs les cartes et le plateau.

Mais j'aimerai via la classe REGLES utilisé l'élement "PLATEAU Table" créé dans la classe GAME.

Je n'arrive pas à trouver, pour le moment j'ai ce message :
error: ‘Table’ was not declared in this scope
Ce qui est normal vu qu'il ne le connait pas encore :( .

J'ai juste inclus la classe GAME.h CARTES.h et PLATEAU.h dans REGLES.h.

Merci pour votre aide.

Cordialement.
17/05/2010, 19h38
koala01

Salut,

Malheureusement, à moins d'avoir plus d'information, il nous sera difficile de t'aider :aie:

Pourrais tu donc nous présenter le code de tes différentes classes :question:

Voilà le contenu de la classe Game.h

Code:

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
#ifndef GAME
#define GAME
#include "Player.h"
#include "Carte.h"
#include "Table.h"
#include "Rules.h"
#include <cstdio>
#include <string>
#include <vector>
#include <iostream>
#include <ctime>
#include <cstdlib>
 
 
using namespace std;
 
class Game{
 
	private:
		int NombrePlayers;
		vector<Player*> ListPlayers;
		vector<Carte*> Pioche;
		Table Plateau;
 
 
	public:
 
};
#endif

Et de la classe Rules.h

Code:

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
/* Fichier de classe pour un joueur */
 
#ifndef RULES
#define RULES
 
#include <iostream>
#include <string>
#include "Carte.h"
#include "Table.h"
#include "Game.h"
#include <vector>
 
using namespace std;
 
class Rules{
 
	private:
		vector<Carte*> Temp;
 
	public:
		bool TestStraight();
		bool TestGroup();
		bool CheckPlateau();
		Rules();
};
 
#endif

Et le fichier Rules.cpp

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
#include "RummikubRules.h"
 
Rules::Rules(){};
 
bool Rules::CheckPlateau()
{
	for ( int i = 0; i < 20; i++ ) 
	{
		for(int j = 0; j < 26; j++ ) 
		{
			while(!Game::Plateau[i][j].IsCarteInCase(i,j) && j < Game::Plateau.Largeur)
			{
				j++;
			}
			while(Game::Plateau[i][j].IsCarteInCase(i,j)  && j < Game::Plateau.Largeur)
			{
				Temp.push_back(Game::Plateau[i][j].ReturnCarteOnPosition(i,j));
				j++;
			}
		}
 
	}
}

Message d'erreur :

Code:

error: ‘Table Game::Plateau’ is private

17/05/2010, 19h54
koala01

Ha, ben, il semble que tu te trouve face au problème "classique" d'inclusions circulaire...

En effet, tu essaye d'inclure Rules.h dans Game.h et ... Game.h dans Rules.h.

Je vais te laisser réfléchir un peu par toi même, mais tu devrais te baser sur cette entrée de la FAQ pour résoudre ton probème (même si ta classe Game est visiblement un singleton... ce qui pourrait facilement être évité... serais tu pris de singletonite? ;)) :D
17/05/2010, 20h12
kanaziwok

Bah, tu vois, justement je sais pas lequel supprimé :) .
Parce que, d'un côté, la fonction checkrules sera lancée depuis Game.cpp et Rules.h a besoin de Game.h pour les variables initialisés dans celle-ci.

J'aurais comme ça, tendance à supprimer celui du Rules.h.

J'ai testé avec un Class Rules : public Game{ }; mais ce n'est pas non plus la bonne méthode aparement :( .
17/05/2010, 20h21
koala01

Et pourquoi, étant donné que tu n'a aucune référence à Game dans le fichier Rules.h, n'inclurais tu pas, tout simplement, Game.h dans ... Rules.cpp et non dans Rules.h :question:

La règle sur le sujet est relativement simple: tu ne dois inclure un fichier que... quand il devient réellement nécessaire :P ;)

Si tu as besoin d'une classe dans une fonction, mais que tu n'en a pas besoin pour définir la classe, inclus le fichier adéquat non pas dans le fichier d'en-tête (*.h / *.hpp) mais... dans le fichier d'implémentation ( *.cpp) ;)

Cela te permet d'éviter les dépendances inutiles ;)
17/05/2010, 20h32
kanaziwok

Ok, je te remercie pour ta réponse.

Mais j'ai toujours le même problème :mrgreen:.

Plateau n'est toujours pas initialisé :( .
17/05/2010, 21h19
koala01

Et, dis moi, étant donné que tu ne m'as donné que la moitié (même pas) des informations que je demandais (:aie:)...

Comment se présente le constructeur de ta classe Table :question:

Ne serait-ce pas, aussi, un singleton :question: (ce qui n'aurait strictement aucun sens :P)

Non pas de singleton là :) , du moins je pense pas, voilà le code du fichier :
Table.h

Code:

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
 
#ifndef TABLE
#define TABLE
 
#include <iostream>
#include <string>
#include "RummikubCarte.h"
#include <vector>
#include <iomanip>
 
 
using namespace std;
 
class Table{
 
	private:
		int NbCarteOnPlateau;
		vector<vector<Carte*> > TabPlateau;
		int Largeur;
		int Hauteur;
 
	public:
		Table():TabPlateau( 20, vector<Carte*>( 26 ) ){};
		void DisplayPlateau();
		void DelCarteInPlateau(int X,int Y);
		void AddCarteInPlateau(int X,int Y, Carte* Carte_);
		Carte* ReturnCarteOnPosition(int X,int Y);
		bool IsCarteInCase(int X,int Y);
};
 
#endif

17/05/2010, 21h54
koala01

Ouppsss... je viens de remarquer que l'erreur que tu obtiens, c'est

Citation:

error: ‘Table Game:: Plateau’ is private

Ca, ca t'indique que Plateau n'est pas accessible de la manière dont tu essaye d'y accéder...

De toute évidence, tu essaye d'accéder à Plateau depuis (ton instance de Rules :question:) l'extérieur de la classe Game, or, Plateau n'est accessible que... depuis les fonctions de Game, vu que c'est un membre privé :aie: :P

Cela ne change en rien le conseil que je t'ai donné précédemment ;)
17/05/2010, 21h57
kanaziwok

J'ai noté tes conseils ;)

Par contre, pour l'erreur, une petite idée pour la résoudre ?
A part de mettre la fonction de Rules.cpp dans Game.cpp, mais dans ce cas, la classe Rules.h devient quasi inutile ...

Ah oui, aussi l'erreur est maintenant celle là : error: ‘Plateau’ was not declared in this scope

En mettant l'include dans rules.cpp
17/05/2010, 22h05
koala01
Citation:

Envoyé par kanaziwok

J'ai noté tes conseils ;)

Par contre, pour l'erreur, une petite idée pour la résoudre ?
A part de mettre la fonction de Rules.cpp dans Game.cpp, mais dans ce cas, la classe Rules.h devient quasi inutile ...

Ah oui, aussi l'erreur est maintenant celle là : error: ‘Plateau’ was not declared in this scope

En mettant l'include dans rules.cpp

Tu dois te dire que, généralement, si tu as décidé de placer un membre en accessibilité privée, c'est que tu avais de bonne raisons...

Si, pour une raison ou une autre, tu souhaites récupérer ce membre (ou, en tous cas, une référence sur ce membre), tu as deux possibilités différentes (dépendant de l'utilisation que tu en fera):
- Soit tu crée un accesseur sur ce membre (une fonction "getPlateau" renvoyant une référence constante sur Plateau, par exemple)
- Soit tu place carrément le membre en accessibilité publique, mais tu cours alors le risque de "perdre" le contrôle sur ce qui est fait à ce membre.
17/05/2010, 22h39
kanaziwok

Je te remercie pour tes réponses,

Je pense que je vais repenser le fonctionnement de cette partie du programme :)