首页 > 解决方案 > 在使用 boost::asio 时得到了 receive_from: Bad file descriptor

问题描述

#include "string"
#include <boost/asio.hpp>
#include <boost/asio/io_context.hpp>
using namespace boost::asio::ip;
 
class A {
    public:
    std::string address;
    std::string port;
    boost::asio::io_context io_context;
    udp::socket socketInstance = udp::socket(io_context);
    udp::endpoint endpointSetup;
 
    explicit A(std::string addrs, std::string port) {
        this->address = addrs;
        this->port = port;
    }
 
    A(const A& a) {
        this->address = a.address;
        this->port = a.port;
    }
 
    void attachListener() {
        endpointSetup = udp::endpoint(make_address(address), std::stoi(port));
        socketInstance = udp::socket(io_context, endpointSetup);
    }
};
 
class B {
    public:
    A aClass;
 
    explicit B(const A& a) : aClass(a) {}
 
    void getData() {
        std::array<char, 1024> recv_buffer{};
 
        aClass.socketInstance.receive_from(boost::asio::buffer(recv_buffer), aClass.endpointSetup);
    }
};
 
int main() {
    A a("192.168.1.49", "5080");
    a.attachListener();
    B b(a);
    b.getData();
}

零件aClass.socketInstance.receive_from(boost::asio::buffer(recv_buffer), aClass.endpointSetup);产生错误Bad file descriptor

我认为io_context财产以class A某种方式被破坏了。基本上,我想要实现的是在单独的类中监听一些 UDP 端口。我该如何解决这个问题?

标签: c++socketsboostnetwork-programmingboost-asio

解决方案


问题在于您在以下位置声明的aClass成员B

class B {
    public:
    A aClass;
 
    explicit B(const A& a) : aClass(a) {}
...

aClass成员是 type A,所以当你这样做时aClass(a),会执行一个副本。您已经为 定义了一个复制构造函数A,但它不会复制已打开的套接字,从而在副本中留下一个默认初始化(因此未打开)的套接字。

考虑以下示例类:

class C {
public:
  std::string foo_ = "foo";
  C(const std::string &foo) : foo_{foo} {}
  C(const C &c) {}
};

如果您要运行以下代码:

  C c{"bar"};
  C c2{c};
  std::cout << c2.foo_ << std::endl;

你会得到输出foo。这是因为 for 的复制构造函数C不会复制该foo_成员。因此,当c2被初始化时,它不会从 中获取值barc而是将foo_成员默认初始化为fooA同样的问题也适用于您班级的套接字成员。

解决方案 1

通过B保持指向 a 的指针来避免复制A

class B {
public:
  A* aClass;

  explicit B(A* a) : aClass(a) {}

  void getData() {
    std::array<char, 1024> recv_buffer{};

    aClass->socketInstance.receive_from(boost::asio::buffer(recv_buffer),
                                       aClass->endpointSetup);
  }
};

然后在main做:

B b(&a);

这样,当您创建 时B,它仍然可以访问您为原始 初始化的套接字A

除了使用 之外A*,您还可以使用智能指针,即std::unique_ptr<A>或者std::shared_ptr<A>,如果您想在拥有 的意义上表达所有权,从而B拥有A由 持有的套接字A

解决方案 2

另一种解决方案是 move-construct A,如果您想传递一个AtoB但又想获得实例B持有的套接字的所有权A,而不使用(智能)指针的间接寻址。

这样您就可以利用udp::socket. 但是,这需要您将io_context移出A,因为io_context没有移动构造函数。

#include <boost/asio.hpp>
#include <boost/asio/io_context.hpp>

using namespace boost::asio::ip;

class A {
public:
  std::string address;
  uint16_t port;

  boost::asio::io_context &io_context;

  udp::socket socketInstance;
  udp::endpoint endpointSetup;

  explicit A(boost::asio::io_context &io_context, std::string addrs,
             uint16_t port)
      : io_context{io_context}, address{addrs}, port{port}, socketInstance{
                                                                io_context} {}

  void attachListener() {
    endpointSetup = udp::endpoint{make_address(address), port};
    socketInstance = udp::socket{io_context, endpointSetup};
  }
};

class B {
public:
  A aClass;

  explicit B(A &&a) : aClass{std::move(a)} {}

  void getData() {
    std::array<char, 1024> recv_buffer{};

    aClass.socketInstance.receive_from(boost::asio::buffer(recv_buffer),
                                       aClass.endpointSetup);
  }
};

int main() {
  boost::asio::io_context io_context{};

  A a{io_context, "127.0.0.1", 5080};
  a.attachListener();
  B b{std::move(a)};
  b.getData();
}

两种解决方案都可以进行测试,例如

netcat --udp localhost 5080

推荐阅读