Dziedziczenie prywatne i używanie metod

Question

Witam 

Mam problem z pokazanie wszystkich danych poprzez metode klasy pochodnej.

class Gra    // klasa bazowa
{
private:
    string name;
public:
...
    void Show();
};


class Gracz : private string
{
private:
    int ration;
public:
...
   void Show1();
};

Show1()

void Gracz::Show1()
{
    void Gra::Show();
    cout<<"Ration: "<<ration<<endl;
}

 Kiedy użyje metody klasy bazowej Show() to pokazuje mi normalnie składową name. Ale jak użyje Show1() to mi tylko pokazuje składową klasy pochodnej ration a name już nie.

 

Za wszelką pomoc dzięki 

Comments

Tak z ciekawości...

class Gracz : private string

Po jaką cholerę dziedziczysz ze stringa, czy to tylko "literówka"?

---

Tak mam w książce i po prostu tak to zrobiłem. Myślę że tak ma być, albo mi się tak wydaje  :P

Książka: "Język C++ Szkoła Programowania Stephen Prata"

---

Dziedziczenie z std::string jest wysoce odradzane, pomijając, że std::string nie ma wirtualnego destruktora. Tak więc nie za dobry przykład w książce...

Answer 1

Podstawowy błąd:

void Gra::Show();

Nie powinno być void. Przecież chcesz wywołać metodę, a nie ją zdefiniować (co nie byłoby raczej nawet możliwe w tym miejscu).

Następnie: metody wywołujemy na rzecz konkretnych obiektów. Tutaj wywołujesz ją "od tak".

Poprawiony kod:

#include <iostream>
#include <string>
using namespace std;

class Gra
{
private:
	string name = "Hello";
public:
	void Show() {
		cout << "Name: " << name << endl;
	}
};


class Gracz : private string
{
private:
	int ration = 0;
public:
	void Show1() {
		Gra obj;
		void (Gra::*pf)() = &Gra::Show;
		(obj.*pf)();

		cout << "Ration: " << ration << endl;
	}
};

int main() {
	Gra obiekt_gra;
	Gracz obiekt_gracz;
	obiekt_gracz.Show1();
	cin.get();
	
}

 

Comments

A to drugi sposób:

#include <iostream>
#include <string>
using namespace std;

class Gra
{
private:
	string name = "Hello";
public:
	void Show() {
		cout << "Name: " << name << endl;
	}
};


class Gracz : private string
{
private:
	int ration = 0;
public:
	void Show1(Gra &ref) {
		ref.Show();

		cout << "Ration: " << ration << endl;
	}
};

int main() {
	Gra obiekt_gra;
	Gracz obiekt_gracz;
	obiekt_gracz.Show1(obiekt_gra);
	cin.get();
	
}

 

Answer 2

Poczytaj o programowaniu obiektywem. Jego teoretycznym celem jest odzwierciedlanie świata rzeczywistego wykorzystując klasy oraz ich instancje. 

Moim zdaniem próbując dziedziczyć ze stringa pokazujesz że nic nie chwytasz. std::string to tak naprawdę base_string<char>, cała jego funkcjonalność jest publiczna. Dziedzicząc prywatnie zamykasz ją w klasie pochodnej i co tam będziesz robić? Pomijam już to, że pochodna nazywa się Gracz, bo to dla mnie pomyłka. 

Btw. Czasem granica między wyborem dziedzicz albo zawieraj jest bardzo cienka. Wtedy lepiej zawierać. 

Answer 3

Zrób pole "name" protected wtedy będzie nie widoczna na zewnątrz klasy ale dziedziczona.

Comments

A jest jakaś może inna możliwość ?

---

Zamiast dziedziczyć utwórz sobie instancję do obiektu "Gra" (inaczej właściwość która będzie grą :-) ).