Discussion :

[basnifo]

Bonjour

J'ai un problème d'include qui me donne ce genre d'erreur :

FicheDlg.obj : error LNK2005: "unsigned long TURQUOISECOLOR" (?TURQUOISECOLOR@@3KA) already defined in fich2.obj
FicheDlg.obj : error LNK2005: "unsigned long SKYBLUECOLOR" (?SKYBLUECOLOR@@3KA) already defined in fich2.obj
...

J'ai donc un fichier Constantes.h qui contient en gros :

Code :

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

1
2
3
4
5
6
7
8
9
10
#ifndef _CONSTDEF
#define _CONSTDEF
 
CBrush SKYBLUEBRUSH;
CBrush TURQUOISEBRUSH;
 
COLORREF SKYBLUECOLOR=RGB(173,216,230);
COLORREF TURQUOISECOLOR=RGB(0,255,255);
 
#endif

J'ai ensuite mes fichiers cpp qui ont un include vers Constantes.h

Je ne comprends pas l'erreur Si j'ai bien compris, #ifndef devrait m'éviter ce genre de pb, non ?

merci

[Aurelien.Regat-Barrel]

Non, car tu définis des variables dans un .h, au lieu de simplement les déclarer. Donc chaque cpp qui utilise ton .h va définir TURQUOISECOLOR etc... et donc on a des définitions multiples.
Tu peux utiliser extern, ou une fonction:

Code :

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

1
2
3
4
5
6
7
8
// .h
#ifndef _CONSTDEF 
#define _CONSTDEF 
 
// déclaration SANS définir
extern CBrush SKYBLUEBRUSH; 
 
CBrush & GetTURQUOISEBRUSH();

Code :

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

1
2
3
4
5
6
7
8
9
10
//.cpp
 
// déclaration ET définition
CBrush SKYBLUEBRUSH; 
 
CBrush & GetTURQUOISEBRUSH()
{
    static CBrush TURQUOISEBRUSH;
    return TURQUOISEBRUSH;
}

idem pour les COLORREF. Mais dans leur cas tu peux rester dans le .h en utilisant une macro:

Code :

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

1
2
#define SKYBLUECOLOR RGB(173,216,230)
#define TURQUOISECOLOR RGB(0,255,255)

[basnifo]

ok merci pour la réponse

Enfin j'aurais pensé que le préprocesseur ne passait pas dans la condition ... mais apparemment il l'évalue quand même ...