Частные Typedefs в заголовках?

Question

Редактировать: Просто забыли добавить «state_manager::« Плохо.

Я пытаюсь создать простую государственную систему. Чтобы сохранить некоторую типизацию и сделать что-то легче изменить позже, я добавил некоторые typedefs в свой state_manager.hpp. Проблема в том, что эти typedefs, похоже, не распознаются в моем state_manager.cpp. Я получаю ошибки, такие как 'element' does not name a type и странно 'states' was not declared in this scope. Я действительно смущен этим.

state_manager.hpp:

#pragma once 
#include <stack> 
#include <memory> 

class state; 

class state_manager{ 
typedef std::unique_ptr<state> element; 
typedef std::stack<element> container; 
protected: 
container states; 
public: 
void push(const element &to_push); 
void pop(); 
void change(const element &change_to); 
}; 

state_manager.cpp:

#include "state_manager.hpp" 
#include "state.hpp" 

void push(const element &to_push){ 
states.push(to_push); 
} 

void pop(){ 
states.pop(); 
} 

void change(const element &change_to){ 
states.pop(); 
push(change_to); 
} 

Answer 1

void push(const element &to_push){ 
    states.push(to_push); 
} 

- вы не определения функций-членов, но те, не являющихся членами, поэтому вы не иметь доступ к частным членам вашего класса. После этого вы получите ошибку компоновщика. Не забудьте добавить префикс:

void state_manager::push(const element &to_push){ 
    states.push(to_push); 
} 

Answer 2

В дополнении недостающей квалификации как функции-членов, unique_ptr s не копируемые, поэтому текущая реализация push и change не будет работать.

Вы можете изменить их следующим образом:

void state_manager::push(element&& to_push) { 
    states.push(std::forward<element>(to_push)); 
} 

, который затем может быть использован как my_state_manager.push(std::make_unique<state>());