0

已解决:http : //pastebin.com/seEaALZh

我试图创建简单的项目系统,我可以通过它的 id 获取项目信息。我不能使用数组,因为项目 ID 可以说是随机的。我想使用声明的项目作为变量,我想通过它的 id 快速找到任何项目信息。我找到的唯一方法是stl map。

所以我有这个简单的代码:

  1. 主文件

    #include <iostream>
    #include <map>
    
    enum
    {
        weapon,
        ammo
    };
    
    class c_items
    {
        bool operator()(const c_items& l, const c_items& r) const
        {
            return (l.id < r.id);
        }
    public:
        c_items(void){};
        c_items(int id, char name[], int type);
        char *name;
        int type;
        int id;
    };
    
    extern std::map<int,c_items> Stuff;
    
    c_items::c_items(int id, char name[], int type) : id(id), type(type), name(name)
    {
        Stuff[id] = c_items(id, name, type);
    }
    
    const c_items
        brass_knuckles          (546, "Brass knuckles", weapon),
        golf_club               (2165, "Gold club", weapon);
    
  2. 主文件

    #include "main.h"
    
    std::map<int,c_items> Stuff;
    
    using namespace std;
    
    int main()
    {
        // cout << Stuff[2165].name.data();
        return 1;
    }
    

并且由于某种原因程序崩溃。如何在类初始化时将类数据正确插入映射?

4

2 回答 2

1

问题是初始化的顺序。brass_knuckles和的静态构造函数golf_club首先运行,在静态构造函数之前运行Stuff,因此它们尝试插入到尚未构造的映射中。

此外,您永远不需要在头文件中使用变量 DEFINITIONS,因为如果您将头文件包含在多个源文件中,您最终会得到多个定义,这充其量会导致链接失败。因此,您应该将 DEFINITIONS 从 .h 文件移到 .cpp 文件中。将它们放在定义之后Stuff将解决初始化问题的顺序。

如果要在其他编译单元中使用它们,可以在头文件中声明变量:

extern const c_items brass_knuckles, golf_club;
于 2013-07-13T21:10:12.630 回答
0

你不能把 c_item 放在那样的东西中

 std::map<int,c_items> Stuff = { {item1.id, item1}, {item2.id, item2}};

但是您还需要@Chris Dodd 提出的所有建议

所以

c_items::c_items(int id, char name[], int type) : id(id), type(type), name(name)
{}

extern const c_items  brass_knuckles,      golf_club;

并在 main.cpp

 const c_items  brass_knuckles = {546, "Brass knuckles", weapon);
    const c_items golf_club = (2165, "Gold club", weapon);
    std::map<int,c_items> Stuff = { {brass_knuckles.id, brass_knuckles}, etc....};
于 2013-07-13T21:18:04.480 回答