我正在尝试将一些结构映射到其他一些实例,例如:
template <typename T>
class Component {
public:
typedef std::map<EntityID, T> instances_map;
instances_map instances;
Component() {};
T add(EntityID id) {
T* t = new T();
instances[id] = *t;
return *t;
};
};
然后我像这样使用它:
struct UnitInfos {
int owner_id;
int health;
float x, y;
};
class LogicComponent : public Component<UnitInfos> {};
问题在于稍后稍后检索数据时,如下所示:
comp.instance[id];
我得到一个breand新对象,其属性初始化为默认值。
这段代码是否存在内在错误,或者我是否遗漏了有关此问题的信息?
根据@aaa建议,我将代码更改为
typedef std::map<EntityID, T> instances_map;
instances_map instances;
T& add(EntityID id) {
instances[id] = T();
return instances[id];
};
但是当我访问它时
UnitInfos &info = logic_c.instances[id];
info.x的值仍为0.任何指针?
问题是我如何在另一个类中存储对LogicComponent的引用。使用LogicComponent logic_c;
代替LogicComponent& logic_c;
。它现在有效,但我在地图中存储指针(而不是@aaa的建议)。这是个坏主意吗?
答案 0 :(得分:4)
可能就是那个
T add(EntityID id) {
T* t = new T();
instances[id] = *t;
return *t; // return value and map instance are not the same anymore
};
应该是
T& add(EntityID id) {
instances[id] = T();
return instances[id];
};
答案 1 :(得分:3)
阐明您要在LogicComponent上执行的操作。假设您正在尝试实现这样的目标:
步骤1:向地图添加新条目:
LogicComponent comp;
EntityID id = 99;
UnitInfos info = comp.add(id);
第2步:初始化信息:
info.x = 10.0;
info.y = 11.0
// etc
第3步:再次获取信息对象:
UnitInfos info2 = comp.instances[id]; // this is uninitialized.
然后,一些代码注释按顺序排列:
comp.add返回的info对象是您添加到地图中的对象的COPY。通过修改它,您不会修改地图中的内容。
最简单的解决方法是创建指向对象而不是对象本身的指针映射。
typedef std::map<EntityID, T*> pinstances_map;
T * add(EntityID id) {
T* t = new T();
instances[id] = t;
return t;
};
// initialize as
UnitInfo *info = comp.add(id);
info->x = 10.0;
info->y = 11.0;
// retrieve as
UnitInfos *info = comp.instances[id];
此外,请使用访问器方法来获取映射值,而不是将地图对象公开为public。使实例变量受保护,并添加一个公共get()方法。
编辑:此代码适用于我:
#include <map>
#include <iostream>
using namespace std;
template<typename T>
class Component
{
public:
typedef map<long, T*> pinstances_map;
pinstances_map instances;
T * add(long id)
{
T *t = new T();
instances[id] = t;
return t;
}
};
struct UnitInfo
{
float x, y;
};
class LogicComponent: public Component<UnitInfo> {};
int main()
{
LogicComponent comp;
UnitInfo *info = comp.add(99);
info->x = 10.0;
info->y = 11.0;
UnitInfo *info2 = comp.instances[99];
cout << info2->x << " " << info2->y;
return 0;
}
答案 2 :(得分:2)
听起来您将索引运算符定义为:
template <typename T>
T& Component::operator[]( EntityID id )
{
return instances[id];
}
或类似的东西。
这可能会产生意想不到的效果,它会自动将默认构造的T
实例插入到地图中,然后将返回给不存在的条目。这是在std::map
中完成的,因此instances[10] = t;
等自然分配语法可以正常工作。
这里的关键点是 constness 。除了按值返回并使用 const
属性外,完全按上述方式定义:
template <typename T>
T Component::operator[]( EntityID id ) const
{
return instances[id];
}
这样,当您尝试使用不存在的密钥进行检索时,您将收到异常。更好的是,只需typedef
它就像吼叫一样,并完成它:
typedef std::map<EntityID,UnitInfos> EntityUnitMap;
其他人已经提到过你不需要动态分配一个对象 - 无论如何你都要在副本中存储一个副本 - 当你这样做时你会泄漏内存。