C# 简化switch语句

Question

我可以以某种方式简化此 switch 语句，因为两种情况都只使用另一个 function 参数做同样的事情吗？

switch (data.Subscriber.Protocol)
{
    case "email json":
        builder.Attachments.Add("Očitanje.json", CraftAttachment(data));
        break;
    case "email text":
        builder.Attachments.Add("Očitanje.txt", CraftAttachment(data));
        break;
    default:
        break;
}

Answer 1

像这样的东西怎么样：

string attachmentName = data.Subscriber.Protocol switch
{
    "email json" => "Očitanje.json",
    "email text" => "Očitanje.txt",
    _ => null
};

if (attachmentName is not null)
{
    builder.Attachments.Add(attachmentName, CraftAttachment(data));
}

开关表达式 | C# 参考

Answer 2

//另一种不使用 Swtich 案例的干净方法：

var ProtocolAndFileMappings = new Dictionary<string, string>()
    {
       {"email json","Očitanje.json"},
       {"email text","Očitanje.json"},
       {"email png","Očitanje.png"},
       {"email Jpeg","Očitanje.Jpeg"}
     };

 builder.Attachments.Add(ProtocolAndFileMappings[data.Subscriber.Protocol], CraftAttachment(data));

Answer 3

另一种使用本地 function 来简化调用的方法：

void add(string s) => if (s != null) builder.Attachments.Add(s, CraftAttachment(data));

add( data.Subscriber.Protocol switch
{
    "email json" => "Očitanje.json",
    "email text" => "Očitanje.txt",
    _ => null
});

（虽然我认为有些人会批评它“太可爱了......）

注意：此解决方案（与其他解决方案一样）存在缺陷。

该代码将始终对 null 进行额外测试，而直接开关不会这样做 - 所以这（非常边缘）效率较低。

我个人会这样做（避免了缺点）：

void add(string s) => builder.Attachments.Add(s, CraftAttachment(data));

switch (data.Subscriber.Protocol)
{
    case "email json": add("Očitanje.json"); break;
    case "email text": add("Očitanje.txt")   break;
    default:           /* Do nothing */      break;
}    

像这样的简单本地函数很可能会被 JIT 编译器内联，所以应该没有开销。