0

Вытаскиваю с БД таблицу в которой есть |id|Code|Type|Value| в модель:

public record MyDto
{
    public Guid Id{ get; set; }
    public Code Code { get; set; }
    public Type Type { get; set; }
    public string Value { get; set; }
}

Необходимо по признаку |Type| вернуть из метода соответствующий тип данных лежащий в поле Value которая парситься всегда в строку, но ее содержимое может быть например "100" или "Привет Мир!". Я ничего лучшего пока не придумал, но очень сомнительный код:

 public async Task<object> GetDataByCodeAsync(Code code, CancellationToken token)
    {
        var result = await _repository.GetDataByCodeAsync(code, token);

        switch (result.Type)
        {
            case Type.String:
                return result.Value.ToString();
            case Type.Decimal:
                return Convert.ToDecimal(result.Value);
            case Type.Integer:
                return Convert.ToInt32(result.Value);
            case Type.Guid:
                return Guid.Parse(result.Value);
            default:
                throw new ArgumentOutOfRangeException();
        }
    }

Можно пока без проверок или TryParse, просто сугубо понимать, как реализовать в нормальном виде.

12
  • 1
    Зачем у метода <T>, если обобщение никак не используется? 17 мая 2022 в 14:54
  • Метод возвращает object. То есть значимые типы боксятся. В целом это выглядит сомнительно и неэффективно. Может, опишете задачу более крупно? Авось удастся найти решение лучше на более высоком уровне. 17 мая 2022 в 14:56
  • Замечание: в соответствии с Framework Design Guidelines параметр типа CancellationToken всегда должен задаваться как опциональный: CancellationToken cancellationToken = default. Он единственный, для которого сделано такое исключение. 17 мая 2022 в 15:00
  • Т это рудимент, метод раньше был обобщённым. Просто копипаста. Задача описана, больше и описать-тл в принципе мне нечего. Т.е по полученному признаку типу из бд сконвертировать в реальный (int, guid, string...).
    – JDo
    17 мая 2022 в 16:47
  • 2
    Если речь идет чисто про то, чтобы вернуть тип с нужным, сконвертированным значением, то для этого в C# есть Convert.ChangeType(....), который отлично подходит под ваш код, ибо вы можете заменить весь switch на нечто такое return Convert.ChangeType(result.Value, result.Type);, но вот с Task<object>, тут уже сложнее, ибо он подразумевает то, что тип будет задан при вызове метода и тут наверно лучше вызывать Convert.ChangeType за пределами метода, там, где он нужен уже сконвертированный, либо Type передавать методу, тогда result.Type заменять на typeof(T).
    – EvgeniyZ
    17 мая 2022 в 19:57

1 ответ 1

5

Можно применить паттерн-матчинг для лаконичности:

return result.Type switch
{
    SettingType.String => result.Value,
    SettingType.Decimal => decimal.Parse(result.Value),
    SettingType.Integer => int.Parse(result.Value),
    SettingType.Guid => Guid.Parse(result.Value),
    _ => throw new ArgumentOutOfRangeException()
};

Давайте разберёмся.
Ваш метод GetDataByCodeAsync возвращает какое-то значение. Что дальше с ним происходит? Очевидно, оно как-то обрабатывается. И, вероятно, обрабатывать нужно по разному в зависимости от типа.

Мне много раз попадались такие вопросы с конвертацией типа из строки и дальнейшим его использованием. На предложения показать проблему более широко практически всегда авторы упираются и хотят непременно чёрную магию: чтобы оно само как-то работало...

В тех задачах, что я видел, обычно код выглядит примерно так:

object value = GetDataByCode(...);
Process(value);

Внутри Process опять приходится определять, какой именно тип имеет параметр value: if (value is int n) ... и т. п. То есть от чего уходили, к тому и вернулись. Разница лишь в том, что теперь тип задан не строкой.

Предлагаю следующее.

public async Task ProcessDataByCodeAsync(Code code, CancellationToken token = default)
{
    var result = await _repository.GetDataByCodeAsync(code, token);

    switch (result.Type)
    {
        case SettingType.String:
            Process(result.Value);
            break;
        case SettingType.Decimal:
            Process(decimal.Parse(result.Value));
            break;
        case SettingType.Integer:
            Process(int.Parse(result.Value));
            break;
        case SettingType.Guid:
            Process(Guid.Parse(result.Value));
            break;
        default:
            throw new ArgumentOutOfRangeException();
    }
}

И несколько перегрузок метода, принимающих нужный тип:

void Process(string value) { }
void Process(decimal value) { }
void Process(int value) { }
void Process(Guid value) { }

Таким образом наш метод выполняет диспетчеризацию в одном-единственном месте. А дальше мы без проблем работаем с конкретными типами.

4
  • ArgumentOutOfRangeException для индексаторов же :) docs.microsoft.com/ru-ru/dotnet/api/system.argumentexception
    – aepot
    17 мая 2022 в 16:26
  • Это конечно красиво, но суть вопроса, как сделать возвращаемое значение по типу.
    – JDo
    17 мая 2022 в 16:44
  • 2
    @JDo используйте Task<dynamic>, если хотите, чтобы метод мог менять тип возвращаемого значения в runtime, но работать будет медленно 17 мая 2022 в 23:17
  • @JDo - посмотрите обновление. 18 мая 2022 в 9:54

Ваш ответ

Нажимая на кнопку «Отправить ответ», вы соглашаетесь с нашими пользовательским соглашением, политикой конфиденциальности и политикой о куки

Всё ещё ищете ответ? Посмотрите другие вопросы с метками или задайте свой вопрос.