WinForm托盘图标ContextMenu第二次打开后立即关闭问题求助

问题根因 #

你遇到的问题由两个典型的WinForms开发错误共同导致，这也解释了你为什么第一次打开菜单正常、后续打开就会闪现消失且Collapse事件不触发的现象：

• 缺少主UI消息循环：WinForms依赖Application.Run()启动消息泵处理所有UI交互事件，你当前的Main方法完成初始化后直接退出，没有启动消息循环，所有UI控件的事件响应逻辑都会异常。
• 跨线程操作UI：后台ping工作线程直接修改NotifyIcon的图标、提示文本，以及MenuItem的显示内容，违反了WinForms"所有UI控件操作必须在创建它的UI线程执行"的要求，会导致UI消息队列紊乱，进而出现菜单被强制销毁的异常表现。

修复步骤 #

1. 补全主消息循环 #

修改Main方法，在初始化完成后调用Application.Run()启动消息泵：

[STAThread]
static void Main()
{
    Application.EnableVisualStyles();
    Application.SetCompatibleTextRenderingDefault(false);
    PingGoogleDNS pinger = new PingGoogleDNS();
    pinger.StartPinger();
    // 新增：启动UI消息循环
    Application.Run();
}

2. 修复跨线程UI操作 #

把后台线程中所有修改UI的逻辑，通过ISynchronizeInvoke封送到UI线程执行，这里我们可以直接复用pingIcon作为同步调用的发起对象：
首先在PingGoogleDNS类里加一个辅助方法，用来安全更新UI：

private void InvokeIfNeeded(Action action)
{
    if (pingIcon != null && pingIcon.SynchronizingObject.InvokeRequired)
    {
        pingIcon.SynchronizingObject.Invoke(action);
    }
    else
    {
        action();
    }
}

然后替换PingSenderThread里所有直接修改UI的代码：

// 网络正常分支的UI修改替换为：
InvokeIfNeeded(() => 
{
    if (reply.RoundtripTime < 500)
        pingIcon.Icon = goodIcon;
    else
        pingIcon.Icon = weakIcon;

    pingIcon.Text = $"8.8.8.8 ping success! ({reply.RoundtripTime} ms)";
    roundTripTime.Text = $"Latest ping: {reply.RoundtripTime} ms";
});

// 网络异常分支的UI修改也同样替换：
InvokeIfNeeded(() => 
{
    pingIcon.Icon = noIcon;
    pingIcon.Text = "8.8.8.8 ping failure.";
    roundTripTime.Text = "Latest ping: N/A ms";
});

3. 优化退出逻辑（可选） #

Thread.Abort()会强制抛出线程终止异常，可能导致资源泄漏，建议改用CancellationTokenSource来控制后台线程退出：
首先在类里新增CancellationTokenSource成员：

private CancellationTokenSource _cts = new CancellationTokenSource();

创建线程的时候传入token，循环中判断token是否取消，退出的时候调用_cts.Cancel()即可。

完成以上修改后菜单就不会出现闪现消失的问题，Collapse事件也会正常触发。

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