Как исправить ошибку?

Question

[Falevi]

Выдаёт такую ошибку:
Необработанное исключение:

System.FormatException: Входная строка имела неверный формат.
в System.Number.StringToNumber(String str, NumberStyles options, NumberBuffer& number, NumberFormatInfo info, Boolean parseDecimal)
в System.Number.ParseInt32(String s, NumberStyles style, NumberFormatInfo info)
в System.Convert.ToInt32(String value)
в Program1.Program.Main() в C:\CSAIO4D\ВК01\CH01\Program1\Program1\Program.cs:строка 42

Вот код:

static void Main()
{
Console.WriteLine("Введите первое число.");
int One = Convert.ToInt32(Console.ReadLine());

Console.WriteLine("Введите второе число.");
int Two = Convert.ToInt32(Console.ReadLine());

Console.WriteLine("Введите Третье число.");
int Three = Convert.ToInt32(Console.ReadLine());

if (One > Two)
{
if(One > Three)
{
Console.WriteLine(One);
}
else
{
Console.WriteLine(Three);
}
}
else if (Two > Three)
{
Console.WriteLine(Two);
}
else
{
Console.WriteLine(Three);
}
int Number = Convert.ToInt32(Console.ReadLine());
Number /= 2;
switch (Number)
{
case 0:
Console.WriteLine("Чётное");
break;
case 1:
Console.WriteLine("Не чётное");
break;
}

Comments

[oleg_ods]

У тебя неправильно определяется четность числа. Тут нужно деление по модулю(%).

[Василий Банников]

oleg_ods, А ещё способ нахождения максимального числа из трёх дичайший :)

Console.WriteLine("Введите первое число.");
int a = Convert.ToInt32(Console.ReadLine());

Console.WriteLine("Введите второе число.");
int b = Convert.ToInt32(Console.ReadLine());

Console.WriteLine("Введите Третье число.");
int c = Convert.ToInt32(Console.ReadLine());

var max = Math.Max(a, Math.Max(b, c));
Console.WriteLine(max);


int number = Convert.ToInt32(Console.ReadLine());
if(number % 2 == 0)
  Console.WriteLine("Чётное");
else
  Console.WriteLine("Не чётное");

(надо бы обработку кривого ввода добавить, но пофиг)

Answer 1

[Василий Банников]

В какой-то момент ты ввёл в консоль число в неверном формате, о чём и сообщается в ошибке.
В стактрейсе можно увидеть, что это произошло при вызове Convert.ToInt32 на 42й строке

System.FormatException: Входная строка имела неверный формат.
в System.Number.StringToNumber(String str, NumberStyles options, NumberBuffer& number, NumberFormatInfo info, Boolean parseDecimal)
в System.Number.ParseInt32(String s, NumberStyles style, NumberFormatInfo info)
в System.Convert.ToInt32(String value)
в Program1.Program.Main() в C:\CSAIO4D\ВК01\CH01\Program1\Program1\Program.cs:строка 42

Чтобы чуть проще было обрабатывать подобные ошибки, используй int.TryParse, вместо Convert.ToInt32

int.Parse и Convert.ToInt32 стоит использовать только тогда, когда ты точно знаешь, что ввод всегда будет корректным.

Answer 2

[freeExec]

Вводить в верном формате, написано же.

Comments

[Василий Банников]

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

[edward_freedom]

Василий Банников, это написано в книге, зачем дублировать книгу на форум?

[Василий Банников]

edward_freedom, вообще тут на такие вопросы уже 100% много раз отвечали. Но я не смог найти.
Имхо, нужно поиск на хабре нормальный сделать.