【发布时间】:2023-03-12 06:23:01
【问题描述】:
我确实看到了一个(对我而言)奇怪的访问冲突异常。我会尽量减少问题。我有一个 A 类和一个单例对象 sing_。代码看起来像这样:
class A {
A();
Sing& sing_;
}
A::A() : sing_(Sing::instance()){
call a method that creates a local copy of Singleton Sing.
....
}
类 Sing 继承自 Singleton:
class Sing : public Singleton<Sing>{
friend class Singleton<Sing>;
....
}
Singleton 本身看起来是这样的(这是 QuantLib 库中的实现)
template <class T>
class Singleton : private boost::noncopyable {
public:
static T& instance();
protected:
Singleton() {}
};
template <class T>
T& Singleton<T>::instance() {
static std::map<Integer, boost::shared_ptr<T> > instances_;
#if defined(QL_ENABLE_SESSIONS)
Integer id = sessionId();
#else
Integer id = 0;
#endif
boost::shared_ptr<T>& instance = instances_[id];
if (!instance)
instance = boost::shared_ptr<T>(new T);
return *instance;
}
我的项目代码嵌入在 Qt Gui 环境中。在调试模式下启动它不会引起任何麻烦。当我尝试以发布模式启动时,情况发生了可怕的变化。这是主要方法:
int main(int argc, char *argv[])
{
QApplication a(argc, argv);
GUI w;
w.show();
w.setup(argc, argv);
return a.exec();
}
最后,类 GUI 看起来像这样缩写:
class GUI : public QMainWindow
{
Q_OBJECT
public:
GUI(QWidget *parent = 0, Qt::WFlags flags = 0);
~GUI();
private:
boost::shared_ptr<A> a_;
};
当我在发布模式下启动此代码时,会发生以下情况:
调用了名为 ___tmainCRTstartup() 的方法。其中调用了方法_WinMain。
在这个方法中 WinMain(甚至在调用 main 方法和 GUI 对象之前) 已创建)调用 A 的构造函数。这意味着成员 sing 将被初始化。
sing_ 在调用 Sing::instance() 期间被初始化。到目前为止一切看起来都很好。
-
A 的构造函数被执行。其中创建了对单例对象 Sing 的本地引用。调用 Sing::instance() 会导致该行的访问冲突
boost::shared_ptr<T>& instance = instances_[id];
当我查看那个地方的instances_[id] 时(在发布模式下调试),地图看起来已经完全被破坏了。这意味着地图中有一个元素。但是key不是0,而是一个非常负的整数,而且值看起来很奇怪。
我完全不知道这里出了什么问题。
将 A 中的成员 sing_ 更改为静态成员可以解决问题:
class A {
A();
static Sing& sing_;
}
Sing& sing_ = Sing::instance();
A::A() {
call a method that creates a local copy of Singleton Sing.
....
}
这当然很好,但我真的很想知道这两种实现之间的“大”区别是什么。为什么第一种方式以访问冲突结束?任何提示表示赞赏。
【问题讨论】:
-
单身人士并不是真正的邪恶,他们有自己的用途。
-
sessionId肯定会返回一些垃圾? PS:迄今为止我见过的对单例反模式最严重的滥用。 -
它在调试而不是发布中工作表明您没有初始化变量。这反过来又导致代码中其他地方的未定义行为。您确实需要正确解决此问题,目前您刚刚隐藏了它,它会回来咬你。调高你的编译器警告级别;并修复所有警告(最好将所有警告视为错误)。
-
不,没有定义 QL_ENABLE_SESSIONS。所以这部分代码应该是无关紧要的。
-
@NicholasSmith Singletons 可以总是被更好的东西取代。
标签: c++ singleton release access-violation