Discussion :

[erroneus]

Bonjour,

Mon soucis est le suivant : j'ai des constantes qui sont utilisées u peu partout dans mon code (typiquement des constantes de physique, genre vitesse de la lumière). Pour le moment, elles sont définies comme const dans les .h de plusieurs classes. je ne trouve pas ça très élégant mais je ne sais pas trop comment y remédier proprement.

Est-ce que je peux toutes les mettre dans un .h que j'appelle dans chaque classe ? Est-ce que je dois définir une classe spécifique où je les définis toutes ?

Bref, je suis preneur de toute solution qui donnerait un code plus propre.

D'avance, merci.

[Invité]

Bonjour,

Elles peuvent très bien être regroupées dans un .h (par exemple: constantes.h) dédié à la déclaration des constantes qui sera ensuite inclus dans dans des .cpp, .C, ... ou d'autres .h qui en auraient besoin.

[erroneus]

J'ai essayé et j'ai un léger soucis

Dans mon fichier Defines.h, j'ai déclaré mes constantes :

Code :

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

1
2
3
4
const float AMU = 931.494028; // MeV.c^-2
const float M_proton = 938.783; // MeV.c^-2
const float M_neutron = 939.565; // MeV.c^-2
const float c0 = 299792458; // m.s^-1

Ensuite, ce Defines.h est appelé par les .h de différentes classes.

À la compilation (sous Linux avec gcc) :

Code :

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

1
2
3
4
5
6
7
8
Defines.h:50: erreur: redefinition of ‘const float AMU’
Defines.h:50: erreur: ‘const float AMU’ previously defined here
Defines.h:51: erreur: redefinition of ‘const float M_proton’
Defines.h:51: erreur: ‘const float M_proton’ previously defined here
Defines.h:52: erreur: redefinition of ‘const float M_neutron’
Defines.h:52: erreur: ‘const float M_neutron’ previously defined here
Defines.h:53: erreur: redefinition of ‘const float c0’
Defines.h:53: erreur: ‘const float c0’ previously defined here

Est-ce que j'ai compris de travers ou oublié quelque chose ?

[valAa]

Salut,
Tu as des inclusions multiples, il faut mettre des gardes dans ton fichier Defines.h (comme dans tout fichier d'entêtes) qui ressemblera alors à :

Code :

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

1
2
3
4
5
6
7
8
9
#ifndef DEFINES_H
#define DEFINES_H
 
const float AMU = 931.494028; // MeV.c^-2
const float M_proton = 938.783; // MeV.c^-2
const float M_neutron = 939.565; // MeV.c^-2
const float c0 = 299792458; // m.s^-1
 
#endif // DEFINES_H

[erroneus]

Merci à vous deux.

[3DArchi]

Salut,
Tu peux utiliser un espace de nom pour les regrouper de façon cohérente :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
 
#ifndef DEFINES_H
#define DEFINES_H
 
namespace physical
{
static const float AMU = 931.494028; // MeV.c^-2
static const float M_proton = 938.783; // MeV.c^-2
static const float M_neutron = 939.565; // MeV.c^-2
static const float c0 = 299792458; // m.s^-1
}
#endif // DEFINES_H

Pourquoi utiliser float plutôt que double ?