标头中的私有Typedef?

时间:2015-10-04 21:35:27

标签: c++ scope header typedef

编辑:忘记添加" state_manager::"我的坏。

我正在尝试创建一个简单的状态系统。为了节省一些打字并让以后更容易更改,我在state_manager.hpp中添加了一些typedef。问题是这些typedef似乎在我的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);
}

2 个答案:

答案 0 :(得分:2)

除了缺少作为成员函数的资格外,unique_ptr不可复制,因此您当前的pushchange实现将无效。

您可以像这样更改它们:

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

然后可以像my_state_manager.push(std::make_unique<state>());

一样使用

答案 1 :(得分:1)

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

- 您没有定义成员函数,而是非成员函数,因此您无权访问类的私有成员。之后你会得到一个链接器错误。不要忘记添加前缀:

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