Discussion :

[SmallFitz]

Salut !

En souhaitant réaliser un Sudoku je suis tombé sur une erreur que je n'arrivais pas à comprendre.
J'ai tout de même réussi à localiser où se trouve le souci et je l'ai extrait du projet pour essayer de le résoudre plus facilement.

Voilà comment je peux illuster le souci. J'essaie de faire ceci :

Soit une classe A qui a pour attribut un pointeur sur une instance de la classe B.
Soit une classe B qui a pour attribut un pointeur sur une instance de la classe A.

Ce qui donne les fichiers suivant : A.h, A.cpp, B.h et B.cpp
Et voilà leur contenu :

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
// FICHIER A.h
#ifndef A_H
#define A_H
 
#include "B.h"
 
class A
{
	private:
		B* m_b;
 
	public:
		A();
};
 
#endif

Code :

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

1
2
3
4
5
6
7
8
9
10
11
// Fichier A.cpp
#include <iostream>
 
using namespace std;
 
#include "A.h"
 
A::A() : m_b(NULL)
{
 
}

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
// Fichier B.h
#ifndef B_H
#define B_H
 
#include "A.h"
 
class B
{
	private:
		A* m_a;
 
	public:
		B();
};
 
#endif

Code :

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

1
2
3
4
5
6
7
8
9
10
11
// Fichier B.cpp
#include <iostream>
 
using namespace std;
 
#include "B.h"
 
B::B() : m_a(NULL)
{
 
}

Voilà les erreurs obtenues lors des compilations :

g++ -Wall -c A.cpp
In file included from A.h:4:0,
from A.cpp:5:
B.h:9:3: error: ‘A’ does not name a type

g++ -Wall -c B.cpp
In file included from B.h:4:0,
from B.cpp:5:
A.h:9:3: error: ‘B’ does not name a type

Pourriez vous m'expliquer pourquoi cela plante ?
Merci davance.
SmallFitz.

[DakM]

Je pense que ca doit etre ca:
Quand ta classe A cherche la classe B, elle n'est pas encore définit.
j'ai eu ce problème une fois, j'ai du ordonner l'ordre de mes fichiers inclus dans mon projet pour que ca marche.

Peut etre que ton idée marcherait avec un template ?

[Xtrem_Voyageur]

Peut-être qu'une simple déclaration de la classe B avant la définition de l'interface de ta classe A règlerait le problème.

Code :

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

1
2
3
4
5
 
class B;
class A{
...
};

[SmallFitz]

Merci à vous deux !

@DakM :
J'ai essayé mais je n'ai pas trouvé une manière de changer l'ordre.
Je ne connais pas encore les templates ! Quelle nouveauté auraient ils pu apporter ?

@Xtrem_Voyageur :
Joli ! J'ai essayé et ça fonctionne niquel.
Quand on y réfléchit bien c'est tellement logique !
Je n'avais jamais vu une simple déclaration dans une interface... C'est beau !

[Emmanuel Deloget]

Merci à vous deux !

@DakM :
J'ai essayé mais je n'ai pas trouvé une manière de changer l'ordre.
Je ne connais pas encore les templates ! Quelle nouveauté auraient ils pu apporter ?

@Xtrem_Voyageur :
Joli ! J'ai essayé et ça fonctionne niquel.
Quand on y réfléchit bien c'est tellement logique !
Je n'avais jamais vu une simple déclaration dans une interface... C'est beau !

C'est certes beau, syntaxiquement valide et tout et tout. Par contre, ça dénote avant tout d'un problème de conception si le but est de régler un soucis de dépendance circulaire.

La relation "A utilise B utilise A" (de manière directe ou indirecte) va te poser des problèmes. Est-ce que tu es sûr que tu n'as pas d'autres solution ?

[oodini]

Ben quand tu as une relation symétrique du genre :

"H est marié à F qui est mariée à H",

il n'y a pas grand chose d'autre à faire.

Bon, certains diront que la mariage n'est pas une relation symétrique...

[SmallFitz]

C'est certes beau, syntaxiquement valide et tout et tout. Par contre, ça dénote avant tout d'un problème de conception si le but est de régler un soucis de dépendance circulaire.

La relation "A utilise B utilise A" (de manière directe ou indirecte) va te poser des problèmes. Est-ce que tu es sûr que tu n'as pas d'autres solution ?

Je crois que je pourrais m'en passer mais j'ai l'impression que l'utilisation des classes sera plus facile si je ne m'en passe pas.

Pour la syntaxe : F.A.Q. Comment créer 2 classes qui font référence l'une à l'autre ?

Pour le design, voir la réponse d'Emmanuel : cette référence croisée est-elle vraiment pertinente ?

Merci pour la FAQ.
J'ignore si elle est pertinente, je suis débutant et je n'ai jamais étudié de près les architectures de classe.

Je vais essayer de présenter ce que je suis en train de réaliser.

Je souhaite représenter en C++ un Sudoku.
Voilà mon analyse : Un Sudoku contient des cases, des lignes horizontales et verticales, et des carrés.
J'ai donc créé les classes suivantes (j'ai allégé le code)

Code :

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

1
2
3
4
5
6
7
8
class Sudoku
{
	private:
		Case* m_cases; // tableau de 81 cases
		Ligne* m_lignesH; // tableau de 9 lignes horizontales
		Ligne* m_lignesV; // 9 lignes verticales
		Carre* m_carres; // 9 carrés
};

Un carré a un index qui le différencie des autres carrés (position différente) et est composé d'un tableau de 9 pointeurs sur des cases, et il connait le Sudoku à qui il appartient

Code :

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

1
2
3
4
5
6
7
8
 
class Carre
{
	private:
		unsigned int m_index;
		Case** m_cases;
		Sudoku* m_sudoku;
};

Une ligne a un index qui la différencie des autres lignes (position différente), et est un tableau de 9 pointeurs sur des cases, et elle connait le Sudoku à qui elle appartient

Code :

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

1
2
3
4
5
6
7
class Ligne
{
	private:
		unsigned int m_index;
		Case** m_cases;
		Sudoku* m_sudoku;
};

Une case a un index qui la différencie des autres cases (position différente), une valeur (1à9), peut être cachée ou non, et elle sait à qui elle appartient

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
 
class Case
{
	private:
		unsigned int m_index;
		unsigned int m_valeur;
		bool m_cachee;
		Carre* m_carre;
		Ligne* m_ligneH;
		Ligne* m_ligneV;
		Sudoku* m_sudoku;
};

Comme on peut le constater Case est utilisée dans chacune des classes et je crois que je ne peux pas l'éviter.

Chaque classe sait dans quelle autre classe elle est incluse. Ca je sais pas encore si c'est indispensable mais j'ai le sentiment que j'en aurais besoin plus tard, dans les constructeurs de copie par exemple, et puis avec ça j'ai l'impression que je ferais pas des instructions à ralonge, tout est accessible depuis n'importe quelle classe (avec les getteurs et setteurs adaptés bien sûr).

Suis je à côté de la plaque ?