Question

我有一个相当大的程序，通常运行得非常好，但是使用疯狂的内存来运行。这是一种收集大量数据的机器学习方法，因此通常没问题，但即使收集了所有数据，内存也会快速增长和增长，因此我使用valgrind massif来找出问题所在。地块顶部的堆树看起来像这样：

99.52% (60,066,179B) (heap allocation functions) malloc/new/new[], --alloc-fns, etc.
->43.50% (26,256,000B) 0x439785: Image::Image(SDL_Surface*) (Image.cpp:95)
| ->43.50% (26,256,000B) 0x437277: EncodedFeature::forwardPass() (EncodedFeature.cpp:65)
....

所以我想，嗯，也许构建的图像不是免费的，但没有：

void EncodedFeature::forwardPass()
{
    // Get image:
    Image* img = new Image(screen);

    // Preprocess:
    if(preprocessor)
        preprocessor->process(img);

    // Do forward pass:
    encoder->encode(img, features);

    delete img;
}

所以对Image构造函数：

Image::Image(SDL_Surface* surface)
{
    this->width = surface->w;
    this->height = surface->h;
    pixels = new int*[width];
    for(int i = 0; i < width; i++)
        pixels[i] = new int[height];

    for(int x = 0; x < surface->w; x++)
        for(int y = 0; y < surface->h; y++)
            pixels[x][y] = getPixelFromSDLSurface(surface, x, y);
}

稍后分配一个在析构函数中释放的像素数组：

Image::~Image()
{
    if(!pixels)
        return;

    for(int x = 0 ; x < width; x++)
        delete[] pixels[x];

    delete[] pixels;
}

所以最后一个罪魁祸首：

Uint32 Image::getPixelFromSDLSurface(SDL_Surface *surface, int x, int y)
{
    if(!surface)
        return 0;

    // Got this method from http://www.libsdl.org/cgi/docwiki.fcg/Pixel_Access
    int bpp = surface->format->BytesPerPixel;
    /* Here p is the address to the pixel we want to retrieve */
    Uint8 *p = (Uint8 *)surface->pixels + y * surface->pitch + x * bpp;

    switch(bpp) {
    case 1:
        return *p;
        break;

    case 2:
        return *(Uint16 *)p;
        break;

    case 3:
        if(SDL_BYTEORDER == SDL_BIG_ENDIAN)
            return p[0] << 16 | p[1] << 8 | p[2];
        else
            return p[0] | p[1] << 8 | p[2] << 16;
        break;

    case 4:
        return *(Uint32 *)p;
        break;

    default:
        return 0;       /* shouldn't happen, but avoids warnings */
    }
}

正如评论中所提到的，我从SDL wiki那里得到了那个，所以我希望那里没有任何漏洞。在我的情况下，bpp实际上总是1，所以它只返回地址p处的int，这听起来不会泄漏给我。

我在这里结束了我的智慧。谁能想到记忆的去向？我的意思是，massif专门针对Image构造函数，但我看不出有什么问题......

非常感谢您查看我的问题！

最高

回答您的意见：

你是对的，我不需要img作为指针。我来自Java背景所以我只想要一切都成为指针:)改变它但没有帮助。

第95行位于构造函数的第一个for循环内： pixels[i] = new int[height];

在其中一个预处理器中我做调整图像大小，但我这样做调用我的reset函数，这应该确保删除旧数组：

void Image::reset(int width, int height)
{
    if(pixels)
    {
        // Delete old image:
        for(int x = 0 ; x < width; x++)
            delete[] pixels[x];

        delete[] pixels;
    }

    this->width = width;
    this->height = height;
    pixels = new int*[width];
    for(int i = 0; i < width; i++)
        pixels[i] = new int[height];
}

之后我重新填充像素值......

没有任何例外被抛出。

你建议我在哪里使用智能指针？

感谢所有答案！

Answer 1

我认为您在图像类中没有正确表示像素。我想你可以使用正确的无符号类型（vector？）的简单1维uint32_t。

（标题中）：

class Image {
protected:
    std::vector<uint32_t> pixels;
    ...
};

（在实施文件中）

size_t Image::offset(unsigned x, unsigned y) {
    return (y * width) + x;
}

Image::Image(const SDL_Surface* surface)
{
    width = surface->w;
    height = surface->h;
    pixels.reserve(width * height);
    for(unsigned x = 0; x < width; x++)
        for(unsigned y = 0; y < height; y++)
            pixels[offset(x, y)] = getPixelFromSDLSurface(surface, x, y);
}

你的析构函数将完全为空，因为没有任何东西可以做：

Image::~Image() {
}

请注意，对于任何给定的x / y对，您需要使用offset()方法为vector获取正确的索引。

Answer 2

最可能的情况是，某些代码路径中没有调用您的重置或析构函数，从而泄漏了该内存。

幸运的是，C ++具有防止此类泄漏的内置功能：它被称为vector。

首先，让pixels看起来像这样：

typedef std::vector<int> PixelCol;
typedef std::vector<PixelCol> Pixels;
Pixels pixels;

现在我们在构造函数中调整它的大小：

Image::Image(SDL_Surface* surface)
: pixels(surface->w, PixelCol(surface->h))
{
    this->width = surface->w;
    this->height = surface->h;

    for(int x = 0; x < surface->w; x++)
        for(int y = 0; y < surface->h; y++)
            pixels[x][y] = getPixelFromSDLSurface(surface, x, y);
}

最后我们更新了reset：

void Image::reset(int width, int height)
{
    Pixels(width, PixelCol(height)).swap(pixels);
}

通过让语言管理你的记忆，你完全消除了任何这样的内存管理错误。[/ p>

我的记忆不是免费的

2 个答案: