c++ - 帮我去掉一个Singleton : looking for an alternative

Question

背景:我有一些类实现了主题/观察者设计模式，我已将其设为线程安全。 subject 将通过一个简单的方法调用通知它的 observers在发出通知的同一线程中构造。但是如果 observer 是在不同的线程中构建的，那么通知将被发布到 queue 中，稍后由构建 observer 然后在处理通知事件时可以进行简单的方法调用。

所以……我有一个关联线程和队列的映射，当线程和队列被构造和销毁时，它会被更新。该映射本身使用互斥锁来保护对它的多线程访问。

map 是单例的。

我过去一直对使用单例感到内疚，因为“这个应用程序中只有一个”，相信我 - 我已经付出了我的苦修!

我的一部分不禁想到应用程序中真的只有一个队列/线程映射。另一个声音说单例不好，应该避免。

我喜欢删除单例并能够为我的单元测试 stub 的想法。问题是，我很难想出一个好的替代解决方案。

过去有效的“常规”解决方案是传递一个指向要使用的对象的指针，而不是引用单例。我认为在这种情况下这会很棘手，因为在我的应用程序中观察者和主题是 10 分钱的，而且必须将队列/线程映射对象传递到每个观察者的构造函数中会非常尴尬。

我很欣赏我的应用程序中可能只有一张 map ，但它不应该在做出该决定的主题和观察者类代码的内部。

也许这是一个有效的单例，但我也很感激任何关于如何删除它的想法。

谢谢。

附言。我读过What's Alternative to Singleton和 this article在接受的答案中提到。我不禁想到 ApplicationFactory 它只是另一个名字不同的单例。我真的看不出有什么好处。

Answer 1

如果尝试摆脱单例的唯一目的是从单元测试的角度来看，也许可以用可以在 stub 中交换的东西替换单例 getter。

class QueueThreadMapBase
{
   //virtual functions
};

class QeueueThreadMap : public QueueThreadMapBase
{
   //your real implementation
};

class QeueueThreadMapTestStub : public QueueThreadMapBase
{
   //your test implementation
};

static QueueThreadMapBase* pGlobalInstance = new QeueueThreadMap;

QueueThreadMapBase* getInstance()
{
   return pGlobalInstance;
}

void setInstance(QueueThreadMapBase* pNew)
{
   pGlobalInstance = pNew
}

然后在您的测试中只需换出队列/线程映射实现。至少这会更多地暴露单例。