我为这篇冗长的帖子道歉。我仍然尽可能小,但仍然传达了问题。
好的,这让我发疯了。我有一个客户端和一个服务器程序,都在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的相关性。
如果你还在阅读这一点,我赞赏你的奉献精神。
一些重要的观察结果:
如上所述,这让我发疯!我总是能够解决我的编程问题,但这个问题让我很难过。所以我在这里请求就此主题提出任何建议或知识。
谢谢。
答案 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),它可以让您轻松地发送和接收整个对象。