Discussion :

[gwinyam]

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 c++ :

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

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

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

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

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

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!

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

[3DArchi]

Petite rectification :

Code c++ :

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

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

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

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

Code :

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

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 :

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

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 :

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

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.

[Lightness1024]

Inclure des headers DANS des namespaces, ca ne m'a pas l'air très "good practice" !
(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 !)