Correct place to initialize class variables?

What you have there is an instance variable. Each instance of the class gets its own copy of myInt. The place to initialize those is in a constructor:

class Foo {
private:
    int myInt;
public:
    Foo() : myInt(1) {}
};

A class variable is one where there is only one copy that is shared by every instance of the class. Those can be initialized as you tried. (See JaredPar's answer for the syntax)

For integral values, you also have the option of initializing a static const right in the class definition:

class Foo {
private:
    static const int myInt = 1;
};

This is a single value shared by all instances of the class that cannot be changed.


To extend on Jared's answer, if you want to initialize it the way it is now, you need to put it in the Constructor.

class Foo
{
public:
    Foo(void) :
    myInt(1) // directly construct myInt with 1.
    {
    }

    // works but not preferred:
    /*
    Foo(void)
    {
        myInt = 1; // not preferred because myInt is default constructed then assigned
                   // but with POD types this makes little difference. for consistency
                   // however, it's best to put it in the initializer list, as above
                   // Edit, from comment: Also, for const variables and references,
                   // they must be directly constructed with a valid value, so they
                   // must be put in the initializer list.
    }
    */

private:
    int myInt;
};

Tags:

C++