首页 文章

将结构实例存储在std :: map中

提问于
浏览 次
5

我正在尝试将一些结构映射到其他一些实例，如下所示：

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; . 它现在有效，但我建议） . 这是一个坏主意吗？

c++ templates stl struct

3 回答

  • 2

    澄清您想要在LogicComponent上执行的操作 . 假设您正在尝试实现以下目标：

    第1步：向 Map 添加新条目：

    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对象是您添加到 Map 中的对象的COPY . 通过修改它，您不会修改 Map 中的内容 .

    最简单的解决方法是创建指向对象而不是对象本身的指针映射 .

    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];

    此外，请使用访问器方法来获取映射值，而不是将 Map 对象公开为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;
}
    回复于
  • 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];
};
    回复于
  • 3

    听起来您将索引运算符定义为：

    template <typename T>
T& Component::operator[]( EntityID id )
{
    return instances[id];
}

    或类似的东西 .

    这可能意外的影响是它会自动将 T 的默认构造实例插入到 Map 中，然后将其返回 for non-exising entries . 这是在 std::map 中完成的，因此 instances[10] = t; 等自然赋值语法有效 .

    这里的关键是 constness . 完全按照上面的定义，除了 returning by value 和 const 属性：

    template <typename T>
T Component::operator[]( EntityID id ) const
{
    return instances[id];
}

    这样，当您尝试使用不存在的密钥进行检索时，您将获得异常 . 更好的是，只是 typedef 它就像吼叫并完成它：

    typedef std::map<EntityID,UnitInfos> EntityUnitMap;

    其他人已经提到过，您不需要动态分配对象 - 无论如何都要将副本存储在容器中 - 并且在执行此操作时会泄漏内存 .

    回复于

相关问题