Создание класса на основе идентификатора

Question

Прямо сейчас у меня есть следующая функция, которая не выглядит очень хорошо ко мне:

string input = "??"; 
Letter output = null; 

switch(input[0]) 
{ 
    case LetterA.Id: // = 'A' 
     output = new LetterA(); 
     break; 

    case LetterB.Id: // = 'B' 
     output = new LetterB(); 
     break; 

    case LetterC.Id: // = 'C' 
     output = new LetterC(); 
     break; 

    // ... 
} 

if (output == null) 
    throw new Exception(); 

output.Parse(input); 

return output; 

Есть ли лучше/короткий путь, чтобы решить эту проблему и создать классы, основанные на Идентификатор?

Answer 1

У вас будет работа, и это не редкость.

Другим вариантом является создание словаря, который сопоставляет строку идентификатора с делегатом, который возвращает букву или словарь, который сопоставляет идентификатор объекту, который получен из письма, из которого можно сделать клонирование (этот выбор может зависеть о том, имеют ли разные производные классы Letter разные параметры конструкции).

//example of second 

Dictionary<string, Letter> PrototypeLetters = new Dictionary<string, Letter>(); 

//... 
output = PrototypeLetters[input]->MakeClone(); 
//where make clone is virtual in Letter returning a Letter (of derived type as Letter) 
//still need to handle case where input is not found 

С другой стороны, вы должны построить эту карту, строит его во время выполнения является более гибким, если вы когда-либо необходимо изменить поведение, но включает в себя гибкость, чтобы представить ошибки, и делает модульное тестирование менее легко.

С другой стороны, код, получающий письмо от идентификатора, не обязательно должен знать о каждом производном типе письма.

Считаете ли вы, что этот вариант лучше, зависит от того, какие критерии вы сравниваете.

Answer 2

Вы можете использовать отражение. Предполагается, что Letter, LetterA, LetterB и т. Д. Все живут в одной и той же сборке.

var letterType = typeof(Letter).Assembly.GetTypes() 
    .Where(t => t.Name == "Letter" + input[0]); 
var output = Activator.CreateInstance(letterType) as Letter; 

Пожалуйста, обратите внимание, что выше должна быть какая-то ошибка проверка добавил к нему в случае, если тип не может быть найден или не может быть приведен к Letter после конкретизации.