1

我正在使用 C++ 和 SFML 2.0 开展一个学校项目,该项目将于周二到期,而我最近刚刚遇到了这个问题。我在最新的调用堆栈所在的位置上放了一个箭头。这些是我怀疑其中可能有问题的代码部分。我对 c++ 也很陌生,我知道我做错了很多事情。

这是在我的Background课堂上,它加载我所有的瓷砖,设置它们的位置和瓷砖类型等等。

GetWayPointList瓷砖矢量返回给我的EnemyManager班级

void Background::Load(QuadTree * quadTree) {

        int tempTiles[32][32] = {... Lots of 1's and 0's in here };

        for(int i = 0; i < 32; i++)
        {
            for(int j = 0; j < 32; j++)
            {
--->                    tile[j][i]->tileType = tempTiles[j][i];
                    tile[j][i]->setPosition(sf::Vector2f((float)(i*32),(float)(j*32)));
                    if(tile[j][i]->tileType == 1)
                    {
                        quadTree->AddObject(tile[j][i]);            
                    }

                else if(tile[j][i]->tileType == 0)
                {
                    tile[j][i]->distanceFromStart = INT_MAX;
                    tile[j][i]->parent = NULL;

                    vector<Tile*> tempTiles;
                    if(tile[j+1][i]->tileType == 0)
                        tempTiles.push_back(tile[j+1][i]);
                    if(tile[j-1][i]->tileType == 0)
                        tempTiles.push_back(tile[j-1][i]);
                    if(tile[j][i+1]->tileType == 0)
                        tempTiles.push_back(tile[j][i+1]);
                    if(tile[j][i-1]->tileType == 0)
                        tempTiles.push_back(tile[j][i-1]);
                    tile[j][i]->setSuccessors(tempTiles);
                    tilesList->push_back(tile[j][i]);
                }
            }
        }
    }

vector<Tile*>* Background::GetWayPointList()
{
    return tilesList;
}

调用堆栈本身说TileShooter v0.1.exe!Background::Load(QuadTree*quadTree) Line 68 + 0x2e bytes。第 68 行是箭头所在的位置。

如果您需要更多代码/信息,请说。任何关于可能出错的地方都会受到极大的赞赏。最好的问候,弗雷德里克 W


编辑

我已经编辑了一些代码,包括tile[j][i] = new Tile();

void Background::Load(QuadTree * quadTree) {

        int tempTiles[32][32] = {... Lots of 1's and 0's in here };

        for(int i = 0; i < 32; i++)
        {
            for(int j = 0; j < 32; j++)
            {
                    tile[j][i] = new Tile();
                    tile[j][i]->tileType = tempTiles[j][i];
                    tile[j][i]->setPosition(sf::Vector2f((float)(i*32),(float)(j*32)));
                    if(tile[j][i]->tileType == 1)
                    {
                        quadTree->AddObject(tile[j][i]);            
                    }

                else if(tile[j][i]->tileType == 0)
                {
                    tile[j][i]->distanceFromStart = INT_MAX;
                    tile[j][i]->parent = NULL;

                    vector<Tile*> tempTiles;
---->                   if(tile[j+1][i]->tileType == 0)
                        tempTiles.push_back(tile[j+1][i]);
                    if(tile[j-1][i]->tileType == 0)
                        tempTiles.push_back(tile[j-1][i]);
                    if(tile[j][i+1]->tileType == 0)
                        tempTiles.push_back(tile[j][i+1]);
                    if(tile[j][i-1]->tileType == 0)
                        tempTiles.push_back(tile[j][i-1]);
                    tile[j][i]->setSuccessors(tempTiles);
                    tilesList->push_back(tile[j][i]);
                }
            }
        }
    }

vector<Tile*>* Background::GetWayPointList()
{
    return tilesList;
}

这并没有真正解决,而是将错误向下移动,我将箭头移到调用堆栈最新调用的位置。

4

1 回答 1

4

所以你的tile成员被定义为Tile* tile[32][32]。这是一个大小为 32x32 的二维数组,其中每个元素都是指向Tile. 但是,它们只是指针,目前并没有特别指向任何地方 - 访问它们是未定义的行为(在您的情况下,访问冲突)。您需要分配实际Tile对象:

for (int i = 0; i < 32; i++) {
  for (int j = 0; j < 32; j++) {
    tile[i][j] = new Tile();
  }
}

不要忘记在Backgroundthat的析构函数中执行相同类型的循环delete tile[i][j];

更好的是,不要动态分配您的图块,只需定义tile为:

Tile tile[32][32];

更好的是(!),使用std::array<std::array<Tile,32>,32>. :)


要回答您的编辑,即使它使我的上述答案无效:您正在j+1访问tile. 什么时候j是 31,您正在访问tile[32]数组范围之外的内容。

于 2012-12-02T14:30:07.343 回答