学习 C++和 SDL- 以下是否会产生内存泄漏？

Question

我正在自学一些 C++，但我不完全确定我对内存管理有很好的掌握。我只懂 Java 以及一点 PHP 和 Python，所以这对我来说有点新鲜。我也在与 SDL 合作——这似乎是一种加速学习过程的有趣方式。不管怎样，我正在尝试编写一个清理函数来释放所有已传递到堆栈的表面（我只是使用堆栈STL）。所以，我有以下代码（缩写）：

#include <stack>

//stack of SDL_Surfaces  
stack<SDL_Surface*> surfaces;

void clean() {      
    SDL_Surface *temp = NULL;

    //loops through the stack depending on its size
    while (surfaces.size() != 0) {
        temp = surfaces.top();
        SDL_FreeSurface(temp);
        surfaces.pop();

    } //while
    if (surfaces.size() == 0) {
        cout << "cleanup worked correctly" << endl;     
    } //if
}

//loading an image (this is in the main function)
background = load_image( "background.bmp" );
surfaces.push(background);

//cleaning time
clean();

我不确定清理方法。我认为这将是实现 SDL_FreeSurface 函数的更好方法，而不是手动指定每个表面。因此，如果我在屏幕上绘制十个图像（例如十艘星舰）然后将它们炸毁，我需要正确删除它们。我会为这 10 艘星舰创建一个堆栈，然后在它们被摧毁时我可以将它们全部消灭，如果这有意义的话。我担心我把事情变得过于复杂并引入了一种全新的方法来导致内存泄漏。

任何反馈/评论将不胜感激！我是 C++ 新手，所以请随意嘲笑我在内存管理方面徒劳的尝试。

Answer 1

您自己的代码没有任何内存泄漏。假设 load_image() 也是安全的，则整个代码不会出现内存泄漏。

请记住，如果不使用 new 运算符，则不会发生内存泄漏。

Answer 2

这似乎没有任何内存泄漏。但是，您需要确保不要将表面保留太久，否则可能会出现泄漏（在 Java 中也可能发生同样的情况）。假设 clean 被调用得足够频繁，那么应该没问题。

这将清理各个表面（通过在适当的时间调用 SDL_FreeSurface）的问题转移到清理表面堆栈（通过在适当的时间调用 clean）。如果您已经有用于管理堆栈的代码（或想法），我会继续运行它。如果没有，我会尝试找到一种单独管理表面寿命的方法，因为这种方法更灵活并且不会引入新概念。

Answer 3

您正在做的事情有点令人困惑，但是，如果 SDL_FreeSurface() 释放对象（？？？），那么我没有看到内存泄漏。

你想做什么？ SDL_FreeSurface() 的作用是什么？ load_image() 是做什么的？如果有的话，它是如何分配内存的？

Answer 4

阅读RAII - C++ 非常擅长于此。一旦您了解发生了什么（基本上是构造函数中的资源分配和析构函数中的释放 - 意味着您的对象会自行清理），您就可以为 SDL 函数创建漂亮的包装器。以下是创建表面的方法：

CBitmapSurface::CBitmapSurface(const std::string &filename) 
{
    m_pSurface = SDL_LoadBMP(filename.c_str());
}

CBitmapSurface::~CBitmapSurface()
{
    SDL_FreeSurface(m_pSurface); 
}

其中 m_pSurface 是 SDL_Surface* 类型的成员变量。

在完美的世界中，您希望使此类不可复制，如果使用 c++0x 则处理移动构造函数并检查错误代码并引发异常。但这应该让你开始。

Answer 5

到目前为止，查找内存泄漏的最佳方法是使用 valgrind。据我所知，它只能在 Linux 上运行，但非常值得花时间研究和使用。