SFML自定义Game类无法正常关闭问题求助

从你给出的代码片段来看，窗口关闭功能失效主要有两个核心问题：事件处理逻辑缺失和**close()方法未完成**。我来帮你修复这两个问题，让窗口能正常关闭。

问题分析 #

1. 你的pollEvent()方法只是简单返回窗口的事件查询结果，但没有处理关键的Event::Closed事件（也就是用户点击窗口关闭按钮的事件）；
2. close()方法代码不完整，没有正确执行窗口关闭的逻辑，也没有同步更新isOpen状态。

修复后的完整代码 #

首先补充Game.h的必要声明（因为你只提供了cpp文件）：

#ifndef GAME_H
#define GAME_H
#include <SFML/Graphics.hpp>
#include <string>

class Game {
private:
    float length;
    float height;
    sf::RenderWindow* window;
    bool isOpen;
    sf::Event e; // 确保事件对象是类成员

public:
    Game(float length, float height, std::string title);
    bool pollEvent();
    void close();
    void display();
    bool getIsOpen() const; // 用于主循环判断窗口状态
};

#endif

然后修改Game.cpp的实现：

#include "Game.h"
#include <SFML/Audio.hpp>
#include <SFML/Graphics.hpp>
#include <SFML/Graphics/Text.hpp>
using namespace sf;

Game::Game(float length, float height, std::string title) {
    this->length = length;
    this->height = height;
    this->window = new RenderWindow(VideoMode(this->length, this->height), title);
    this->isOpen = true;
}

bool Game::pollEvent() {
    if (window->pollEvent(e)) {
        // 处理关闭事件
        if (e.type == Event::Closed) {
            this->close();
        }
        return true;
    }
    return false;
}

void Game::close() {
    // 先关闭SFML窗口
    if (window->isOpen()) {
        window->close();
    }
    // 更新状态标记，让主循环能正确退出
    this->isOpen = false;
    // 注意：如果是用new创建的window，记得在程序结束时手动delete，避免内存泄漏
}

void Game::display() {
    window->display();
}

bool Game::getIsOpen() const {
    return this->isOpen;
}

使用示例（主函数） #

你需要在主循环中持续检查事件和窗口状态：

int main() {
    Game game(800, 600, "My Game");
    
    while (game.getIsOpen()) {
        // 处理所有待处理事件
        while (game.pollEvent()) {
            // 这里可以扩展添加其他事件处理逻辑，比如键盘、鼠标输入
        }
        
        // 游戏渲染流程
        game.window->clear(Color::White);
        // ... 绘制你的游戏元素 ...
        game.display();
    }
    
    // 释放window的内存，避免泄漏
    delete game.window;
    return 0;
}

关键修改点说明 #

• 在pollEvent()中新增了Event::Closed事件的处理，当用户点击窗口关闭按钮时自动触发关闭逻辑；
• 补全了close()方法，先调用SFML窗口自身的close()方法，再同步更新isOpen状态，确保主循环能正确退出；
• 添加了getIsOpen()方法，让外部主循环可以判断窗口是否还处于开启状态；
• 额外提醒内存管理：因为window是用new创建的，记得在程序结束时手动delete，避免内存泄漏。

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