Discussion :

[Nauvisse]

Bonjour,

J'ai un ordi Windows 64 bits et je programme sous QT Creator. J'ai un problème de compilation lorsque je déclare des variables globales ou définis des fonctions dans un header. Le message est le suivant :

multiple definition of 'alphabetL'
first defined here

Mais je n'ai pas le message avec la variable NAN_E.
J'ai consulté quelques forums (https://stackoverflow.com/questions/...in-header-file), mais je n'ai pas trop compris.
On y lit qu'on ne doit pas trop écrire ce genre de code dans un header, mais cela se fait quand même.

~~~~~~
Fichiers.h
~~~~~~
- string.h

Code :

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

1
2
3
4
5
6
7
#ifndef STRING_H
#define STRING_H
 
#include "graph.h"
const char* alphabetL = "abcdefghijklmnopqrstuvwxyz"; // il y a un problème.
...
#endif // STRING_H

- graph.h

Code :

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

1
2
3
4
5
6
#ifndef GRAPH_H
#define GRAPH_H
 
#include "draw.h"
...
#endif // GRAPH_H

- draw.h

Code :

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

1
2
3
4
5
6
#ifndef DRAW_H
#define DRAW_H
 
#include "list.h"
...
#endif // DRAW_H

- list.h

Code :

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

1
2
3
4
5
6
#ifndef LIST_H
#define LIST_H
 
#include "math.h"  // It is not <math.h>.
...
#endif // LIST_H

- math.h

Code :

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

1
2
3
4
5
6
7
8
#ifndef MATH_H
#define MATH_H
 
#include "misc.h"
 
const ent NAN_E   = 0x8000000000000000LL; // Il n'y pas de problème.
...
#endif // MATH_H

- misc.h

Code :

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

1
2
3
4
5
#ifndef MISC_H
#define MISC_H
 
// pas de #include "..."
#endif // MISC_H

- main.h

Code :

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

1
2
3
4
5
6
#ifndef MAIN_H
#define MAIN_H
 
#include "../Graph/string.h"
...
#endif // MAIN_H

~~~~~~~
Fichiers.cpp
~~~~~~~
- string.cpp

Code :

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

#include "string.h"

- graph.cpp

Code :

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

#include "graph.h"

- draw.cpp

Code :

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

#include "draw.h"

- misc.cpp

Code :

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

#include "misc.h"

- main.cpp

Code :

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

#include "main.h"

Est-ce qu'un aurait une solution s'il vous plaît ?
Merci d'avance.

[CGi]

Il ne faut pas définir une variable globale dans un fichier entête. Car elle sera définie dans tous les modules où tu as inclus le fichier entête en question.
Tu la défini dans un fichier source et tu la déclares extern dans le fichier entête :

Code :

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

extern const char* alphabetL;

[Nauvisse]

Merci beaucoup. ça marche.

J'ai fini par comprendre :
1. La variable NAN_E est acceptée parce qu'il n'y a pas de fichier "math.cpp" (il y a juste un fichier "math.h").
2. La variable alphabetL est dupliquée puisque je mets #include dans le fichier "string.cpp" et puisque "string.h" est aussi compilé.

Merci encore.

[dalfab]

Les variables et les fonctions ne doivent pas être définies dans un entête.

Il y a des exceptions, les entités qui sont inline, template ou constexpr.
Et par extension aussi les constantes entières qui sont implicitement considérées comme si on avait utilisé constexpr.

Code :

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

1
2
3
4
5
6
7
const long long  NAN_E = 0x8000000000000000LL;
// est vu comme étant
constexpr long long  NAN_E = 0x8000000000000000LL;      // est constante et est inline
// Mais
const double  PI = 3.14159265358979323846264338328;     // PROVOQUE une redéfinition!
// Alors que
constexpr double  PI = 3.14159265358979323846264338328; // possible depuis C++20

[Nauvisse]

Merci beaucoup.