错误C2280:'std :: thread :: thread(const std :: thread&)':尝试引用已删除的函数

时间:2013-12-21 21:08:27

标签: c++ multithreading visual-c++ c++11 visual-studio-2013

我在尝试创建使用C ++ 11标准线程的VC ++静态库时遇到问题。

我目前有两个类,我能够声明并稍后在我的起始类(最后声明的)上定义一个线程。在此阶段,代码只是一个套接字侦听器,然后创建另一个类的对象来处理接受的每个客户端。这些子对象应该创建并行数据捕获,编码和传输所需的线程。

问题是:如果我在我的其他类上声明了一个std :: thread,即使我完全像在我的start类上那样,无论如何,我在构建error C2280: 'std::thread::thread(const std::thread &)' : attempting to reference a deleted function [...]\vc\include\functional 1124 1上都会出现此错误

我能够解决这个错误的唯一方法是根据我想要它做的事情,根本不在后一个类中声明std::thread对象,这是不可能的。

我正在使用VS2013,我的来源是:

stdafx.h中

#pragma once
#include "targetver.h"
#define WIN32_LEAN_AND_MEAN             // Exclude rarely-used stuff from Windows headers
#include <Windows.h>
#include <WinSock2.h>
#include <WS2tcpip.h>
#include <thread>
#include <iostream>
#include <vector>

StreamServer.h

#pragma once
#define DEFAULT_BUFLEN 65535
#define DEFAULT_PORT "5649"

class StreamServerClient
{
public:
    bool* terminate;
    //std::thread client;     //If I comment this line out, it builds just fine.
    void DoNothing();
    StreamServerClient(SOCKET clientSock, bool* ptTerm);
    StreamServerClient();
    ~StreamServerClient();
};

class StreamServer
{
public:
    bool terminate;
    std::thread Listener;
    std::vector<StreamServerClient> clients;
    void CreateClient(SOCKET, bool*);
    void Listen();
    StreamServer();
    ~StreamServer();
};

StreamServer.cpp

#include "stdafx.h"
#include "StreamServer.h"

StreamServerClient::StreamServerClient(SOCKET clientSock, bool* ptTerm)
{
    terminate = ptTerm;
    //client = std::thread(&StreamServerClient::DoNothing, this);     //Same thing as the declaration
}

StreamServerClient::StreamServerClient()
{
    *terminate = false;
    //client = std::thread(&StreamServerClient::DoNothing, this);     //Same thing as the declaration
}

void StreamServerClient::DoNothing()
{
}

StreamServerClient::~StreamServerClient()
{
}

void StreamServer::Listen()
{
    {...}
    do {
        clients.push_back(StreamServerClient::StreamServerClient(accept(listenSock, NULL, NULL), &terminate));
        std::cout << "accepted a client!" << std::endl;
    } while (!terminate);
}

StreamServer::StreamServer()
{
    terminate = false;
    Listener = std::thread(&StreamServer::Listen, this);
    Listener.detach();
}

StreamServer::~StreamServer()
{
}

2 个答案:

答案 0 :(得分:11)

无法复制std::thread类型的对象。最好只是初始化成员初始化列表中的对象:

class StreamServerClient
{
public:
    bool* terminate;
    std::thread client;
    void DoNothing();
    StreamServerClient(SOCKET clientSock, bool* ptTerm);
    StreamServerClient(StreamServerClient&& other);
    ~StreamServerClient();
};

StreamServerClient::StreamServerClient(SOCKET clientSock, bool* ptTerm)
    : terminate(ptTerm)
    , client(std::thread(&StreamServerClient::DoNothing, this)) {
}
StreamServerClient::StreamServerClient(StreamServerClient&& other)
    : terminate(other.terminate)
    , client(std::move(other.client)) {
}

我省略了默认构造函数(注意你的版本不起作用,因为它试图为取消引用未初始化指针的结果赋值)而是添加了一个移动构造函数:当推回std::vector<...>时当提供看起来像临时的东西时(即, 是临时的或者看起来像是一个,例如,使用std::move()),将调用此构造函数。

答案 1 :(得分:7)

std::thread对象不可复制。调用此行时:

clients.push_back(StreamServerClient::StreamServerClient(accept(listenSock, NULL, NULL), &terminate));

您创建的StreamServerClient对象需要添加到clients向量中,但唯一的方法是复制它,因为您的StreamServer对象没有移动构造函数定义。生成StreamServerClient的默认复制构造函数,以及C ++ 11中的默认复制构造函数是什么,它调用所有数据成员的复制构造函数。

在这种情况下,它正在调用std::thread数据成员的复制构造函数,该成员将被删除。

无法复制std::thread的原因是std::thread对象对应于执行线程。您应该重构您的代码,IMO。您可以将StreamServerClient的移动构造函数混合在一起,但我认为有更清晰的解决方案,例如,使用指向std::thread的指针替换std::thread并使用完全独立的调用初始化线程

编辑:一般来说,我会说在对象的构造函数中分支新线程是不明智的,因为它操作太精细了。 (它可能会失败,那么你必须处理异常等)尽管可能C ++纯粹主义者不同意我的看法。

编辑:有一个愚蠢的术语。