Does anyone know of a way to make derived classes automatically instantiate a static variable with a template type (this either has to require nothing from the writer of the derived class, or force him to call this static method in order to make the derived class definition valid).
This is probably impossible to understand, I'll try and define it better.
Basically I have a global factory class with a templated function called registerType. For every class derived from Entity, I need this function to be called with the template parameter of the derived type. At the moment, I have to manually do it in some init function, which results in a large block of calls to this function, which kind of goes against the principle of templates for me.
So I have this:
class Factory
{
template <typename EntityType>
registerEntityType();
};
void someInitFunction()
{
/// All of these are derived from Entity
gFactory.registerEntityType<EntityType1>();
gFactory.registerEntityType<EntityType2>();
gFactory.registerEntityType<EntityType3>();
/// and so on
}
whereas I would rather have this:
class Factory
{
template <typename EntityType>
registerEntityType();
};
class Entity // Abstract
{
/// This function should be called automatically with the derived
/// type as a parameter
SomeStaticConstructor<MDerivedType>()
{
gFactory.registerEntityType<MDerivedType>();
}
};
EDIT: This is the static recurring template code that isn't working:
This is my base class, and the class for automatically registering stuff
template <typename DerivedType>
class Registrar
{
public:
Registrar();
void check();
};
template <typename Product, typename DerivedType>
class AbstractFactory: public AbstractFactoryBase<Product>
{
public:
AbstractFactory();
~AbstractFactory();
private:
static Registrar<DerivedType> registrar;
};
The registrar's constructor
template <typename DerivedType>
Registrar<DerivedType>::Registrar()
{
std::cout << DerivedType::name() << " initialisation" << std::endl;
g_AbstractFactories.registerFactoryType<DerivedType>(DerivedType::name());
}
And a derived type
class CrateFactory : public AbstractFactory<Entity, CrateFactory>
{
public:
CrateFactory(FactoryLoader* loader);
virtual ~CrateFactory();
Entity* useFactory(FactoryParameters* parameters);
static std::string name()
{
return "CrateFactory";
}
You might be able to get what you want using a mix-in and the CRTP.
But first, you need to take care of the "order of initialization" problem. To ensure the
gFactory
exists before you try to use it, you really need to make it a proper "singleton" class, like this:Then the "mix-in" would look like this:
And you would use it like this:
[Update]
You can also take the Xeo/Merlyn idea of creating an
EntityBase
, renameEntityMixin
toEntity
, and avoid the need to inherit from two places. I actually think my original proposal is more clear; you could even call the mixinFactoryMixin
and tack it on to any class you want to register.But the Xeo/Merlyn version would look like so:
The keys to any solution are the CRTP and careful use of static local variables to avoid the order-of-initialization problem.
I'd recommend a CTRP-backed approach:
Though, as seen, you now need to get your factory through a
GetGlobalFactory
function, which lazy initializes the factory to ensure that it has been initialized before the enforced registration happens:If anyone is still interested, I figured it out. Static template member variables are not automatically instantiated unless they are used. I needed it to be instantiated before the constructor was called, so I couldn't make it a static local. The solution is to make it a static template member variable, and then use it (just call an empty function on it if you want) in a member function (I use the constructor). This forces the compiler to instantiate the static for every template parameter ever declared, because the instantiated constructor code uses it, for example:
My registry class, with its blank function for calling
My class I want registered.
The registrar's constructor
And my classes constructor