[C++] Header en cpp files

[BlackHawkDesign]

Goeiemiddag heren,

Ik ben ooit al eens eerder begonnen met C++ en heb toen ook wat in elkaar gedraaid. Alleen door de jaren heen is die kennis weer vergaan. Nu wil ik toch weer opnieuw c++ gaan oppakken. Nu lees ik overal dat ik me interface van me class in de header file moet stoppen en de implementatie in de cpp file.

Alleen ik krijg steeds build errors. Ik heb overal gezocht en ik kom steeds de code tegen die exact hetzelfde is als mijn code. Wat doe ik dan verkeerd? Zouden jullie mij op weg willen helpen?

main.cpp

C++:

#include "stdafx.h";
#include <iostream>;
#include "MouseHandler.cpp";

using namespace std;

int _tmain(int argc, _TCHAR* argv[])
{

    MouseHandler mouseHandler;
    mouseHandler.PreformClick();

    cin.get();

    return 0;
}

MouseHandler.h

C++:

class MouseHandler {
    public:
        void SetLocation(double x, double y);
        void PreformClick();
};

MouseHandler.cpp

C++:

#include "MouseHandler.h";
#include "stdafx.h";
#include <iostream>;

using namespace std;

void MouseHandler::SetLocation(double x, double y){
    cout<<"SetLocation is preformed";
}

void MouseHandler::PreformClick(){
    cout<<"click is preformed";
}

De errors die ik krijg zijn dat MouseHandler in de mouseHandler.cpp een class of een namespace hoort te zijn, maar ik heb dat *** al gedefinieerd als een class.

Wat doe ik nou verkeerd?

[Woy]

Je moet niet MouseHandler.cpp includen maar MouseHandler.h.

De compiler heeft genoeg aan alleen de interface definitie, en de linker zal later de implementatie er bij zoeken.

Aangezien je nu alleen MouseHandler.cpp include, zal de compiler op dat moment niet weten wat de interface is van MouseHandler, en zal de implementatie dus voor de compiler ook nergens op slaan.

[ Voor 35% gewijzigd door Woy op 26-10-2010 13:58 ]

[NMe]

Woy schreef op dinsdag 26 oktober 2010 @ 13:57:
Je moet niet MouseHandler.cpp includen maar MouseHandler.h.

Mooie circular include ook als dit zou werken. Ah wacht, ik kijk verkeerd.

offtopic:
"Perform" spel je niet zoals jij het tot drie keer toe doet.

[ Voor 5% gewijzigd door NMe op 26-10-2010 13:59 ]

[Woy]

overigens moet je in je header file ook een Include Guard opnemen, anders is de kans groot dat je binnen de kortste keren foutmeldingen krijgt over dubbele definitie van je class

[BlackHawkDesign]

Ja van die include guard was ik me bewust, wilde die gaan toevoegen zodra dit werkte Evengoed bedankt voor de tip

Oke aangepast, maar de error blijft:

Om exact te zijn:
error C2653: 'MouseHandler' : is not a class or namespace name mousehandler.cpp

[ Voor 6% gewijzigd door BlackHawkDesign op 26-10-2010 14:18 ]

[BlackHawkDesign]

En opgelost.

Deze include #include "stdafx.h";

Moet bovenaan staan, de includes die hierboven staan, worden overgeslagen.

Wat een vage bug.

[Woy]

BlackHawkDesign schreef op dinsdag 26 oktober 2010 @ 14:30:
Wat een vage bug.

Niet echt een bug, maar expected behavior
Wikipedia: Precompiled header

[CodeCaster]

BlackHawkDesign schreef op dinsdag 26 oktober 2010 @ 14:30:
En opgelost.

Deze include #include "stdafx.h";

Moet bovenaan staan, de includes die hierboven staan, worden overgeslagen.

Wat een vage bug.

quote: http://en.wikipedia.org/wiki/Precompiled_header#stdafx.h

Visual C++ will not compile anything before the #include "stdafx.h" in the source file, unless the compile option /Yu'stdafx.h' is unchecked (by default); it assumes all code in the source up to and including that line is already compiled.

Met deze code:

C++:

#include "MouseHandler.h";
#include "stdafx.h"; 

Gaat de compiler er dus al van uit dat MouseHandler.h geprecompileerd is. Trouwens ook de tweede hit op Google.

Woy!

[ Voor 14% gewijzigd door CodeCaster op 26-10-2010 14:37 ]

[BlackHawkDesign]

True, ik had beter moeten zoeken. Maja bij een nieuwe taal is het altijd weer je weg vinden. Het is al moeilijk genoeg om de neiging te weerhouden om de nieuwe taal uit te schelden en weer terug te gaan naar een van je vertrouwde talen