Хранение экземпляров структуры в std :: map - PullRequest
Новая
       16

Хранение экземпляров структуры в std :: map

5 голосов
/ 31 июля 2010

Я пытаюсь сопоставить некоторые структуры другим экземплярам, ​​например: 

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

Я получаю новый объект со свойствами, инициализированными по умолчанию.

Что-то не так с этим фрагментом кода или я пропускаю информацию о проблеме?

Согласно предложению @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). Это плохая идея?

Ответы [ 3 ]

4 голосов
/ 31 июля 2010

может быть, что 

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 голосов
/ 31 июля 2010

Уточните операции, которые вы хотите выполнить на 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, является копией объектаВы добавили на карту.Изменяя его, вы не изменяете то, что находится на карте.

Самое простое решение - создать карту указателей на объект, а не на сам объект.

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

Кроме того, используйте метод доступа, чтобы получить сопоставленное значение, вместо того, чтобы выставлять объект карты как открытый.Сделайте переменную экземпляров защищенной и добавьте публичный метод 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 голосов
/ 31 июля 2010

Похоже, вы определили свой оператор индексирования следующим образом: 

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;

Другие уже упоминали, что вам не нужно динамически выделять объект - вы все равно сохраняете копию в контейнере - ичто вы теряете память при этом.

...