使这个代码片段变得更好

Question

我有一个包含以下代码的程序：

foreach (string section in DataAccessLayer.AcceptedSections)
{
    switch (section)
    {
        case "Section1":
            Console.WriteLine("Section 1");
            break;
        case "Section2":
            Console.WriteLine("Section 2");
            break;
        case "Section3":
            Console.WriteLine("Section 3");
            break;
        default:
            Console.WriteLine("Default section");
            break;
    }                    
}

我是否可以执行此代码的操作，而无需在案例中再次提供该部分的字符串？ DataAccessLayer.AcceptedSections 是动态的，我不想必须在我的代码中添加另一个部分案例，每次有新部分出现时重建和重新部署。今天是星期五，我的思维不太好。

例如： 当第 4 节添加到数据库时，我不想添加以下代码：

case "Section4":
    Console.WriteLine("Section 4");
     break;

Answer 1

如果字符串始终是“SectionN”，您可以直接处理它：

if (section.StartsWith("Section"))
    Console.WriteLine(section.Insert(7, " "));
else
    Console.WriteLine("Default Section");

Answer 2

有一个由 section 键控的 Dictionary>。这将完全取代 switch 语句。

调用相应的动作：

foreach (string section in DataAccessLayer.AcceptedSections)
{
    myActionsDictionary[section]();
}

Answer 3

如果这都是数据驱动的，我建议您从数据库中返回一些其他显示值以及标识符字符串

Table AcceptedSections

Name = "Section1"
DisplayName = "Section 1"

然后您可以只返回 DisplayName

---

如果它是不是您必须像现在一样处理这个问题，或者您可以创建一个带有用于显示的属性的枚举：

public enum AcceptedSections
{
    [Description("Default Section")]
    Default,
    [Description("Section 1")]
    Section1,
    [Description("Section 2")]
    Section2,
    [Description("Section 3")]
    Section3,
    [Description("Section 4")]
    Section4
}
// writing this made me kind woozy... what a terrible enum

这将允许您编写如下内容：

foreach (AcceptedSections section in AcceptedSections.GetValues())
{
    Console.WriteLine(section.GetDescription());
}

其中 GetDescription() 是返回枚举上的自定义属性的简单方法