处理曲面向量时如何正确使用 SDL_FreeSurface

Question

我在 SDL 中设置了一个小型射击游戏作为自己的教程。我有一个弹丸结构

struct projectile
{
    SDL_Surface* surface;
    int x;
    int y;
};

，我把它放入一个向量中。

vector<projectile> shot;
projectile one_shot;

当我按下空格键时，我会创建一个新的射弹并将其添加到矢量中，然后在渲染时将其传输。

这工作正常，但我在看似随机的情况下收到“程序已停止工作”错误。

所以我想知道释放表面的正确方法是什么。

• 之后我要把它们全部释放吗？
• 当每个镜头退出屏幕时，我是否可以释放它？
• 或者还有其他选择？

更新：

我发现当我退出时，当我开了几枪并且它们都退出屏幕时，它崩溃了。我尝试用“正确的复制方式”替换将表面添加到向量的代码，如

这就是我释放表面的方法。

if(shot.at(i).y < 0 - shot.at(i).surface->h)
{
    SDL_FreeSurface(shot.at(i).surface);
    shot.erase(shot.begin() + i);
}

任何人都有一个想法或一些示例代码，我可以查看以解决这个问题。

Answer 1

如果多个射弹使用相同的精灵（就像在几乎所有基于精灵的游戏中一样），那么最好使用包含游戏使用的所有图像的图像缓存，并仅在那里进行内存管理。在启动时或按需填充，并在退出时冲洗。然后射弹只需要向该缓存请求一个指向“arrow.png”的指针，缓存就会加载它（如果需要）并返回表面指针。

这样的缓存可以是一个简单的 std::map<字符串，SDL___Surface* >仅具有 get_surface(string) 和flush() 等函数。

编辑：这个想法的实现：

class image_cache{
    map<string, SDL_Surface*> cache_;
    public:
    SDL_Surface* get_image(string file){
        map<string, SDL_Surface*>::iterator i = cache_.find(file);
        if(i == cache_.end()) {
            SDL_Surface* surf = SDL_LoadBMP(file.c_str());
            i = cache_.insert(i, make_pair(file, surf));
        }
        return i->second;
    } 
    void flush(){
        map<string, SDL_Surface*>::iterator i = cache_.begin();
        for(;i != cache_.end();++i)
            SDL_FreeSurface(i->second);
        cache_.clear();
    }
    ~image_cache() {flush();}
};

image_cache images;
// you can also use specialized caches instead of a global one
image_cache projectiles_images;

int main()
{
    ...
    SDL_Surface* surf = images.get_image("sprite.png");
    ...
}

Answer 2

当你摧毁射弹时，你应该释放表面。何时摧毁射弹是游戏设计的决定；最晚可能是在它离开屏幕时，当然也可能是在它击中目标时。

Answer 3

您在其他地方也使用相同的表面吗？因为如果是这样，只要它在其他地方使用，您就无法释放它。

如果您不这样做：在射弹的构造函数/析构函数中构建/加载表面并释放。 IE：

struct projectile {
    SDL_Surface* surface;

    projectile() : surface(NULL) {
        surface = LoadImage(...);
    }

    ~projectile() {
        if(surface) {
             SDL_FreeSurface(surface);
             surface = NULL;
        }
    }

};