基于C++与SFML开发太空侵略者游戏 主菜单回车无法启动求助

现存核心错误 #

• Menu类语法错误：selectedItemIndex = 0; 代码位于Menu构造函数的花括号外部，落在全局作用域中，既不符合C++语法规则，也导致私有成员selectedItemIndex未被正确初始化，读取值为栈上随机垃圾值。
• 变量名笔误：main.cpp中创建GameObjectManager实例时，指针命名为gameObjectManagerManager，后续调用update()、render()时错写为未定义的gameObjectManager，该调用实际不会被执行。
• 缺失状态流转逻辑：这是游戏无法启动的根本原因。当前主循环固定执行清屏、绘制背景、绘制菜单的流程，按下回车触发的事件处理中，仅在单帧事件处理瞬间调用了一次游戏对象的更新与渲染，事件处理结束后，主循环立刻回到固定菜单绘制流程，那帧转瞬即逝的游戏画面直接被菜单完全覆盖，无法形成持续的游戏流程。
• 内存泄漏风险：每次按下回车都会new一个GameObjectManager对象，全程没有匹配的delete操作，会持续占用堆内存。
• 菜单功能缺失：当前代码没有实现上下切换菜单项的交互逻辑，选中项固定为第一个，无法切换到Exit选项。

正确实现流程 #

采用状态机逻辑拆分不同运行阶段，通过枚举值标记当前处于主菜单、游戏运行还是退出状态，主循环根据当前状态分支执行事件处理、逻辑更新、界面渲染，从结构上完成菜单到游戏的跳转。

代码修正要点 #

1. 引入游戏状态枚举，将GameObjectManager提前初始化为栈对象，避免在事件处理中动态申请堆内存

// 游戏状态标记
enum GameState {
    MainMenu,
    GameRunning,
    ExitGame
};
// 提前初始化游戏资源，避免事件触发时临时加载
GameState currentState = MainMenu;
GameObjectManager gameObjectManager(&window);

2. 主循环内根据当前状态分发逻辑，替换原有固定绘制菜单的写法

while (window.isOpen() && currentState != ExitGame)
{
    sf::Event event;
    while (window.pollEvent(event))
    {
        // 跨状态通用事件处理
        if (event.type == sf::Event::Closed)
            currentState = ExitGame;
        if (event.type == sf::Event::KeyPressed && event.key.code == sf::Keyboard::Escape)
            currentState = ExitGame;

        // 主菜单状态下的专属事件处理
        if (currentState == MainMenu)
        {
            if (event.type == sf::Event::KeyPressed && event.key.code == sf::Keyboard::Return)
            {
                int selectedItem = menu.GetPressedItem();
                if (selectedItem == 0) // 选中Play选项
                    currentState = GameRunning;
                if (selectedItem == 1) // 选中Exit选项
                    currentState = ExitGame;
            }
            // 可在此处补充上下按键切换菜单项的逻辑：修改selectedItemIndex，同步更新菜单文字的选中颜色
        }
    }

    // 按状态分发渲染逻辑
    window.clear();
    window.draw(galaxyBackground);
    if (currentState == MainMenu)
    {
        menu.draw(window);
    }
    else if (currentState == GameRunning)
    {
        gameObjectManager.update();
        gameObjectManager.render(window);
    }
    window.display();
}

3. 修正Menu.cpp的语法错误，将selectedItemIndex的初始化移入构造函数内部

Menu::Menu(float width, float height)
{
    if (!font.loadFromFile("arial.ttf"))
    {
        cout << "can't load font" << endl;
    }
    // 成员初始化移入构造函数内部
    selectedItemIndex = 0;

    // 原有菜单项初始化逻辑保持不变
    menu[0].setFont(font);
    menu[0].setColor(sf::Color::Red);
    menu[0].setString("Play");
    menu[0].setPosition(sf::Vector2f(width / 2, height / (MAX_NUMBER_OF_ITEMS + 1) * 1));

    menu[1].setFont(font);
    menu[1].setColor(sf::Color::White);
    menu[1].setString("Exit");
    menu[1].setPosition(sf::Vector2f(width / 2, height / (MAX_NUMBER_OF_ITEMS + 1) * 2));
}

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