我已经使用 TcpClient 类实现了套接字客户端。所以我可以发送和接收数据,一切正常。但我问了一些专家:)我的实现有问题吗?也许有更好的做事方式。特别是,如何处理断开连接?是否有一些指示器(或者也许我可以自己写一个)告诉我套接字已断开连接?
我还研究了 Socket 类的 async wait 功能,但无法理解“SocketAsyncEventArgs”,为什么它首先存在。 为什么我不能只是: await Client.SendAsync("data"); ?
public class Client
{
private TcpClient tcpClient;
public void Initialize(string ip, int port)
{
try
{
tcpClient = new TcpClient(ip, port);
if (tcpClient.Connected)
Console.WriteLine("Connected to: {0}:{1}", ip, port);
}
catch (Exception ex)
{
Console.WriteLine(ex.Message);
Initialize(ip, port);
}
}
public void BeginRead()
{
var buffer = new byte[4096];
var ns = tcpClient.GetStream();
ns.BeginRead(buffer, 0, buffer.Length, EndRead, buffer);
}
public void EndRead(IAsyncResult result)
{
var buffer = (byte[])result.AsyncState;
var ns = tcpClient.GetStream();
var bytesAvailable = ns.EndRead(result);
Console.WriteLine(Encoding.ASCII.GetString(buffer, 0, bytesAvailable));
BeginRead();
}
public void BeginSend(string xml)
{
var bytes = Encoding.ASCII.GetBytes(xml);
var ns = tcpClient.GetStream();
ns.BeginWrite(bytes, 0, bytes.Length, EndSend, bytes);
}
public void EndSend(IAsyncResult result)
{
var bytes = (byte[])result.AsyncState;
Console.WriteLine("Sent {0} bytes to server.", bytes.Length);
Console.WriteLine("Sent: {0}", Encoding.ASCII.GetString(bytes));
}
}
用法:
static void Main(string[] args)
{
var client = new Client();
client.Initialize("127.0.0.1", 8778);
client.BeginRead();
client.BeginSend("<Names><Name>John</Name></Names>");
Console.ReadLine();
}
好吧,我花了 10 秒才找到你本可以解决的最大问题:
public void BeginRead()
{
var buffer = new byte[4096];
var ns = tcpClient.GetStream();
ns.BeginRead(buffer, 0, buffer.Length, EndRead, buffer);
}
但是别担心,这就是我们这样做的原因。
首先让我解释一下为什么这是一个如此大的问题。
假设您发送的消息长度为 4097 字节。您的缓冲区只能接受 4096 字节,这意味着您无法将整个消息打包到该缓冲区中。
假设您发送的消息长度为 12 字节。您仍在内存中分配 4096 字节 只是为了存储 12 字节。
如何处理?
每次使用网络时,您都应该考虑制定某种协议(有些人称之为消息框架,但它只是一个协议)来帮助您识别传入的包。
协议示例可以是:
[1B = 消息类型][4B = 长度][XB = 消息]
- where X == BitConvert.ToInt32(length);
接收者:
byte messageType = (byte)netStream.ReadByte();
byte[] lengthBuffer = new byte[sizeof(int)];
int recv = netStream.Read(lengthBuffer, 0, lengthBuffer.Length);
if(recv == sizeof(int))
{
int messageLen = BitConverter.ToInt32(lengthBuffer, 0);
byte[] messageBuffer = new byte[messageLen];
recv = netStream.Read(messageBuffer, 0, messageBuffer.Length);
if(recv == messageLen)
{
// messageBuffer contains your whole message ...
}
}
发件人:
byte messageType = (1 << 3); // assume that 0000 1000 would be XML
byte[] message = Encoding.ASCII.GetBytes(xml);
byte[] length = BitConverter.GetBytes(message.Length);
byte[] buffer = new byte[sizeof(int) + message.Length + 1];
buffer[0] = messageType;
for(int i = 0; i < sizeof(int); i++)
{
buffer[i + 1] = length[i];
}
for(int i = 0; i < message.Length; i++)
{
buffer[i + 1 + sizeof(int)] = message[i];
}
netStream.Write(buffer);
其余代码看起来没问题。但在我看来,在你的情况下使用异步操作是没有用的。您可以对同步调用执行相同的操作。
这很难回答,因为这里没有确切的问题,但更多的是某种代码审查。但还是有一些提示:
TcpClient.Connected
不会阻塞。因此,当连接正在进行时,它通常会失败,然后您重新开始。您应该切换到使用阻塞或异步 Connect
方法。Async
的 Task
方法,因为它们可以轻松地与 async/await 结合使用。使用 TPL 方法读取代码(未经测试):
public async Task Initialize(string ip, int port)
{
tcpClient = new TcpClient;
await tcpClient.ConnectAsync(ip, port);
Console.WriteLine("Connected to: {0}:{1}", ip, port);
}
public async Task Read()
{
var buffer = new byte[4096];
var ns = tcpClient.GetStream();
while (true)
{
var bytesRead = await ns.ReadAsync(buffer, 0, buffer.Length);
if (bytesRead == 0) return; // Stream was closed
Console.WriteLine(Encoding.ASCII.GetString(buffer, 0, bytesRead));
}
}
在初始化部分你会做:
await client.Initialize(ip, port);
// Start reading task
Task.Run(() => client.Read());
对于使用同步方法,请删除所有
Async
事件并将任务替换为线程。
您缺少一个重要的属性: TcpClient.ReceiveBufferSize = 4096
设置接收缓冲区的大小。
如果消息大于缓冲区,则会自动为您拆分消息,无需任何额外的代码逻辑! 从上面的示例来看,缓冲区大小为 4096,要发送的消息长度为 4097,那么 2 个数据包将到达接收方,一个数据包的长度为 4096(完整缓冲区),另一个数据包的长度为 1 字节。此时,您可以将其放入并发队列中,然后稍后进行检查(以避免在套接字事件内进行编程,这可能会产生不良错误...延迟等)。这里有人说收到的消息的长度=0是异步错误10054,小于0则为其他类型的错误。请记住,避免接收事件上出现过多代码...获取数据。放入队列,此时仅此而已。在服务器端几乎是同样的事情,并且该队列必须是并发队列。在这种情况下,您应该添加一个对象来说明数据属于谁以及部分缓冲区。