Discussion :

[kanaziwok]

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.

[koala01]

Salut,

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

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

[kanaziwok]

Voilà le contenu de la classe Game.h

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
#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 :

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
/* 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 :

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
#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 :

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

error: ‘Table Game::Plateau’ is private

[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? )

[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 .

[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

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

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