我正在测试我的应用程序中遇到的一个奇怪的错误,最终能够创建一个简单的复制品:

using System;
using System.Windows.Forms;

static class Program
{
    [STAThread]
    static void Main()
    {
        Application.EnableVisualStyles();
        Application.SetCompatibleTextRenderingDefault(false);
        var notifyIcon1 = new NotifyIcon();
        notifyIcon1.Icon = new Form().Icon;
        notifyIcon1.Visible = true;
        var contextMenuStrip1 = new ContextMenuStrip();
        ToolStripMenuItem menu1 = new ToolStripMenuItem();
        menu1.Text = "test";
        contextMenuStrip1.Items.Add(menu1);
        contextMenuStrip1.Items.Add("t1");
        contextMenuStrip1.Items.Add("t2");

        notifyIcon1.ContextMenuStrip = contextMenuStrip1;
        var timer = new System.Timers.Timer();
        timer.Interval = 3000;
        timer.Elapsed += (sender, e) =>  /* Runs in a different thread from UI thread.*/
        {
            if (contextMenuStrip1.InvokeRequired)
                contextMenuStrip1.Invoke(new Action(() =>
                {
                    menu1.DropDownItems.Add(e.SignalTime.ToString() + "extra");
                    menu1.DropDownItems.Add(e.SignalTime.ToString());
                }));
            else
            {
                menu1.DropDownItems.Add(e.SignalTime.ToString() + "extra");
                menu1.DropDownItems.Add(e.SignalTime.ToString());
            }
        };
        timer.Start();
        Application.Run();
    }
}


请注意,上下文菜单不会打开,但是执行以下任何一项操作都可以打开它:


删除每次执行添加的“额外”下拉项。 (准确地说
每次执行仅添加0或1)
删除InvokeRequired == false上的部分代码(这允许每次执行添加多个项目)
删除t1t2元素。 (它仍然可以在没有
根目录中的其他项)


这是一个错误还是我做错了什么?

编辑:
找到的其他条件(由于@derape):


如果将else分支移到单独的方法,则可以使用,但是如果在InvokeRequired分支中使用相同的方法,则不能使用。但是,使用具有不同名称和相同代码的2种方法有效。


可能的解决方法是在满月跳舞时穿老虎的皮。

最佳答案

如果查看InvokeRequired,则会看到对IsHandleCreated的明确检查,该检查返回了false。返回的值并不意味着您不必调用,而仅意味着您不能调用。

使您更加困惑:您必须调用,但是还不能。

如果尚未创建句柄(并且只是错过项目),您可以决定不执行任何操作,或者组织单独的队列来存储项目直到句柄可用,类似于:

var items = new List<string>();
timer.Elapsed += (sender, e) =>
{
    if (contextMenuStrip1.IsHandleCreated)  // always invoke, but check for handle
        contextMenuStrip1.Invoke(new Action(() =>
        {

            menu1.DropDownItems.Add(e.SignalTime.ToString() + "extra");
            menu1.DropDownItems.Add(e.SignalTime.ToString());
            contextMenuStrip1.Refresh();
        }));
    else
    {
        lock (items)
        {
            items.Add(e.SignalTime.ToString() + "extra");
            items.Add(e.SignalTime.ToString());
        }
    }
};
contextMenuStrip1.HandleCreated += (s, e) =>
{
    lock (items)
    {
        foreach (var item in items)
            menu1.DropDownItems.Add(item);
        contextMenuStrip1.Refresh();
    }
    items = null;
};


另一个注意事项:如果在打开菜单的同时向子菜单添加了项目,则需要调用Refresh,但子菜单尚未打开,这是Winforms的奇怪之处。

10-04 14:35