Discussion :

[mulfycrowh]

Bonjour,

J'ai un fichier Const_Msg.h qui contient des déclarations de numéros de message :

Code :

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

1
2
3
4
5
6
7
8
 
 
int ID_MSG_ENTER_NEW_RESOLUTION	= 0;
int ID_MSG_WRONG_RESOLUTION= 1;
int ID_MSG_MODE= 2;
int ID_MSG_FIXED_OUTPUT= 3;
int ID_MSG_WIDTH	= 4;
int ID_MSG_HEIGHT= 5;

Ce fichier est inclus dans le fichier dxManager.cpp pour utilisation de ces messages.
D'autre part, winmain.cpp utilise une fonction X (déclarée dans dxManager.h et explicitée dans dxManager.cpp) qui utilise des messages. J'ai donc inclu dans winmain.cpp Const_Msg.h

A l'édition de lien, erreur LNK2005 :

"int ID_MSG_ENTER_NEW_RESOLUTION" (?ID_MSG_ENTER_NEW_RESOLUTION@@3HA) already defined in dxManager.obj

...


car double édition de lien sur Const_Msg et j'ai donc modifié Const_Msg.h :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
 
#ifndef CONST_MSG_H
#define CONST_MSG_H
 
int ID_MSG_ENTER_NEW_RESOLUTION	= 0;
int ID_MSG_WRONG_RESOLUTION= 1;
int ID_MSG_MODE= 2;
int ID_MSG_FIXED_OUTPUT= 3;
int ID_MSG_WIDTH	= 4;
int ID_MSG_HEIGHT= 5;
 
#endif

L'erreur subsiste, pourquoi ? La directive de compilation à priori fait en sorte que Const_Msg.h ne soit incluse qu'une fois ?

Merci à tous

[10_GOTO_10]

- Est-ce que tu compiles avec l'option "entêtes précompilés" ? Si oui, essaye de reconstruire le projet.

- Est-ce que le .h ne contient que ça ? N'y a-t-il pas des includes en plus ?

- Essaye plutôt avec la directive "#pragma once". Le #ifndef marche pour la compilation d'un cpp, mais si le .h est inclu dans deux cpp, le #define se remet à zéro au début de la compilation du second.

[mulfycrowh]

Le point h contient plusieurs include.
Il y a pragma once dans chaque fichier .h

Ou gère t'on les en têtes précompilées ?

[mulfycrowh]

Le problème vient du fait que l'édition de lien voit deux fois la déclaration des messages.
Je n'ai pas encore trouvé la solution car j'ai besoin des messages à de nombreux endroits du projet ...

[3DArchi]

Bonsoir,
Tes variables sont définis autant de fois que le fichier d'en-tête est inclus dans un fichier cpp compilé. Il te faut séparer la déclaration des tes variables de leur définition.

Tu as différentes façon d'aborder ton problème :
1/Utilisation d'une énumération (ce qui m'a l'air de correspondre à ce que tu fais) :

Code :

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

1
2
3
4
5
6
7
8
enum E_ID_Message{
   ID_MSG_ENTER_NEW_RESOLUTION=1,
   ID_MSG_WRONG_RESOLUTION,
   ID_MSG_MODE,
   ID_MSG_FIXED_OUTPUT,
   ID_MSG_WIDTH	,
   ID_MSG_HEIGHT
};

Ici tes valeurs ne sont plus contenues das des variables mais directement portées par un type : l'énumération E_ID_Message.

2/ Séparer la déclaration de la définition :
déclaration dans l'en-tête : Const_Msg.h

Code Const_Msg.h :

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

1
2
3
4
5
6
7
8
9
#ifndef CONST_MSG_H
#define CONST_MSG_H
extern const int ID_MSG_ENTER_NEW_RESOLUTION;
extern const int ID_MSG_WRONG_RESOLUTION;
extern const int ID_MSG_MODE;
extern const int ID_MSG_FIXED_OUTPUT;
extern const int ID_MSG_WIDTH;
extern const int ID_MSG_HEIGHT;
#endif

Définition dans un .cpp pour ne les avoir qu'une seule fois :

Code Const_Msg.cpp :

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

1
2
3
4
5
6
const int ID_MSG_ENTER_NEW_RESOLUTION	= 0;
const int ID_MSG_WRONG_RESOLUTION= 1;
const int ID_MSG_MODE= 2;
const int ID_MSG_FIXED_OUTPUT= 3;
const int ID_MSG_WIDTH	= 4;
const int ID_MSG_HEIGHT= 5;

Mais, il me semble quand même que l'enum soit la bonne solution.

[mulfycrowh]

Merci bcp

[mulfycrowh]

La solution 1/ avec enum fonctionne très bien.
MERCI !