C#中UDP Socket接收方法偶发丢失第二个数据包，如何修正？

我看了你的UDP接收代码，确实能发现几个导致丢包的问题，尤其是你提到的“丢失第二个数据包”的情况，咱们一步步来拆解和修复：

现有代码的问题分析 #

先看你写的接收逻辑：

private static byte[] ReceivePlainData(Socket socket) { 
    var recievedData = new List<byte>(); 
    var buffer = new byte[1024]; 
    do { 
        int recievedAmount; 
        while (true) { 
            try { 
                recievedAmount = socket.Receive(buffer); 
            } catch (SocketException) { 
                continue; 
            } 
            break; 
        } 
        recievedData.AddRange(buffer.Take(recievedAmount)); 
    } while (socket.Available > 0); 
    return recievedData.ToArray(); 
}

还有Socket初始化代码：

using (var socket = new Socket(SocketType.Dgram, ProtocolType.Udp) { ReceiveTimeout = 1000 })

主要问题集中在这几点：

• 超时异常处理逻辑错误：当Receive抛出超时异常时，你直接continue回到循环，这会跳过当前的接收尝试——哪怕此时缓冲区已经有数据，或者超时后刚有数据包进来，都会被直接错过。
• socket.Available的不可靠性：Available返回的是当前缓冲区已接收的字节数，但UDP是按完整数据包传输的，Available大于0只能说明有数据，但你无法确定是一个还是多个包；更关键的是，两次Receive之间，新数据包可能还在传输途中，Available会返回0，导致你提前退出循环，丢了后续到达的包。
• 忽略UDP的报文边界特性：UDP是面向报文的，每个Receive调用对应一个完整的UDP数据包，你的代码直接把多个包的内容拼接，不仅可能不符合业务需求，还会因为循环提前退出导致未读取的包直接丢失。

修改后的代码方案 #

针对这些问题，我们可以调整接收逻辑，同时保留超时机制：

方案1：保留UDP数据包边界（推荐，符合UDP特性） #

如果你需要区分每个独立的UDP数据包，用这个方案：

private static List<byte[]> ReceiveUdpPackets(Socket socket)
{
    var receivedPackets = new List<byte[]>();
    var buffer = new byte[1024];
    bool timeoutTriggered = false;

    while (!timeoutTriggered)
    {
        try
        {
            // 接收一个完整的UDP数据包
            int receivedBytes = socket.Receive(buffer);
            // 复制有效数据，避免下一次Receive覆盖当前包内容
            byte[] singlePacket = new byte[receivedBytes];
            Array.Copy(buffer, singlePacket, receivedBytes);
            receivedPackets.Add(singlePacket);
        }
        catch (SocketException ex)
        {
            // 只处理超时异常，其他异常直接抛出（比如网络错误）
            if (ex.SocketErrorCode == SocketError.TimedOut)
            {
                timeoutTriggered = true;
            }
            else
            {
                throw;
            }
        }
    }

    return receivedPackets;
}

方案2：合并所有接收的字节（如果业务需要） #

如果你确定发送端是连续发送小数据包，需要合并成一个字节数组，用这个方案：

private static byte[] ReceiveCombinedUdpData(Socket socket)
{
    var combinedData = new List<byte>();
    var buffer = new byte[1024];
    bool timeoutTriggered = false;

    while (!timeoutTriggered)
    {
        try
        {
            int receivedBytes = socket.Receive(buffer);
            combinedData.AddRange(buffer.Take(receivedBytes));
        }
        catch (SocketException ex)
        {
            if (ex.SocketErrorCode == SocketError.TimedOut)
            {
                timeoutTriggered = true;
            }
            else
            {
                throw;
            }
        }
    }

    return combinedData.ToArray();
}

关键修改说明 #

• 精准处理超时：只在捕获到TimedOut错误时才退出循环，其他Socket异常直接抛出，避免忽略关键的网络错误。
• 移除对socket.Available的依赖：UDP的Available无法可靠判断后续是否有数据包（数据包可能在检查后才到达），我们用超时窗口来控制接收时长，在超时前尽可能接收所有到达的包。
• 避免缓冲区覆盖：方案1中用Array.Copy复制每个数据包的有效内容，防止下一次Receive操作覆盖当前包的数据。

另外要注意：UDP本身是不可靠协议，如果网络环境差，仍可能出现丢包，但你遇到的是代码逻辑导致的丢包，上面的修改可以解决“丢失第二个数据包”的问题。

内容的提问来源于stack exchange，提问作者Jake Smokie