boost :: asio :: async_write не завершает свою задачу - PullRequest
0 голосов
/ 17 июня 2010

Я пытаюсь понять, как асинхронное чтение и запись работает в boost asio, манипулируя примером echo.В настоящее время у меня есть сервер, который должен при отправке предложения отвечать только первым словом.Тем не менее, boost :: asio :: async_write никогда не завершается, хотя вызывается обработчик записи.Может кто-нибудь объяснить, пожалуйста, что происходит?Вот код:

#include <cstdlib>
#include <iostream>
#include <boost/bind.hpp>
#include <boost/asio.hpp>

using boost::asio::ip::tcp;

class session
{
public:
  session(boost::asio::io_service& io_service)
    : socket_(io_service)
  {
  }

  tcp::socket& socket()
  {
    return socket_;
  }

  void start()
  {
 std::cout<<"starting"<<std::endl;
  boost::asio::async_read_until(socket_, buffer, ' ',
        boost::bind(&session::handle_read, this,
          boost::asio::placeholders::error,
          boost::asio::placeholders::bytes_transferred));
  }

  void handle_read(const boost::system::error_code& error,
      size_t bytes_transferred)
  {

// std::ostringstream ss;
// ss<<&buffer;

 char* c = new char[bytes_transferred]; 

 //std::string s;
 buffer.sgetn(c,bytes_transferred);
 std::cout<<"data: "<< c<<" bytes: "<<bytes_transferred<<std::endl;

 if (!error)
    {
      boost::asio::async_write(socket_,
          boost::asio::buffer(c,bytes_transferred),
          boost::bind(&session::handle_write, this,
            boost::asio::placeholders::error));
    }
    else
    {
      delete this;
    }
  }

  void handle_write(const boost::system::error_code& error)
  {
 std::cout<<"handling write"<<std::endl;
    if (!error)
    {
    }
    else
    {
      delete this;
    }
  }

private:
  tcp::socket socket_;
  boost::asio::streambuf buffer;
};

class server
{
public:
  server(boost::asio::io_service& io_service, short port)
    : io_service_(io_service),
      acceptor_(io_service, tcp::endpoint(tcp::v4(), port))
  {
    session* new_session = new session(io_service_);
    acceptor_.async_accept(new_session->socket(),
        boost::bind(&server::handle_accept, this, new_session,
          boost::asio::placeholders::error));
  }

  void handle_accept(session* new_session,
      const boost::system::error_code& error)
  {
    if (!error)
    {
      new_session->start();
      new_session = new session(io_service_);
      acceptor_.async_accept(new_session->socket(),
          boost::bind(&server::handle_accept, this, new_session,
            boost::asio::placeholders::error));
    }
    else
    {
      delete new_session;
    }
  }

private:
  boost::asio::io_service& io_service_;
  tcp::acceptor acceptor_;
};

int main(int argc, char* argv[])
{
  try
  {
    if (argc != 2)
    {
      std::cerr << "Usage: async_tcp_echo_server <port>\n";
      return 1;
    }

    boost::asio::io_service io_service;

    using namespace std; // For atoi.
    server s(io_service, atoi(argv[1]));

    io_service.run();
  }
  catch (std::exception& e)
  {
    std::cerr << "Exception: " << e.what() << "\n";
  }

  return 0;
}

Спасибо!

Ответы [ 3 ]

2 голосов
/ 18 июня 2010

Вы неправильно используете boost :: asio :: streambuf, внимательно прочитайте пример клиента HTTP *1002*.Для вашего кода вы должны сделать что-то вроде:

buffer.commit( bytes_transferred );
std::istream is( &buffer );
std::string data;
is >> data;
std::cout << "data: " << data << " bytes: " << bytes_transferred << std::endl;

, а затем использовать отдельный boost :: asio :: streambuf для вашего ответа.Как и в предыдущем ответе, вы теряете память.Я бы предложил использовать boost :: shared_ptr и shared_from_this.

0 голосов
/ 13 марта 2011
  1. Чтобы избежать проблем с возможными утечками памяти, связанными со сессией, вы должны рассмотреть возможность использования boost::enable_shared_from_this, как показано ниже:

    class session : public boost::enable_shared_from_this<session>
    

    Вместо использования "this" используйте shared_from_this(),

    Не использовать delete this.

  2. Вы распределяете таблицу по new, но я не вижу, где свободна память.

    char* c = new char[bytes_transferred]; 
    

    delete c не вызывается

  3. Возможно, лучшим решением будет использование дополнительного буфера m_bufferForRead вместо таблицы выделения c.

0 голосов
/ 17 июня 2010

Как видите, вы просто не закрываете сокет и даже не удаляете сеанс. После успешной записи и завершения записи обработчик оставляет сеансовый объект, делающий утечку памяти. Также char* c = new char[bytes_transferred]; вызывает утечку памяти, так как он никогда больше не используется, boost::asio::buffer не освобождает память для вас

Просто измените обработчик записи:

void handle_write(const boost::system::error_code& error)
{
  std::cout<<"handling write and exiting."<<std::endl;
  delete this;
}
...