您会在长开关/枚举声明中使用区域吗？

发布于 2024-07-25 09:19:41 字数 703 浏览 5 评论 0原文

我最近发现自己需要（是的，需要）在 C# 代码中定义荒谬的长 switch 语句和 enum 声明，但我想知道人们认为最好的是什么将它们分成逻辑小节的方法。在我的情况下，枚举值和案例（基于枚举值）都有相当清晰的分组，但我有点不确定如何在代码中反映这一点。

请注意，在我的代码中，我大约有 5 组，每组有 10 到 30 个枚举值/案例。

我可以设想的三个模糊合理的选项是：

在声明中围绕所有逻辑组/枚举值定义 #region 块（可以选择用空行分隔）。
用其名称注释每个组，每个组名称注释前有一个空行。
不执行任何操作 - 只需将开关/枚举保留为大量案例/值列表即可。

你更喜欢哪一个？你会分开对待枚举和开关吗？（这对我来说似乎有点奇怪。）现在，我不会说这个问题有任何正确/错误的答案，尽管我仍然很有兴趣听到普遍共识的观点是什么。

注 1： 不幸的是，这种情况是不可避免的（与 switch 类似），因为我正在尝试编写一个词法分析器（标记器），因此，出于多种原因，这似乎是最合理的方法。

注2：我完全意识到关于是否在通用代码中使用区域（主要用于构造类）的问题已经存在几个重复的问题，但我觉得我在这里的问题更加具体和尚未得到解决。

原文

分享到QQ

分享到微博

如果你对这篇内容有疑问，欢迎到本站社区发帖提问参与讨论，获取更多帮助，或者扫码二维码加入 Web 技术交流群。

发布评论

需要登录才能够评论，你可以免费注册一个本站的账号。

昔梦 2024-08-01 09:19:42

当然，将这些事情划分出来。它们可能不会发生太大变化，当它们发生变化时，您可以扩展该区域，进行更改，折叠它，然后继续处理文件的其余部分。

它们的存在是有原因的，利用它们为你带来优势。

回复收藏 0 原文

总以为 2024-08-01 09:19:42

您还可以有一个 Dictionary<[your_enum_type], Action> （或 Func 而不是 Action）或类似的东西（考虑到您的函数具有类似的签名）。然后你可以不使用开关，而是：

        switch (item)
        {
            case Enum1: func1(par1, par2)
                break;
            case Enum2: func2(par1, par2)
                break;
        }

你可以有类似的东西：

public class MyClass
{
    Dictionary<int, Action<int, int>> myDictionary;
    //These could have only static methods also
    Group1Object myObject1;
    Group2Object myObject2;

    public MyClass()
    {
        //Again, you wouldn't have to initialize if the functions in them were static
        myObject1 = new Group1Object();
        myObject2 = new Group2Object();
        BuildMyDictionary();
    }

    private Dictionary<int, Action<int, int>> BuildMyDictionary()
    {
        InsertGroup1Functions();
        InsertGroup2Functions();
        //...
    }

    private void InsertGroup2Functions()
    {
        myDictionary.Add(1, group2.AnAction2);
        myDictionary.Add(2, group2.AnotherAction2);
    }

    private void InsertGroup1Functions()
    {
        myDictionary.Add(3, group1.AnAction1);
        myDictionary.Add(4, group1.AnotherAction1);
    }


    public void DoStuff()
    {
        int t = 3; //Get it from wherever
        //instead of switch
        myDictionary[t](arg1, arg2);
    }
}

You could also have a Dictionary<[your_enum_type], Action> (or Func instead of Action) or something like that (considering your functions have a similar signature). Then you could instead of using a switch, instead of:

        switch (item)
        {
            case Enum1: func1(par1, par2)
                break;
            case Enum2: func2(par1, par2)
                break;
        }

you could have something like:

public class MyClass
{
    Dictionary<int, Action<int, int>> myDictionary;
    //These could have only static methods also
    Group1Object myObject1;
    Group2Object myObject2;

    public MyClass()
    {
        //Again, you wouldn't have to initialize if the functions in them were static
        myObject1 = new Group1Object();
        myObject2 = new Group2Object();
        BuildMyDictionary();
    }

    private Dictionary<int, Action<int, int>> BuildMyDictionary()
    {
        InsertGroup1Functions();
        InsertGroup2Functions();
        //...
    }

    private void InsertGroup2Functions()
    {
        myDictionary.Add(1, group2.AnAction2);
        myDictionary.Add(2, group2.AnotherAction2);
    }

    private void InsertGroup1Functions()
    {
        myDictionary.Add(3, group1.AnAction1);
        myDictionary.Add(4, group1.AnotherAction1);
    }


    public void DoStuff()
    {
        int t = 3; //Get it from wherever
        //instead of switch
        myDictionary[t](arg1, arg2);
    }
}

回复收藏 0 原文