views:

1262

answers:

3

Greetings everyone.

I seem to be snagging on a fundimental but I cant find the solution anywhere. Anywho, will go ahead and explain.

I have a program consisting of three files; main.ccp, add.h, add.cpp.

I declare the class 'SA' in add.h and have all my functions defined in add.cpp

additional.h

class SA {
    ...
public
    int x;
} Obj1, Obj2;

main.ccp

#include "additional.h" 

int main() {

    Obj1.x = 5;

    ...
}

This gives me a link error on compiling: error LNK2005: "class SA Obj1" (?Obj1@@3VSA@@A) already defined in main.obj

The only deffinition of the object occurs in add.h, and no where else. The program compiles just fine if declare the objects in the main and not the header:

main.ccp

#include "additional.h" 

int main() {

    SA Obj1;
    Obj1.x = 5;

    ...
}

The issue is that I want to use the objects primarially within add.cpp, but still need to initialise several public values through main.cpp. Any words of wisdom?

+4  A: 

Use extern keyword. Declare these public objects as extern in header, then define instances in one of the cpps.

Like this:

extern SA Obj1; // in header

SA Obj1;// in any one (no more than one) cpp
sharptooth
+9  A: 

Declare Obj1 and Obj2 in your .cpp instead of at .h

add.h

class SA {
 ...
public
    int x;
};

main.cpp

#include "additional.h" 

SA Obj1, Obj2;

int main() {

 Obj1.x = 5;

 ...
}

If you want to declare Obj1 and Obj2 in your .h file, add extern in the .h file like so:

extern SA Obj1, Obj2;

but you should declare the objects in a .cpp file in your project:

main.cpp

SA Obj1, Obj2;

The reason for this is that everytime you include the .h file, you are declaring Obj1 and Obj2. So if you include the .h file two times, you will create two instance of Obj1 and Obj2. By adding the keyword extern, you are telling the compiler that you have already decalred the two variables somewhere in your project (preferably, in a .cpp file).

MrValdez
perfect. Thank you!
Raugnar
A: 

You can also have the following line of code in your .h file:

static SA Obj1, Obj2;

this will create only 1 instance of each object at run time and prevent redefinitions. However, keep in mind that all files that include the .h file will be using the same instance of those two objects so you will need to be careful when you are accessing them.

Ben