1

在我的程序中，我有一个列表框，当用户双击一个对象时，它会查看 switch 语句以查看应该发生的事件。随着列表开始变大，我很好奇是否有办法避免必须在 2 个位置维护对象列表（一次在添加到列表框的列表中，一次在 switch 语句中。有没有办法索引/读取/存储我的 switch 语句的各种案例，然后将它们作为对象添加到我的列表框中？

示例：（不起作用，只是一个理论）

Switch (n)
ForEach (Case c in Cases)
{
   arrayCases.Add(c);
}
listbox.Items.AddRange(arrayCases);

编辑：

继续我现在的字典建议：

public void SetDictionary()
    {
       //add entries to the dictionary
        dict["cat"] = new Action(Cat);
        dict["dog"] = new Action(Dog);

        //add each dictionary entry to the listbox.
        foreach (string key in dict.Keys)
        {
            listboxTest.Items.Add(key);
        }                            
    }

     //when an item in the listbox is double clicked
     private void listboxTest_DoubleClick(object sender, EventArgs e)
     {
         testrun(listboxCases.SelectedItem.ToString());             
     }

     public void testrun(string n)
     {
         //this is supposed to receive the item that was double clicked in the listbox, and run it's corresponding action as defined in the dictionary.
         var action = dict[n] as Action action();
     }

我相信我上面的代码大部分是正确的并且我理解它，但是操作行： var action = dict[n] as Action action();

显示一个错误，指出“操作”需要一个“;”。我这里的逻辑准确吗？如果是这样，为什么动作调用不正确？

4

5 回答 5

4

Dictionary<string, Action>是避免的方法。Dictionary.Keys变成ListBox.Items.

switch(n) 变成

var action = dict[n] as Action
action();
于 2013-01-09T19:06:18.220 回答
3

我建议将您的操作移动到单独的类中。为您的操作创建一个基类，如下所示。我为表单添加了一个字段，因为您可能必须与表单进行交互。如果需要，您还可以传入其他对象。

internal abstract class Operation
{
    protected readonly MyForm form = null;

    protected Operation(MyForm form)
    {
        this.form = form;
    }

    public abstract String DisplayName { get; }

    internal abstract void Execute();
}

然后为每个操作派生一个类。

internal sealed class DoThis : Operation
{
    internal DoThis(MyForm form) : base(form) { }

    public override String DisplayName
    {
        get { return "Do this!"; }
    }

    internal override void Execute()
    {
        // Code to do this. You can use this.form to interact with
        // your form from this operation.
    }
}

internal sealed class DoSomethingElse : Operation
{
    internal DoSomethingElse(MyForm form) : base(form) { }

    public override String DisplayName
    {
        get { return "Do something else!"; }
    }

    internal override void Execute()
    {
        // Code to do something else.
    }
}

现在您可以将所有操作添加到列表框中

this.lsitBox.Items.Add(new DoThis(this));
this.lsitBox.Items.Add(new DoSomethingElse(this));

并设置显示成员属性。

this.listBox.DisplayMember = "DisplayName";

最后在事件处理程序中执行选中的操作。

((Operation)this.listBox.SelectedItem).Execute();

这种模式在所有操作之间提供了清晰的分离，并使未来的扩展变得简单而干净。例如，CanExecute如果您必须检查某个操作当前是否可用，您可以将一个属性添加到所有操作。或者，如果您必须支持本地化，则可以轻松添加逻辑以在当前 UI 语言中显示操作名称。

另一个容易支持的场景是，如果您有一些所有操作共有的代码，例如日志记录、安全检查、性能测量等。

internal abstract class Operation
{
    protected readonly MyForm form = null;

    protected Operation(MyForm form)
    {
        this.form = form;
    }

    public abstract String DisplayName { get; }

    protected abstract void ExecuteCore();

    internal void Execute()
    {
        Logger.Log("Executing operation " + this.DisplayName);

        try
        {
            this.ExecuteCore();

            Logger.Log("Executing operation " + this.DisplayName + " succeeded.");
        }
        catch (Exception exception)
        {
            Logger.Log("Executing operation " + this.DisplayName + " failed.", exception);

            throw;
        }
    }
}

请注意，您现在必须覆盖ExecuteCore()而不是Execute().

最后一个想法 - 使用接口IOperation代替或与抽象基类结合使用也可能会有所帮助。这消除了所有操作都从同一个基类继承的需要，因为这有时可能不方便。但我省略了这一点，以免过度设计。

于 2013-01-09T19:27:20.980 回答
2

是的，有一种方法可以通过制作 lambda 字典来做到这一点。

void Main()
{
  // set up your dictionary
  Dictionary<string,Action> myList = new Dictionary<string,Action> {
     { "one", () => { Console.WriteLine("One function"); } },
     { "two",  () => { Console.WriteLine("Two function"); }},
     { "three", () => { Console.WriteLine("Three function"); }}
  };

  // do a "switch" (that is invoke a function that corresponds to a name)
  myList["one"]();

  // loop the list of keys (that is get a list of all the names)
  foreach (string key in myList.Keys)
    Console.WriteLine(key);
}

该程序的输出：

One function
one
two
three

还要注意——你可以像这样动态地添加到这个“开关”中（这很酷，而且你不能用经典的 switch 语句来做。）

myList.Add("four",() => { Console.WriteLine("Four function is dynamic"); });
于 2013-01-09T19:02:35.653 回答
2

你不能*用普通代码枚举case。switch

您可以做的是将“动作名称”映射替换switch为“动作处理程序”，然后您将能够将此映射重用于动作名称列表框列表。有关示例，请参阅 Tilak 的答案。

*) 如果你真的很好奇，你可以列举switch. C# 代码被转换为 IL 并且 IL 可以用代码读取。因此，您可以为方法获取 IL，为 IL 编写（或获取现有的 - Parser for C#）解析器并在方法内部找到实现switch，选择所有情况。您甚至可以在构建时直接访问 C# 源代码 - 但它比 IL 解析更复杂。

于 2013-01-09T19:03:26.730 回答
1

在我看来，您的交换机中的案例数量将会发生很大变化。如果这是真的，那么您可能需要考虑使用 switch 语句以外的机制。也许您想做一些像 Alexi Levenkov 建议的事情，然后迭代存储的动作名称列表并执行关联的处理程序。这样，您将避免必须将动作名称添加到动作映射，然后将其添加到开关。

于 2013-01-09T19:06:59.110 回答