C# Why I get "Unhandled Exception: System.FormatException: Input string was not in a correct format." when I use TryParse?

Question

I get "Unhandled Exception: System.FormatException: Input string was not in a correct format." but actually I catch the Exception with TryParse.

using System;
using System.Linq;
using System.Collections.Generic;

class MinAndMax
{
    static void Main()
    {
        // Task 3 - Write a program that reads from the console
        // a sequence of N integer numbers and returns the minimal
        // and maximal of them.

        int n;
        double num = 0, counter = 0, minNum = 0, maxNum = 0;
        List<double> numbers = new List<double>();

        Console.Write("How many numbers will you enter: ");
        bool isNum = int.TryParse(Console.ReadLine(), out n);

        if (isNum)
        {
            for (counter = 1; counter <= n; counter++)
            {
                Console.Write("Enter number {}: ", counter);
                bool isValid = double.TryParse(Console.ReadLine(), out num);

                if (isValid == false)
                {
                    Console.WriteLine("Invalid input!");
                }
                else
                {
                    numbers.Add(num);
                }
        }

        minNum = numbers.Max();
        maxNum = numbers.Min();

        Console.WriteLine("The maximal of the numbers is: " + maxNum);
        Console.WriteLine("The minimal of the numbers is: " + minNum);
        }
        else
        {
            Console.WriteLine("Invalid input!");
        }
    }
}

When the input is string it goes to the else block(so it catches the exception), but when the input is an integer I get Unhandled Exception: System.FormatException: Input string was not in a correct format.

The exception is probably thrown by Console.Write("Enter number {}: ", counter); — Lee
– Lee, Commented Nov 28, 2013 at 20:23

Lee · Accepted Answer · 2013-11-28 20:25:15Z

3

The line

Console.Write("Enter number {}: ", counter);

will throw an exception, you should change it to

Console.Write("Enter number {0}: ", counter);

answered Nov 28, 2013 at 20:25

Lee

145k20 gold badges243 silver badges292 bronze badges

Sign up to request clarification or add additional context in comments.

1 Comment

Beatris Boneva Over a year ago

Obviously.. How I could not have seen that! Very stupid mistake.. Thank you very much!

Guffa · Accepted Answer · 2013-11-28 20:25:25Z

1

It's your format string that is causing the error message. Put an index between the brackets:

Console.Write("Enter number {0}: ", counter);

answered Nov 28, 2013 at 20:25

Guffa

703k112 gold badges760 silver badges1k bronze badges

Comments

Inisheer · Accepted Answer · 2013-11-28 20:29:03Z

0

In addition to the Console.Write error already fixed by previous posters, you will also get a System.InvalidOperationException here if the user enters only strings because numbers list will be empty.

minNum = numbers.Max();
maxNum = numbers.Min();

answered Nov 28, 2013 at 20:29

Inisheer

20.9k9 gold badges55 silver badges83 bronze badges

1 Comment

Beatris Boneva Over a year ago

Yes you are right I put minNum = numbers.Max(); maxNum = numbers.Min(); in the else block.

Collectives™ on Stack Overflow

C# Why I get "Unhandled Exception: System.FormatException: Input string was not in a correct format." when I use TryParse?

3 Answers 3

1 Comment

Comments

1 Comment

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

3 Answers 3

1 Comment

Comments

1 Comment

Your Answer

Sign up or log in

Post as a guest

Related