boost:线程没有执行接收信号后发布的处理程序

时间:2012-07-10 17:50:06

标签: c++ boost boost-thread

我熟悉了提升线程和信号。因此,我实现了这个简单的示例,我只发布了一个示例类的cpp文件,该类实现了一个能够在触发Signal1时执行方法的线程。信号在Package1Signals单例中定义(请原谅我们这些名称,它们是从模型生成的)

Class1.hpp

#ifndef CLASS1_HEADER
#define CLASS1_HEADER
#include <boost/asio/io_service.hpp>
#include <boost/asio/ip/udp.hpp>
#include <boost/asio/signal_set.hpp>
#include "Package1.hpp"
#include <boost/thread.hpp>


class Class1{
private:
    boost::asio::io_service service;
public:
    boost::thread  thread;
    Class1();
    void classifierBehavior();
    void Reception1(Signal1 signal1);
    void Reception1Behavior();
};


#endif

Class1.cpp

#include "Class1.hpp"
#include <boost/thread.hpp>
#include <boost/date_time.hpp>
#include "Package1.hpp"
#include "Package2.hpp"
#include "Package1.hpp"

Class1::Class1(){
 //boost::thread thread(boost::bind(&Class1::classifierBehavior,boost::ref(*this)));
 //thread.join();
 thread = boost::thread(&Class1::classifierBehavior,this);
};

void Class1::classifierBehavior(){
  Package1Signals::getInstance()->signal1.connect(boost::bind(&Class1::Reception1, 
  this,_1));
  service.run();

};

 void Class1::Reception1(Signal1 signal1){
std::cout<<"Signal received\n";
service.post(boost::bind(&Class1::Reception1Behavior, this));
 }

 void Class1::Reception1Behavior(){
std::cout<<"Behavior executed\n";
 }

Package1.hpp

#ifndef PACKAGE1_HEADER
#define PACKAGE1_HEADER
#include <boost/signal.hpp>

struct Signal1{   };

class Package1Signals{
private:
    Package1Signals();
    static Package1Signals * instance;
public:
    boost::signal<void(Signal1)> signal1;
    static Package1Signals * getInstance();
 };
 #endif

Package1.cpp

 #include "Package1.hpp"
 Package1Signals * Package1Signals::instance = NULL;
 Package1Signals::Package1Signals(){}
 Package1Signals * Package1Signals::getInstance(){
    if(!instance){
            instance = new Package1Signals();
        }
    return instance;
 }

这是执行它的代码

int main() {
    Class1 test;
    Package1Signals::getInstance()->signal1();
    test.thread.join();
    int k =0;
    std::cin>>k;
    return 0;
 }

我可以看到线程运行,信号被拦截但是没有执行发布的处理程序。我究竟做错了什么?

2 个答案:

答案 0 :(得分:3)

  

我可以看到线程运行,信号被拦截

你确定吗?我认为信号不是发送的事件。

  

但是未执行发布的处理程序。我做错了什么?

您的构造函数创建一个新线程,然后等待它完成,因此构造函数不会返回,直到新线程退出。这意味着main中的这一行在接收者线程退出之前不会执行:

 Package1Signals::getInstance()->signal1();

也许您希望boost::thread成为您的类的成员,因此它与您的类具有相同的生命周期,而不是构造函数中的局部变量。

对于样式注意事项,您不需要使用boost::ref(*this)只能传递this,并且您不需要使用boost:bind来创建线程,请阅读{ {3}}告诉你构造一个带有多个参数的thread等同于将这些参数传递给bind并构造带有结果的线程,即你可以说

thread(&Class1::classifierBehavior, this);

哪个更简单易读!

更新:现在您已经修复了构造函数,因此它不会阻止您在将接收器连接到信号之间存在竞争条件(在新线程中发生)并发布信号,一旦构造函数完成,就会在主线程中发生。如果新线程需要几毫秒才能开始执行,那么它将太晚并且错过信号,因为在主线程中构造函数已经完成并且信号已经被发出。

尝试在启动新线程之前连接接收器

Class1::Class1(){
  Package1Signals::getInstance()->signal1.connect(boost::bind(&Class1::Reception1,   this,_1));
 thread = boost::thread(&Class1::classifierBehavior,this);
};

void Class1::classifierBehavior(){
  service.run();
};

这样接收器就会在发出信号之前连接。

认为即使在服务运行之前事件被发布,io_service仍将获得事件,但您可能想要检查,否则那里会有另一个竞争条件。

答案 1 :(得分:1)

thread.join();语句基本上等待线程退出。所以本质上发生的是你的构造函数在线程退出之前不会完成,这相当于在构造函数中运行线程代码。

修复是创建线程并在类中保留它的句柄。不要在等待线程完成并加入主线程之前调用thread.join();

如果创建的线程由service运行,则需要运行该服务以使线程开始工作。所以你必须在构造函数中做service.run();而不是在线程运行的函数内。

另外,如果我没记错,服务只能在创建它的同一个线程中运行。