C/C++ private array initialization in the header file

Question

I have a class called Cal and it's .cpp and .h counterpart

Headerfile has

class Cal {
    private:
        int wa[2][2];

    public:
        void do_cal();
};

.cpp file has

#include "Cal.h"
void Cal::do_cal() {
   print(wa) // where print just itterates and prints the elements in wa
}

My question is how do I initialize the array wa ? I just can't seem to get it to work.

I tried with :

int wa[2][2] = {
                {5,2},
                {7,9}
               };

in the header file but I get errors saying I cant do so as it's against iso..something.

Tried also to initialize the array wa in the constructor but that didnt work either.. What am I missing ?

Thanks

ephemient · Accepted Answer · 2009-02-06 20:46:58Z

10

If it can be static, you can initialize it in your .cpp file. Add the static keyword in the class declaration:

class Cal {
    private:
        static int wa[2][2];
    public:
        void do_cal();
};

and at file scope in the .cpp file add:

#include "Cal.h"
int Cal::wa[2][2] = { {5,2}, {7,9} };
void Cal::do_cal() {
   print(wa) // where print just itterates and prints the elements in wa
}

If you never change it, this would work well (along with making it const). You only get one that's shared with each instance of your class though.

edited Feb 6, 2009 at 20:46

ephemient

206k39 gold badges292 silver badges402 bronze badges

answered Feb 6, 2009 at 18:50

Rob K

8,9662 gold badges35 silver badges37 bronze badges

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

Comments

Perchik · Accepted Answer · 2009-02-06 16:12:49Z

7

You cannot initialize array elements in a class declaration. I recently tried to find a way to do just that. From what I learned, you have to do it in your initialize function, one element at a time.

Cal::Cal{
   wa[0][0] = 5;
   wa[0][1] = 2;
   wa[1][0] = 7;
   wa[1][1] = 9;
}

It's possible (and probable) that there's a much better way to do this, but from my research last week, this is how to do it with a multi dimensional array. I'm interested if anyone has a better method.

answered Feb 6, 2009 at 16:12

Perchik

5,6501 gold badge21 silver badges22 bronze badges

Comments

Community · Accepted Answer · 2017-05-23 12:06:33Z

5

You can't do it easily. If you don't want to specify each element individually like in Perchik's answer, you can create one static array and memcpy that (which will probably be faster for non-trivial array sizes):

namespace
{
    const int default_wa[2][2] = {{5, 2}, {7, 9}};
}

Cal::Cal
{
    memcpy(&wa[0][0], &default_wa[0][0], sizeof(wa));
}

edited May 23, 2017 at 12:06

CommunityBot

11 silver badge

answered Feb 6, 2009 at 16:17

Adam Rosenfield

402k102 gold badges524 silver badges600 bronze badges

5 Comments

Mark Ransom Over a year ago

memcpy is dangerous here. If someone in the future changes the dimensions or type of one array without reflecting those changes in the other...

Adam Rosenfield Over a year ago

True, but you'd have the same problem with any other solution. You could easily throw in an assert(sizeof(wa) == sizeof(default_wa)) here to protect against that in this case.

Christoph Over a year ago

You can make sure that the array sizes always match if you define them like this: int default_wa[][2] = {{ 5, 2 }, { 7, 9 }}; int wa[count(default_wa)][count(default_wa[0])]; with #define count(ARRAY) ((sizeof (ARRAY))/(sizeof (ARRAY[0])))

Mark Ransom Over a year ago

Good point Christoph, I've done that before. I hate #defining count() every time I need it - should have been a built-in feature.

Mark Ransom Over a year ago

Should also point out that you can use a static within the constructor, rather than putting the defaults in a namespace outside. But only if you don't need an inline constructor. It enhances the readability.

Collectives™ on Stack Overflow

C/C++ private array initialization in the header file

3 Answers 3

Comments

Comments

5 Comments

Your Answer

Linked

Hot Network Questions

Collectives™ on Stack Overflow

3 Answers 3

Comments

Comments

5 Comments

Your Answer

Sign up or log in

Post as a guest

Linked

Related