[c++]podział programu na pliki

[ocb]

Mam problem ponieważ potrzebuje dodać bardzo obszerna funkcje do programu. Nie chcę żeby znajdowała się w pliku z funkcja main() ponieważ robi się nieczytelnie. Jednak mimo moich starań kompilator nie puszcza tego co nabazgrałem. O to kod przedstawiający mój problem.

//dziedziczenie_test.cpp
#include <iostream>
#include "klasa_pierwsza.h"

void test();

int main()

{
	klasa_pierwsza p1;
	p1.a = 1;
	std::cout << "p1.a: " << p1.a << std::endl;
	test();
	system("pause");
	return 0;
}

//klasa_pierwsza.h
ifndef klasa_pierwsza_h
#define klasa_pierwsza_h

#include <iostream>

class klasa_pierwsza
{

public:
	int a;
	void wywal();
};

void wywal()
{
	std::cout << "wywal1" << std::endl;
}
#endif

//klasa_dwa.h
#ifndef klasa_dwa_h
#define klasa_dwa_h

#include <iostream>
#include "klasa_pierwsza.h"

class klasa_dwa : public klasa_pierwsza
{
	public:
	int b;
	void wywal2();
};
void wywal2()
{
	std::cout << "wywal2" << std::endl;
}
#endif

//klasa_dwa.cpp
#include <iostream>
#include "klasa_dwa.h"

void test()
{
	klasa_dwa k2;
	k2.wywal2();
}

To mi pluje kompilator i ciężko mi się ustosunkować do tego co mówi. Błędu 2019 w ogóle nie rozumiem...

1>klasa_dwa.obj : error LNK2005: "void __cdecl wywal(void)" (?wywal@@YAXXZ) already defined in dziedziczenie_test.obj
1>klasa_dwa.obj : error LNK2019: unresolved external symbol "public: void __thiscall klasa_dwa::wywal2(void)" (?wywal2@klasa_dwa@@QAEXXZ) referenced in function "void __cdecl test(void)" (?test@@YAXXZ)
1>C:\Users\kk\documents\visual studio 2010\Projects\dziedziczenie_test\Debug\dziedziczenie_test.exe : fatal error LNK1120: 1 unresolved externals

[Orb]

Mogę się mylić, ale:

#include <iostream>

Chyba nie powinno się powtarzać w plikach nagłówkowych, skoro było już w pliku źródłowym.

[ocb]

next please

[Orb]

Faktycznie dodatkowe załączniki do iostreama nie przeszkadzały, chociaż nie mam pojęcia po co zamieszczasz je w każdym pliku. Dziś miałem chwilę, aby zerknąć dokładniej i sprawdzić na kompilatorze, czy mam racje.

Po pierwsze w pliku klasa_pierwsza.h nie było # przed ifndef.
Po drugie nie było nigdzie includa do pliku klasa_dwa.cpp, więc kompliator mi słusznie krzyczał, że nie zdefiniowano funkcji test(). Dodałem odpowiednią linijkę do dziedziczenie_test.cpp i to już grało.
Po trzecie zdefiniowałeś funkcje wywal2(), jako funkcje globalną a nie składnik klasy, przez co kompilator słusznie narzekał, że funkcja nie została zdefiniowana, podczas próby wywołania jej jako składnika klasy, która zawierała jedynie jego deklaracje. Powinno to wyglądać tak:

void klasa_dwa::wywal2()
{
	std::cout << "wywal2" << std::endl;
}

Jednak osobiście polecam robić tak jak ja. Wszędzie tam, gdzie nie ma konieczności oddzielania deklaracji od definicji funkcji ( jeszcze się nie spotkałem z przypadkiem w którym to konieczne ), definiować ją w miejscu deklaracji.

class klasa_dwa : public klasa_pierwsza
{
	public:
	int b;
	void wywal2()
	{
		std::cout << "wywal2" << std::endl;
	}
};

To pozwala wyeliminować tego typu pomyłki.

Po tych zabiegach na moim kompilatorze wszystko gra.

[ocb]

Hasza nie ma ponieważ nie zaznaczyłem go widocznie druga sprawa, faktycznie nie zdefiniowałem funkcji jako składnik klasy(w tym kodzie, jest naskrobany na potrzebę przedstawienia problemu, oryginał ma poprawnie zdefiniowane ) trzecia sprawa to wklej cały kod na pastebin lub tu ponieważ po tych zmianach dalej ni w ząb nie mogę skompilować tego kodu. Czwarta sprawa to nie widzę powody czemu nie miał bym wywalać definicji metody poza klasę, w której jest ta metoda zadeklarowana.. Nie sprawiało mi to problemu dopóki nie miałem potrzeby dodać funkcji i przenieść jej do drugiego pliku .cpp.
Dziękuję za zaangażowanie.

[Orb]

Czwarta sprawa to nie widzę powody czemu nie miał bym wywalać definicji metody poza klasę,

Nie ma żadnego, poza tym, który podałem w poprzednim poście. Najłatwiej unikać pomyłek uniemożliwiając ich popełnienie. A oto poprawiony kod, który u mnie pod kompilatorem g++ chodzi elegancko.

//dziedziczenie_test.cpp
#include <iostream>
#include "klasa_pierwsza.h"
#include "klasa_dwa.cpp"

void test();

int main()

{
	klasa_pierwsza p1;
	p1.a = 1;
	std::cout << "p1.a: " << p1.a << std::endl;
	test();
	return 0;
}

system("pause") usunąłem, bo mam linuxa a pod nim nie ma takiej funkcji. U siebie śmiało możesz to dodać. Nie powinno przeszkadzać.

//klasa_pierwsza.h
#ifndef klasa_pierwsza_h
#define klasa_pierwsza_h

#include <iostream>

class klasa_pierwsza
{

public:
	int a;
	void wywal();
};

void wywal()
{
	std::cout << "wywal1" << std::endl;
}
#endif

//klasa_dwa.h
#ifndef klasa_dwa_h
#define klasa_dwa_h

#include <iostream>
#include "klasa_pierwsza.h"

class klasa_dwa : public klasa_pierwsza
{
	public:
	int b;
	void wywal2();
};
void klasa_dwa::wywal2()
{
	std::cout << "wywal2" << std::endl;
}
#endif

//klasa_dwa.cpp
#include "klasa_dwa.h"

void test()
{
	klasa_dwa k2;
	k2.wywal2();
}