游戏循环冻结与解冻异常：触发重启后陷入无限循环求助

需求背景 #

需要实现游戏循环的临时暂停与恢复功能，计划通过传递布尔变量指针控制冻结与解冻状态。游戏循环核心逻辑伪代码如下：

while(!quit)  
     if(!freeze):  
     [handle events()] -> [move()] ->[render()] (REPEAT)

现有C++实现代码 #

void game_over(int *flag);
void restart_game(int *flag);

void game_over(int *flag)  
{  
    *flag = 0;  
}  
void restart_game(int *flag)  
{  
    *flag = 1;  
}

int main()  
{  
    //freeze boolean flag initially false  
    int freeze = 0;  
    SDL_event e;  
    //game loop  
    int quit = 0;  
    while(!quit)  
    {  
        if(!freeze)  
            {  
                 while(SDL_PollEvent(&e) != 0)  
                 {  
                      switch(e.key.keysym.sym)  
                      {  
                           case SDLK_t:  
                               game_over(&freeze);//passing freeze address  
                               break;  
                           case SDLK_r:  
                               restart_game(&freeze);  
                               break;  
                      }  
                 }  
                 move();  
                 render();  
            }  
    }  
    close();  
    return 0;
}

问题原因分析 #

按下R键后程序卡顿甚至无法关闭，核心原因有两点：

• 函数功能与命名完全倒置：restart_game函数将freeze设为1（冻结状态），而game_over函数将freeze设为0（恢复运行），逻辑完全反了，导致按下R键后游戏直接进入冻结状态。
• 冻结状态下无事件处理与CPU控制：当freeze=1时，代码不会进入if(!freeze)分支，既不处理SDL事件（包括窗口关闭、按键输入等），也没有任何延迟逻辑。外层while(!quit)会无限空转，CPU占用率拉满导致卡顿，同时无法响应退出指令，程序无法正常关闭。

修复方案 #

1. 修正函数逻辑与命名 #

先把函数的功能和命名对应上，让代码逻辑清晰：

// 暂停游戏：设置freeze为1
void pause_game(int *flag)  
{  
    *flag = 1;  
}  
// 恢复游戏：设置freeze为0
void resume_game(int *flag)  
{  
    *flag = 0;  
}

同时修改主函数中的按键绑定：

switch(e.key.keysym.sym)  
{  
    case SDLK_t:
        pause_game(&freeze); // 按T键暂停
        break;  
    case SDLK_r:
        resume_game(&freeze); // 按R键恢复
        break;  
    case SDLK_ESCAPE: // 增加退出按键
        quit = 1;
        break;
}

2. 冻结状态下处理事件并控制CPU占用 #

把事件处理逻辑移到if(!freeze)分支外面，确保无论是否冻结都能处理事件；同时在冻结时添加短暂延迟，避免空循环占用CPU：

int main()  
{  
    int freeze = 0;  
    SDL_Event e;  
    int quit = 0;  
    while(!quit)  
    {  
        // 无论是否冻结，都处理所有事件
        while(SDL_PollEvent(&e) != 0)  
        {  
            if(e.type == SDL_QUIT) { // 处理窗口关闭事件
                quit = 1;
                break;
            }
            if(e.type == SDL_KEYDOWN) {
                switch(e.key.keysym.sym)  
                {  
                    case SDLK_t:
                        pause_game(&freeze);
                        break;  
                    case SDLK_r:
                        resume_game(&freeze);
                        break;
                    case SDLK_ESCAPE:
                        quit = 1;
                        break;
                }
            }
        }  

        if(!freeze)  
        {  
            move();  
            render();  
        } else {
            // 冻结时添加短延迟，降低CPU占用
            SDL_Delay(10);
        }
    }  
    close();  
    return 0;
}

更优实现思路 #

• 使用bool类型代替int：C++中用bool freeze = false;更符合布尔语义，代码可读性更高。
• 用引用代替指针：C++函数参数用引用比指针更简洁安全，比如void pause_game(bool &flag) { flag = true; }。
• 分离事件处理模块：把事件处理抽成单独函数，让游戏循环逻辑更清晰。
• 添加帧率控制：即使在运行状态，也应该添加帧率控制（比如固定60帧），避免不必要的CPU占用，例如在render()后调用SDL_Delay(16)（约60帧间隔）。

内容的提问来源于stack exchange，提问作者Mahdi Habibi