Discussion :

[mamok]

Bonjour,

Je suis en train de m'embrouiller avec des #ifndef
Mon inputOutput.cpp compile avec le code suivant

Code :

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

1
2
3
4
5
6
7
 
// InputOutput.h: interface for the CInputOutput class.
//////////////////////////////////////////////////////////////////////
//#ifndef Date_h
//#define Date_h
#include "CDate.h"
//#endif

mais pas avec celui-la

Code :

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

1
2
3
4
5
6
7
 
// InputOutput.h: interface for the CInputOutput class.
//////////////////////////////////////////////////////////////////////
#ifndef Date_h
#define Date_h
#include "CDate.h"
#endif

Alors que le 2eme normalement est mieux, non???

Merci

[Médinoc]

Généralement, on met juste les gardes d'inclusion à l'intérieur d'un fichier d'en-tête, pas à l'extérieur...

Si ça ne compile pas dans le second cas, il est possible que Date_h soit utilisé dans le fichier d'en-tête...

[mamok]

juste qques précisons les gardes d'inclusion sont les #ifndef et #define
c ca ?

pour toi a l'extérieur=.cpp??

et le second cas
mon .h utilse mon CDate.h donc je comprends pas trop en fait
désolé

[mamok]

exemple

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
 
 
#ifndef CThirty360_h
#define CThirty360_h
 
#ifndef CDayCounter_h
#define CDayCounter_h
#include "CDayCounter.h"
#endif
 
 
class CThirty360 : public CDayCounter  
{
public:
	CThirty360();
	virtual ~CThirty360();
	double yearFraction(CDate &d1, CDate &d2);
	};
 
#endif

et mon .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
 
// CThirty360.cpp: implementation of the CThirty360 class.
//
//////////////////////////////////////////////////////////////////////
 
#include "CThirty360.h"
 
//////////////////////////////////////////////////////////////////////
// Construction/Destruction
//////////////////////////////////////////////////////////////////////
 
CThirty360::CThirty360():CDayCounter("30360")
{
//	m_dayCountType = "30360";
}

et j'ai les erreurs suivantes

Compiling...
CThirty360.cpp
c:\my_models\dll_pricing_v2.0\headers\cdaycounter.h(22) : error C2629: unexpected 'class CDayCounter ('
c:\my_models\dll_pricing_v2.0\headers\cdaycounter.h(22) : error C2238: unexpected token(s) preceding ';'

mon CDayCounter est bien inclus???

merci

[Médinoc]

juste qques précisons les gardes d'inclusion sont les #ifndef et #define
c ca ?

Oui.

pour toi a l'extérieur=.cpp??

Oui. Et à l'intérieur = à l'intérieur du .h gardé.

Dans ton dernier exemple, le #ifndef CThirty360_h est au bon endroit, mais le #ifndef CDayCounter_h n'a rien à faire ici : Il devrait être dans CDayCounter.h et pas ailleurs.
Edit: Si tu l'as mis dans les deux, c'est normal que ça foire.

De plus, l'accolade fermante de ta classe CThirty360 est mal placée : Elle devrait être sur le même niveau d'indentation que l'accolade ouvrante.

[mamok]

ok donc pour résumer

dans tous les .h, il faut mettre

#ifndef NOM.H_H
#define
corps du .h

#endif

et dans les .cpp #include "headers.h"

[Médinoc]

C'est ça.
Et dans les .h qui incluent d'autres .h, on entoure le fichier lui-même de gardes, mais pas ses inclusions.

[mamok]

super merci
c'est clair maintenant