Zaprzyjaźnione klasy i pobieranie danych miedzy nimi

Question

Hej!

Chce stworzyć małą gierke, i chcę aby klasy abstrakcyjne ( w tym wypadku wojownik ) pobrał jedną wartość od innej klasy abstrakcyjnej (miecz) , i tu się pojawia mój problem bo nawet nie wiem czy mój pomysł podziału i podpięcia wszystkiego pod VirtualGameObject ma jakiś sens a co dopiero zaprzyjaźnienie dwóch klas ze sobą,

Link do mojego kodu

https://gist.github.com/krutek/6d6fdbdf48e0652fae613e266ca677dd

Potrzebuje odpowiedzi na następujące pytania:

1) Czy w ogóle mój pomysł dziedziczenia ma jakiś sens ?

2)Czy można tak jak jak jest w kodzie zrobić klase abstrakcyjną od klasy abstrakcyjnej i czy ma to jakikolwiek sens

3) Jak przesłać z klasy bron lub ekwipunek dane o mieczu do klasy wojownik

Dzięki!

Answer 1

1. Wygląda sensownie
2. Tak, można i ma sens

3. 	void PobierzWartoscAtakuBroni(int obrazenia , int sila , Miecz )
   	{
   		obrazenia = sila + Miecz.WartoscAtakuBroni;
   
   	};

   Nie wiem co próbujesz tu robić. Nie nadałeś nazwy ostatniemu argumentowi. Stwórz sobie w klasie Miecz jakąś metode w stylu `int getAttackVal()` (albo po prostu zostaw WartoscAtakuBroni jako public) i wtedy
   możesz 
    

   	void PobierzWartoscAtakuBroni(int obrazenia , int sila , const Miecz& miecz )
   	{
   		obrazenia = sila + miecz.getAttackVal();
   
   	};

   Druga sprawa, to.. co ma ta funkcja PobierzWartoscAtakuBroni wlasciwie robić? Obecnie nie robi w praktyce nic, bo wynik sumy przypisujesz do zmiennej lokalnej.

Comments

Funkcja PobierzWartoscAtakuBroni  była moją próbą pobrania wartosci ataku z klasy miecz i przekazanie jej do klasy wojownik.

Zmieniłem wartość " WartoscAtakuBroni " na public oraz zastosowałem wpisanie const Miecz& miecz. do funkcji PobierzWartoscAtakuBroni

Tylko że nie wiem co ma robić ta funkcja getAttackVal(); w zasugerowanym przez ciebie kodzie, gdy zamiast funkcji getAttackVal(); wpsiałem WartoscAtakuBroni wyskakują następujące błędy

-brak specyfikatora typu - założono, że int. Uwaga: C++ nie obsługuje domyślnie typu int   
-Błąd    "WartoscAtakuBroni": niezadeklarowany identyfikator

 - "miecz": niezadeklarowany identyfikator       

po lewej ".WartoscAtakuBroni" musi być typ struct/union

błąd składniowy: brakuje „,” przed „&”

---

Zwykły getter. Jeśli zrobiłeś sobie WartoscAtakuBroni  public, to nie ma potrzeby żeby mieć getAttackVal(). No anyway - chodziło o:

int getAttackVal() const { return WartoscAtakuBroni; }

Wracając.. skoro masz tamto public, to:

void PobierzWartoscAtakuBroni(int obrazenia , int sila , const Miecz& miecz )
{
    obrazenia = sila + miecz.WartoscAtakuBroni;
 
};

---

Właśnie tak zrobiłem i dalej wywala mi masę błędów , dokładnie te same o których pisałem wyżej

---

Zupdejtuj kod w gistcie, to spojrze. I napisz w których liniach te błędy

---

https://gist.github.com/krutek/6d6fdbdf48e0652fae613e266ca677dd
Błąd    C2228    po lewej ".WartoscAtakuBroni" musi być typ struct/union  wojownik.h  wiersz   20   
Błąd    C4430    brak specyfikatora typu - założono, że int. Uwaga: C++ nie obsługuje domyślnie typu int    wojownik.h    wiersz 18  
Błąd    C2143    błąd składniowy: brakuje „,” przed „&”   wojownik.h   wiersz  18
Błąd    C2065    "miecz": niezadeklarowany identyfikator    wojownik.h    wiersz 20   
Błąd    C2065    "WartoscAtakuBroni": niezadeklarowany identyfikator    wojownik.cpp wiersz   26    i 31

---

Wojownik.h i Miecz.h includują sie nawzajem co kończy się nieskonczoną pętlą includowania i stąd dziwne błędy. W pliku wojownik.h tylko zadeklaruj klase Miecz gdzieś na początku pliku:

class Miecz;

A wszystkie definicje funkcji nalezacych do klasy Wojownik wywal do wojownik.cpp tak jak to zrobiłeś z całą resztą jego funkcji poza `PobierzWartoscAtakuBroni`. natomiast sam #include "Miecz.h" wrzuć tez do wojownik.cpp. Analogicznie zrób dla klasy Miecz. Najlepiej jakby tak to wyglądało wszędzie - deklaracja klasy jeśli potrzebna w *.h, a include w *.cpp.

Dodatkowo masz zupełnie bezsensowny kod w wojownik.cpp:

void Wojownik::Wyswietl()
{
    SprawdzObrazenia();
    cout << "Parametry twojej postaci" << endl;
    cout << "imie : " << imie << endl << "lvl : " << lvl << endl;
    cout << "punkt zycia : " << zdrowie << endl << "punkt energi :" << energia << endl <<
        "sila : " << sila << endl << "atak od broni : " << WartoscAtakuBroni << endl << "obrazenia w walce: " << obrazenia << endl;
    cout << " Nastepny level za  : " << exp << " punktow doswiadczenia " << endl;
}
void Wojownik::SprawdzObrazenia()
{
    obrazenia = sila + WartoscAtakuBroni;
}

W klasie Wojownik nie ma czegoś takiego jak WartoscAtakuBroni, więc...

No i ta funkcja PobierzWartoscAtakuBroni. Nie chodzilo ci przypadkiem o cos takiego:

    int PobierzWartoscAtakuBroni(int sila, const Miecz& miecz)
    {
        return sila + miecz.WartoscAtakuBroni;

    };

lub 

    void PobierzWartoscAtakuBroni(int& obrazenia, int sila, const Miecz& miecz)
    {
        obrazenia = sila + miecz.WartoscAtakuBroni;

    };

? To co jest obecnie nie ma sensu, bo `obrazenia` jest zmienną lokalną o czym już mówiłem.

---

Dzięki za odpowiedź! wszystko ruszyło :) i tak o to dokładnie mi chodziło z funkcją Pobierz...

Tylko teraz jeszcze jedno pytanie mam, bo do klasy wojownik jest zaincludowana klasa miecz a gdybym miał więcej takich klas to jest miecz, topór, włócznia i później analogicznie dla maga jakieś rodzaje magicznych lasek to czy mogę wszędzie tam gdzie zainkludowałem miecz zainkludować po prostu klase abstrakcyjną bron ?

---

Jeśli nigdzie w kodzie nie będziesz potrzebował się odwoływać bezpośrednio do klas pochodnych od Bron (wspomniane miecz, topor, wlocznia...), to tak.

---

Ok, Jeszcze raz Wielkie Dzięki!

---

W sumie to chciałbym zapytać cię o jeszcze jedną rzecz, mianowicie jak wywołać  to

int PobierzWartoscAtakuBroni(int sila, const Miecz& miecz)
{
    return sila + miecz.WartoscAtakuBroni;
 
};

 

---

Jak to jak wywolac? Jak kazdą inną funkcje

PobierzWartoscAtakuBroni(5, Miecz());

 

btw po definicji funkcji nie stawia się średnika