0

我正在设计一个非常简单的视频游戏来帮助学习使用 SDL。我遇到了碰撞检测问题。为了绘制游戏地图（使用图块），我在一个名为 的类中创建了一个Map指向名为 的类的指针的二维数组Tile。该类Map有一个名为的函数，该函数通过从文件中set_tiles读取它们来定义所有图块。.MAP我还有一个名为 的类Hero，其中包含有关英雄的所有函数和变量。一切都很琐碎。但是，当我尝试从其他任何地方读取数组时，它会产生奇怪的结果（通过错误测试，我发现 x 和 y 值相差数千，但我可能在那里犯了错误）。这是代码。

SDL_Surface* Map::set_tiles ( SDL_Surface* tile_image ) {

Setup setup;

//Make a temporary map to draw the tiles to
Uint32 rmask, gmask, bmask, amask;
if ( SDL_BYTEORDER == SDL_BIG_ENDIAN ) {
    rmask = 0x00000000;
    gmask = 0x00000000;
    bmask = 0x00000000;
    amask = 0x00000000;
}
else {
    rmask = 0x00000000;
    gmask = 0x00000000;
    bmask = 0x00000000;
    amask = 0x00000000;
}
SDL_Surface* temp_map = SDL_CreateRGBSurface(SDL_SWSURFACE, MAP_WIDTH, MAP_HEIGHT, 32, rmask, gmask, bmask, amask);

//Open the map
std::ifstream map ( "Test.map" );

//Catch any errors
if ( map.fail() ) return NULL;

//Initialize the tiles
for ( int y = 0; y < MAP_HEIGHT / TILE_HEIGHT; y++ ) {
    for ( int x = 0; x < MAP_WIDTH / TILE_WIDTH; x++ ) {
        //Determines the tile type
        int tile_type = -1;

        //Read the tile type from the map
        map >> tile_type;

        //Make sure it's a real tile
        if ( tile_type < 0 || tile_type >= TILE_SPRITES ) {
            map.close();
            return NULL;
        }

        //Error check for the .map file
        if ( map.fail() ) {
            map.close();
            return NULL;
        }

        //Add the tile to the array
        tile_array[x][y] = &Tile ( x, y, tile_type );

        //Create the temp. image crop
        SDL_Rect* temp_crop = &tile_array[x][y]->get_crop();

        //Edit the temp. map
        setup.apply_surface ( x * TILE_WIDTH, y * TILE_HEIGHT, tile_image, temp_map, temp_crop );
    }

}

map.close();

//Return the modified map
return temp_map;

}

现在，如果我尝试在其他地方阅读它，我就有问题了。IE

bool Hero::collision_check ( Map map ) {
for ( int y = 0; y < MAP_HEIGHT / TILE_HEIGHT; y++ ) {
    for ( int x = 0; x < MAP_WIDTH / TILE_WIDTH; x++ ) {
        Tile* tile = map.tile_array[x][y];
        if ( collision ( box, tile->get_box() ) ) {
            //Switch the effect based on the tile type
            switch ( tile->get_type() ) {
            case TILE_RED:
            case TILE_GREEN:
            case TILE_BLUE:
                return false;
                break;
            case TILE_CENTER:
            case TILE_TOP:
            case TILE_TOPRIGHT:
            case TILE_RIGHT:
            case TILE_BOTTOMRIGHT:
            case TILE_BOTTOM:
            case TILE_BOTTOMLEFT:
            case TILE_LEFT:
            case TILE_TOPLEFT:
                return true;
                break;
            default:
                return false;
            }
        }
    }
}
return false;
}

在指针（或大多数其他事情）方面，我不是大师。是否有任何明显的缺陷可以解释我的问题？

4

2 回答 2

1

这将是一个超出范围的临时指针：

 tile_array[x][y] = &Tile ( x, y, tile_type );

您可能正在寻找

 tile_array[x][y] = new Tile ( x, y, tile_type );

PS1：顶部的 if 语句似乎不是很有用

PS2：下次如果你发布一个最小的例子而不是你的所有代码会有所帮助——人们倾向于忽略有太多代码无法阅读的帖子

编辑：

你在做什么基本上是：

Foo* fp;
// a Foo() is created, its address assigned to fp,
// but it is then destroyed at the end of the statement
fp = &Foo(2);
// now fp is a pointer to unallocated memory and using it causes undefined behaviour
fp.val; // this value is now undefined
于 2013-02-20T21:30:35.660 回答
1

在行

tile_array[x][y] = &Tile ( x, y, tile_type );

您正在将临时地址分配给您的向量。一旦该表达式结束，临时对象就会被销毁，并且您保存的指针现在指向一个无效对象。我不会存储指向 Tile 对象的指针，而是将 tile 对象本身存储在tile_array.

于 2013-02-20T21:32:43.863 回答