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

Posted on 2003-11-21
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.)


Question by:Philluminati
Welcome to Experts Exchange

Add your voice to the tech community where 5M+ people just like you are talking about what matters.

  • Help others & share knowledge
  • Earn cash & points
  • Learn & ask questions
  • 2
  • 2
LVL 12

Accepted Solution

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


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


Author Comment

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).


Phillip Taylor

Author Comment

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


LVL 12

Expert Comment

ID: 9809531

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!

Featured Post

Free Tool: Port Scanner

Check which ports are open to the outside world. Helps make sure that your firewall rules are working as intended.

One of a set of tools we are providing to everyone as a way of saying thank you for being a part of the community.

Question has a verified solution.

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

Written by John Humphreys C++ Threading and the POSIX Library This article will cover the basic information that you need to know in order to make use of the POSIX threading library available for C and C++ on UNIX and most Linux systems.   [s…
This article will show you some of the more useful Standard Template Library (STL) algorithms through the use of working examples.  You will learn about how these algorithms fit into the STL architecture, how they work with STL containers, and why t…
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 be introduced to the member functions push_back and pop_back of the vector class. The video will teach the difference between the two as well as how to use each one along with its functionality.

627 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