任何人都可以在c ++中为我提供Singleton样本吗？

Question

我按照以下方式编写单例c ++:

class A {
    private:
        static A* m_pA;
        A();
        virtual ~A();

    public:
        static A* GetInstance();
        static void FreeInstance();

        void WORK1();
        void WORK2();
        void WORK3();
    }
}

A* A::GetInstance() {
    if (m_pA == NULL)
        m_pA = new A();
    return m_pA;
}

A::~A() {
    FreeInstance()  // Can I write this? are there any potential error?
}

void A::FreeInstance() {
    delete m_pA;
    m_pA = NULL;
}

谢谢!Evan Teran和sep61.myopenid.com的回答是正确的,非常好!我的方式是错的,我希望任何人写这样的代码都可以避免我的愚蠢错误.

我的项目中的单例A有一个智能指针向量,另一个线程也可以编辑这个向量,所以当应用程序关闭时,即使我添加了很多CMutex,它总是变得不稳定.多线程错误+单身错误浪费了我一天.

// ------------------------------------------------ -----------新单例,如果您认为以下示例中存在任何问题,欢迎您进行编辑:

class A {
    private:
        static A* m_pA;
        explicit A();
        void A(const A& a);
        void A(A &a);
        const A& operator=(const A& a);
        virtual ~A();

    public:
        static A* GetInstance();
        static void FreeInstance();

        void WORK1();
        void WORK2();
        void WORK3();
    }
}

A* A::GetInstance() {
    if (m_pA == NULL){
        static A self;
        m_pA = &self;
    }
    return m_pA;
}

A::~A() {
}

Answer 1

为什么每个人都希望将单例作为指针返回？
返回它作为参考似乎更合乎逻辑!

你永远不能手动释放单身人士.你怎么知道谁在提到单身人士？如果您不知道(或不能保证)没有人有引用(在您的情况下通过指针),那么您没有业务释放该对象.

在函数方法中使用static.
这保证了它只被创建和销毁一次.它还免费为您提供延迟初始化.

class S
{
    public:
        static S& getInstance()
        {
            static S    instance;
            return instance;
        }
    private:
        S() {}
        S(S const&);              // Don't Implement.
        void operator=(S const&); // Don't implement
 };

请注意,您还需要将构造函数设为私有.还要确保覆盖默认的复制构造函数和赋值运算符,这样就不能复制单例(否则它不会是单例).

另请阅读:

• /sf/answers/70580261/
• Singleton:如何使用它
• C++ Singleton设计模式

确保您使用单身人员是出于正确的原因.

虽然在一般情况下技术上不是线程安全的,但请参阅:
C++函数中静态变量的生命周期是多少？

海湾合作委员会有一个明确的补丁来弥补这一点:http:
//gcc.gnu.org/ml/gcc-patches/2004-09/msg00265.html

Answer 2

您可以通过使用这样的静态对象来避免需要删除它:

if(m_pA == 0) {
    static A static_instance;
    m_pA = &static_instance;
}