我想知道为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表达式.
退出表单时,您需要执行相反的操作来分离事件处理程序,但这并不比附加代码更难.如果你很聪明,你可以在编写附件时编写准备分离的代码,以便以后轻松分离.