initialization error- obsolete C++ constructor initialization syntax

Question

4.00/5 (1 vote)

See more:

The error message says I need to initialize ARRAYSIZE in constructor base/member initializer list. I have. Not sure why I am getting the error. Any help would be appreciated. Thanks.

C++

class CRsSortPPage : public CPropertyPage{

public:
    const unsigned short ARRAYSIZE;


}


CRsSortPPage::CRsSortPPage()
    : CPropertyPage(CRsSortPPage::IDD),
    ARRAYSIZE(5)
{
}

Error Message

 error C2351: obsolete C++ constructor initialization syntax
 error C2758: 'CRsSortPPage::ARRAYSIZE' : must be initialized in constructor base/member initializer list
 see declaration of 'CRsSortPPage::ARRAYSIZE'

Posted 17-Feb-11 8:36am

23_444

Updated 17-Feb-11 8:48am

fjdiewornncalwe

v2

Add a Solution

1 solution

Add a Solution

Add your solution here

Treat my content as plain text, not as HTML

Preview 0

…

Existing Members

Sign in to your account

...or Join us

Download, Vote, Comment, Publish.

Your Email
Password
Forgot your password?

Your Email
This email is in use. Do you need your password?
Optional Password

I have read and agree to the Terms of Service and Privacy Policy
Please subscribe me to the CodeProject newsletters

When answering a question please:

Read the question carefully.
Understand that English isn't everyone's first language so be lenient of bad spelling and grammar.
If a question is poorly phrased then either ask for clarification, ignore it, or edit the question and fix the problem. Insults are not welcome.
Don't tell someone to read the manual. Chances are they have and don't get it. Provide an answer or move on to the next question.

Let's work to help developers, not make them feel stupid.

This content, along with any associated source code and files, is licensed under The Code Project Open License (CPOL)

Nish Nishant · Accepted Answer · 2011-02-17T09:00:00

Solution 1

ARRAYSIZE is a macro. That's why you get the error. The following code will give the same error:

C++

class Test
{
public:
    Test()  : ARRAYSIZE(0)
    {
    }

    short ARRAYSIZE;
};

You can fix this by either renaming ARRAYSIZE to something else, maybe ARRSIZE?

Or doing this before the class definition:

C++

#undef ARRAYSIZE

[Update]
~~~~~~~~~

In response to your comment, here's how ARRAYSIZE is used.

C++

int test[100];
cout << ARRAYSIZE(test) << endl;

That will print 100.

Posted 17-Feb-11 9:00am

Nish Nishant

Updated 17-Feb-11 9:09am

v3

Comments

23_444 17-Feb-11 15:05pm

It worked. Thank you. Not sure how ARRAYSIZE is a macro? I just made up the name as the name of the constant and can't find it anywhere in Help. Please tell me more. Thanks again.

Nish Nishant 17-Feb-11 15:09pm

Updated my answer with more info.

23_444 17-Feb-11 15:11pm

Thanks, I would have been stuck on that for some time!

Nish Nishant 17-Feb-11 15:12pm

No problem, you are welcome!

Sergey Alexandrovich Kryukov 17-Feb-11 15:17pm

Easy enough? My 5,
--SA

Nish Nishant 17-Feb-11 15:20pm

Thank you, SA!

Espen Harlinn 17-Feb-11 18:32pm

Nicely spotted, my 5 :)

Nish Nishant 17-Feb-11 19:22pm

Thanks Espen.

Albert Holguin 17-Feb-11 22:47pm

typically names with all capitals are used for macros/definitions, to avoid this in the future, don't use all capital names for variables... its probably not a good idea to #undef a macro if you didn't define it in the first place

Albert Holguin 17-Feb-11 22:48pm

...and good catch for nishant...