Powrot po dlugiej przerwie i maly problem

Question

Witam,

wracam po dluzszej przerwie do programowania w C++. Moim nowym projektem bedzie gra, poki co hcialbym zrobic zwykle MMO bez grafiki, ktora zostanie dodana pozniej, ale mam problem. Gdy odwoluje sie do klasy mojej postaci:

https://pastebin.com/fZf75Kht

tak jak jest pokazane w kodzie przy nacisnieciu "W", to wyskakkuje nastepujacy blad:

error: 'mage' is not a member of 'Mage'|

Klasa mojego bohatera wyglada nastepujaco:

https://pastebin.com/NRcSMNqN

 

Blad sam w sobie nie rozumiem, osattni raz odpalalem projekt przed 6 miesiacami jak nie lepiej. Wczesniej uzywalem singletonu, ale odradzono mi go, z tego pwoodu musialem zmeinic moje klasy, i nie wiem gdzielezy w tym momencie blad. Gdyby mogl mi ktos pomoc, podpowiedziec jak to rozwiazac bylbym wdzieczny.

 

Z gory dziekuje, zycze milego dnia i Swiat Wielkanocnych :)

Answer 1

Twoja klasa Mage nie posiada statycznej funkcji mage().

Comments

Wystarczy dodac static mage, albo static obojetnie jaka klasa, tzn funkcja?

 

EDIT:

Gdzie dodac static? Czy moze byc to w dowolnym pliku czy tylko w tym ktor ma dzialac?

---

Jeżeli chcesz naprawić kod na szybko to powinieneś przywrócić singleton (statyczna funkcja Mage::mage() powinna zwracać referencję na twój obiekt)
Jeżeli chcesz to zrobić poprawnie to powinieneś całkowicie przerobić swój kod z wykorzystaniem dziedziczenia i polimorfizmu.
Przykład:
 

class Hero {
public:
    Hero(int x, int y, int hpMax) {
        posX = x;
        posY = y;
        hp = this->hpMax = hpMax;
    }
    void moveUp()    { ++posY; }
    void moveDown    { --posY; }
    void moveRight() { ++posX; }
    void moveLeft()  { --posX; }
    int x() const { return posX; }
    int y() const { return posY; }    
    int hp() const { return hp; }
    virtual void printPosition() const = 0;
private:
    int posX, posY, hp, hpMax;
}
class Mage : public Hero {
public:
    Mage(int x, int y) : Hero(x, y, 10) {}
    virtual void printPosition() const {
        cout << "Potezny mag {" << x() << "," << y() << "} HP: " << hp() << endl;
    }
}
class Warior : public Hero {
public:
    Mage(int x, int y) : Hero(x, y, 50) {}
    virtual void printPosition() const {
        cout << "Dziarski wojownik {" << x() << "," << y() << "} HP: " << hp() << endl;
    }
}

 

---

Nie jestem jeszcze tak daleko, ale dziekuje, pki co chce skocczyc to jak najszybciej, poniewaz bedzie to temat mojej pracy do szkoly. Ale oczywiscie pozniej bede to przerabial, bo i poniewaz, chce stworzyc MMORPG, wiec dodatkowo dojdzie obsluga bazy ddanych, bede musial wiec razy wykorzystywac jedna klase, z mozliwoscia odrozanienia postaci itd, ale wiadomo w bazie bede wszytsko zapisywal. Od czerwca, mam w planach przepisaie kodu, skrocenie do max i powolne zaczecie z grafika lub dzialaniem w oknie.

Dziekuje Ci bardzo i kod na 100% przeanalizuje i wykorzystam w pozniejszej pracy :)

Pozdrawiam

---

W sumie to zapomniałem o najważniejszej rzeczy (użyciu poprzednich klas w kodzie):
 

Hero *hero = nullptr;
int wyborKlasy;
cout << "Kim chcesz zostac?" << endl;
cout << "1. Mag" << endl;
cout << "2. Wojownik" << endl;
cin >> wyborKlasy;
if (wyborKlasy == 1)
    hero = new Mage(0,0);
else if (wybor == 2)
    hero = new Warior(0,0);
else {
   cout << "Nie ma takiej klasy!" << endl;
   exit(1);
}
hero->printPosition();