Solved

100 points: 1 class can't use another - #include gives link errs

Posted on 2003-11-21
4
263 Views
Last Modified: 2010-04-01
I am writing my my first ever serious Visual C++ 6 visual application. To give you a run down, i have created a default dialog program and created some extra dialogs.

I have core functional classes named Hobby, Client, DatingAgency (im sure you see where this is going)

To jump straight to the problem, if i try to use:    #include "Client.h"   i get the following error:
(this happens during linking)

WindowTestDlg.obj : error LNK2005: "public: __thiscall Client::Client(void)" (??0Client@@QAE@XZ) already defined in WindowTest.obj

This is repeated for every method of the Client object. If i don't use #include then i can't access the client class (undeclared identifier).

Now i deleted Client.cpp and wrote the declaration and the implementations in the header file, because im using #include <list> and using the STL List object.

Does anybody recognise this error? And know it's solution?

(extra info that may help: MS Visual C++ 6.0 Ent. Ed., Windows XP Pro)

(If you answer this corretly i will point you to a question that i posted for 70 points and never gave to anybody because i solved it myself. You can add a comment and i will give you those points as well.)

Thanks

Phill
0
Comment
Question by:Philluminati
  • 2
  • 2
4 Comments
 
LVL 12

Accepted Solution

by:
andrewjb earned 30 total points
ID: 9799397
We'll probably need to see some code snippets.

In general, your .h files should

a) Contain the class definitions only - no code, thanks

b) Be protected against multiple #includ-ing, typically by something like
  #ifndef Client_H
  #define Client_H

  ... rest of the header file goes here

  #endif

which, I presume, VC++ puts in for you anyway.


Your .cpp files should contain the implemnentations of the classes etc.



It sound like you've got some implemenation, or some functions written in the .h files? Is that it?

Or, I guess, you've got the imlpemneation of the Client consturctor (Client::Client) in two .cpp files, namely WindowTest and WindowTestDialog.

You don't #include a .cpp anywhere do you? A .cpp should #include it's .h, and either a .cpp or .h can #include other .h files. Nothing should #include a .cpp anywhere.



Ah ! you say:
Now i deleted Client.cpp and wrote the declaration and the implementations in the header file, because im using #include <list> and using the STL List object.


So you have implemented Client in a .h file. That's not right, unless it's a template class itself. Implement in .cpp, define in .h


0
 

Author Comment

by:Philluminati
ID: 9799589
I don't believe it!!!

I'm shocked, my answers been correctly solved in less than an hour. Thats fantastic!

Thank-you very much, problems like that are a real pain. You know it's obvious but you don't have enough experience to know what it is. Thanks mate.

My end of the bargin: I promised you 100 points so copy and paste the path into address bar, leave a small comment and ill assign the 70 points there for you. (30 from this question).

http://www.experts-exchange.com/Applications/MS_Office/Outlook/Q_20790470.html#9700755

Thanks

Phillip Taylor
0
 

Author Comment

by:Philluminati
ID: 9799624
sorry - i never told you how i solved the problem. I added client.cpp back into the project and copy and pasted the definition in to its own file. Then it worked!

That easy

Thanks

Phill
0
 
LVL 12

Expert Comment

by:andrewjb
ID: 9809531
Phil..

You ought to ask in Community Support to close that other question and get your points back. Then post a new question in _this_ group, with a topic line of 'Points for andrewjb' and a reference to _this_ question. Then I'll comment in the new one, and you can give me the points.

Someone's going to be annoyed if you accept me as an answer in some unrelated Q, and you'll get told off by the admins!
0

Featured Post

Is Your Active Directory as Secure as You Think?

More than 75% of all records are compromised because of the loss or theft of a privileged credential. Experts have been exploring Active Directory infrastructure to identify key threats and establish best practices for keeping data safe. Attend this month’s webinar to learn more.

Question has a verified solution.

If you are experiencing a similar issue, please ask a related question

Often, when implementing a feature, you won't know how certain events should be handled at the point where they occur and you'd rather defer to the user of your function or class. For example, a XML parser will extract a tag from the source code, wh…
IntroductionThis article is the second in a three part article series on the Visual Studio 2008 Debugger.  It provides tips in setting and using breakpoints. If not familiar with this debugger, you can find a basic introduction in the EE article loc…
The goal of the video will be to teach the user the difference and consequence of passing data by value vs passing data by reference in C++. An example of passing data by value as well as an example of passing data by reference will be be given. Bot…
The viewer will learn how to pass data into a function in C++. This is one step further in using functions. Instead of only printing text onto the console, the function will be able to perform calculations with argumentents given by the user.

920 members asked questions and received personalized solutions in the past 7 days.

Join the community of 500,000 technology professionals and ask your questions.

Join & Ask a Question

Need Help in Real-Time?

Connect with top rated Experts

16 Experts available now in Live!

Get 1:1 Help Now