c# – 处理MenuItem Click事件的最佳方法?

我想知道为MenuItems处理30个左右Click事件的最佳方法是什么?

我的第一个想法显然是为每个MenuItem创建一个事件监听器,如下所示:

XAML:

<Menu Name="MainMenu" IsMainMenu="True" Height="25">
        <MenuItem Header="_File" >
            <MenuItem Name="New" Header="_New" Click="MenuItem_NewClick" />
            <MenuItem Name="Open" Header="_Open" Click="MenuItem_OpenClick" />
            <MenuItem Name="Save" Header="_Save" Click="MenuItem_SaveClick" />
        </MenuItem>
</Menu>

C#1:

private void MenuItem_NewClick(object sender, RoutedEventArgs e)
{           
    //Do work...
}
private void MenuItem_OpenClick(object sender, RoutedEventArgs e)
{
    //Do work...
}
private void MenuItem_SaveClick(object sender, RoutedEventArgs e)
{
    //Do work...
}

但这看起来很混乱,特别是对于不需要像Copy或Paste这样的代码的MenuItems.

我可以使用一个事件监听器并使用if / case来检查MenuItem并消除所有额外的事件监听器,如下所示:

C#2:

private void MenuItem_FileClick(object sender, RoutedEventArgs e)
    {
        MenuItem item = e.OriginalSource as MenuItem;
        switch (item.Name)
        {
            case "New":
                MessageBox.Show("New File Created.");
                break;
            case "Open":
                MessageBox.Show("File Opened Created.");
                break;
            case "Save":
                MessageBox.Show("File Saved.");
                break;
        }
    }

这似乎更清晰,但同时也是凌乱如果MenuItems需要做更多的工作,而不仅仅是打开另一个表单或清除一些控件.如果需要复杂的逻辑,我可以调用一个函数,但我认为这会导致意大利面条代码?

处理这个问题最优雅的方法是什么?

谢谢,
皮匠

最佳答案 我经常做这种事情来设置动作,我试图将UI与正在执行的动作分开.

        var actions = new Dictionary<string, Func<MenuItem, RoutedEventHandler>>()
        {
            { "New", mi => (s, e) => { MessageBox.Show("New File Created."); }},
            { "Open", mi => (s, e) => { MessageBox.Show("File Opened."); }},
            { "Save", mi => (s, e) => { MessageBox.Show("File Saved."); }},
        };

        foreach (MenuItem mi in FileMenu.Items)
        {
            if (actions.ContainsKey(mi.Name))
            {
                mi.Click += actions[mi.Name](mi);
            }
        }

关于这一点的好处是你明确地拥有对mnu项的强类型引用.例如,在定义每个项目时,您可以执行此操作:

            {
                "New",
                mi =>
                    (s, e) =>
                    {
                        MessageBox.Show("New File Created.");
                        MessageBox.Show(
                            String.Format("You clicked the {0} menu.", mi.Name));
                    }
            },

您可以看到mi变量传递给lambda表达式.

退出表单时,您需要执行相反的操作来分离事件处理程序,但这并不比附加代码更难.如果你很聪明,你可以在编写附件时编写准备分离的代码,以便以后轻松分离.

点赞