Как обработать функцию и записать все ее ошибки и вывести итог?

Question

[Ledington]

Такой вопрос.
У меня есть цикл, который обрабатывает входной параметр, следит за тем, что вводит пользователь.
Т.е. он должен вводить четные числа, если он введет символ, дробное число, букву и т.д., то выпадает сообщение об ошибке. И так на каждую переменную. А как сделать, чтобы он не выдавал сразу сообщение, а записал все что ввел пользователь и в конце только выдал общее сообщение об ошибке, что несколько параметров было введено не правильно?

Код цикла

static int GetInput(string paramName, Severity ex)
        {
            //задаем произвольное начальное значение
            int result = Int32.MinValue;

            //цикл обработки значения
            while (true) 
            {
                try
                {
                    Console.Write($"\nВведите значение <{paramName}>: ");
                    var x = Console.ReadLine();
                    int xi = int.Parse(x);
                    result = xi;

                    if (xi % 2 == 0)

                    {
                        break;
                    }

                    Throw(ex);
                    throw new System.FormatException();
                }

                //сообщение об ошибке на ввод нечетного числа 
                catch (ArgumentNullException) { } 

                //сообщение об ошибке на ввод букв, дробных значений и т.д.
                catch (System.FormatException) 
                {
                    Console.BackgroundColor = ConsoleColor.Red;
                    Console.ForegroundColor = ConsoleColor.Black;
                    Console.WriteLine("\n***********************************************************************");
                    Console.WriteLine("*Ошибка: Один из идентифицированных элементов имел недопустимый формат*");
                    Console.WriteLine("***********************************************************************");
                    Console.ResetColor();
                } 
            }
            return result;
        }

Отработка кода

Введите значение <a>: вап

***********************************************************************
*Ошибка: Один из идентифицированных элементов имел недопустимый формат*
***********************************************************************

Как должно быть

Введите значение <a>: вап
Введите значение <b>: 1.5
Введите значение <c>: ...

***********************************************************************
*Ошибка: Один из идентифицированных элементов имел недопустимый формат*
***********************************************************************
Введенное значение <a>: вап
Введенное значение <b>: 1.5
Введенное значение <c>: ...
***********************************************************************

Введите значение <a>:

Answer 1

[Griboks]

static int GetInput(string paramName)
        {
            while (true) 
            {
                Console.Write($"\nВведите значение <{paramName}>: ");
                if (int.TryParse(Console.ReadLine(), out int xi) && xi % 2 == 0) return xi;
                else 
                {
                    Console.BackgroundColor = ConsoleColor.Red;
                    Console.ForegroundColor = ConsoleColor.Black;
                    Console.WriteLine("\n***********************************************************************");
                    Console.WriteLine("*Ошибка: Один из идентифицированных элементов имел недопустимый формат*");
                    Console.WriteLine("***********************************************************************");
                    Console.ResetColor();
                } 
            }
        }

Comments

[Ledington]

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

[Griboks]

Ledington, я думал, вы опечатались, а вы действительно хотите это сделать. В таком случае придётся ввести дополнительную функцию или модифицировать эту.

[Ledington]

Думали шучу? =))) Но нет... Вот я никак не могу понять как это сделать.

[Griboks]

Ledington, вот канонический пример через out, хотя мне больше нравятся кортежи.

static bool TryGetInput(string paramName, out string input, out int value)
        {
                Console.Write($"\nВведите значение <{paramName}>: ");
                input = Console.ReadLine();
                return int.TryParse(input, out value) && value % 2 == 0;
        }

static bool TryGetAllInputs(List<string> paramNames, out List<string> inputs, out List<int> values)
{
    inputs = new List<string>(paramNames.Count);
    values= new List<int>(paramNames.Count);
    return paramNames.Select(paramName => {
        var result = TryGetInput(paramName, out string input, out int value));
        inputs.Add(input);
        values.Add(value);
        return result;
    }).All();
}

static List<int> GetAllInputs(List<string> paramNames)
{
    while(!TryGetAllInputs(paramNames, out List<string> inputs, out List<int> values))
    {
        Console.BackgroundColor = ConsoleColor.Red;
        Console.ForegroundColor = ConsoleColor.Black;                                    
        Console.WriteLine("\n***********************************************************************");
        Console.WriteLine("*Ошибка: Один из идентифицированных элементов имел недопустимый формат*");                            
        Console.WriteLine("***********************************************************************");
        Console.WriteLine(String.Join("\r\n", paramNames.Zip(inputs, (p,i) => $"Введенное значение <{p}>: {i}")));
        Console.WriteLine("***********************************************************************");
        Console.ResetColor();
    }
    return values;
}

p.s.
Лучше, конечно, использовать массивы, но со списками красивее.

[Ledington]

он жалуется на:
inputs.Add(input);
values.Add(value);
Нельзя использовать параметр с ключевым словом ref или out в параметре в анонимном методе, лямбда-выражении или выражении запроса.

и

return values; - Имя "values" не существует в текущем контексте.

Почему? Первое там наверное надо задать какие-нибудь переменные я так понял и сделать пересылку?
А на второе я пока не понял. Вроде имя есть, а он ругается...

[Griboks]

Ledington,

А на второе я пока не понял. Вроде имя есть, а он ругается...

Да, точно, надо вынести values перед циклом.

Первое там наверное надо задать какие-нибудь переменные я так понял и сделать пересылку?

Хмм, мне казалось, что в стрелочных функциях их можно использовать. Короче, предлагаю не париться и вместо канонического вида переписать через современные кортежи.

Answer 2

[oleg_ods]

Создать массив строк и вместо вывода ошибок на консоль писать в него.