С++ лучший способ определить константы между файлами

Избавьтесь от extern, и все готово.

Этот код отлично работает в заголовке, потому что все «действительно постоянно» и, следовательно, имеет внутреннюю связь:

const int BEGINNING_HEALTH = 10;
const int BEGINNING_MANA = 5;
const char BEGINNING_NAME[] = "Fred";
const char *const BEGINNING_NAME2 = "Barney";

Этот код нельзя безопасно поместить в заголовочный файл, потому что каждая строка имеет внешнюю связь (либо явно, либо из-за того, что она не является действительно постоянной):

extern const int BEGINNING_HEALTH = 10;
extern const int BEGINNING_MANA = 5;
const char *BEGINNING_NAME = "Wilma";  // the characters are const, but the pointer isn't

Как насчет перечислений?

константы.hpp

  enum {
    BEGINNING_HEALTH = 10,
    BEGINNING_MANA = 5
  }

Используйте «static const int» в вашем файле .hpp и ничего не добавляйте в файл .cpp (кроме любого другого кода, который у вас есть, конечно).

использовать пространства имен:

namespace GameBeginning {
    const int HEALTH = 10;
    const int MANA   = 5; 
};

тогда вы можете использовать как player.health = GameBeginning::HEALTH;

Большинство компиляторов просто не выделяют место для значений const POD. Они оптимизируют их и обращаются с ними так, как если бы они были #defined, не так ли?

Что случилось с простым:

#define BEGINNING_HEALTH 10

Чувак, это были дни.
Ой, подождите, эти до сих пор дни!

возможно, что-то вроде статического класса?

class CONSTANTS {
public:
static inline int getMana() { return 10;};
};

В качестве быстрого ответа на вопрос, заданный в заголовке, можно сказать, что шаблон singleton — это, возможно, наилучший способ C++ для определения межфайловых констант и обеспечения только одного экземпляра объекта.

Что касается проблемы с параметром шаблона, вам нужно передать тип, а не значение. Ваш тип "int".