espaces de noms pour les types d'enum - meilleures pratiques

Question

Souvent, on a besoin de plusieurs types énumérés ensemble. Parfois, il y a un conflit de noms. Deux solutions à ce problème viennent à l'esprit : utiliser un espace de noms, ou utiliser des noms d'éléments d'énumération "plus grands". Cependant, la solution de l'espace de noms a deux implémentations possibles : une classe fictive avec des énumérations imbriquées ou un espace de noms à part entière.

Je cherche les avantages et les inconvénients de ces trois approches.

Exemple :

// oft seen hand-crafted name clash solution
enum eColors { cRed, cColorBlue, cGreen, cYellow, cColorsEnd };
enum eFeelings { cAngry, cFeelingBlue, cHappy, cFeelingsEnd };
void setPenColor( const eColors c ) {
    switch (c) {
        default: assert(false);
        break; case cRed: //...
        break; case cColorBlue: //...
        //...
    }
 }

// (ab)using a class as a namespace
class Colors { enum e { cRed, cBlue, cGreen, cYellow, cEnd }; };
class Feelings { enum e { cAngry, cBlue, cHappy, cEnd }; };
void setPenColor( const Colors::e c ) {
    switch (c) {
        default: assert(false);
        break; case Colors::cRed: //...
        break; case Colors::cBlue: //...
        //...
    }
 }

 // a real namespace?
 namespace Colors { enum e { cRed, cBlue, cGreen, cYellow, cEnd }; };
 namespace Feelings { enum e { cAngry, cBlue, cHappy, cEnd }; };
 void setPenColor( const Colors::e c ) {
    switch (c) {
        default: assert(false);
        break; case Colors::cRed: //...
        break; case Colors::cBlue: //...
        //...
    }
  }

Answer 1

Réponse originale de C++03 :

El avantage d'un namespace (sur un class ) est que vous pouvez utiliser using quand vous le souhaitez.

El problème avec l'utilisation d'un namespace est que les espaces de noms peuvent être étendus ailleurs dans le code. Dans un grand projet, vous ne pouvez pas être sûr que deux enums distincts ne pensent pas tous les deux s'appeler eFeelings

Pour un code plus simple, j'utilise un struct car vous souhaitez sans doute que le contenu soit public.

Si vous suivez l'une de ces pratiques, vous êtes en avance sur votre temps et vous n'avez probablement pas besoin de vous pencher davantage sur la question.

Conseil plus récent, C++11 :

Si vous utilisez C++11 ou une version ultérieure, enum class mettra implicitement les valeurs de l'enum dans le nom de l'enum.

Avec enum class vous perdrez les conversions et comparaisons implicites avec les types entiers, mais en pratique, cela peut vous aider à découvrir du code ambigu ou bogué.

Answer 2

FYI Dans C++0x il y a une nouvelle syntaxe pour les cas comme celui que vous avez mentionné (voir Page wiki C++0x )

enum class eColors { ... };
enum class eFeelings { ... };

Answer 3

J'éviterais définitivement d'utiliser une classe pour cela ; utilisez plutôt un espace de noms. La question se résume à savoir s'il faut utiliser un espace de noms ou des identifiants uniques pour les valeurs de l'enum. Personnellement, j'utiliserais un espace de noms pour que mes identifiants soient plus courts et, je l'espère, plus explicites. Le code de l'application pourrait alors utiliser une directive "using namespace" et rendre le tout plus lisible.

D'après votre exemple ci-dessus :

using namespace Colors;

void setPenColor( const e c ) {
    switch (c) {
        default: assert(false);
        break; case cRed: //...
        break; case cBlue: //...
        //...
    }
}

Answer 4

Une différence entre l'utilisation d'une classe et d'un espace de nom est que la classe ne peut pas être rouverte comme le peut un espace de nom. Cela évite la possibilité d'abuser de l'espace de noms à l'avenir, mais il y a aussi le problème que vous ne pouvez pas non plus ajouter à l'ensemble des énumérations.

Un avantage possible de l'utilisation d'une classe est qu'elle peut être utilisée comme argument de type de modèle, ce qui n'est pas le cas pour les espaces de noms :

class Colors {
public:
  enum TYPE {
    Red,
    Green,
    Blue
  };
};

template <typename T> void foo (T t) {
  typedef typename T::TYPE EnumType;
  // ...
}

Personnellement, je ne suis pas un fan de en utilisant et je préfère les noms pleinement qualifiés, donc je ne vois pas vraiment cela comme un avantage pour les espaces de noms. Cependant, ce n'est probablement pas la décision la plus importante que vous prendrez dans votre projet !

Answer 5

Étant donné que les enums ont une portée limitée à la portée qui les englobe, il est probablement préférable de les envelopper dans un fichier quelque chose pour éviter de polluer l'espace de nom global et pour éviter les collisions de noms. Je préfère un espace de noms à une classe simplement parce que namespace ressemble à un sac de rétention, alors que class ressemble à un objet robuste (cf. struct vs. class débat). Un avantage possible d'un espace de noms est qu'il peut être étendu ultérieurement - utile si vous avez affaire à du code tiers que vous ne pouvez pas modifier.

Tout cela est bien sûr discutable lorsque nous aurons des classes enum avec C++0x.