非常奇怪的问题通过C#中的套接字发送数据

时间:2011-01-31 19:17:31

标签: c# arrays sockets byte

我为这篇冗长的帖子道歉。我仍然尽可能小,但仍然传达了问题。

好的,这让我发疯了。我有一个客户端和一个服务器程序,都在C#中。服务器通过Socket.Send()将数据发送到客户端。客户端通过Socket.BeginReceive和Socket.Receive接收数据。我的伪协议如下:服务器发送一个双向(短)值,表示实际数据的长度,紧接着是实际数据。客户端异步读取前两个字节,将字节转换为short,并立即同步从套接字读取多个字节。

现在每隔几秒左右就可以正常运行一个周期,但是当我提高速度时,情况会变得奇怪。似乎客户端在尝试从两个字节长度读取时会随机读取实际数据。然后它尝试将这些任意两个字节转换为short,这会导致完全不正确的值,从而导致崩溃。以下代码来自我的程序,但修剪为仅显示重要的行。

用于发送数据的服务器端方法:

private static object myLock = new object();
private static bool sendData(Socket sock, String prefix, byte[] data)
{
    lock(myLock){
        try
        {
            // prefix is always a 4-bytes string
            // encoder is an ASCIIEncoding object    
            byte[] prefixBytes = encoder.GetBytes(prefix);
            short length = (short)(prefixBytes.Length + data.Length);

            sock.Send(BitConverter.GetBytes(length));
            sock.Send(prefixBytes);
            sock.Send(data);

            return true;
        } 
        catch(Exception e){/*blah blah blah*/}
    }
}

接收数据的客户端方法:

private static object myLock = new object();
private void receiveData(IAsyncResult result)
{
    lock(myLock){
        byte[] buffer = new byte[1024];
        Socket sock = result.AsyncState as Socket;
        try
        {
            sock.EndReceive(result);
            short n = BitConverter.ToInt16(smallBuffer, 0);
            // smallBuffer is a 2-byte array

            // Receive n bytes
            sock.Receive(buffer, n, SocketFlags.None);

            // Determine the prefix.  encoder is an ASCIIEncoding object
            String prefix = encoder.GetString(buffer, 0, 4);

            // Code to process the data goes here

            sock.BeginReceive(smallBuffer, 0, 2, SocketFlags.None, receiveData, sock);
        }
        catch(Exception e){/*blah blah blah*/}
    }
}

服务器端代码可靠地重新创建问题:

byte[] b = new byte[1020];  // arbitrary length

for (int i = 0; i < b.Length; i++)
    b[i] = 7;  // arbitrary value of 7

while (true)
{
    sendData(socket, "PRFX", b);
    // socket is a Socket connected to a client running the same code as above
    // "PRFX" is an arbitrary 4-character string that will be sent
}

查看上面的代码,可以确定服务器将永远发送数字1024,包括前缀在内的总数据的长度为短(0x400),后跟ASCII二进制中的“PRFX”,后跟一个一堆7(0x07)。客户端将永远读取前两个字节(0x400),将其解释为1024,将该值存储为n,然后从流中读取1024个字节。

这确实是它在前40次左右的迭代中的作用,但是,自发地,客户端将读取前两个byes并将它们解释为1799,而不是1024!十六进制中的1799是0x0707,这是连续两个7!那是数据,而不是长度!这两个字节发生了什么变化?这种情况发生在我放在字节数组中的任何值,我只选择7,因为很容易看到与1799的相关性。

如果你还在阅读这一点,我赞赏你的奉献精神。

一些重要的观察结果:

  • 减少b的长度会增加问题发生前的迭代次数,但不会阻止问题的发生
  • 在循环的每次迭代之间添加显着延迟可以防止问题发生
  • 当在同一主机上同时使用客户端和服务器并通过环回地址连接时,不会发生。

如上所述,这让我发疯!我总是能够解决我的编程问题,但这个问题让我很难过。所以我在这里请求就此主题提出任何建议或知识。

谢谢。

5 个答案:

答案 0 :(得分:3)

我怀疑您假设整个邮件都是在receiveData的一次通话中发送的。一般情况下,情况并非如此。碎片,延迟等最终可能意味着数据以运球形式到达,因此您可以在仅有的情况下调用receiveData函数,例如准备好900字节。您的呼叫sock.Receive(buffer, n, SocketFlags.None);说“将数据提供给缓冲区,最多n字节” - 您可能会收到更少的数据,实际读取的字节数由Receive返回。

这解释了为什么减少b,增加更多延迟或使用相同的主机似乎“修复”了问题 - 使用这些方法一次性到达整个消息的可能性大大增加(较小{{ 1}},较小的数据;添加延迟意味着管道中的总体数据较少;本地地址没有网络。)

要查看这确实是问题,请记录b的返回值。如果我的诊断正确,它偶尔会小于1024。要修复它,您需要等到本地网络堆栈的缓冲区包含所有数据(不理想),或者只是在数据到达时接收数据,将其存储在本地,直到整个消息准备就绪并然后处理它。

答案 1 :(得分:2)

听起来像我这里的主要问题是没有检查 EndReceive的结果,它是读取的字节数。如果套接字已打开,则可以是您指定的最大值> 0<=。假设因为你要求2个字节,读取2个字节是不安全的。在阅读实际数据时同样如此。

您必须始终循环,累积您期望的数据量(并减少剩余计数,并相应地增加偏移量)。

同步/异步的混合无助于简化:£

答案 2 :(得分:2)

sock.Receive(buffer, n, SocketFlags.None);

您没有检查函数的返回值。套接字将返回最多'n'个字节,但只返回可用的内容。您有可能无法通过此阅读获得全部“数据”。因此,当您执行下一个套接字读取时,实际上是获取剩余数据的前两个字节而不是下一个传输的长度。

答案 3 :(得分:1)

使用套接字时,必须预期套接字可能传输的字节数少于预期。您必须循环.Receive方法以获取剩余的字节。

通过套接字发送字节时也是如此。您必须检查发送了多少字节,然后循环发送直到发送所有字节。

此行为是由于网络层将邮件拆分为多个数据包。如果您的消息很短,那么您就不太可能遇到此消息。但是你应该总是为它编码。

每个缓冲区使用更多字节,您很可能会看到发送方的消息吐出多个数据包。每次读取操作都将收到一个数据包,这只是您邮件的一部分。但是小缓冲区也可以分开。

答案 4 :(得分:0)

我的猜测是smallBuffer在receive方法之外声明,并且正在从多个线程重用。换句话说,这不是线程安全的。

编写好的套接字代码很难。由于您正在编写客户端和服务器,因此您可能需要查看Windows Communication Foundation(WCF),它可以让您轻松地发送和接收整个对象。