C# Socket线程实现本地P2P通信遇循环问题，点击时应用冻结

嘿，我看了你这段代码，一眼就瞅出问题出在哪儿了——这个无限while(true)循环直接把UI线程给堵死了！咱们来一步步拆解问题，再给你修正方案：

问题根源分析 #

你这段代码犯了两个关键错误：

1. 把阻塞操作放在了UI线程：listener.AcceptTcpClient()是个阻塞方法，它会一直等着客户端连接，要是在UI线程里跑这个，UI根本没法处理用户的点击、刷新这些操作，直接就冻住了。
2. 反复创建TcpListener：你每次循环都新建一个TcpListener并调用Start()，这不仅浪费资源，还可能导致端口被重复占用，引发额外的异常。

修复后的代码示例 #

// 把监听器初始化移到循环外面，只做一次
TcpListener listener = new TcpListener(IPAddress.Parse(STD_IP), HOST_PORT);
listener.Start();

// 把整个监听逻辑放到后台线程里，别占着UI线程
Thread listenThread = new Thread(() =>
{
    while (true)
    {
        try
        {
            // AcceptTcpClient在这里阻塞，但因为是后台线程，不会影响UI
            using (TcpClient client = listener.AcceptTcpClient())
            using (NetworkStream nwStream = client.GetStream())
            {
                // 处理接收消息的线程
                Thread receiveThread = new Thread(() =>
                {
                    string message = Receive(nwStream);
                    // 跨线程更新UI，必须用Invoke切换到UI线程
                    this.Invoke((Action)(() =>
                    {
                        updateBoard(message);
                    }));
                });
                receiveThread.Start();

                // 发送响应
                string response = stanje;
                Send(nwStream, response);
            }
        }
        catch (Exception er)
        {
            // 同样，弹窗也要在UI线程里执行
            this.Invoke((Action)(() =>
            {
                MessageBox.Show($"Communication error:\n{er.Message}\n{er.StackTrace}");
            }));
        }
    }
});
// 设置为后台线程，这样程序关闭时线程会自动终止
listenThread.IsBackground = true;
listenThread.Start();

关键改动说明 #

• 监听器只初始化一次：TcpListener只需要创建并启动一次，它可以持续监听端口上的新连接，没必要每次循环都重建。
• 监听逻辑移到后台线程：把整个无限循环和阻塞的AcceptTcpClient放到后台线程里，让UI线程能专心处理用户交互，再也不会冻结。
• 跨线程更新UI：updateBoard和MessageBox.Show都是UI操作，必须通过Invoke切换到UI线程执行，否则会引发跨线程操作异常。

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