Question

我做了一个程序在屏幕上移动一个点。我基本上只是重新创建lazyfoo sdl教程。问题是我的程序吃了内存。它的内存使用率以200kb / s的速率缓慢增加。它会加班加点。

我使用视觉检漏仪测试它，它说没有内存泄漏。那我做错了什么？请帮我。它让我发疯了。

这里是代码：lazyfoo的信用。如果代码太长或不清楚，我会根据需要进行编辑。谢谢。

//Main Stage
int main(int argc, char* args[])
{
    if (!init())
    {
        printf("failed to init! \n");
    }
    else
    {
        cTile *tileSet[TOTAL_TILES];

        if (!loadMedia(tileSet))
        {
            printf("failed to load media! \n");
        }
        else
        {
            bool running = true;
            SDL_Rect wall;
            wall.x = 200;
            wall.y = 200;
            wall.h = 150;
            wall.w = 70;
            cDot Dot1(1270, 720);
            cDot Dot2(SCREEN_HEIGHT / 4, SCREEN_WIDTH / 4);
            cTimer timer;
            SDL_Event e;
            SDL_Color textColor = { 0, 0, 120, 120 };

            std::vector<SDL_Rect> camera;
            camera.resize(1);
            camera[0] = { 0, 0, SCREEN_WIDTH, SCREEN_HEIGHT };

            Uint32 startTime = 0;
            std::stringstream timeText;
            int countedFrames = 0;
            timer.start();

            while (running)
            {
                while (SDL_PollEvent(&e) != 0)
                {
                    if (e.type == SDL_QUIT)
                    {
                        running = false;
                    }

                    Dot1.handleEvent(e);

                    for (int i = 0; i < TOTAL_BUTTONS; i++)
                    {
                        buttons[i].buttonEvent(&e);
                    }
                }

                float avgFPS = countedFrames / (timer.getTicks() / 1000.f);
                if (avgFPS > 2000)
                {
                    avgFPS = 0;
                }

                if (!textTexture.loadTextFromFile(timeText.str().c_str(), textColor))
                {
                    printf("unable to render text ypoooooo");
                }

                timeText.str("");
                timeText << "fps " << avgFPS;

                Dot1.move(tileSet);
                Dot1.setCamera(camera[0]);

                /*camera.x = (Dot1.getPosX() + cDot::DOT_WIDTH / 2) - SCREEN_WIDTH / 2;
                camera.y = (Dot1.getPosY() + cDot::DOT_HEIGHT / 2) - SCREEN_HEIGHT / 2;*/

                /*if (camera.x < 0){ camera.x = 0; }
                if (camera.y < 0){ camera.y = 0; }
                if (camera.x > LEVEL_WIDTH - camera.w){ camera.x = LEVEL_WIDTH - camera.w; }
                if (camera.y > LEVEL_HEIGHT - camera.h){ camera.y = LEVEL_HEIGHT - camera.h; }*/

                SDL_SetRenderDrawColor(renderer, 0xFF, 0xFF, 0xFF, 0xFF);
                SDL_RenderClear(renderer);

                //SDL_SetRenderDrawColor(renderer, 0x00, 0x00, 0x00, 0xFF);
                //SDL_RenderDrawRect(renderer, &wall);
                for (int i = 0; i < TOTAL_TILES; ++i)
                {
                    tileSet[i]->render(camera);
                }
                //BGTexture.render(0, 0, &camera);
                Dot1.render(camera[0]);

                textTexture.render(250, 250);
                timeTextTexture.render(250, 250 + textTexture.getWidth());
                for (int i = 0; i < 2; i++)
                {
                    //buttons[i].render();
                }


                //Dot2.render(camera.x, camera.y);

                SDL_RenderPresent(renderer);
                if (timer.started())
                {
                    countedFrames++;
                }
                else if (!timer.started())
                {
                    countedFrames = 0;
                }
            }
        }
        close(tileSet);
    }
    return 0;
}

编辑：

解决方案非常简单，感谢保罗和天气！显然我把这些代码行放在一个循环中。其主要功能基本上是创建纹理。并且它一遍又一遍地加载纹理，因为它在循环内部。我只需要删除它。这些行：

if (!textTexture.loadTextFromFile(timeText.str().c_str(), textColor))
                    {
                        printf("unable to render text ypoooooo");
                    }

谢谢你！ :) 对不起，我不能投票，显然它需要更多的声誉来投票答复。：|

Answer 1

我怀疑这只会在NULL时删除。

for (int i = 0; i < TOTAL_TILES; ++i)
{
    if (tiles[i] == NULL)
    {
        delete tiles[i];
        tiles[i] = NULL;
    }
}

修改

以下是另一个镜头：您在主循环重复中调用loadTextFromFile()，但只能在最终SDL_DestroyTexture()中调用close()。

这是一个类似的问题，Memory leak SDL while using SDL_CreateTextureFromSurface

随着C ++程序的运行，内存使用量不断增加

1 个答案: