System.FormatException 'Input string was not in a correct format.' - Correct format is entered

T

TheBoxyBear•10/28/22, 2:42 PM

Can't find the issue right now, but in the meantime you should rewrite the ReadLine to avoid crashing when the input is incorrect

T

TheBoxyBear•10/28/22, 2:42 PM

$tryparse

TTheBoxyBear $tryparse

M

MODiX•10/28/22, 2:42 PM

The TryParse pattern is considered best practice of parsing data from a string:

a TryParse method returns
```
true
```
true
```
true
```
true
or
```
false
```
false
```
false
```
false
to inform you if it succeeded or not, so you can use it directly in a condition,
since C# 7 you can declare a variable that will be used as an
```
out
```
out
```
out
```
out
argument inline in an argument list,
it forces you to check if the
```
out
```
out
```
out
```
out
argument contains valid data afterwards,

Avoid: Convert.ToInt32 — it's a bad choice for parsing an

int

int

int

int. It exists only for backwards compatibility reasons and should be considered last resort.

return Convert.ToInt32(null); //returns 0. null should not be considered as 0 ever
return Convert.ToInt32("asdf"); //throws FormatException

return Convert.ToInt32(null); //returns 0. null should not be considered as 0 ever
return Convert.ToInt32("asdf"); //throws FormatException

return Convert.ToInt32(null); //returns 0. null should not be considered as 0 ever
return Convert.ToInt32("asdf"); //throws FormatException

return Convert.ToInt32(null); //returns 0. null should not be considered as 0 ever
return Convert.ToInt32("asdf"); //throws FormatException

(Note: Convert does contain useful conversion methods:

To/FromBase64String

To/FromBase64String

To/FromBase64String

To/FromBase64String,

To/FromHexString

To/FromHexString

To/FromHexString

To/FromHexString,

ToString(X value, int toBase)

ToString(X value, int toBase)

ToString(X value, int toBase)

ToString(X value, int toBase),

ToX(string? value, int fromBase)

ToX(string? value, int fromBase)

ToX(string? value, int fromBase)

ToX(string? value, int fromBase))

Avoid: int.Parse — you have to use a

try

try

try

try/

catch

catch

catch

catch statement to handle invalid input, which is a less clean solution.

var number = int.Parse("abc"); //throws FormatException
var number = int.Parse(""); //throws FormatException

var number = int.Parse("abc"); //throws FormatException
var number = int.Parse(""); //throws FormatException

var number = int.Parse("abc"); //throws FormatException
var number = int.Parse(""); //throws FormatException

var number = int.Parse("abc"); //throws FormatException
var number = int.Parse(""); //throws FormatException

Use int.TryParse https://docs.microsoft.com/en-us/dotnet/api/system.int32.tryparse?view=net-5.0#System_Int32_TryParse_System_String_System_Int32__

if (int.TryParse(someInput, out var result))
{
  Console.WriteLine($"Thanks for giving me the following number: {result}!");
}
else
{
  Console.WriteLine("You didn't give me a valid number :c");
}

if (int.TryParse(someInput, out var result))
{
  Console.WriteLine($"Thanks for giving me the following number: {result}!");
}
else
{
  Console.WriteLine("You didn't give me a valid number :c");
}

if (int.TryParse(someInput, out var result))
{
  Console.WriteLine($"Thanks for giving me the following number: {result}!");
}
else
{
  Console.WriteLine("You didn't give me a valid number :c");
}

if (int.TryParse(someInput, out var result))
{
  Console.WriteLine($"Thanks for giving me the following number: {result}!");
}
else
{
  Console.WriteLine("You didn't give me a valid number :c");
}

Int32.TryParse Method (System)

Converts the string representation of a number to its 32-bit signed integer equivalent. A return value indicates whether the operation succeeded.

TTheBoxyBear $tryparse

M

maroonlolOP•10/28/22, 4:01 PM

Unfortunatly I have tried that an no luck

M

maroonlolOP•10/28/22, 4:27 PM