Initializing a private static member variable in the cpp file. error: member is private

Question

Here is a simple example illustrating my problem.

IDGenerator.hpp

#ifndef IDGENERATOR_HPP_
#define IDGENERATOR_HPP_

class IDGenerator {

   private:
      static int s_nNextID;

      // make the IDGenerator class a static class with a private default constructor
      IDGenerator();

   public:
      static int GetNextID();

};

#endif

IDGenerator.cpp

#include "IDGenerator.hpp"

// initialize the static id
int IDGenerator::s_nNextID(1);

int GetNextID()
{
   return IDGenerator::s_nNextID++;
}

I have tried initializing both explicitly (int IDGenerator::s_nNextID = 1;) and implicitly as shown. Here is the compile command and the error

g++ -c IDGenerator.cpp
IDGenerator.cpp: In function ‘int GetNextID()’:
IDGenerator.cpp:11:5: error: ‘int IDGenerator::s_nNextID’ is private
 int IDGenerator::s_nNextID(1);
     ^
IDGenerator.cpp:15:22: error: within this context
  return IDGenerator::s_nNextID++;

I have also tried to compile with -Wall and std=gnu++11. Same error

@user3528438 It's being defined and initialised there, absolutely correctly. It's a static member. — Angew is no longer proud of SO
– Angew is no longer proud of SO, Commented Mar 14, 2015 at 17:09

Sebastian Redl · Accepted Answer · 2015-03-14 16:54:10Z

3

The error isn't about the initialization. It just points to the initialization as the point where s_nNextID comes from.

The real error is on line 15, where you access s_nNextID from a normal global function, because you forgot the IDGenerator:: in the definition head of GetNextID.

answered Mar 14, 2015 at 16:54

Sebastian Redl

73.9k8 gold badges137 silver badges177 bronze badges

Sign up to request clarification or add additional context in comments.

Comments

Gilead Silvanas · Accepted Answer · 2015-03-14 16:58:37Z

0

s_nNextID is private so you can't just access it like this. You need to make GetNextID a member of the class IDGenerator.

answered Mar 14, 2015 at 16:58

Gilead Silvanas

976 bronze badges

Collectives™ on Stack Overflow

Initializing a private static member variable in the cpp file. error: member is private

2 Answers 2

Comments

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Related