Static const double in c++

Question

Is this the proper way to use a static const variable? In my top level class (Shape)

#ifndef SHAPE_H
#define SHAPE_H

class Shape
{
public:

    static cons         


        

Answer 1

Because const double Shape::pi = 3.14159265; is the definition of Shape::pi and C++ only allows a single definition of a symbol (called the one-definition-rule which you may see in it's acronym form ODR). When the definition is in the header file, each translation unit gets it's own definition which breaks that rule.

By moving it into the source file, you get only a single definition.

Answer 2

If you have a way to add C++0x flag to your compiler, you would've been able to do:

ifndef SHAPE_H
#define SHAPE_H

class Shape
{
public:

    static constexpr double pi = 3.14159265;
private:
    double originX;
    double originY;
};

#endif

In C++0x you are able to use const expressions to types other than integral ones. This enables you to declare and define in place your constant variable.

Answer 3

For primitive data types (like int, double but not char[]) you may also define the constant within the class definition within the header file, e.g.:

class Shape
{
public:
    static const double pi = 3.14159265;

private:
    double originX;
    double originY;
};

This will allow better compiler optimisation.

Edit: As Dennis pointed out below this is only allowed for integral types and not for double or float data types (however some compilers will allow it).

Answer 4

Static floating-point data members must be defined and initialized in a source file. The one-definition rule forbids a definition outside the class {} block in the header, and only integral data members are allowed to be initialized inside the class {} block.

This is also unfortunate because, being an algebraic value, having the immediate value on hand could be nice for optimization, rather than loading from a global variable. (The difference is likely to be inconsequential, though.)

There is a solution, though!

class Shape
{
public:
    static double pi()
        { return 3.14159265; }

private:
    double originX;
    double originY;
};

Inline function definitions, including static ones, are allowed inside the class{} block.

Also, I recommend using M_PI from <math.h>, which you should also get from <cmath>.

Answer 5

The line const double Shape::pi = 3.14159265; should be in your Shape.cpp file. The header file is for declaring the variables. You can only define a variable once, therefore it must be done in the .cpp. The header file says how to use these variables and functions, the cpp file says what to do.

Answer 6

It happens because you can't define Shape::pi more than once. It's defined once when you include Shape.h in Shape.cpp, and then again every other time you use Shape.h in another cpp file. When you go to link you program together the linker will barf because of multiple definitions.