C++: inclure plusieurs fichiers d'en-tête ayant le même nom d'espaces de noms différents

Question

Comment résolvez-vous créé en incluant un fichier d'en-tête du même nom qu'un autre fichier d'en-tête déjà indirectement inclus à la suite d'un autre include?

Par exemple:

// src/blah/a.hpp 
#ifndef A_HPP 
#define A_HPP 

namspace blah 
{ 

class A 
{ 
} 

} 

#endif 

// src/blah/b.hpp 
#ifndef B_HPP 
#define B_HPP 

#includes "a.hpp" 

namspace blah 
{ 

class B 
{ 
} 

} 

#endif 

// src/foo/a.hpp 
#ifndef A_HPP 
#define A_HPP 

namspace foo 
{ 

class A 
{ 
} 

} 

#endif 

// src/foo/c.hpp 
#ifndef C_HPP 
#define C_HPP 

#includes "../b.hpp" 
#includes "a.hpp"  // won't be included due to multiple inclusion prevention 

namspace foo 
{ 

class C 
{ 
} 

} 

#endif 

Dans le dernier fichier d'en-tête, a.hpp ne seront pas inclus en raison des multiples agents de préprocesseur d'inclusion. Vraiment, cela devrait être correct mais puisque les classes sont dans des espaces de noms différents. Je me rends compte que le moyen le plus simple est de simplement changer le nom de foo/a.hpp ou juste de lui donner un faux nom dans la garde d'inclusion multiple. Y a-t-il un meilleur moyen?

EDIT Je comprends que vous pouvez résoudre ce problème en utilisant un nom plus descriptif dans les directives #define et #ifndef (par exemple FOO_A_HPP et BLAH_A_HPP), mais je veux savoir si cela est la meilleure façon recommandée ou . Est-ce que certaines personnes recommandent d'utiliser un nom de fichier différent comme meilleure solution ou n'est-ce pas important? Recommanderiez-vous en utilisant la convention:

<NAMESPACE>_<CLASS>_HPP 

au lieu de

<CLASS>_HPP 

pour donner une meilleure chance d'éviter ces problèmes en général?

Answer 1

vous résolvez cela, simplement, ne pas utiliser le même #define au sommet ...

Il serait préférable d'utiliser BLAH_A_HPP et FOO_A_HPP etc afin que le #define comprend également le nom d'espace de noms.

Edit: Eh bien, je recommande de faire ce qui suit:

1) Ne nommez pas les mêmes en-têtes (c.-à-utiliser un autre nom de fichier ... cela ne suffit pas toujours) et utiliser des noms différents #define.
2) Ne nommez pas les classes de la même façon. Mettez 1 classe par en-tête et nommez l'en-tête après la classe
3) Si elles sont différenciées par un espace de noms, utilisez cet espace de noms dans le nom de fichier ET #define
4) Ajoutez un ID #define qui vous est propre (Je pourrais utiliser GOZ par exemple)
5) Utiliser #pragma une fois. C'est utile pour les compilateurs qui l'utilisent.

C'est une question de goût. Choisissez un système qui fonctionne pour vous et respectez-le. Il n'y a pas de vrai ou de faux. Tant que cela fonctionne et est cohérent.

Answer 2

Bien qu'il n'y ait pas d'espaces de noms dans C, je pense que cela fonctionne à la fois pour C et C++.

#ifndef BLAH_B_HPP 

et

#ifndef FOO_B_HPP 

Answer 3

Lorsque MSVC crée une classe, il ajoutera un GUID à votre nom de garde pour éviter ce genre de problème.

Answer 4

Comme vous l'avez suggéré vous-même, la manière normale de procéder consiste à rendre vos identifiants de garde uniques, en ajoutant un GUID, un chemin complet ou autre.

Dans VC++ il y a aussi un pragma pour prendre soin d'inclusion multiples pour vous, afin que vous n'avez pas besoin d'écrire le #if defined(HUMPTYDUMPTY) vous bit, il va comme

#pragma once 

a été autour pendant quelques itérations VC++ maintenant, mais pour les autres compilateurs, vous devez vérifier. Cela vous évite également de chercher des noms uniques.

Édité: Juste jeté un coup d'oeil et l'Internet pense que c'est aussi dans GCC. On dirait que les gardes de style #include sont très 20ème siècle ...

Answer 5

Je suis surpris que personne ne mentionne la solution simple d'utilisation #undef A_HPP comme ci-dessous

//src/foo/c.hpp 
#ifndef C_HPP 
#define C_HPP 

#includes "../b.hpp" 
#undef A_HPP 
#includes "a.hpp" 
... 

Ceci est particulièrement bon si foo/c.hpp est le seul fichier que l'on peut ou est autorisé à changer.