处理菜单项点击事件的最佳方法是什么？

Question

处理菜单项点击事件的最佳方法是什么？

6

我想知道处理大约30个菜单项的Click事件最好的方式是什么？

我的第一个想法显然是为每个菜单项创建一个事件侦听器，如下所示：

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这样不需要太多代码的菜单项。

相反，我可以使用一个事件监听器，并使用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;
        }
    }

这种方式看起来更加干净，但如果菜单项需要完成的工作不仅仅是打开另一个窗体或清空几个控件，那么它也会变得混乱。如果需要复杂的逻辑处理，我可以调用一个函数，但我认为这会导致代码混乱难懂。

如何以最优雅的方式解决这个问题？

谢谢， Tanner

- Tanner.R

1个回答

网页内容由stack overflow 提供, 点击上面的

可以查看英文原文，
原文链接

- Enigmativity · Accepted Answer

我经常这样做，设置操作以将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表达式中。

当你退出表格时，你需要执行相反的操作来分离事件处理程序，但这并不比附加代码更难。如果你足够聪明，你可以编写代码，在附加代码时准备分离，使其非常容易以后分离。