我目前使用下面這個簡單的單例類:辛格爾頓模板設計問題
template<class T>
class singleton
: boost::noncopyable
{
public:
static T& get_instance()
{
assert(sm_instance != nullptr);
return *static_cast<T*>(sm_instance);
}
protected:
singleton()
{
assert(sm_instance == nullptr);
sm_instance = this;
}
virtual ~singleton()
{
assert(sm_instance != nullptr);
sm_instance = nullptr;
}
private:
static singleton<T>* sm_instance;
};
template<class T> singleton<T>* singleton<T>::sm_instance = nullptr;
class example_one
: public singleton<example_one>
{
static example_one instance;
};
example_one example_one::instance;
class example_two
: singleton<example_two>
{
static example_two instance;
};
example_two example_two::instance;
// Usage:
example_one& x = example_one::get_instance();
example_two& y = example_two::get_instance(); // not accessible because 'example_two' uses 'private' to inherit from 'singleton<T>'
不過,我想調整一些東西。我不喜歡那get_instance()
被繼承到派生類。
我願做這樣的事情(非工作代碼):
template<class T>
T& get_singleton();
template<class T>
class singleton
{
friend T& get_singleton()
{
assert(sm_instance != nullptr);
return *static_cast<T*>(sm_instance);
}
}
// Usage:
example_two& x = get_singleton<example_two>();
如果你能避免以任何方式,不使用單身 - http://stackoverflow.com/questions/1392315/problems-with-singleton-pattern –
您的代碼不很有道理。第一個例子檢查它的實例字段是否爲空,在CONSTRUCTOR(爲什麼?),然後析構函數更糟。你的析構函數應該刪除sm_instance,或者根本不需要析構函數。在析構函數內部設置一個instacne字段爲空也是毫無意義的。 –
@ AngelO'Sphere - 我認爲你應該再次閱讀代碼並思考它。 – 0xbadf00d