"error:invalid use of undefined type" ok, mais je comprends pas

Version imprimable

Salut à tous,

J'ai un bug sur un script, le compilo me renvoie "error:invalid use of undefined type". Bon j'ai bien compris que cette erreur veut dire que je me sers mal de mon bazar, mais là je comprends pas en quoi...

Alors je vous mets les fichiers pour faire simple:
FaEntretien.h

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
 
#ifndef FAENTRETIEN_H
#define FAENTRETIEN_H
 
namespace CSC_Entretien
{
 
#include "cctrlentretien.h"
class CCtrlEntretien;
 
class FaEntretien
{
	public:
		FaEntretien(int p_SeuilPermanent, int p_SeuilClignotant, int p_SeuilVidange, int p_SeuilFiltreAir, int p_SeuilGeneral);
		~FaEntretien();
	private:
		CCtrlEntretien* controleur;
};
 
}
#endif

FaEntretien.cpp

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
#include "faentretien.h"
#include <iostream>
 
namespace CSC_Entretien
{
 
FaEntretien::FaEntretien(int p_SeuilPermanent, int p_SeuilClignotant, int p_SeuilVidange, int p_SeuilFiltreAir, int p_SeuilGeneral)
{
	CCtrlEntretien* ctrl = new CCtrlEntretien(p_SeuilPermanent, p_SeuilClignotant, p_SeuilVidange, p_SeuilFiltreAir, p_SeuilGeneral, this);
	this->controleur = ctrl;
	std::cout << "Construction facade entretien : " << this << std::endl;
}
 
FaEntretien::~FaEntretien()
{
}
 
}

CCtrlEntretien.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
#ifndef CCTRLENTRETIEN_H
#define CCTRLENTRETIEN_H
 
namespace CSC_Entretien
{
 
#include "faentretien.h"
class FaEntretien;
 
class CCtrlEntretien
{
	public:
		CCtrlEntretien(int p_SeuilPermanent, int p_SeuilClignotant, int p_SeuilVidange, int p_SeuilFiltreAir, int p_SeuilGeneral, FaEntretien* facade);
		~CCtrlEntretien();
	private:
		FaEntretien* facade;
		int m_Compteur;
		int m_SeuilClignotant;
		int m_SeuilPermanent;
};
 
}
 
#endif

CCtrlEntretien.cpp

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
#include "cctrlentretien.h"
#include <iostream>
 
namespace CSC_Entretien
{
 
CCtrlEntretien::CCtrlEntretien(int p_SeuilPermanent, int p_SeuilClignotant, int p_SeuilVidange, int p_SeuilFiltreAir, int p_SeuilGeneral, FaEntretien* p_Facade)
{
	this->facade = p_Facade;
	this->m_Compteur = 0;
	this->m_SeuilClignotant = p_SeuilClignotant;
	this->m_SeuilPermanent = p_SeuilPermanent;
	std::cout << "Construction controleur entretien : " << this << std::endl;
}
 
CCtrlEntretien::~CCtrlEntretien()
{
}
 
}

Mon IDE (DevCpp) me dit que c'est dans FaEntretien.cpp sur la ligne:

Code:

CCtrlEntretien* ctrl = new CCtrlEntretien(p_SeuilPermanent, p_SeuilClignotant, p_SeuilVidange, p_SeuilFiltreAir, p_SeuilGeneral, this);

Mais je comprends pas. Quelqu'un a une idée?
Merci d'avance
Bonne nuit à tous!

19/01/2010, 07h17
3DArchi

Salut,
En regardant rapidement, je dirais qu'il manque l'inclusion de CCtrlEntretien.h dans FaEntretien.cpp. La déclaration anticipée est valide tant que la classe est utilisée pour déclarer des références ou des pointeurs. Dès que tu alloues, tu a besoin de connaître la classe d'où la nécessité d'inclure l'en-tête contenant la déclaration complète de la classe.

Petite rectification :

Citation:

Envoyé par gwinyam

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
#ifndef FAENTRETIEN_H
#define FAENTRETIEN_H

namespace CSC_Entretien
{

// #include "cctrlentretien.h" -> A mettre dans le CPP et pas ici (ou alors avant le namespace) !
class CCtrlEntretien;

class FaEntretien
{
	public:
		FaEntretien(int p_SeuilPermanent, int p_SeuilClignotant, int p_SeuilVidange, int p_SeuilFiltreAir, int p_SeuilGeneral);
		~FaEntretien();
	private:
		CCtrlEntretien* controleur;
};

}
#endif

FaEntretien.cpp

Code:

1
2
3
#include "faentretien.h"
#include "cctrlentretien.h" 
#include <iostream>

Idéalement, j'aurais fait un CCtrlEntretienFwd.h :

Code:

1
2
3
4
5
6
7
8
#ifndef CCTRLENTRETIENFWD_H
#define CCTRLENTRETIENFWS_H
 
namespace CSC_Entretien
{
   class CCtrlEntretien;
}
#endif /*CCTRLENTRETIENFWS_H*/

Puis :
FaEntretien.h

Code:

1
2
3
4
5
6
7
8
#ifndef FAENTRETIEN_H
#define FAENTRETIEN_H
 
#include "CCtrlEntretienFwd.h"
namespace CSC_Entretien
{
   class FaEntretien
   {

Et :
FaEntretien.cpp

Code:

1
2
3
4
5
6
#include "faentretien.h"
#include "CCtrlEntretien.h"
#include <iostream>
 
namespace CSC_Entretien
{

Et tu as le problème symétrique dans CCtrlEntretien.h qui inclue #include "faentretien.h" dans le namespace CSC_Entretien.

21/01/2010, 00h24
Lightness1024

Inclure des headers DANS des namespaces, ca ne m'a pas l'air très "good practice" ! :calim2:
(bien que j'adorerais pouvoir faire ca avec windows.h si ils avaient pas autant de fichus put*** de defines qui ne sont pas scopées !)