Предложения по очистке этого API

Question

Для забавного проекта я пытаюсь реализовать спецификацию BitTorrent, и сейчас я работаю над частью BEncoding.

Кодировка в основном может кодировать из int / string / dictionary -> string для передачи. У меня есть все различные кодировки, написанные / протестированные / работающие как перегруженные методы Encode (...), и у меня есть отдельные методы декодирования, написанные / протестированные / работающие как DecodeString (...), DecodeInt (... ) и т. д.

Я не могу найти способ иметь 1 метод декодирования для всех декодирований, чтобы сохранить API для кодирования / декодирования как можно более чистым (2 открытых метода, топы, на данный момент).

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

Код клиента, прямо сейчас должен выглядеть примерно так каждый раз, когда он хочет декодировать сообщение:

string s = ...; // Encoded string
Type t = Encoder.GetDecodedType(s);
if (t == typeof(int))
    process(Encoder.DecodeInt(s));
else if (t == typeof(string))
    process(Encoder.DecodeString(s));
else if (t == typeof(Dictionary<string, string>))
    process(Encoder.DecodeStringDictionary(s));
else if (t == typeof(Dictionary<string, int>))
    process(Encoder.DecodeIntDictionary(s)):

и я бы хотел почистить это так:

string s = ...; // Encoded string
process(Encoder.Decode(s));

где в обоих случаях процесс (...), вероятно, будет перегружен функциями на стороне клиента, принимая 4 типа декодированных значений.

Answer 1

Вы можете позволить DLR сделать это за вас.

public static void Process(int i) { ... }
public static void Process(string s) { ... }
public static void Process(Dictionary<string, string> dic) { ... }
public static void Process(Dictionary<string, int> dic) { ... }

[...]

public dynamic Decode(string input)     // or 'object' if you prefer
{
    var t = GetDecodedType(input);
    if (t == typeof(int))
        return DecodeInt(input);
    else if (t == ...)
        // ...
}

[...]

string s = ...; // Encoded string
Process(Encoder.Decode(s));            // if you used 'dynamic' above
Process((dynamic)Encoder.Decode(s));   // if you used 'object' above

Answer 2

Если вы пишете библиотеку / фреймворк ... это будет самый бесценный ресурс в вашем начинании :-) У меня есть бумажная копия, и я прочитал ее на обложке, чтобы покрыть:
Руководство по разработке для разработки библиотек классов от Microsoft

Answer 3

Я в замешательстве. почему бы просто не выполнить логику GetDecodedType в общедоступном методе Decode и определить тип, а затем сделать разные вызовы после определения?

Answer 4

Я бы сказал, что вы должны следовать принципу подстановки Лискова здесь и создать метод для каждого типа данных.Таким образом, вы не добавляете к страданиям использование typeof, когда начинаете передавать пользовательские объекты.Перечитав свой вопрос еще раз, вы уже знаете, какой тип ему передается, что дополнительно поддерживает необходимость удаления операции typeof