使用_getch()的循环未按预期结束，线程中断场景下如何处理？

咱们先说说你代码里的核心问题，再给你针对性的解决方案：

1. 循环逻辑搞反了，还被阻塞输入坑了 #

你写的while (done || ((c = _getch()) != EOF))完全搞反了逻辑——这个条件的意思是只要done为true，或者用户按下了按键，就进入循环体。但你真正想要的是：等待到操作完成（done变true），或者用户按按键中断，然后结束等待。

更致命的是，_getch()是个阻塞函数：当操作还没完成（done是false）时，主线程会死死卡在这个函数里等按键，哪怕另一个线程已经把done改成true了，主线程也看不到，因为它根本没机会检查done的值！

2. done变量没同步，主线程可能看不到它的变化 #

如果done是普通的bool，编译器可能会把它的值缓存到寄存器里，另一个线程对done的修改，主线程可能根本感知不到。必须用线程安全的类型来保证可见性。

这里给你两种可行的方案，根据你的需求选：

方案一：非阻塞按键检查（简单易实现） #

这个方案用_kbhit()先检查有没有按键，避免主线程被卡死，同时用原子变量保证done的可见性：

#include <thread>
#include <conio.h> // 要包含这个头文件用_kbhit和_getch
#include <atomic>
#include <chrono>

std::atomic<bool> done = false; // 原子变量，保证多线程间可见
std::promise<void> exit_signal;

void Start(const std::string& folder_path) {
    // 模拟你的长时间操作
    for (int i = 0; i < 10; ++i) {
        // 每隔1秒检查一次是否要退出
        if (exit_signal.get_future().wait_for(std::chrono::seconds(1)) == std::future_status::ready) {
            done = true;
            return;
        }
    }
    done = true; // 操作正常完成
}

int main() {
    std::string folder_path = "./your_folder";
    std::thread t(Start, folder_path);
    char c;

    // 循环逻辑：只要操作没完成，就检查按键
    while (!done) {
        // 非阻塞检查是否有按键按下
        if (_kbhit()) {
            c = _getch();
            std::cout << "Operation aborted" << std::endl;
            exit_signal.set_value(); // 通知工作线程退出
            break;
        }
        // 短暂休眠，避免CPU空转
        std::this_thread::sleep_for(std::chrono::milliseconds(100));
    }

    t.join();
    std::cout << "Operation finished" << std::endl;
    return 0;
}

关键改进：

• 用std::atomic<bool>代替普通bool，确保工作线程修改done后，主线程能立刻看到
• 循环条件改成while (!done)：操作没完成就继续等，完成了直接退出循环
• 用_kbhit()非阻塞检查按键，让主线程有机会定期查看done的值，不会被卡死

方案二：用条件变量实现高效等待（适合对CPU占用敏感的场景） #

如果不想让主线程频繁休眠占用CPU，可以用条件变量让主线程“睡”着，直到工作完成或者有按键输入：

#include <thread>
#include <conio.h>
#include <atomic>
#include <condition_variable>
#include <mutex>

std::atomic<bool> done = false;
std::promise<void> exit_signal;
std::condition_variable cv;
std::mutex mtx;

void Start(const std::string& folder_path) {
    // 模拟长时间操作
    for (int i = 0; i < 10; ++i) {
        if (exit_signal.get_future().wait_for(std::chrono::seconds(1)) == std::future_status::ready) {
            done = true;
            cv.notify_one(); // 通知主线程可以醒了
            return;
        }
    }
    done = true;
    cv.notify_one(); // 操作完成，通知主线程
}

int main() {
    std::string folder_path = "./your_folder";
    std::thread t(Start, folder_path);

    // 单独开个线程处理按键输入，避免阻塞主线程
    std::thread input_thread([&]() {
        _getch(); // 阻塞等按键
        std::cout << "Operation aborted" << std::endl;
        exit_signal.set_value();
        done = true;
        cv.notify_one(); // 通知主线程
    });

    // 主线程等待，直到done变为true
    std::unique_lock<std::mutex> lock(mtx);
    cv.wait(lock, []{ return done.load(); });

    // 等待所有线程结束
    input_thread.join();
    t.join();
    std::cout << "Operation finished" << std::endl;
    return 0;
}

这个方案里，主线程大部分时间在休眠，只有当工作完成或者有按键时才会被唤醒，CPU占用极低。

• 循环逻辑完全搞反：你需要的是“当done为false时等待”，但你写的是“当done为true时进入循环”
• 阻塞的_getch()让主线程失去了检查done的机会，哪怕done已经变true，主线程还在等按键
• 普通bool没有同步机制，主线程可能看不到工作线程对done的修改

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