Я изучаю SDL с помощью онлайн-учебника и обнаружил проблему, которую не понимаю.
В соответствии с руководством рекомендуется всегда правильно освобождать ресурсы, используемые вашим приложением. В моем случае я хочу освободить память, используемую поверхностями SDL.
Однако, один SDL_FreeSurface вылетает из моей программы на выходе, если я не установлю указатель, который будет освобожден до nullptr ДО вызова функции.
Этого не происходит с любыми другими поверхностями. Даже решая проблему путем изменения порядка утверждений, мне не нравится «культ груза». Я хочу понять, ПОЧЕМУ в данном конкретном случае мне нужно сделать указатель nullptr ДО освобождения памяти.
Это код:
#define SDL_MAIN_HANDLED
#include<SDL2/SDL.h>
int main (void) {SDL_SetMainReady();
SDL_Init(SDL_INIT_VIDEO);
enum image_list{ /* set enum */ };
SDL_Window * ptr_Window = nullptr;
SDL_Surface * ptr_Canvas = nullptr;
SDL_Surface * ptr_Mask = nullptr; // this is the pointer with problem
SDL_Surface * array_list_of_images[enum_Total]; /* each array initialized correctly */
ptr_Mask = array_list_of_images[enum_Image_Default]; // here the problematic pointer is set to array[0]
/* code */
// Here is where the pointer is used:
switch (union_events_holder.key.keysym.sym){
case SDLK_UP : ptr_Mask = array_list_of_images[enum_Image_Up]; break;
case SDLK_DOWN : ptr_Mask = array_list_of_images[enum_Image_Down]; break;
case SDLK_LEFT : ptr_Mask = array_list_of_images[enum_Image_Left]; break;
case SDLK_RIGHT : ptr_Mask = array_list_of_images[enum_Image_Right]; break;
default : ptr_Mask = array_list_of_images[enum_Image_Default]; break;
/* more code */
SDL_BlitSurface(ptr_Mask , NULL , ptr_Canvas , NULL );
SDL_UpdateWindowSurface(ptr_Window);
/* more code */
// HERE OCCURS THE PROBLEM.
// WITHOUT POINTING ptr_Mask TO NULL BEFORE CALLING SDL_FreeSurface,
// THE APP CRASHES.
ptr_Mask = nullptr;
SDL_FreeSurface(ptr_Mask); // This cannot be called before previous statement. Why?
for ( int ctd = 0 ; ctd < enum_Total ; ++ctd){
SDL_FreeSurface(array_list_of_images[ctd]);
array_list_of_images[ctd] = nullptr; // Here it is safe to set pointer to nullptr AFTER freeing memory.
}
SDL_FreeSurface(ptr_Canvas);
SDL_DestroyWindow(ptr_Window);
ptr_Canvas = nullptr; // Here also it is safe to set pointer to nullptr AFTER freeing memory.
ptr_Window = nullptr;
SDL_Quit();
return (0);}
Примечания:
Помечен как C, потому что SDL. Я кодирую с C ++.
Найдена ошибка, пошаговая инструкция по шагам с помощью std::cerr + std::cin.get()
SDL_GetError() не может вывести информацию, потому что окно и консоль аварийно завершают работу при вызове функции.
Я подозревал о неопределенном поведении разыменования nullptr, но это не имеет смысла, если функция работает точно только при установке на nullptr.
РЕДАКТИРОВАТЬ - Это полный код (переменные на бразильском португальском языке)
#define SDL_MAIN_HANDLED
#include<SDL2/SDL.h>
int main (void) {SDL_SetMainReady();
SDL_Init(SDL_INIT_VIDEO);
SDL_Window * ptr_Janela = nullptr;
SDL_Surface * ptr_Tela = nullptr;
enum lista_de_imagens{
LIMG_Imagem_Default,
LIMG_Imagem_Cima,
LIMG_Imagem_Baixo,
LIMG_Imagem_Esquerda,
LIMG_Imagem_Direita,
LIMG_Total_de_imagens
};
SDL_Surface * l_lista_de_imagens[LIMG_Total_de_imagens];
l_lista_de_imagens[LIMG_Imagem_Default] = SDL_LoadBMP("Lesson4/Default.bmp" );
l_lista_de_imagens[LIMG_Imagem_Cima] = SDL_LoadBMP("Lesson4/Cima.bmp" );
l_lista_de_imagens[LIMG_Imagem_Baixo] = SDL_LoadBMP("Lesson4/Baixo.bmp" );
l_lista_de_imagens[LIMG_Imagem_Esquerda] = SDL_LoadBMP("Lesson4/Esquerda.bmp");
l_lista_de_imagens[LIMG_Imagem_Direita] = SDL_LoadBMP("Lesson4/Direita.bmp" );
l_lista_de_imagens[LIMG_Total_de_imagens ]= nullptr;
SDL_Surface * ptr_mascara = nullptr;
ptr_Janela = SDL_CreateWindow("Lesson 4 - Show image as keyboard inputs",
SDL_WINDOWPOS_UNDEFINED,
SDL_WINDOWPOS_UNDEFINED,
640, //SCREEN_WIDTH,
480, //SCREEN_HEIGHT,
SDL_WINDOW_SHOWN);
ptr_Tela = SDL_GetWindowSurface(ptr_Janela);
SDL_BlitSurface(l_lista_de_imagens[LIMG_Imagem_Default] , NULL , ptr_Tela , NULL );
SDL_UpdateWindowSurface(ptr_Janela);
SDL_Event u_Gerenciador_de_eventos;
bool sair = false;
ptr_mascara = l_lista_de_imagens[LIMG_Imagem_Default];
while(!sair){
while(SDL_PollEvent(&u_Gerenciador_de_eventos) !=0){
if(u_Gerenciador_de_eventos.type == SDL_QUIT){sair = true;}
if( u_Gerenciador_de_eventos.type == SDL_KEYDOWN){
switch (u_Gerenciador_de_eventos.key.keysym.sym){
case SDLK_UP : ptr_mascara = l_lista_de_imagens[LIMG_Imagem_Cima]; break;
case SDLK_DOWN : ptr_mascara = l_lista_de_imagens[LIMG_Imagem_Baixo]; break;
case SDLK_LEFT : ptr_mascara = l_lista_de_imagens[LIMG_Imagem_Esquerda]; break;
case SDLK_RIGHT : ptr_mascara = l_lista_de_imagens[LIMG_Imagem_Direita]; break;
default : ptr_mascara = l_lista_de_imagens[LIMG_Imagem_Default]; break;
/*end switch*/}
/*end if*/}
/*end laço de eventos*/}
SDL_BlitSurface(ptr_mascara , NULL , ptr_Tela , NULL );
SDL_UpdateWindowSurface(ptr_Janela);
/*end laço principal*/}
ptr_mascara = nullptr;
SDL_FreeSurface(ptr_mascara);
for ( int ctd = 0 ; ctd < LIMG_Total_de_imagens ; ++ctd){
SDL_FreeSurface(l_lista_de_imagens[ctd]);
l_lista_de_imagens[ctd] = nullptr;
/*end for*/}
SDL_FreeSurface(ptr_Tela);
SDL_DestroyWindow(ptr_Janela);
ptr_Tela = nullptr;
ptr_Janela = nullptr;
SDL_Quit();
return (0);
/*end main*/}