C++ std::thread构造函数匹配错误排查求助

我来帮你拆解这些错误的根源，以及对应的修复方法：

核心错误原因 #

你遇到的std::thread构造不匹配问题，主要来自两个关键问题：

1. 错误的线程构造方式 #

你写的thread tme((timer()));是直接调用了timer()函数，而不是把timer作为线程要执行的任务传递给std::thread。而且timer函数需要一个int类型的参数hit，你这里调用时完全没传参数，这本身就会触发编译错误，更别说线程构造了。

std::thread的正确用法是：传入可调用对象（比如函数名/函数指针），然后跟上该对象需要的所有参数，格式为std::thread 线程名(函数名, 参数1, 参数2, ...);。

2. 函数参数不匹配 #

你的timer函数定义是void timer(int hit)，但你在创建线程时，既没有给timer传必要的hit参数，也没有按照线程构造的要求传递函数和参数。

修复后的代码示例 #

下面是修正后的代码，我标注了关键修改点：

#include <iostream>
#include <thread>
#include <string>
#include <cstdlib>
#include <windows.h> // 注意：Sleep是Windows平台特有的函数

// 建议尽量避免全局变量，这里先保留你的原有变量逻辑
int i = 0;
int s = 5; // 补充s的初始值，原代码未定义

void f1opp(int hit) {
    // 关键修改1：传递函数名timer，以及它需要的参数hit
    std::thread tme(timer, hit);
    
    std::string choi; // 补充choi的定义，原代码未声明
    std::getline(std::cin, choi);
    
    if (choi == "punch") {
        std::cout << "You try to punch him" << std::endl;
        int ran = rand() % 3; // 将ran定义为局部变量，避免全局污染
        if (ran == 1) {
            std::cout << "You hit him for 3 damage" << std::endl;
            i = 1 + 123456789;
        } else {
            std::cout << "You miss" << std::endl;
            Sleep(1000);
        }
    }
    
    tme.join();
}

// the timer program
void timer(int hit) {
    if (hit >= 3) {
        for (i = 1; i <= 10000; i++) {
            if (s == 0) {
                std::cout << "They were able to hit you" << std::endl;
                hit = hit + 1;
                break;
            }
            system("cls");
            s--;
            std::cout << "He attacks you in:" << s << std::endl;
            Sleep(1000);
        }
    }
}

额外注意事项 #

• 我补上了代码里缺失的变量定义（比如choi、ran），这些也会导致编译错误；
• 使用rand()前建议调用srand(time(nullptr))初始化随机数种子，否则每次运行的随机序列都是固定的；
• 全局变量i、s容易引发线程安全问题，建议尽量用局部变量或者通过线程参数传递/返回的方式处理；
• Sleep()是Windows平台API，若要跨平台，建议用std::this_thread::sleep_for(std::chrono::seconds(1))替代。

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