0

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

private Type GetColumnType(List<string> columns)
{
    int matches = 0;
    foreach (var col in columns)
    {
        if (!DateTime.TryParse(col, out var res))
            break;
        else
            matches++;
    }
    if (matches == columns.Count)
        return typeof(DateTime);
}

Идея в том что бы проверять сначала на DateTime, потом на int, потом на double. И если ничего из этого не получится оставлять строку.

Мне не нравится, что получается простыня if'ов. Может быть, есть какая то идея по-лучше?

1 ответ 1

2
bool areAllDateTime = columns.Any(x => !DateTime.TryParse(x, out _));

И т.д.

5
  • 2
    Спасибо. Но TryParse принимает 2 аргумента. Сделал DateTime.TryParse(x, out _)
    – banana
    8 фев 2022 в 9:57
  • 1
    Так же, Any возвращает true если хотя бы один элемент соответствует условию
    – banana
    8 фев 2022 в 11:06
  • 1
    @banana там отрицание
    – aepot
    8 фев 2022 в 14:06
  • @aepot var columns = new List<string>() { "123", "32", "asd" }; bool areAllDateTime = columns.Any(x => !int.TryParse(x, out _)); Console.WriteLine(areAllDateTime); Выводит в консоль "True". Нужно использовать All или исправить именование переменной)
    – banana
    8 фев 2022 в 14:36
  • 1
    @banana columns.All(x => int.TryParse(x, out _)) попробуйте
    – aepot
    8 фев 2022 в 14:37

Ваш ответ

By clicking “Отправить ответ”, you agree to our terms of service and acknowledge that you have read and understand our privacy policy and code of conduct.

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