C++ Object不为null,但其内容是否被破坏？

Question

我一直在VS2012教自己C++,并且遇到了一些真正让我摸不着头脑的东西.当创建一个对象并在另一个对象中添加对它的引用时,该对象似乎被破坏或者某些东西.它不是 null(空指针检查的每个组合我可以确认这一点),但其内容很时髦并且试图访问其中一个成员导致"0xC0000005:访问冲突读取位置0xCCCCCCE8".

我的程序的要点是我有一个Tile对象的2D矢量,每个对象都可以引用一个站点.当我在具有网站的tile的draw()函数中时,它将尝试以其文明所属的颜色绘制该网站.

世界一代:

std::vector<std::vector<Tile>> world;
// ... vector is sized ...
// Populate world with tiles
for (int h=0; h<height; h++) {
    for (int w=0; w<width; w++) {
        Tile t(PLAINS);
        world[h][w] = t;
    }
}

创建文明时,会为它们创建一个简单的大写网站,并将其引用放在指定x,y的图块中:

void Game::createCivilisation(std::string name, sf::Color colour, int x, int y) {

    // Create the civ
    Civilisation civ(name, colour);

    // Create their capital and link it up
    Site capital("Capital", year, &civ);  // when I comment these five lines out
    civ.capital = &capital;               // the program doesn't crash, so I know
    civ.sites.push_back(capital);         // that the dodgy site originates here.

    // Add capital to tile
    world[y][x].site = &capital;
    capital.location = &world[y][x];
}

然而,当调用Tile的draw()函数时,它会在尝试访问该站点的成员时死亡.这只是一个文明.

if (site != nullptr) { // have also tried (site) (site!=0) (site!=NULL)
    sf::CircleShape s(tileWidth*0.5);
    s.setFillColor(site->civilisation->colour);  // womp womp

上图是提供的调试信息.如你所见,这都是垃圾."资本"已成为几十亿字符串的垃圾,一切都是错误的,所有对其他对象的引用也都消失了.当我评论出资本网站链接时

我想我已经提供了所有相关的代码片段(为了简洁起见/为了她的乐趣).我已经在这个工作了几个小时,它让我难过.所有索引都应该是正确的,createCivilisation()中生成的网站绝对是唯一存在的网站等等......

Answer 1

这是错的

Site capital("Capital", year, &civ);  // when I comment these three lines out
civ.capital = &capital;               // the program doesn't crash, so I know
civ.sites.push_back(capital);         // that the dodgy site originates here.

// Add capital to tile
world[y][x].site = &capital;

您存储的指针&capital到本地对象(和做两次,似乎).当你的createCivilisation方法退出时,本地对象被销毁,所以你最终会得到垃圾指针.

简单的答案是您复制对象本身而不是尝试存储指针.如果对象很大并因此复制它们是一个问题,那么您应该查看动态堆分配和智能指针.