Obiekt typu wyliczeniowego znajdującego się w klasie A jako argument funkcji z klasy B.

Question

Cześć to znowu ja. Dzisiaj natknąłem się na pewien problem związany z przekazywaniem obiektu typu wyliczeniowego do funkcji pewnej klasy, oto kod:

//GameManager.hpp
enum PlayerMovementDirection
{
	LEFT,
	RIGHT,
	JUMP,
	ARE_NOT_MOVING
};
//...oraz stworzony obiekt tegoż typu, na razie niezainicjalizowany
PlayerMovementDirection playerMovementDirection;
//Player.hpp
//W tym pliku mam funkcję move o następującej deklaracji:
void move(const PlayerMovementDirection direction, const float FPS);
//a wywołuję ją tak:
player.move(playerMovementDirection, FPS);

Błędy są następujące:

Player.hpp:30:18: error: ‘PlayerMovementDirection’ does not name a type
  void move(const PlayerMovementDirection direction, const float FPS);
                  ^
In file included from GameManager.hpp:9:0,
                 from main.cpp:1:
Player.hpp:30:18: error: ‘PlayerMovementDirection’ does not name a type
  void move(const PlayerMovementDirection direction, const float FPS);
                  ^
In file included from Player.cpp:1:0:
Player.hpp:30:18: error: ‘PlayerMovementDirection’ does not name a type
  void move(const PlayerMovementDirection direction, const float FPS);
                  ^
Player.cpp:21:25: error: ‘PlayerMovementDirection’ does not name a type
 void Player::move(const PlayerMovementDirection direction, const float FPS)

Zadeklarowanie klasy GameManager na początku pliku Player.hpp nie pomogło, jak i również używanie przedrostka GameManager, np. tak: 

void move(const GameManager::PlayerMovementDirection direction, const float FPS);

Proszę o pomoc.

Answer 1

Po kolei.. Po pierwsze.. Dlaczego przesyłasz enum'a jako argument funkcji? Przez tego const mogę się domyślać, że po prostu pomieszało ci się trochę czym dokładnie jest enum. Po prostu wsadź ten kod:
enum PlayerMovementDirection
{
    LEFT,
    RIGHT,
    JUMP,
    ARE_NOT_MOVING
};
Do jakiegoś osobnego pliku nagłówkowego (np. Keys.h) i dołączaj go do każdego pliku (.cpp), w którym będzie ci potrzebny ten enum (nie musisz nigdzie tworzyć obiektu typu PlayerMovementDirection). Teraz jeżeli chcesz wykorzystać jedną z tych (stałych) wartości wystarczy, że zrobisz np. tak:
Player.Move(PlayerMovementDirection::LEFT);
//Gdzie argumentem w metodzie Move jest zwykły int
Jeżeli jednak koniecznie chcesz przesyłać enum.. to poczytaj o czymś takim jak "enum class".. jak wygooglujesz znajdziesz na pewno.
Przykład użycia enum class:
http://cpp.sh/93mhv

Comments

Bardzo dziękuję, umieszczenie enum'a w osobnym pliku pomogło.

Answer 2

Program nie działa, ponieważ w deklaracji metody chcesz pobrać stały obiekt, a przesyłasz mu zmienny. Żeby to naprawić usuń const z deklaracji funkcji lub dodaj &. Wtedy będziesz pobierał referencję do stałego obiektu i przesłanie zwykłego powinno zadziałać. Całość powinna wyglądać tak

void move(PlayerMovementDirection direction, const float FPS);

lub

void move(const PlayerMovementDirection& direction, const float FPS);

Mam nadzieję, że pomogłem. Pozdrawiam.

Comments

Nie pomogło, teraz mam błędy typu

Player.hpp:30:12: error: ‘PlayerMovementDirection’ has not been declared
  void move(PlayerMovementDirection direction, const float FPS);

---

@Dubgron, to tak nie działa. 

const przy argumencie w funkcji/metodzie, to deklaracja, że nie będzie on zmieniany.

---

@Criss, Zgadza się, mój błąd.

Answer 3

Samo zadeklarowanie klasy nie ma jak czegokolwiek dać, bo to tylko deklaracja. Player wciąż nie wie, że wewnątrz GameManager jest taki enum. Musisz zaincludować cały GameManager w Player.h.

Jeśli enum jest zadeklarowany wewnątrz GameManager, to musisz pisać GameManager::PlayerMovementDirection. No i oczywiście musi być public.

Answer 4

czy ten enum jest w środku jakiejś klasy?
jeśli tak to musi to byc tak:
 

void move(GameManager::PlayerMovmentDirection, float FPS);