Asio : TCP Async_read

Bonjour bonjour, j'ai (encore et toujours) des probl�mes avec Asio :oops:

Je m'explique : j'ai lu et me suis inspir� de l'article de Pierre Schwartz (http://khayyam.developpez.com/articl.../architecture/), et je suis en train de r�aliser un jeu comportant un serveur, qui ne fait que �a (et non, comme dans l'article, un client qui joue aussi le r�le de serveur), et des clients qui s'y connectent. Autant pour le serveur que pour le client, j'utilise ceci pour mettre en place l'�coute sur le socket TCP, en ayant v�rifi� que le socket soit bien connect� avant :

Code:

---

1 2 3

m_tcpSocket->async_receive(asio::buffer(m_networkBuffer), boost::bind(&NetworkMachine::tcpAsyncReceive, this, asio::placeholders::error, asio::placeholders::bytes_transferred)); asio::thread(boost::bind(&asio::io_service::run, &m_io));

---

Ensuite, ma fonction tcpAsyncReceive :

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 23 24 25 26

void NetworkMachine::tcpAsyncReceive(const asio::error_code& pE, size_t pBytesReceived) { std::cout << "tcpAsyncReceive size : " << m_networkBuffer.size() << std::endl; std::cout << "pE.value() : " << pE.value() << std::endl; if (pE.value() != 0) { EngineEvent e; e.m_type = EngineEvent::REMOVE_MACHINE; e.m_iData["ID"] = m_id; m_parent->pushReceivedEvent(e); return; } // let's deserialize the message if (m_networkBuffer.size() > 0) { std::string strData(&m_networkBuffer[0], m_networkBuffer.size()); std::istringstream archiveStream(strData); boost::archive::text_iarchive archive(archiveStream);   EngineEvent ne; archive >> ne;   // add the event to the received event queue m_parent->pushReceivedEvent(ne); std::cout << "Something received from " << getNick() << " and added to NetworkEngine received queue." << std::endl; } return; }

---

J'ai bien d'affich� "tcpAsyncReceive size : 0", 0 erreurs une premi�re fois � l'�tablissement de la connexion, puis... Plus rien, alors que le client continue d'�mettre des donn�es, et que le socket n'est pas ferm� (Ce que je sais pour la simple et bonne raison que si je quitte le serveur, le client me dit qu'il est d�connect�, mais pas avant).

Ma question est donc, ai-je oubli� quelque chose ? Faut-il remettre en place l'�coute apr�s que la fonction ait �t� appell�e une fois ?

Merci.

Salut,

A vue de nez je dirais qu'il faut re-demander � recevoir apr�s avoir re�u quelque chose.

MAT.

C'est � dire, mettre m_tcpSocket->async_receive(asio::buffer(m_networkBuffer), boost::bind(&NetworkMachine::tcpAsyncReceive, this, asio::placeholders::error, asio::placeholders::bytes_transferred));
� la fin de tcpAsyncReceive ?

Ne serait-ce pas appropri� d'avoir un thread qui s'occupe de boucler en lisant s'il y a quelque chose � lire, puis qui attend par exemple 50ms avant de retourner voir. D�s qu'il y a quelque chose, il r�cup�re, et il met dans un conteneur de messages, qui sera trait� et vid�o au fur et � mesure par le thread principal ?

Parce que l�, ta solution am�nerait � un appel r�cursif dont on ne peut dire s'il termine (si tu modifies encore ta fonction).

Citation:

---

Envoy� par Alp

Ne serait-ce pas appropri� d'avoir un thread qui s'occupe de boucler en lisant s'il y a quelque chose � lire, puis qui attend par exemple 50ms avant de retourner voir. D�s qu'il y a quelque chose, il r�cup�re, et il met dans un conteneur de messages, qui sera trait� et vid�o au fur et � mesure par le thread principal ?

---

N'est-ce pas ce qui est fait avec m_tcpSocket->async_receive(asio::buffer(m_networkBuffer), boost::bind(&NetworkMachine::tcpAsyncReceive, this, asio::placeholders::error, asio::placeholders::bytes_transferred)); ? (Je ne connais pas encore vraiment Asio, donc j'essaie...)

Moui.

Je viens de retrouver cet exemple qui devrait bien t'aider : http://www.boost.org/doc/libs/1_37_0...hat_server.cpp

Le probl�me c'est que je fais quasiment la m�me chose que sur l'exemple justement, sauf que j'ai tout regroup� en une seule fonction... J'ai essay� de remettre le async_receive � la fin de ma fonction : sur le client �a marche, sans soucis, mais sur le serveur... Elle est bien appel�e en boucle d�s la premi�re r�ception :/

Justement il ne faut pas le faire en une fonction !
Boost a fait en sorte que tu confies, pour la r�ception depuis un socket, la gestion d'erreur � une fonction et la gestion de r�cup�ration du contenu � une autre. Garde cette s�paration et fais vraiment comme ils font eux, �a ira ;)

Ok, j'essaie �a et je te dis ;)

Edit : non, �a boucle toujours... V'la le code :

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 23 24 25 26 27 28 29 30 31 32 33

void NetworkMachine::tcpAsyncReceive(const asio::error_code& pE, size_t pBytesReceived) { std::cout << "tcpAsyncReceive" << std::endl; if (pE.value() != 0) { EngineEvent e; e.m_type = EngineEvent::REMOVE_MACHINE; e.m_iData["ID"] = m_id; m_parent->pushReceivedEvent(e); return; } m_tcpSocket->async_receive(asio::buffer(m_networkBuffer), boost::bind(&NetworkMachine::tcpAsyncReceiveD, this, asio::placeholders::error, asio::placeholders::bytes_transferred)); }   void NetworkMachine::tcpAsyncReceiveD(const asio::error_code& pE, size_t pBytesReceived) { std::cout << "tcpAsyncReceiveD" << std::endl; if (pE.value() != 0) { return; } // let's deserialize the message if (m_networkBuffer.size() > 0) { std::string strData(&m_networkBuffer[0], m_networkBuffer.size()); std::istringstream archiveStream(strData); boost::archive::text_iarchive archive(archiveStream);   EngineEvent ne; archive >> ne;   // add the event to the received event queue m_parent->pushReceivedEvent(ne); std::cout << "Something received from " << getNick() << " and added to NetworkEngine received queue." << std::endl; } m_tcpSocket->async_receive(asio::buffer(m_networkBuffer), boost::bind(&NetworkMachine::tcpAsyncReceive, this, asio::placeholders::error, asio::placeholders::bytes_transferred)); }

---

D�s r�ception du premier paquet, tcpAsyncReceive tcpAsyncReceiveD en continu, je vois pas quoi faire de plus :/

Pourquoi refais-tu un async_receive dans la deuxi�me fonction ?
Il ne faut pas.

C'est pourtant ce qui est fait sur l'exemple que tu as link�, si je me trompe pas :

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 23 24 25 26 27 28 29 30 31

void handle_read_header(const boost::system::error_code& error) { if (!error && read_msg_.decode_header()) { boost::asio::async_read(socket_, boost::asio::buffer(read_msg_.body(), read_msg_.body_length()), boost::bind(&chat_session::handle_read_body, shared_from_this(), boost::asio::placeholders::error)); } else { room_.leave(shared_from_this()); } }   void handle_read_body(const boost::system::error_code& error) { if (!error) { room_.deliver(read_msg_); boost::asio::async_read(socket_, boost::asio::buffer(read_msg_.data(), chat_message::header_length), boost::bind(&chat_session::handle_read_header, shared_from_this(), boost::asio::placeholders::error)); } else { room_.leave(shared_from_this()); } }

---

Et, si je ne le mets pas sur la deuxi�me, toutes les donn�es re�ues iront sur la deuxi�me, non ?

Peut-�tre qu'il faut utiliser async_read, genre comme dans l'exemple, plut�t que d'aller taper directement dans le async_receive de la socket ?
En tous cas c'est comme �a que je proc�de syst�matiquement pour du TCP.

MAT.

En utilisant ceci :

Code:

---

1 2 3 4 5 6

asio::async_read(m_tcpSocket, asio::buffer(m_networkBuffer), boost::bind(&NetworkMachine::tcpAsyncReceive, this, asio::placeholders::error, asio::placeholders::bytes_transferred));

---

Je me retrouve avec des erreurs de compilations plut�t obscures...

Citation:

---

Envoy� par G++

/usr/include/asio/impl/read.ipp: In function �void asio::async_read(AsyncReadStream&, const MutableBufferSequence&, CompletionCondition, ReadHandler) [with AsyncReadStream = asio::basic_stream_socket<asio::ip::tcp, asio::stream_socket_service<asio::ip::tcp> >*, MutableBufferSequence = asio::mutable_buffers_1, CompletionCondition = asio::detail::transfer_all_t, ReadHandler = boost::_bi::bind_t<void, boost::_mfi::mf2<void, NetworkMachine, const asio::error_code&, long unsigned int>, boost::_bi::list3<boost::_bi::value<NetworkMachine*>, boost::arg<1> (*)(), boost::arg<2> (*)()> >]�:
/usr/include/asio/impl/read.ipp:211: instantiated from �void asio::async_read(AsyncReadStream&, const MutableBufferSequence&, ReadHandler) [with AsyncReadStream = asio::basic_stream_socket<asio::ip::tcp, asio::stream_socket_service<asio::ip::tcp> >*, MutableBufferSequence = asio::mutable_buffers_1, ReadHandler = boost::_bi::bind_t<void, boost::_mfi::mf2<void, NetworkMachine, const asio::error_code&, long unsigned int>, boost::_bi::list3<boost::_bi::value<NetworkMachine*>, boost::arg<1> (*)(), boost::arg<2> (*)()> >]�
network_machine.cpp:41: instantiated from here
/usr/include/asio/impl/read.ipp:200: erreur: request for member �async_read_some� in �s�, which is of non-class type �asio::basic_stream_socket<asio::ip::tcp, asio::stream_socket_service<asio::ip::tcp> >*�

---

Tu utilises quelle version de boost ?
Pourquoi c'est asio:: et non boost::asio:: tiens d'ailleurs ?

MAT.

J'ai Boost 1.35, et je pense que je viens de trouver la cause de pas mal de soucis... J'ai aussi Asio 1.1.0, alors qu'Asio fait maintenant partie de Boost, je devrais pas en avoir besoin je me trompe ?

Asio fait partie de boost depuis justement la version 1.35 donc oui en effet tu devrais utiliser boost/asio et boost::asio plut�t.

MAT.

Au passage, r�cup�re la 1.37. Parce que par exemple je me suis bas� sur sa doc pour les exemples donn�s etc.

En effet, j'ai de jolis conflits entre Asio et Boost::asio en fait... Je fais du m�nage, et je vous donne des nouvelles ^^

Voil�, j'ai donc vir� Asio, et mit Boost en 1.37. Je compile avec ma ligne d'avant, les deux fonctions sont appel�es en boucle.
J'essaie donc avec

Code:

---

1 2 3 4 5 6

boost::asio::async_read(m_tcpSocket, boost::asio::buffer(m_networkBuffer), boost::bind(&NetworkMachine::tcpAsyncReceive, this, boost::asio::placeholders::error, boost::asio::placeholders::bytes_transferred));

---

et GCC me r�pond toujours de mani�re barbare... :

Citation:

---

Envoy� par GCC

/usr/include/boost/asio/impl/read.ipp: In function �void boost::asio::async_read(AsyncReadStream&, const MutableBufferSequence&, CompletionCondition, ReadHandler) [with AsyncReadStream = boost::asio::basic_stream_socket<boost::asio::ip::tcp, boost::asio::stream_socket_service<boost::asio::ip::tcp> >*, MutableBufferSequence = boost::asio::mutable_buffers_1, CompletionCondition = boost::asio::detail::transfer_all_t, ReadHandler = boost::_bi::bind_t<void, boost::_mfi::mf2<void, NetworkMachine, const boost::system::error_code&, long unsigned int>, boost::_bi::list3<boost::_bi::value<NetworkMachine*>, boost::arg<1> (*)(), boost::arg<2> (*)()> >]�:
/usr/include/boost/asio/impl/read.ipp:232: instantiated from �void boost::asio::async_read(AsyncReadStream&, const MutableBufferSequence&, ReadHandler) [with AsyncReadStream = boost::asio::basic_stream_socket<boost::asio::ip::tcp, boost::asio::stream_socket_service<boost::asio::ip::tcp> >*, MutableBufferSequence = boost::asio::mutable_buffers_1, ReadHandler = boost::_bi::bind_t<void, boost::_mfi::mf2<void, NetworkMachine, const boost::system::error_code&, long unsigned int>, boost::_bi::list3<boost::_bi::value<NetworkMachine*>, boost::arg<1> (*)(), boost::arg<2> (*)()> >]�
network_machine.cpp:75: instantiated from here
/usr/include/boost/asio/impl/read.ipp:216: erreur: request for member �get_io_service� in �s�, which is of non-class type �boost::asio::basic_stream_socket<boost::asio::ip::tcp, boost::asio::stream_socket_service<boost::asio::ip::tcp> >*�
/usr/include/boost/asio/impl/read.ipp:232: instantiated from �void boost::asio::async_read(AsyncReadStream&, const MutableBufferSequence&, ReadHandler) [with AsyncReadStream = boost::asio::basic_stream_socket<boost::asio::ip::tcp, boost::asio::stream_socket_service<boost::asio::ip::tcp> >*, MutableBufferSequence = boost::asio::mutable_buffers_1, ReadHandler = boost::_bi::bind_t<void, boost::_mfi::mf2<void, NetworkMachine, const boost::system::error_code&, long unsigned int>, boost::_bi::list3<boost::_bi::value<NetworkMachine*>, boost::arg<1> (*)(), boost::arg<2> (*)()> >]�
network_machine.cpp:75: instantiated from here
/usr/include/boost/asio/impl/read.ipp:221: erreur: request for member �async_read_some� in �s�, which is of non-class type �boost::asio::basic_stream_socket<boost::asio::ip::tcp, boost::asio::stream_socket_service<boost::asio::ip::tcp> >*�
make: *** [network_machine.o] Erreur 1

---

Au temps pour moi, j'avais pas remarqu� la p'tite �toile... J'ai donc �a maintenant :

Code:

---

1 2 3 4 5 6

boost::asio::async_read((*m_tcpSocket), boost::asio::buffer(m_networkBuffer), boost::bind(&NetworkMachine::tcpAsyncReceive, this, boost::asio::placeholders::error, boost::asio::placeholders::bytes_transferred));

---

Qui se compile, s'ex�cute... Et boucle toujours :/