处理 MenuItem 单击事件的最佳方法？

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...
}

但这似乎很混乱，尤其是对于不需要太多代码（如复制或粘贴）的 MenuItem。

我可以改为使用一个事件侦听器并使用 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 表达式中。

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