一个static变量出现两个实例,单例模式出现了两个实例 5C

主程序会依赖一个dll库,单例在dll库中实现,单例的get接口使用dllexport修饰,以便在主程序中获得。

现在发现从主程序第一次调用get和dll库中第一次调用get都会新创建实例。
而后在主程序和dll库中get单例返回的分别是他们第一次创建的实例。代码如下:

dll:test.h
#ifdef EXPORTS
#define dllAPI __declspec(dllexport)
#else
#define dllAPI __declspec(dllimport)
#endif
namespace test:
{
    class testclass
    {
    public:
        dllAPI static int* getinstance();
    private:
        static int* instance;
    }
}
dll:test.cpp
#include test.h
namespace test:
{
    int* testclass::instance;
    int* testclass::getinstance()
    {
            if(!instance)
                instance = new int(0);
            return instance;
    }

    void testclass::function()
    {
        ...
        int* g_instance = getinstance();
        ...
    }
}

main:
#include test.h
int main()
{
    ...
    int* g_instance = test::testclass::getinstance();
    ...
}

不知道这个写法是不是有问题?还是有什么别的需要注意的?

5个回答

你创建单例判断指针等没意见加锁。以及双层空指针判断。

你有用到多线程么?如果有,需要对
if(!instance)
instance = new int(0);
这里做同步。

另外,dll是每个进程实例一份的,不可以用这个代码实现跨进程的单例,如果你要所有进程单例,需要用互斥量来实现。

xiaoxiangyue_lib
xiaoxiangyue_lib 可以确定是一个进程,所以dll应该只有一份。代码里的确是有多线程,但是可以确定不会发生线程竞争。我加了同步以后依然有上述两个实例的问题
2 年多之前 回复
xiaoxiangyue_lib
xiaoxiangyue_lib 谢谢!
2 年多之前 回复

consolog 没有清屏 你试试

xiaoxiangyue_lib
xiaoxiangyue_lib 能不能进一步解释一下?
2 年多之前 回复
xiaoxiangyue_lib
xiaoxiangyue_lib 什么意思?
2 年多之前 回复

你的instance这个没有初始化啊。

单例模式需要优化,double check能解决出现其他实例的问题
标准线程安全的单例代码演示

xiaoxiangyue_lib
xiaoxiangyue_lib 谢谢,我添加了double check之后还是有上述问题。这样看来这个问题似乎不是多线程问题
2 年多之前 回复
Csdn user default icon
上传中...
上传图片
插入图片
抄袭、复制答案,以达到刷声望分或其他目的的行为,在CSDN问答是严格禁止的,一经发现立刻封号。是时候展现真正的技术了!
立即提问
相关内容推荐