Как упростить этот кусок кода?

Question

У вас есть идея, как упростить этот простой «механизм перевода»?

Является ли Хэш-стол полезным?

char translateChar(char strIn) 
    { 
     char strOut = '?'; 

     if (strIn == 'A') strOut = '1'; 
     else if (strIn == 'B') strOut = '2'; 
     else if (strIn == 'C') strOut = '3'; 
     else if (strIn == 'D') strOut = '4'; 
     else if (strIn == 'E') strOut = '5'; 
     else if (strIn == 'F') strOut = '6'; 
     else if (strIn == 'G') strOut = '7'; 
     else if (strIn == 'H') strOut = '8'; 
     else if (strIn == 'I') strOut = '9'; 
     else if (strIn == 'J') strOut = '@'; 
     else if (strIn == 'K') strOut = 'A'; 
     else if (strIn == 'L') strOut = 'B'; 
     else if (strIn == 'M') strOut = 'C'; 
     else if (strIn == 'N') strOut = 'D'; 
     else if (strIn == 'O') strOut = 'E'; 
     else if (strIn == 'P') strOut = 'F'; 
     else if (strIn == 'Q') strOut = 'G'; 
     else if (strIn == 'R') strOut = 'H'; 
     else if (strIn == 'S') strOut = 'I'; 
     else if (strIn == 'T') strOut = 'J'; 
     else if (strIn == 'U') strOut = 'K'; 
     else if (strIn == 'V') strOut = 'L'; 
     else if (strIn == 'W') strOut = 'M'; 
     else if (strIn == 'X') strOut = 'N'; 
     else if (strIn == 'Y') strOut = 'O'; 
     else if (strIn == 'Z') strOut = 'P'; 
     else if (strIn == '2') strOut = 'X'; 
     else if (strIn == '1') strOut = 'Y'; 
     else if (strIn == '_') strOut = '_'; 

     return strOut; 
    } 

Answer 1

I подумайте, что это поможет вам ...

char[] strIN = { 'A', 'B', 'C', 'D', 'E', 'F', 'G', 'H', 'I', 'J', 'K', 'L', 'M', 'N', 'O', 'P', 'Q', 'R', 'S', 'T', 'U', 'V', 'W', 'X', 'Y', 'Z', '2', '1', '_' }; 
     char[] strOut = { '2', '3', '4', '5', '6', '7', '8', '9', '@', 'A', 'B', 'C', 'D', 'E', 'F', 'G', 'H', 'I', 'J', 'K', 'L', 'M', 'N', 'O', 'P', 'X', 'Y', '_' }; 


     char init = 'C'; 

     int index = Array.IndexOf(strIN, init); 
     char output = strOut[index]; 

Answer 2

Создать Dictionary<char,char>, заполнить его с вашими переводами, то просто:

return (translationDictionary.ContainsKey(strIn))? translationDictionary[strIn] : null 

Answer 3

Вы можете использовать Dictionary<char, char> так:

private Dictionary<char, char> mTranslationMappings = new Dictionary<char, char>(); 

// ... in .ctor ... 
mTranslationMappings.Add('2', 'X'); 
// ... add other mappings ... 

char translateChar(char strIn) 
{ 
    return mTranslationMappings[strIn]; 
} 

Это не может быть лучшим подходом, но это решение.

Answer 4

Используйте Dictionary<char, char>, чтобы сопоставить каждый strIn с его значением strOut. Возможно, вы захотите сделать словарь приватным полем, которое вы инициализируете в конструкторе класса.

public class MyClass 
{ 
    private Dictionary<char, char> dict = new Dictionary<char, char>(); 

    public MyClass() 
    { 
     dict.Add('A', '1'); 
     dict.Add('B', '2'); 
     // ... and so on ... 
    } 

    public char TranslateChar(char input) 
    { 
     char result; 
     if (dict.TryGetValue(input, out result)) 
     { 
      return result; 
     } 
     return '?'; 
    } 
} 

Использование:

var myClass = new MyClass(); 
Console.WriteLine(myClass.TranslateChar('A')); 
Console.WriteLine(myClass.TranslateChar('@')); 

EDIT: В ответ на комментарий, нет встроенного способ определения ключа для конкретного значения. Для этого вы можете использовать этот подход:

char value = '@'; 
foreach (var kvp in dict) 
{ 
    if (kvp.Value == value) 
    { 
     Console.WriteLine("Key found: " + kvp.Key); 
     break; 
    } 
} 

Или, вы можете добавить метод в TryGetKey расширения, который имитирует TryGetValue:

public static class MyExtensions 
{ 
    public static bool TryGetKey<TKey, TValue>(
     this IDictionary<TKey, TValue> dict, 
     TValue value, 
     out TKey key) 
    { 
     key = default(TKey); 

     bool isKeyFound = false; 
     foreach (var kvp in dict) 
     { 
      if (EqualityComparer<TValue>.Default.Equals(kvp.Value, value)) 
      { 
       isKeyFound = true; 
       key = kvp.Key; 
       break; 
      } 
     } 

     return isKeyFound; 
    } 
} 

TryGetKey расширение использования метода:

char value = '@'; 
char keyResult;  
if (dict.TryGetKey(value, out keyResult)) 
{ 
    Console.WriteLine("Key found: " + keyResult); 
} 
else 
{ 
    Console.WriteLine("Key doesn't exist for value: " + value); 
} 

Answer 5

Если вы посмотрите на вход и выход функции, вы увидите несколько ясных диапазонов: A-I, J, K-Z, 2, 1 и Y. Если вы используете их в своих операторах if, ваш код будет проще. И даже меньше, чем использование Словаря и его заполнение.

Answer 6

Предполагая, что кодирование ASCII:

char translateChar(char strIn) 
{ 
    if (strIn >= 'A' && strIn <= 'I') return strIn - 'A' + '1' ; 
    if (strIn >= 'J' && strIn <= 'Z') return strIn - 'J' + '@' ; 
    if (strIn == '2') return 'X'; 
    if (strIn == '1') return 'Y'; 
    if (strIn == '_') return '_'; 

    return '?'; 
}