c＃switch语句返回适合替换中断

Question

这是处理c＃switch语句的合适方式，还是需要显式中断吗？ reference

public static string ToRegistryString(AliceKey.AliceKeyPaths aliceKeyPath) 
    { 
    switch (aliceKeyPath) 
     { 
      case AliceKey.AliceKeyPaths.NET_CLR_DATA: 
       return @"\.NET CLR Data\"; 
      case AliceKey.AliceKeyPaths.NET_CLR_NETWORKING: 
       return @"\.NET CLR Networking\"; 
      case AliceKey.AliceKeyPaths.NET_DATA_PROVIDER_MSSQL: 
       return @"\.NET Data Provider for SqlServer\"; 
      case AliceKey.AliceKeyPaths.NET_DATA_PROVIDER_ORACLE: 
       return @"\.NET Data Provider for Oracle\"; 
     } 
     return new string(new char[0]); 
    } 

Answer 1

这很好。重点是，一个案例块的结束应该是无法访问 - 它在这里，因为你已经返回。

为什么你要返回new string(new char[0])而不是只是“”或string.Empty虽然？如果你试图确保每次都是不同的字符串，你实际上会遇到一个非常奇怪的角落案例 - 尽管调用new string(...)代码将始终实际返回相同的参考...

最后：我会其实建议改变这个开关/ case块逼到Dictionary<AliceKey.AliceKeyPaths, string>：

private static readonly Dictionary<AliceKey.AliceKeyPaths, string> RegistryMap = 
    new Dictionary<AliceKey.AliceKeyPaths, string> 
{ 
    { AliceKey.AliceKeyPaths.NET_CLR_DATA, @"\.NET CLR Data\" }, 
    { AliceKey.AliceKeyPaths.NET_CLR_NETWORKING, @"\.NET CLR Networking\" }, 
    // etc 
}; 

public static string ToRegistryString(AliceKey.AliceKeyPaths aliceKeyPath) 
{ 
    string value; 
    return RegistryMap.TryGetValue(aliceKeyPath, out value) ? value : ""; 
} 

Answer 2

你不需要专门使用一个break声明只是一个改变控制的流程，所以goto或return应该工作。

查看MSDN了解更多信息：http://msdn.microsoft.com/en-us/library/06tc147t(VS.71).aspx