在一个线程中产生线程与可调用对象

时间:2018-04-25 04:20:04

标签: c++ multithreading callable callable-object

我在多个场合都看到过这个问题,似乎它出现在Windoes(visual studio)和Linux(gcc)中。 这是它的简化版本:

class noncopyable
{
public:
    noncopyable(int n);
    ~noncopyable();
    noncopyable(const noncopyable&) = delete;
    noncopyable& operator=(const noncopyable&) = delete;
    noncopyable(noncopyable&&);
    int& setvalue();
private:
    int* number;
};

class thread_starter
{
public:
    template<typename callable>
    bool start(callable & o);
};

template<typename callable>
inline bool thread_starter::start(callable & o)
{
    std::thread t(
        [&]() {
        int i = 10;
        while (i-- > 0)
        {
            noncopyable m(i);
            std::thread child(o, std::move(m));
            child.detach();
        }
    });
    return true;
}

class callable
{
public:
    virtual void operator()(noncopyable &m);
};

void callable::operator()(noncopyable & m) { m.setvalue()++; }


int main()
{
    thread_starter ts;
    callable o;
    ts.start(o);
}

代码似乎足够合法,但它不会编译。

Visual Studio 中,它将提供:

error C2893: Failed to specialize function template 'unknown-type std::invoke(_Callable &&,_Types &&...) noexcept(<expr>)'

GCC 中,它会给出:

error: no type named ‘type’ in ‘class std::result_of<callable(int)>’....

我想我知道问题在于某种形式的复制或引用机制,但所有语法似乎都是正确的。

我错过了什么?

我已经改变了一些例子,我为这种混乱道歉。我试图尽可能地重现问题,而且我自己也不能完全理解它。

2 个答案:

答案 0 :(得分:2)

std::thread的调用会创建一个元组。无法使用引用初始化元组。因此,您必须使用伪引用std::ref(i)来编译并使用int-refs int&调用callables。

template <typename callable>
bool thread_starter::start(callable &o)
{
    // Nonsense
    std::thread t(
        [&]() {
        int i = 10;
        while (i-- > 0)
        {
            std::thread child(o, std::ref(i));
            child.detach();
        }
    });
    return true;
}

然而,结果代码毫无意义。产生的线程与while循环竞争。循环递减索引i,而线程尝试递增它。无法保证何时会发生这些事情。增量和减量不是原子的。在lambda完成后,线程可能会尝试增加索引。

简而言之,如果你进行编译,结果是未定义的行为有很多原因。

你究竟想做什么?

答案 1 :(得分:1)

我做了一个类似的程序来弄清楚会发生什么。这是它的外观:

class mynoncopy 
{
public:

    mynoncopy(int resource)
        : resource(resource)
    {

    }

    mynoncopy(mynoncopy&& other)
        : resource(other.resource)
    {
        other.resource = 0;
    }

    mynoncopy(const mynoncopy& other) = delete;

    mynoncopy& operator =(const mynoncopy& other) = delete;

public:

    void useResource() {}

private:
    int resource;
};

class mycallablevaluearg
{
public:

    void operator ()(mynoncopy noncopyablething)
    {
        noncopyablething.useResource();
    }
};

class mycallableconstrefarg
{
public:

    void operator ()(const mynoncopy& noncopyablething)
    {
        //noncopyablething.useResource(); // can't do this becuase of const :(
    }
};

class mycallablerefarg
{
public:

    void operator ()(mynoncopy& noncopyablething)
    {
        noncopyablething.useResource();
    }
};

class mycallablervaluerefarg
{
public:

    void operator ()(mynoncopy&& noncopyablething)
    {
        noncopyablething.useResource();
    }
};

class mycallabletemplatearg
{
public:

    template<typename T>
    void operator ()(T&& noncopyablething)
    {
        noncopyablething.useResource();
    }
};

当您发布std::thread(callable, std::move(thenoncopyableinstance))时,这两件事情将在内部使用模板魔术发生:

  1. 使用您的callable和所有args创建一个元组。
    std::tuple<mycallablerefarg, mynoncopy> thetuple(callable, std::move(thenoncopyableinstance));
    在这种情况下将复制callable。

  2. std::invoke()用于调用callable,arg通过移动语义从元组传递给它。
    std::invoke(std::move(std::get<0>(thetuple)), std::move(std::get<1>(thetuple)));

  3. 因为使用了移动语义,所以它会期望callable接收一个rvalue引用作为参数(在我们的例子中是mynoncopy&&)。这限制了我们以下参数签名:

    1. mynoncopy&&
    2. const mynoncopy&
    3. T&&其中T是模板参数
    4. mynoncopy不是引用(这将调用move-constructor)
    5. 这些是使用不同类型的callables的编译结果:

      mynoncopy testthing(1337);
      
      std::thread t(mycallablerefarg(), std::move(testthing)); // Fails, because it can not match the arguments. This is your case.
      std::thread t(mycallablevaluearg(), std::move(testthing)); // OK, because the move semantics will be used to construct it so it will basically work as your solution
      std::thread t(mycallableconstrefarg(), std::move(testthing)); // OK, because the argument is const reference
      std::thread t(mycallablervaluerefarg(), std::move(testthing)); // OK, because the argument is rvalue reference 
      std::thread t(mycallabletemplatearg(), std::move(testthing)); // OK, because template deduction kicks in and gives you noncopyablething&&
      std::thread t(std::bind(mycallablerefarg(), std::move(testthing))); // OK, gives you a little bit of call overhead but works. Because bind() does not seem to use move semantics when invoking the callable
      std::thread t(std::bind(mycallablevalue(), std::move(testthing))); // Fails, because bind() does not use move semantics when it invokes the callable so it will need to copy the value, which it can't.