Question

更新：我明白了！有点......我不确定这是怎么解决的，但确实如此。而不是使用：

NetworkMessage msg;
msg.AddByte(0x01);
sendmessage(msg);

我试过这个并且它有效（对某些功能进行了一些更改）

NetworkMessage* msg = new NetworkMessage();
msg.AddByte(0x01);
sendmessage(msg);

这修复了它，但仍然会感谢任何人解释为什么这首先发生的原因。发布它作为答案，所以我可以标记它。谢谢。

我一直在研究TCP服务器，但我遇到了一些问题..

有一个发送大约600的循环？消息，一次发送1个（从排队消息列表中读取）。

出于某种原因，大约200个没有错误地到达我的客户端，然后我收到System.10014错误。每次在不同的消息上，即使消息每次都相同。此外，如果我只是在System.10014错误后重新发送消息，它工作正常。所以很明显，消息不是问题所在。但我不知道是什么。昨天我花了整整一天的时间试图找出它并且仍然没有运气。所以我在这里。试。

void outputMessage(NetworkMessage& msg, bool forceTrue = false)
{
    if(!connectionLocked || forceTrue) {
        connectionLocked = true;

        std::cout << "Message: ";
        char* buf = msg.getOutputBuffer();
        for (int i = 0; i < msg.size(); i++){
            std::cout << (buf[i] + 0) << ":"; //Never any errors
        }

        boost::asio::async_write(socket_, boost::asio::buffer(msg.getOutputBuffer(), msg.size()),boost::bind(&Session::handle_write, shared_from_this(), msg, boost::asio::placeholders::error));

    }
    else {
        //Queue message
        _msgqueue.push_back(msg);
    }

}
void handle_write(NetworkMessage& msg, const boost::system::error_code& error)
{
    if (error)
    {
        cout << "Error: " << error << endl;
        outputMessage(msg, true); //works, even though it's the same message
    }
    else {
        if(_msgqueue.size() != 0){
            //cout << "Sending queued message #" << _msgqueue.size() << endl;
            outputMessage(_msgqueue.front(), true);
            _msgqueue.pop_front();
        }
        else
            connectionLocked = false;
    }
}

这是我发送函数的代码。它通常工作正常。出于某种原因，当我连续发送大量消息时，会发生此错误（System.10014）。我尝试阅读Winsock中的错误，但我真的不理解它（它说的是关于无效指针的东西，但我很确定该消息是有效的，因为再次发送它可以正常工作吗？）。我对编程经验很少，特别是使用C ++和指针，所以我遇到了一些麻烦。

生成消息的循环有点像这样。我确认X和Y是有效的，所以我很确定问题不存在。

map->cursor = map->begin;
while(map->cursor != NULL)
{
    //I'm not sure if pointers like this are supposed to be deleted?
    //I suck when it comes to memory management, and most other things :D
    Tile* tile = map->cursor->tile;
    if(tile){
        NetworkMessage msg;
        msg.AddByte(0x03);
        msg.AddByte(tile->posx);
        msg.AddByte(tile->posy);
        msg.AddByte(1);
        msg.AddByte(2);
        outputMessage(msg);
    }
    else
        break;
    map->cursor = map->cursor->next;

}

此外，这似乎不太相关，但如果我尝试将outputMessage更改为：

void outputMessage(NetworkMessage& msg)
{
    boost::asio::async_write(socket_, boost::asio::buffer(msg.getOutputBuffer(), msg.size()),boost::bind(&Session::handle_write, shared_from_this(), msg, boost::asio::placeholders::error));
}
void handle_write(NetworkMessage& msg, const boost::system::error_code& error)
{
    if (error)
    {
        cout << "Error: " << error << endl;
    }
}

我没有收到任何错误，但发送的邮件都是克隆。所以600条消息看起来都一样。这看起来很奇怪，所以我认为这可能与我现在遇到的问题有关。

另外这里是NetworkMessage.h，我也将它发布在你可以在评论中看到的pastebin上。

#ifndef _NETWORK_MESSAGE_H
#define _NETWORK_MESSAGE_H
class NetworkMessage
{
    public:
        enum { header_length = 2 };
        enum { max_body_length = NETWORKMESSAGE_MAXSIZE - header_length};
        NetworkMessage()
        {
            buf_size=0;
            m_ReadPos=header_length;
        }
        void AddByte(char byte_)
        {
            if (!canAdd(1)) return;
            data_[m_ReadPos++] = byte_;
            //data_[m_ReadPos] = '\0';
            buf_size++;
        }
        uint16_t PeekU16(){
            return (data_[1] | uint16_t(data_[0]) << 8);
        }
        uint16_t GetU16(){
            m_ReadPos += 2;
            return (data_[m_ReadPos+1]) | (uint16_t(data_[m_ReadPos]) << 8);
        }

        uint8_t  GetByte(){return data_[m_ReadPos++];}
        char* getBuffer()
        {
            return (char*)&data_[m_ReadPos];
        }
        char* getOutputBuffer()
        {
            writeMessageLength();
            return (char*)&data_[0];
        }

        void setMessageLength(uint16_t v)
        {
            buf_size=v;
        }
        void writeMessageLength()
        {
            *(uint8_t*)(data_) = (buf_size >> 8);
            *(uint8_t*)(data_+1) = buf_size;
        }
        void reset()
        {
            m_ReadPos = 0;
            buf_size = 0;
        }
        uint16_t size(){return buf_size+header_length;}
    protected:
        inline bool canAdd(uint32_t size)
        {
            return (size + m_ReadPos < max_body_length);
        };
    private:
        uint8_t data_[NETWORKMESSAGE_MAXSIZE];
        uint16_t buf_size;
        uint16_t m_ReadPos;
};
#endif //_NETWORK_MESSAGE_H

更新：好吧，遇到一些奇怪的事情......

function error_func(NetworkMessage& msg){
    cout << &msg << endl; //0x2882e8(not 0x274c0e8)
    send(msg.getOutputBuffer(), msg.size(), error_func, msg); //no error
}
cout << &msg << endl; //0x274c0e8
send(msg.getOutputBuffer(), msg.size(), error_func, msg); //error system.10014

显然这不是真正的代码，但你明白了......我想这就是为什么从error_func重新发送它的原因。毕竟问题可能出在NetworkMessage上。我一直在搞错所有错误的地方：D仍然需要帮助，哈哈。

非常感谢任何帮助，我被困住了。如果你到目前为止，感谢阅读..

Answer 1

void outputMessage(NetworkMessage& msg, bool forceTrue = false)
{
    if(!connectionLocked || forceTrue) {
        connectionLocked = true;

        std::cout << "Message: ";
        char* buf = msg.getOutputBuffer();
        for (int i = 0; i < msg.size(); i++){
            std::cout << (buf[i] + 0) << ":"; //Never any errors
        }

        boost::asio::async_write(socket_, boost::asio::buffer(msg.getOutputBuffer(), msg.size()),boost::bind(&Session::handle_write, shared_from_this(), msg, boost::asio::placeholders::error));

    }
    else {
        //Queue message
        _msgqueue.push_back(msg);
    }
}

传递给async_write的缓冲区需要在整个操作期间有效。文档为very explicit here

包含要写入的数据的一个或多个缓冲区。虽然缓冲区对象可以根据需要复制，底层的所有权内存块由调用者保留，必须保证在调用处理程序之前它们仍然有效。

当NetworkMessage对象被销毁时，你已经传递了一个不再有效的指针，我猜测它是在async_write操作完成之前。传递给NetworkMessage完成处理程序时会复制handle_write()对象。

C ++ Server，System.10014错误，使用Boost

1 个答案: