在构造函数上启动线程时调用不匹配

Posted 2023-02-22

【中文标题】在构造函数上启动线程时调用不匹配

【英文标题】：No match for call while launching thread on constructor

【发布时间】：2015-07-27 07:15:40

【问题描述】：

我有一个class Foo，它必须在构造函数上启动一个线程（这是我的类上的一个方法loop()）。

我的类在头文件上是这样的：

class Foo

public:
  Foo();
  ~Foo();
private
  void loop();

  std::thread m_thread;

  //more stuff.

而源文件是：

Foo::Foo()

   m_thread(&Foo::loop, Foo()); // <---- No match for call '(std::thread) (void (Foo::*)(), Foo)'
   m_thread.join();


Foo::~Foo()

  m_thread.kill(); // <---- ??? how to do this?


void Foo::loop()

   while()
   
       //do stuff.
   

问题：

1 - 如何修复Foo constructor 上的错误？编译时出现"no match for call" 错误。

额外问题：

2 -如何杀死对象析构函数上的线程？

【问题讨论】：

1 - 将构造函数公开

...这不是问题，但我想你还是想拥有一个公共 ctr，对吧？

请注意，您的线程调用 loop() 是在一个单独的 Foo 对象上，因此您将无限递归到越来越多的 Foo 构造函数中。您可能希望将 this 指针传递给线程。

对不起...构造函数是公开的...在这里打错了。

【参考方案1】：

1) 你试图调用一个线程对象，而不是在这里初始化它：

m_thread(&Foo::loop, Foo());

您应该设置为它分配适当的值：

m_thread = std::thread(&Foo::loop, Foo());

但这将默认构造另一个Foo 对象，该对象将启动一个线程，该线程将默认构造一个Foo 对象，该对象将启动一个线程，该线程将...

所以，也许你的意思是在实例本身上启动线程，在这种情况下你需要

m_thread = std::thread(&Foo::loop, this);

请注意，通常最好在构造函数初始化列表中初始化数据成员，但由于线程调用Foo的成员，最好在构造对象后启动它.

2) 你可以在析构函数中加入线程。只要确保没有异常传播到析构函数之外。

【参考方案2】：

你也可以像这样在构造函数初始化列表中定义你的 m_thread 对象：

Foo::Foo() : m_thread(&Foo::loop, this), End(false)

   m_thread.join();

结束线程的一种简单方法是使用成员变量 End，在构造函数中将其设置为 true，然后在线程中循环

Foo::~Foo()

    End = true;
    m_thread.join(); // <---- ??? how to do this?


void Foo::loop(Foo* This)

   while(This->End == false)
   
      /*Do something*/