c# - 处理 MenuItem Click 事件的最佳方式？

Question

我想知道处理 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...
}

但这看起来很乱，特别是对于不需要太多代码(如复制或粘贴)的菜单项。

我可以改为使用 one 事件监听器并使用 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 需要做更多的工作而不只是打开另一个窗体或清除几个控件，那么它也会变得困惑。如果需要复杂的逻辑，我可以调用一个函数，但我认为这会导致意大利面条式代码？

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

谢谢，坦纳

Answer 1

我经常做这种事情来设置操作，我试图将 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 表达式。

当您退出表单时，您确实需要执行相反的操作来分离您的事件处理程序，但这并不比附加代码难。如果您很聪明，您可以编写代码，在执行附加操作时准备分离，以便以后非常容易分离。