Unresolved external symbol linkage error for a singleton object

Question

I'm trying to understand the meaning of the error I get.

This is a singleton implementation:

class Singleton
{
    private:
        Singleton():m_value(0){};
        static Singleton * m_instance;
        int m_value;

    public:
        static Singleton * GetInstance()
            {
                if(!m_instance)
                {
                    m_instance = new Singleton;
                }

                return m_instance;
            }
        void SetValue(int x){m_value = x;}
        int GetValue(){return m_value;}

        ~Singleton()
        {
            if(m_instance)
                delete m_instance;
        }
    };



Singleton* Singleton::m_instance = 0;


void main()
{
    Singleton * s1 = Singleton::GetInstance();
}

The code compiles and run successfully.

When I remove the line Singleton* Singleton::m_instance = 0;, I get the error:

 error LNK2001: unresolved external symbol "private: static class Singleton * Singleton::m_instance"

I guess the meaning of that line is to set the static variable m_instance to 0.

So I don't understand the syntax of that line- why can't I write just Singleton::m_instance = 0;? and also why do I get linkage error when removing that line?

The better idiomatic way: http://stackoverflow.com/questions/1008019/c-singleton-design-pattern — πάντα ῥεῖ, Jan 31 '17 at 09:21

Soeren · Answer 1 · 2017-01-31T09:18:11.250

You have to initialize the static variable:

Singleton* Singleton::m_instance = 0;

We can only call static class members on the class and not on an object of the class. And that is possible, even if no instance exist. That's why every static member instance must be initialize, usually in the cpp file.

And because of the static variable is initialized outside the class scope, we have to call the variable by the full name (e.g. Singleton::m_instance).

songyuanyao · Answer 2 · 2017-01-31T09:14:14.450

0

So I dont understand the syntax of that line- why can't I write just Singleton::m_instance = 0;?

The syntax here is the definition of the static member, which need to be defined out of the class definition. That's why if you remove it you'll get the undefined link error.

class X { static int n; }; // declaration (uses 'static')
int X::n = 1;              // definition (does not use 'static')

edited Jan 31 '17 at 09:14

answered Jan 31 '17 at 09:08

songyuanyao

169,198
16
310
405

score 0 · Answer 3 · answered Jan 31 '17 at 09:24

This is specified in C++ standard

9.4.2 Static data members

2 The declaration of a static data member in its class definition is not a definition and may be of an incomplete type other than cv-qualified void. The definition for a static data member shall appear in a namespace scope enclosing the member’s class definition. In the definition at namespace scope, the name of the static data member shall be qualified by its class name using the :: operator. The initializer expression in the definition of a static data member is in the scope of its class

So the declaration in the class scope is not considered a definition, that is why you get the linker error.

Unresolved external symbol linkage error for a singleton object

3 Answers3