c++ - 在 std::map 中存储结构实例

Question

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

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;。它现在可以工作了，但我在 map 中存储了指针(而不是@aaa 的建议)。这是个坏主意吗？

Answer 1

阐明您要对 LogicComponent 执行的操作。假设您正在尝试实现这样的目标:

第 1 步:向 map 添加新条目:

LogicComponent comp; 
EntityID id = 99;
UnitInfos info = comp.add(id);

第 2 步:初始化信息:

info.x = 10.0;
info.y = 11.0
// etc

第三步:再次获取信息对象:

UnitInfos info2 = comp.instances[id]; // this is uninitialized.

然后，按顺序进行一些代码注释:

comp.add 返回的信息对象是您添加到 map 的对象的拷贝。通过修改它，您不会修改 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];

此外，请务必使用访问器方法来获取映射值，而不是将映射对象公开为公共(public)对象。使实例变量 protected ，并添加一个公共(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;
}