Утечка памяти в SDL

@Igor Fender · Регистрация: 09.07.2014

Студворк — интернет-сервис помощи студентам

Здравствуйте!

Для проверки завершения работы я использую условие на нулевую ссылку

C++
1
2
3
4
5
6
7
8
9
10
int Game::exec()
{
    while (m_pWindow != NULL) //Чтобы не создавать переменную bool isRunning
    {
        processEvents();
        update();
        render();
    }
    return 0;
}

В функции обработки событий при завершении работы я вызываю функцию quit

C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
void Game::processEvents()
{
    SDL_Event event;
    while (SDL_PollEvent(&event))
    {
        switch (event.type) {
            case SDL_QUIT:
                quit();
                break;
            default:
                break;
        }
    }
}

А в функции quit я завершаю все процессы и зануляю переменные рендера и окна, чтобы в функции exec выйти из главного цикла.

C++
1
2
3
4
5
6
7
8
9
void Game::quit()
{
    SDL_DestroyWindow(m_pWindow);
    SDL_DestroyRenderer(m_pRenderer);
    
    m_pWindow = NULL;   // ???
    m_pRenderer = NULL; // ???
    SDL_Quit();
}

Вопрос: Нужно ли самостоятельно очищать память m_pWindow и m_pRenderer или функции SDL сами все делают?? Можно ли занулять переменные?
Просто когда я не занулял их, цикл не завершался, что свидетельствует о том, что в переменных что-то лежит даже после
SDL_DestroyWindow(m_pWindow);
SDL_DestroyRenderer(m_pRenderer);

@smartpointer · 18.05.2015, 22:59

Наверное вы имели ввиду нулевой указатель, и проверять на нуль m_pWindow не есть хорошо, поскольку освобождением памяти занимается SDL_DestroyWindow() и вы не можете знать будет ли указателю присвоен ноль, в итоге получаем бесконечный цикл.

@Igor Fender · 18.05.2015, 23:24 **[ТС]**

Сообщение от smartpointer

поскольку освобождением памяти занимается SDL_DestroyWindow() и вы не можете знать будет ли указателю присвоен ноль, в итоге получаем бесконечный цикл.

Так я же самостоятельно присваиваю переменной NULL после вызова SDL_DestroyWindow. Как только вызывается функция quit, моя переменная становится NULL и программа выходит из главного цикла. Почему зацикливание?

@smartpointer · 19.05.2015, 00:11

Все нормально, только обратите внимание что у вас после обработки эвентов после которого вызывается update() и render() сам m_pRenderer уже перестанет существовать - то есть в функции render() будет обращение к нулевому указателю и только после этого завершается цикл exec(), как-то так

@Igor Fender · 19.05.2015, 00:17 **[ТС]**

smartpointer, Ладно, а если просто processEvent переместить в самый конец??

@smartpointer · 19.05.2015, 00:29

Нет, это очередь эвентов самой библиотеки, где обрабатывается input и прочее - по идее она стоит в правильном месте, лучше в функции render() проверять на null m_pRenderer то есть как-то так

C++
1
2
3
4
5
6
7
void Game::render()
{
   if(m_pRenderer)
   {
      //рисуем кадр
   }
}

Либо вынести уничтожение окна и рендера сразу за циклом, но тогда нужен будет bool для остановки цикла

@Igor Fender · 19.05.2015, 00:43 **[ТС]**

smartpointer, Все-таки лучшее, что можно использовать - это внешнюю переменную bool для остановки цикла

Новые блоги и статьи Все статьи Все блоги /
Модель микоризы: классовый агентный подход anaschu 02.01.2026 Раньше это было два гриба и бактерия. Теперь три гриба, растение. И на уровне агентов добавится между грибами или бактериями взаимодействий. До того я пробовал подход через многомерные массивы,. . .	Учёным и волонтёрам проекта «Einstein@home» удалось обнаружить четыре гамма-лучевых пульсара в джете Млечного Пути Programma_Boinc 01.01.2026 Учёным и волонтёрам проекта «Einstein@home» удалось обнаружить четыре гамма-лучевых пульсара в джете Млечного Пути Сочетание глобально распределённой вычислительной мощности и инновационных. . .	Советы по крайней бережливости. Внимание, это ОЧЕНЬ длинный пост. Programma_Boinc 28.12.2025 Советы по крайней бережливости. Внимание, это ОЧЕНЬ длинный пост. Налог на собак: https:/ / ********/ gallery/ V06K53e Финансовый отчет в Excel: https:/ / ********/ gallery/ bKBkQFf Пост отсюда. . .	Кто-нибудь знает, где можно бесплатно получить настольный компьютер или ноутбук? США. Programma_Boinc 26.12.2025 Нашел на реддите интересную статью под названием Anyone know where to get a free Desktop or Laptop? Ниже её машинный перевод. После долгих разбирательств я наконец-то вернула себе. . .	Thinkpad X220 Tablet — это лучший бюджетный ноутбук для учёбы, точка. Programma_Boinc 23.12.2025 Рецензия / Мнение/ Перевод Нашел на реддите интересную статью под названием The Thinkpad X220 Tablet is the best budget school laptop period . Ниже её машинный перевод. Thinkpad X220 Tablet —. . .
PhpStorm 2025.3: WSL Terminal всегда стартует в ~ and_y87 14.12.2025 PhpStorm 2025. 3: WSL Terminal всегда стартует в ~ (home), игнорируя директорию проекта Симптом: После обновления до PhpStorm 2025. 3 встроенный терминал WSL открывается в домашней директории. . .	Как объединить две одинаковые БД Access с разными данными VikBal 11.12.2025 Помогите пожалуйста !! Как объединить 2 одинаковые БД Access с разными данными.	Новый ноутбук volvo 07.12.2025 Всем привет. По скидке в "черную пятницу" взял себе новый ноутбук Lenovo ThinkBook 16 G7 на Амазоне: Ryzen 5 7533HS 64 Gb DDR5 1Tb NVMe 16" Full HD Display Win11 Pro	Музыка, написанная Искусственным Интеллектом volvo 04.12.2025 Всем привет. Некоторое время назад меня заинтересовало, что уже умеет ИИ в плане написания музыки для песен, и, собственно, исполнения этих самых песен. Стихов у нас много, уже вышли 4 книги, еще 3. . .	От async/await к виртуальным потокам в Python IndentationError 23.11.2025 Армин Ронахер поставил под сомнение async/ await. Создатель Flask заявляет: цветные функции - провал, виртуальные потоки - решение. Не threading-динозавры, а новое поколение лёгких потоков. Откат?. . .

@smartpointer 70 / 64 / 40 Регистрация: 17.02.2014 Сообщений: 265
	18.05.2015, 22:59
	Наверное вы имели ввиду нулевой указатель, и проверять на нуль m_pWindow не есть хорошо, поскольку освобождением памяти занимается SDL_DestroyWindow() и вы не можете знать будет ли указателю присвоен ноль, в итоге получаем бесконечный цикл. 0

@smartpointer 70 / 64 / 40 Регистрация: 17.02.2014 Сообщений: 265
	19.05.2015, 00:11
	Все нормально, только обратите внимание что у вас после обработки эвентов после которого вызывается update() и render() сам m_pRenderer уже перестанет существовать - то есть в функции render() будет обращение к нулевому указателю и только после этого завершается цикл exec(), как-то так 0

@Igor Fender 1 / 1 / 0 Регистрация: 09.07.2014 Сообщений: 167
	19.05.2015, 00:17 [ТС]
	smartpointer, Ладно, а если просто processEvent переместить в самый конец?? 0

@Igor Fender 1 / 1 / 0 Регистрация: 09.07.2014 Сообщений: 167
	19.05.2015, 00:43 [ТС]
	smartpointer, Все-таки лучшее, что можно использовать - это внешнюю переменную bool для остановки цикла 0