在 C++ 中创建 std 线程会使程序崩溃

Question

每当我使用线程执行以下代码时，程序都会出现此错误：

调试错误！ 程序：... /path/to/.exe

abort() 已被调用

我想创建一个调用成员函数的线程。这是我正在使用的功能：

void ServerVote::createConnexionThreads()
{
    for (int i = 0; i <= 50; ++i)
    {
        m_connexionThreads.push_back(&(std::thread(&ServerVote::acceptConnection,*this, i)));
    }
    for (int i = 0; i <= 50; ++i)
    {
        m_connexionThreads[i]->join();
    }
}

如果需要，我可以提供额外的代码。使用调试器时，我发现程序在创建第一个线程后立即崩溃，在线程被 push_back 之后。然后调用 ~thread() 并在此函数内崩溃。这是向量声明：

std::vector<std::thread*> m_connexionThreads;

我使用的是 Visual Studio 2015。acceptConnection 函数内部有一个 while(true)，计划稍后终止。

编辑：

感谢您的回答，但是当使用线程对象而不是指针时我无法编译。所以当我尝试推入这个向量时：

std::vector<std::thread> m_connexionThreads;

for (int i = 0; i <= 50; ++i)
{
        m_connexionThreads.push_back((std::thread(&ServerVote::acceptConnection,*this, i)));
}

编译时出现此错误：

error C2280: 'std::thread::thread(const std::thread &)': attempting to reference a deleted function

Answer 1

您不应尝试在任何情况下使用临时地址。事实上，这是 MSVC 中允许此代码的错误。任何符合标准的编译器都会在此处产生错误。

相反，您应该像这样使用线程对象（请参阅我在代码下方的编辑，了解为什么这是首选）：

#include <thread>
#include <vector>

void acceptConnection(int);

void foo() {
  std::vector<std::thread> vec;
  for (int i = 0; i <= 50; ++i)
        vec.push_back(std::thread(acceptConnection, i));
}

为什么这种方法优于使用分配的指向线程对象的指针？有多种好处：

1. 打字少了——即使没有别的，一切都是平等的（尽管它们不是！）少打字胜过多打字。
2. 使用指针需要谨慎。例如，你不应该使用原始指针作为向量数据类型，你应该使用unique_ptr 来确保自动内存清理——这使得语法更加丑陋！
3. 使用动态分配的内存会拖累性能。你被击中了两次——第一次是分配内存，第二次是释放它。为什么要遭受这种惩罚？

Answer 2

您正在堆栈中创建线程的本地实例，获取其地址并将其推送到向量。线程对象将在方法退出时被删除，因此您将留下一个指向已删除对象的指针。

您应该使用new 在堆中创建线程对象，这样它就不会在方法退出时被删除，或者不使用指向线程对象的指针。