Podział projektu na pliki a programowanie obiektowe

Cześć, próbuję zrobić mój pierwszy projekt używając programowania obiektowego. Dlaczego w menu.cpp mogę zdefiniować funkcję "adam" z pliku gra.cpp, a nie mogę w pliku menu.cpp stworzyć obiektu "gracz" klasy "adam" z pliku gra.cpp?

Funkcja:

//main.cpp
#include "menu.hpp"

int main()
{
    menu();
    return 0;
}

//menu.hpp
#ifndef MENU_HPP_INCLUDED
#define MENU_HPP_INCLUDED
#include "gra.hpp"

#include <iostream>
using namespace std;
void menu();

#endif // MENU_HPP_INCLUDED

// menu.cpp

#include "menu.hpp"
void menu(){
    adam(5);
}

//gra.hpp
#ifndef GRA_HPP_INCLUDED
#define GRA_HPP_INCLUDED

#include <iostream>
using namespace std;
void adam(short pkt);

#endif // GRA_HPP_INCLUDED

//gra.cpp
#include "gra.hpp"
void adam(short pkt){
    cout << "Adam ma " << pkt << " pkt";
}

Klasa:

//main.cpp
#include "menu.hpp"

int main()
{
    menu();
    return 0;
}

//menu.hpp
#ifndef MENU_HPP_INCLUDED
#define MENU_HPP_INCLUDED
#include "gra.hpp"

#include <iostream>
using namespace std;
void menu();

#endif // MENU_HPP_INCLUDED

// menu.cpp

#include "menu.hpp"
void menu(){
    adam gracz(5);
    cout << "Adam ma " << gracz.ilepkt();
}

//gra.hpp
#ifndef GRA_HPP_INCLUDED
#define GRA_HPP_INCLUDED

#include <iostream>
using namespace std;
class adam;

#endif // GRA_HPP_INCLUDED

//gra.cpp
#include "gra.hpp"

class adam{
private:
    short pkt;
public:
    adam(short p){
    pkt = p;
    }

    short ilepkt(){
        return pkt;
    }
};

Zwracany błąd:

E:\Moje Dokumenty\Programowanie\c++\gra\menu.cpp|5|error: variable 'adam gracz' has initializer but incomplete type|

Przedstawię ci inny model podziału na pliki. Jest o niebo lepszy, wszystko się ładnie łączy, standardowe nagłówki są użyte tylko raz i koniec.

Ok. Załóżmy że mamy plik main.cpp.
Następnie tworzymy plik Includes.h/hpp, oraz tworzymy plik nagłówkowy i plik źródłowy dla klasy.
Niech to będzie klasa CSamochod.

I teraz robimy tak: W includes.h zawieramy biblioteki standardowe oraz plik nagłówkowy klasy.
W kodzie źródłowym klasy zawieramy plik includes.h, w mainie zawieramy includes.h, a w pliku nagłówkowym klasy nie zawieramy niczego. Kompilator sam poskłada pliki tak, że w Includes znajdą się wszystkie definicje klas + na początku będą domyślne nagłówki, przez co daje to swobodny dostęp.

No i przykład:
Main.cpp:

#include "Includes.h"

int main(int, char**)
{
	CSamochod * pSamochod = new CSamochod();
	pSamochod->Method();
	delete pSamochod;
	
	return 0;
}
 

Includes.h:

#pragma once // dla zawarcia pliku tylko raz! można użyć ifndef..itd.
#include <cstdio>
#include <Windows.h>
#include <cstdlib>
#include <cmath>
// inne

#include "Samochod.h"
 

Samochod.h:

#pragma once
class CSamochod
{
public:
	CSamochod();
	~CSamochod();

	void Method();
};

Samochod.cpp:

 
#include "Includes.h"

CSamochod::CSamochod()
{
}

CSamochod::~CSamochod()
{
}

void CSamochod::Method()
{
	printf("CSamochod::Method()");
}

W ten sposób, możesz dołączać kolejno następne pliki nagłówkowe z klasami do includes.h, dodawać nowe kody źródłowe do projektu includujące includes.h, w których są metody.
Sam używam takiego czegoś, i jestem w sumie zadowolony.

gośćabc napisał(a):

ten pomysł z includes nie jest zbyt dobry,

w plikach cpp powinniśmy includować tylko to co nam potrzebne

a w nagłówkach wprowadzać forward deklarację, gdzie tylko się da

Mówisz o pomyśle mwl4? W takim razie jak rozwiązać mój problem?

@mwl4 Dla mnie też nie wydaje się to do końca najlepsze rozwiązanie - chciałbym mieć w danych plikach dołączane tylko to co w tych plikach wykorzystuje i to powinno być według mnie jasno zaznaczone.

W sumie można by zapytać czy wszystkie niezbędne rzeczy można by zaincludować w pliku .h? Teoretycznie tak a praktycznie są przeciwwskazania?

gośćabc napisał(a):

klasy definiujemy w plikach nagłówkowych, czyli na bank nie w gra.cpp, przenieś definicję​ do gra.h

i zaincluduj gra.h w plikach cpp, w których używasz klasy Adam czyli w menu.cpp

miej świadomość, że tworzysz tam Adama lokanie więc po wykonaniu menu() Adam zniknie

Super, dzięki działa ;) ale skoro jest to Twoje "ale", to jak polecałbyś to zrobić w inny sposób?