119

Hi i am getting undefined reference error in the following code:

class Helloworld{
  public:
     static int x;
     void foo();
};
void Helloworld::foo(){
     Helloworld::x = 10;
};

I don't want a static foo() function. How can I access static variable of a class in non-static method of a class?

BaCaRoZzo
  • 7,502
  • 6
  • 51
  • 82
Aqeel Raza
  • 1,729
  • 4
  • 15
  • 24
  • Possible duplicate of [Undefined reference to static class member](https://stackoverflow.com/questions/272900/undefined-reference-to-static-class-member) – underscore_d Apr 10 '18 at 16:06

3 Answers3

150

I don't want a static foo() function

Well, foo() is not static in your class, and you do not need to make it static in order to access static variables of your class.

What you need to do is simply to provide a definition for your static member variable:

class Helloworld {
  public:
     static int x;
     void foo();
};

int Helloworld::x = 0; // Or whatever is the most appropriate value
                       // for initializing x. Notice, that the
                       // initializer is not required: if absent,
                       // x will be zero-initialized.

void Helloworld::foo() {
     Helloworld::x = 10;
};
Andy Prowl
  • 124,023
  • 23
  • 387
  • 451
77

The code is correct, but incomplete. The class Helloworld has a declaration of its static data member x, but there is no definition of that data member. Somehwere in your source code you need

int Helloworld::x;

or, if 0 isn't an appropriate initial value, add an initializer.

Pete Becker
  • 74,985
  • 8
  • 76
  • 165
51

Old question, but;

Since c++17 you can declare static members inline and instantiate them inside the body of class without the need of an out-of-class definition:

class Helloworld{
  public:
     inline static int x = 10;
     void foo();
};
Eligum
  • 27
  • 7
pvc
  • 1,070
  • 1
  • 9
  • 14
  • This also makes it significantly easier to initialize a private static variable (instance pointer) for a singleton class. – Connell Hagen Mar 09 '23 at 20:33