Initialize an array in C++ header file

Question

I'm trying to initialize an array in a class' header file:

class C {
    private:
        static float p[26] = {
            0.09, 0.02, 0.02, 0.04, 0.12, 0.02, 0.03, 
            0.02, 0.09, 0.01, 0.01, 0.04, 0.02, 0.06, 
            0.08, 0.02, 0.01, 0.06, 0.04, 0.06, 0.04, 
            0.02, 0.02, 0.01, 0.02, 0.01
        };
...

I'm getting the following error(s) from g++:

C.h:15:33: error: a brace-enclosed initializer is not allowed here before ‘{’ token

C.h:20:9: sorry, unimplemented: non-static data member initializers

C.h:20:9: error: ‘constexpr’ needed for in-class initialization of static data member ‘p’ of non-integral type

I'm forced to use c++0x; how can I define this array without doing p[0], p[1], etc.?

Make it constexpr or initialize it outside the class declaration. — user657267
– user657267, Commented Jan 30, 2015 at 2:43
@user657267 when I initialize it in the class constructor I get: "error: assigning to an array from an initializer list" — sheppardzw
– sheppardzw, Commented Jan 30, 2015 at 2:55
outside the class declaration. doesn't mean inside a constructor, please search on how to initialize static members. — user657267
– user657267, Commented Jan 30, 2015 at 2:56

R Sahu · Accepted Answer · 2015-01-30 03:01:26Z

3

You can declare the static member variable of the class in a header file using:

class C {
   private:
      static float p[26];
};

You can define it in a .cpp file using:

float C::p[26] = {
            0.09, 0.02, 0.02, 0.04, 0.12, 0.02, 0.03, 
            0.02, 0.09, 0.01, 0.01, 0.04, 0.02, 0.06, 
            0.08, 0.02, 0.01, 0.06, 0.04, 0.06, 0.04, 
            0.02, 0.02, 0.01, 0.02, 0.01
        };

answered Jan 30, 2015 at 3:01

R Sahu

207k15 gold badges168 silver badges291 bronze badges

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

Comments

Community · Accepted Answer · 2017-05-23 11:49:28Z

2

Unfortunately, you cannot initialize your static members in the class declaration (exceptions are made for const integral and const enum types).

You should define (and initialize) your static member in one separate compilation unit (usually it's done in the *.cpp file corresponding to your class).

So, you'll have:

// code in C.h
class C {
static float p[]; // declaration of C::p
}

and

// code in C.cpp
float C::p[] = { /* put your numbers here */ }; // definition of C::p

If the terms "declaration" and "definition" are confusing to you, read the answers to this question.

edited May 23, 2017 at 11:49

CommunityBot

11 silver badge

answered Jan 30, 2015 at 2:55

nicebyte

1,5781 gold badge11 silver badges22 bronze badges

Collectives™ on Stack Overflow

Initialize an array in C++ header file

2 Answers 2

Comments

Comments

Your Answer

Linked

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Linked

Related