我在多个场合都看到过这个问题,似乎它出现在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)>’....
我想我知道问题在于某种形式的复制或引用机制,但所有语法似乎都是正确的。
我错过了什么?
我已经改变了一些例子,我为这种混乱道歉。我试图尽可能地重现问题,而且我自己也不能完全理解它。
答案 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))
时,这两件事情将在内部使用模板魔术发生:
使用您的callable和所有args创建一个元组。
std::tuple<mycallablerefarg, mynoncopy> thetuple(callable,
std::move(thenoncopyableinstance));
在这种情况下将复制callable。
std::invoke()
用于调用callable,arg通过移动语义从元组传递给它。
std::invoke(std::move(std::get<0>(thetuple)), std::move(std::get<1>(thetuple)));
因为使用了移动语义,所以它会期望callable接收一个rvalue引用作为参数(在我们的例子中是mynoncopy&&
)。这限制了我们以下参数签名:
mynoncopy&&
const mynoncopy&
T&&
其中T是模板参数mynoncopy
不是引用(这将调用move-constructor)这些是使用不同类型的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.