3

我正在写一个服务器应用程序boost::asio

  1. 在这里我实例化一个Server拥有一个io_service
  2. 服务器由server.start()调用的方法启动,甚至在它获得新连接之前Server::accept()创建一个新的Session
  3. 调用_acceptor.async_acceptthats设置一个回调,该回调调用Session::handler()执行握手的方法。

现在可以在获得新客户端之前创建一个套接字吗?在这段代码中,会话在编写"Hello "消息后自动销毁,在 HTTP 的情况下没问题,但我想进行有状态通信。所以套接字必须async_wait在它刚刚写完之后才能阅读"Hallo "。我也想知道这个设计好不好?或者它有任何设计缺陷。

这是我的可编译代码

~

class Session: public boost::enable_shared_from_this<Session>, private boost::noncopyable{
  private:
    size_t _id;
    boost::asio::ip::tcp::socket   _socket;
  public:
    typedef boost::shared_ptr<Session> pointer;
    static pointer create(boost::asio::io_service& ios){
      return pointer(new Session(ios));
    }
  private:
  explicit Session(boost::asio::io_service& ios): _socket(ios){
    static size_t counter = 0;
    _id = counter++;
    std::cout << ">> Session " << id() << " constructing" << std::endl;
  }
  public:
    void handler(const boost::system::error_code &ec){
      const std::string message = (boost::format("HTTP/1.1 200 OK\r\nContent-Length: %2%\r\n\r\nHello, %1%!") % id() % (7+boost::lexical_cast<std::string>(id()).length())).str();
      if(!ec){
        boost::asio::async_write(_socket, boost::asio::buffer(message), boost::bind(&Session::write_handler, this));
      }else{
        std::cout << ec.message() << std::endl;
      }
    }
    void write_handler(){

    }
    size_t id() const{
      return _id;
    }
    boost::asio::ip::tcp::socket& socket(){
      return _socket;
    }
    virtual ~Session(){
      std::cout << ">> Session " << id() << " destructing" << std::endl;
    }
    };

class Server: public boost::enable_shared_from_this<Server>, private boost::noncopyable{
  private:
    boost::asio::io_service        _ios;
    boost::asio::ip::tcp::acceptor _acceptor;
  public:
    explicit Server(boost::asio::ip::tcp::endpoint& endpoint):_acceptor(_ios, endpoint){

    }
    void start(){
      accept();
      _ios.run();
    }
    void accept(){
      std::cout << "accepting " << std::endl;;
      Session::pointer session = Session::create(_ios);
      _acceptor.async_accept(session->socket(), boost::bind(&Server::handler, this, session, boost::asio::placeholders::error));
    }
    void handler(Session::pointer session, const boost::system::error_code &ec){
      if(!ec){
        session->handler(ec);
      }else{
        //possible destroy session ? but how to destroy a shared pointer ?
      }
      accept();
    }
};


int main(){  
  const unsigned int port = 5050;
  boost::asio::ip::tcp::endpoint endpoint(boost::asio::ip::tcp::v4(), port);

  Server server(endpoint);
  server.start();

  return 0;
}
4

1 回答 1

3

整体设计看起来还可以,但是有一些实现错误:

  • Session::handler(): message,提供的底层缓冲区,在被调用async_write()之前可能会超出范围。async_write()这违反了 API 要求的保证。考虑为 制作message一个成员变量Session,或者static在成员函数中制作它。这是文档的相关摘录:

    尽管缓冲区对象可以根据需要被复制,但底层内存块的所有权由调用者保留,它必须保证它们在调用处理程序之前保持有效。以下是文档的相关摘录:

  • 对于继承自 的对象,在将实例句柄传递给 时boost::enable_shared_from_this使用shared_from_this()而不是指针。否则,可能会在处理程序运行之前删除指向的对象实例。thisboost::bindthis

另外,Server::handler()为了Session解决boost::shared_ptr.

于 2012-06-13T21:10:38.293 回答