Question

我在visual studio中创建了一个C＃方法，它只包含一个switch语句，每个case返回一个值。根据个人习惯，我提出了类似于以下代码的内容：

private string SwitchMethod(int num)
    {
        switch (num)
        {
            case 0:
                return "result 1";
            case 1:
                return "result 2";
            case 2:
                return "result 3";
        }
        return "no result";
    }

我的问题是：哪个代码会有更好的性能？上面或下面的代码，或者是相同的？为什么？

我认为由于编译器优化......它们可能只是相同......但我真的不知道。

private string SwitchMethod(int num)
    {
        switch (num)
        {
            case 0:
                return "result 1";
            case 1:
                return "result 2";
            case 2:
                return "result 3";
            default:
                return "no result";
        }
    }

REVISION：看来我应该更具体一点：编译时......一个或另一个会生成效率较低的代码吗？

我意识到性能上的差异可能微不足道......我真的很好奇。

Answer 1

public static string foo(int num)
        {
            switch (num)
            {
                case 0:
                    return "result 1";
                case 1:
                    return "result 2";
                case 2:
                    return "result 3";
            }
            return "no result";
        }

变为：

.method public hidebysig static string  foo(int32 num) cil managed
{
  // Code size       57 (0x39)
  .maxstack  1
  .locals init ([0] string CS$1$0000,
           [1] int32 CS$4$0001)
  IL_0000:  nop
  IL_0001:  ldarg.0
  IL_0002:  stloc.1
  IL_0003:  ldloc.1
  IL_0004:  switch     ( 
                        IL_0017,
                        IL_001f,
                        IL_0027)
  IL_0015:  br.s       IL_002f
  IL_0017:  ldstr      "result 1"
  IL_001c:  stloc.0
  IL_001d:  br.s       IL_0037
  IL_001f:  ldstr      "result 2"
  IL_0024:  stloc.0
  IL_0025:  br.s       IL_0037
  IL_0027:  ldstr      "result 3"
  IL_002c:  stloc.0
  IL_002d:  br.s       IL_0037
  IL_002f:  ldstr      "no result"
  IL_0034:  stloc.0
  IL_0035:  br.s       IL_0037
  IL_0037:  ldloc.0
  IL_0038:  ret
} // end of method Program::foo

将返回移至默认情况：

.method public hidebysig static string  foo(int32 num) cil managed
{
  // Code size       57 (0x39)
  .maxstack  1
  .locals init ([0] string CS$1$0000,
           [1] int32 CS$4$0001)
  IL_0000:  nop
  IL_0001:  ldarg.0
  IL_0002:  stloc.1
  IL_0003:  ldloc.1
  IL_0004:  switch     ( 
                        IL_0017,
                        IL_001f,
                        IL_0027)
  IL_0015:  br.s       IL_002f
  IL_0017:  ldstr      "result 1"
  IL_001c:  stloc.0
  IL_001d:  br.s       IL_0037
  IL_001f:  ldstr      "result 2"
  IL_0024:  stloc.0
  IL_0025:  br.s       IL_0037
  IL_0027:  ldstr      "result 3"
  IL_002c:  stloc.0
  IL_002d:  br.s       IL_0037
  IL_002f:  ldstr      "result 4"
  IL_0034:  stloc.0
  IL_0035:  br.s       IL_0037
  IL_0037:  ldloc.0
  IL_0038:  ret
} // end of method Program::foo

完全一样。没有性能差异。为了确保代码重新生成，我将“无结果”更改为结果4。显然，C＃编译器对它进行了优化，或者它最终只是等效的。

Answer 2

优良作法是始终在交换机的基础上包含默认情况，以防其他任何情况都无效。

这不是一个效率问题，因为如果之前的一个案例被击中 - 你的程序将不会检查任何其他情况（它相当于使用if / else if / else - 其中最后的else是默认值）。

希望这有帮助。

Answer 3

使用通用字典而不是函数和switch语句，其中键是KofaxEnvironment，其值与从交换机返回的值相同。类似的东西：

Dictionary<KofaxEnvironment, string>

或

Dictionary<int, string>

我也不担心表现。正确应该是你的第一个目标。

但是，如果您坚持使用开关，请使用抛出异常的默认值：

default:
    throw new ArgumentException("Serious programmer error!");

至于性能，开关默认值和下降到返回值之间的差异（如果有的话）可以忽略不计。

C＃Switch语句：更高效不使用默认值？

3 个答案: