c++ how to have same enum members name in different enum names without getting err:redefinition; previous definition was 'enumerator'

Question

I have config file that I include in all my files there I have different enums but inside each enum there are same element names for example: config.h

enum GameObjectType {      NINJA_PLAYER  }; enum GameObjectTypeLocation {     NONE,     MASSAGE_ALL,  //this is for ComponentMadiator     NINJA_PLAYER   }; 

But when I try to compile the project with calling the enums with the proper enum name

m_pNinjaPlayer = (NinjaPlayer*)GameFactory::Instance().getGameObj(GameObjectType::NINJA_PLAYER);     ComponentMadiator::Instance().Register(GameObjectTypeLocation::NINJA_PLAYER,m_pNinjaPlayer); 

I am getting compilation error:

error C2365: 'NINJA_PLAYER' : redefinition; previous definition was 'enumerator' (..\Classes\GameFactory.cpp) 2>          d:\dev\cpp\2d\cocos2d-x-3.0\cocos2d-x-3.0\projects\lettersfun\classes\config.h(22) : see declaration of 'NINJA_PLAYER' 

How can I keep in the config.h several enums with different names BUT with same elements names ?

Answer 1

The problem is that old-style enumerations are unscoped. You can avoid this problem (provided your compiler has the relevant C++11 support) by using scoped enumerations:

enum class GameObjectType { NINJA_PLAYER };  enum class GameObjectTypeLocation { NONE, MASSAGE_ALL, NINJA_PLAYER }; 

Alternatively, you can put your old-school enumerations in namespaces:

namespace foo {   enum GameObjectType { NINJA_PLAYER }; } // namespace foo  namespace bar {   enum GameObjectTypeLocation { NONE, MASSAGE_ALL, NINJA_PLAYER }; } // namespace bar 

Then your enum values will be foo::NINJA_PLAYER, bar::NINJA_PLAYER etc.

Answer 2

If you have the possibility to use C++11 I would recommend to use enum class feature to avoid collisions:

enum class GameObjectType {      NINJA_PLAYER  }; enum class GameObjectTypeLocation {     NONE,     MASSAGE_ALL,  //this is for ComponentMadiator     NINJA_PLAYER   }; 

Edit: If you do not have this ability, then you will need to use two different namespaces for each enum.