QT事件循环异常：快速点击QPushButton后循环无法终止问题排查

你的问题出在多次触发button_hold槽函数导致的共享变量竞争，以及手动循环+processEvents的设计缺陷上。

让我们拆解一下快速点击两次时发生的流程：

• 第一次点击按钮，pressed信号触发button_hold，函数进入while循环，此时key_is_released是false，循环持续执行。
• 在第一次循环的processEvents()调用时，快速点击第二次按钮，pressed信号再次触发button_hold，新的函数实例开始执行：它首先把key_is_released重置为false（因为上一次松开可能已经把它设为true了），然后又进入一个新的while循环。
• 当你松开按钮，button_released把key_is_released设为true，此时第二个button_hold的循环会退出，然后执行key_is_released = false;——这就把标志位又改回了false！
• 第一个button_hold的循环检测到!key_is_released为true，会继续无限循环下去，永远停不下来。

另外，你的代码还有个问题：手动调用QThread::msleep()会阻塞主线程，虽然用了processEvents()让事件能处理，但这种方式很容易引发状态混乱，不是Qt推荐的做法。

方案1：修复原代码的标志位竞争问题 #

添加一个额外的标志位is_holding_in_progress，确保同一时间只有一个button_hold循环在执行：

// 在类中添加成员变量
bool is_holding_in_progress = false;
bool key_is_released = false;
int i = 0;

void My_class::button_hold(){ 
    if (is_holding_in_progress) {
        // 如果已经有循环在执行，直接返回，避免重复触发
        return;
    }
    is_holding_in_progress = true;
    key_is_released = false; // 重置标志位
    QThread::msleep(200);
    int wait_lock = 500;
    i++;
    cout<<i<<endl;
    QCoreApplication::processEvents(QEventLoop::AllEvents);
    while(!key_is_released){
        QThread::msleep(wait_lock);
        i++;
        cout<<i<<endl;
        if(wait_lock > 50) wait_lock -= 50;
        QCoreApplication::processEvents(QEventLoop::AllEvents);
    }
    key_is_released = false;
    is_holding_in_progress = false; // 释放标志位
}

void My_class::button_released(){
    key_is_released = true;
}

方案2：使用Qt定时器（更优雅、更Qt化的做法） #

放弃手动循环和sleep，改用QTimer来实现按住时递增速度加快的效果，这完全避免了主线程阻塞和状态竞争：

// 在类中添加成员变量
QTimer* holdTimer = nullptr;
int wait_lock = 500;
int i = 0;

// 构造函数中初始化定时器
My_class::My_class(QWidget *parent) : QWidget(parent) {
    // ... 其他初始化代码 ...
    holdTimer = new QTimer(this);
    connect(holdTimer, &QTimer::timeout, this, &My_class::onHoldTimeout);
    connect(ui->button, &QPushButton::pressed, this, &My_class::onButtonPressed);
    connect(ui->button, &QPushButton::released, this, &My_class::onButtonReleased);
}

void My_class::onButtonPressed() {
    // 第一次触发直接执行一次
    i++;
    cout<<i<<endl;
    // 初始化定时器间隔
    wait_lock = 500;
    holdTimer->start(wait_lock);
}

void My_class::onHoldTimeout() {
    i++;
    cout<<i<<endl;
    // 缩短间隔，最小到50ms
    if (wait_lock > 50) {
        wait_lock -= 50;
        holdTimer->setInterval(wait_lock);
    }
}

void My_class::onButtonReleased() {
    holdTimer->stop();
}

这种方式的优势：

• 不会阻塞主线程，UI始终保持响应
• 没有共享变量竞争的问题，定时器只有一个实例在运行
• 代码逻辑更清晰，符合Qt的事件驱动模型

内容的提问来源于stack exchange，提问作者Antonio Del Sannio