Оператор switch С# является возвратом, подходящим для замены break

Является ли это подходящим способом обработки операторов switch С# или требуется еще один явный разрыв? ссылка

  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]);
     }

Ответы

Ответ 1

Это хорошо. Дело в том, что конец блока case должен быть недоступен - он здесь, потому что вы вернулись.

Почему вы возвращаете new string(new char[0]), а не только "" или string.Empty? Если вы каждый раз пытаетесь сделать это по-разному, вы на самом деле столкнулись с очень странным угловым случаем - несмотря на вызов new string(...), этот код всегда будет возвращать ту же ссылку...

Наконец: я бы предпочел изменить этот блок switch/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 : "";
}

Ответ 2

Вам не нужно специально использовать оператор break только тот, который изменяет поток управления, поэтому должны работать goto или return.

См. MSDN для получения дополнительной информации: http://msdn.microsoft.com/en-us/library/06tc147t(VS.71).aspx