C#•3y ago

❔ ✅ Issues With Type Conversion When Validating User Input

List<string> initialText = new()
{
    "What Would You Like To Do?",
    "--------------------------",
    "** Submit Answer As 1, 2, or 3 **"
};

List<string> initialSelection = new()
{
    "1) Play Game",
    "2) View Previous Games",
    "3) View Statistics",
};

foreach (string item in initialText)
{
    Console.WriteLine(item);
};

foreach (string ite in initialSelection)
{
    Console.WriteLine(ite);
};

int userSelection;

if (!int.TryParse(Console.ReadLine(), out userSelection))
{
    tries--;
    Console.WriteLine("Input Must Be An Integer Of 1, 2, or 3. Try Again! " + tries + " Left.");
    SelectionScreen(tries);
}
else if (int.TryParse(Console.ReadLine(), out userSelection))
{
    int selectedItem = userSelection;

    if (initialSelection[selectedItem-1])
    {

    }
}

List<string> initialText = new()
{
    "What Would You Like To Do?",
    "--------------------------",
    "** Submit Answer As 1, 2, or 3 **"
};

List<string> initialSelection = new()
{
    "1) Play Game",
    "2) View Previous Games",
    "3) View Statistics",
};

foreach (string item in initialText)
{
    Console.WriteLine(item);
};

foreach (string ite in initialSelection)
{
    Console.WriteLine(ite);
};

int userSelection;

if (!int.TryParse(Console.ReadLine(), out userSelection))
{
    tries--;
    Console.WriteLine("Input Must Be An Integer Of 1, 2, or 3. Try Again! " + tries + " Left.");
    SelectionScreen(tries);
}
else if (int.TryParse(Console.ReadLine(), out userSelection))
{
    int selectedItem = userSelection;

    if (initialSelection[selectedItem-1])
    {

    }
}

List<string> initialText = new()
{
    "What Would You Like To Do?",
    "--------------------------",
    "** Submit Answer As 1, 2, or 3 **"
};

List<string> initialSelection = new()
{
    "1) Play Game",
    "2) View Previous Games",
    "3) View Statistics",
};

foreach (string item in initialText)
{
    Console.WriteLine(item);
};

foreach (string ite in initialSelection)
{
    Console.WriteLine(ite);
};

int userSelection;

if (!int.TryParse(Console.ReadLine(), out userSelection))
{
    tries--;
    Console.WriteLine("Input Must Be An Integer Of 1, 2, or 3. Try Again! " + tries + " Left.");
    SelectionScreen(tries);
}
else if (int.TryParse(Console.ReadLine(), out userSelection))
{
    int selectedItem = userSelection;

    if (initialSelection[selectedItem-1])
    {

    }
}

List<string> initialText = new()
{
    "What Would You Like To Do?",
    "--------------------------",
    "** Submit Answer As 1, 2, or 3 **"
};

List<string> initialSelection = new()
{
    "1) Play Game",
    "2) View Previous Games",
    "3) View Statistics",
};

foreach (string item in initialText)
{
    Console.WriteLine(item);
};

foreach (string ite in initialSelection)
{
    Console.WriteLine(ite);
};

int userSelection;

if (!int.TryParse(Console.ReadLine(), out userSelection))
{
    tries--;
    Console.WriteLine("Input Must Be An Integer Of 1, 2, or 3. Try Again! " + tries + " Left.");
    SelectionScreen(tries);
}
else if (int.TryParse(Console.ReadLine(), out userSelection))
{
    int selectedItem = userSelection;

    if (initialSelection[selectedItem-1])
    {

    }
}

In this section of code, I print items to the screen and request user input. I'm trying to basically go

if user input is not an integer, decrement tries and recall function. If user input is an integer but that integer does not match an index of List<string> initialSelection, then decrement tries and recall function. Otherwise, if user input is an integer and is a correct index of the list, then start next function

if user input is not an integer, decrement tries and recall function. If user input is an integer but that integer does not match an index of List<string> initialSelection, then decrement tries and recall function. Otherwise, if user input is an integer and is a correct index of the list, then start next function

if user input is not an integer, decrement tries and recall function. If user input is an integer but that integer does not match an index of List<string> initialSelection, then decrement tries and recall function. Otherwise, if user input is an integer and is a correct index of the list, then start next function

if user input is not an integer, decrement tries and recall function. If user input is an integer but that integer does not match an index of List<string> initialSelection, then decrement tries and recall function. Otherwise, if user input is an integer and is a correct index of the list, then start next function

but I'm having issues with the validation and type conversion. Thanks

JakenVeina•5/27/23, 10:19 PM

I'm having issues with the validation and type conversion.

what issues?

MekOP•5/27/23, 10:19 PM

it's not working? I don't really know how to explain it other than what I'm trying to do

If user input is an integer but that integer does not match an index of List<string> initialSelection, then decrement tries and recall function. Otherwise, if user input is an integer and is a correct index of the list, then start next function

JakenVeina•5/27/23, 10:23 PM

what's not working?

JakenVeina•5/27/23, 10:23 PM

which part of that?

JakenVeina•5/27/23, 10:23 PM

checking whether the user input is an integer?

JakenVeina•5/27/23, 10:24 PM

checking whether it's an index of a

List<string>

List<string>

JakenVeina•5/27/23, 10:24 PM

decrementing and retrying?

JakenVeina•5/27/23, 10:24 PM

starting the next function?

MekOP•5/27/23, 10:52 PM

int userSelection;
if (!int.TryParse(Console.ReadLine(), out userSelection))
{
  tries--;
  Console.WriteLine("Input Must Be An Integer Of 1, 2, or 3. Try Again " + tries + " Left.");
  SelectionScreen(tries);
}

int userSelection;
if (!int.TryParse(Console.ReadLine(), out userSelection))
{
  tries--;
  Console.WriteLine("Input Must Be An Integer Of 1, 2, or 3. Try Again " + tries + " Left.");
  SelectionScreen(tries);
}

this works. However, when the user actually puts in an integer, and checking if it's an index of the list like

else if (int.TryParse(Console.ReadLine(), out userSelection))
{
  int selectedItem = userSelection; // this works...well no errors in ide
  // now checking if it's an index
  if (initialSelection[selectedItem - 1])
  {}
}

else if (int.TryParse(Console.ReadLine(), out userSelection))
{
  int selectedItem = userSelection; // this works...well no errors in ide
  // now checking if it's an index
  if (initialSelection[selectedItem - 1])
  {}
}

it tells me that I

Cannot implicitly convert type 'string' to 'bool'

Cannot implicitly convert type 'string' to 'bool'

but I'm not trying to do that. I'm trying to check if it's an index of the list.

MekOP•5/27/23, 10:53 PM

like my else if part is to check if the integer entered is an index of the list. Since the index is 0-2 and the user enters a number 1-3, I subtract one from the users input to check if it's an index, but it's not letting me

MekOP•5/27/23, 10:54 PM

just like if I write it like this

else if (int.TryParse(Console.ReadLine(), out userSelection))
{
  if (initialSelection[userSelection-1])
  {}
}

else if (int.TryParse(Console.ReadLine(), out userSelection))
{
  if (initialSelection[userSelection-1])
  {}
}

it tells me the same thing. So I'm having issues converting the users input to an integer. I have also tries to wrap

Convert.ToInt32()

Convert.ToInt32()

around user selection and it won't let me do that either

MekOP•5/27/23, 10:55 PM

so my question is, how do I check if the users input is an integer, and if not -> show error, if so -> check if its an index and if not -> show error while decrementing the same tries variable

MekOP•5/27/23, 10:57 PM

checking whether the user input is an integer?

that part works

checking whether it's an index of a List<string>?

can't get conversion right to take user input as an integer to then check the index

decrementing and retrying?

it's not decrementing the same tries variable. Instead it'll decrement one version for when it's not an integer and then reset when it is an integer. It needs to be the same variable

starting the next function?

no I got that part down

MekOP•5/27/23, 10:59 PM

like it keeps reading

int.TryParse(Console.ReadLine(), out userSelection)

int.TryParse(Console.ReadLine(), out userSelection)

as a string instead of an integer so I'm trying to convert it back to an integer so that I can use it to check if the index is correct.

MekOP•5/27/23, 11:06 PM

I just want to get the users input, check if it's an integer and if so then check if that integer is an index of the list. If it's not an integer then decrement tries and recall function. If it is an integer but not an index, then decrement the same tries count and recall the function.

JJakenVeina starting the next function?

MekOP•5/27/23, 11:09 PM

JakenVeina•5/27/23, 11:11 PM

alright

JakenVeina•5/27/23, 11:11 PM

JakenVeina•5/27/23, 11:12 PM

if (initialSelection[selectedItem - 1])

if (initialSelection[selectedItem - 1])

JakenVeina•5/27/23, 11:12 PM

yes, you are getting a type error here

JakenVeina•5/27/23, 11:13 PM

Cannot implicitly convert type 'string' to 'bool'

Cannot implicitly convert type 'string' to 'bool'

JakenVeina•5/27/23, 11:13 PM

that's becase you are passing a string value to a statement that expects a bool

MekOP•5/27/23, 11:15 PM

ok so then if I Have

int userSelection;

if (int.TryParse(Console.ReadLine(), out userSelection))
{
  // do something if input is an integer
}
else
{
  // show error message and decrement tries
}

int userSelection;

if (int.TryParse(Console.ReadLine(), out userSelection))
{
  // do something if input is an integer
}
else
{
  // show error message and decrement tries
}

how do I go inside of my if statement and check if it's a correct index of the list?

JakenVeina•5/27/23, 11:16 PM

what counts as a correct index of the list?

MekOP•5/27/23, 11:17 PM

List<string> initialSelection = new()
{
  "1) Play Game",
  "2) View Previous Game",
  "3) View Statistics"
};

List<string> initialSelection = new()
{
  "1) Play Game",
  "2) View Previous Game",
  "3) View Statistics"
};

if the user inputs a number 1, 2, or 3 then it would subtract 1 from the users input to check if the input matches index 0, 1, or 2. If it doesn't, that means they're out of list index range and the error message needs to show, decrement tries, and recall the function <- or allow the user to try again in other words

JakenVeina•5/27/23, 11:17 PM

JakenVeina•5/27/23, 11:18 PM

what are the valid index values that you can accept?

MekOP•5/27/23, 11:18 PM

..... 1, 2, or 3

JakenVeina•5/27/23, 11:18 PM

great

JakenVeina•5/27/23, 11:18 PM

check for those

MekOP•5/27/23, 11:19 PM

yep great idea

JakenVeina•5/27/23, 11:19 PM

better yet

JakenVeina•5/27/23, 11:19 PM

check in a way that accounts for you chaning the list in the future

JakenVeina•5/27/23, 11:20 PM

I.E. don't assume the list has 3 items

MekOP•5/27/23, 11:24 PM

int userSelection;

if (int.TryParse(Console.ReadLine(), out userSelection))
{
    if (userSelection - 1 >= 0 && userSelection - 1 <= 3)
    {
        string selectedAction = initialSelection[userSelection - 1];
        Console.WriteLine("Selected Action: " + selectedAction);
    }
    else
    {
        tries--;
        Console.WriteLine("Input Must Be An Integer Of 1, 2, or 3. Try Again. " + tries + " Left.");
        SelectionScreen(tries);
    }
}
else
{
    tries--;
    Console.WriteLine("Input Must Be An Integer Of 1, 2, or 3. Try Again. " + tries + " Left.");
    SelectionScreen(tries);
}

int userSelection;

if (int.TryParse(Console.ReadLine(), out userSelection))
{
    if (userSelection - 1 >= 0 && userSelection - 1 <= 3)
    {
        string selectedAction = initialSelection[userSelection - 1];
        Console.WriteLine("Selected Action: " + selectedAction);
    }
    else
    {
        tries--;
        Console.WriteLine("Input Must Be An Integer Of 1, 2, or 3. Try Again. " + tries + " Left.");
        SelectionScreen(tries);
    }
}
else
{
    tries--;
    Console.WriteLine("Input Must Be An Integer Of 1, 2, or 3. Try Again. " + tries + " Left.");
    SelectionScreen(tries);
}

so I did this, but when I put in 4 as a response, it background the console window and told me the list index was out of range inside of the IDE instead of displaying the error message

MekOP•5/27/23, 11:25 PM

check in a way that accounts for you chaning the list in the future

I don't know what this means

MekOP•5/27/23, 11:26 PM

but if I put in 3 for my user input, it works just fine

MekOP•5/27/23, 11:28 PM

I changed my nested if statement to say

if (userSelection >= 0 && userSelection <=4)

if (userSelection >= 0 && userSelection <=4)

and it seems to work ok

MekOP•5/27/23, 11:47 PM

thank you for your help

MMek I changed my nested if statement to say `if (userSelection >= 0 && userSelection...

JakenVeina•5/28/23, 3:20 AM

that would allow for values 0, 1, 2, 3, and 4

JakenVeina•5/28/23, 3:20 AM

which is not 1, 2, and 3

JakenVeina•5/28/23, 3:20 AM

ergo, exception

JJakenVeina that would allow for values 0, 1, 2, 3, and 4

MekOP•5/28/23, 3:27 AM

because the menu option shows 1, 2, 3, 4 but the list indexes 0, 1, 2, 3

MekOP•5/28/23, 3:27 AM

the return subtracts 1 from the users input to get the correct index

JakenVeina•5/28/23, 3:27 AM

okay

JakenVeina•5/28/23, 3:27 AM

so, why is 0 accepted?

MekOP•5/28/23, 3:28 AM

i fixed it to say 1

JakenVeina•5/28/23, 3:29 AM

okay

JakenVeina•5/28/23, 3:29 AM

so... then it's fixed

Accord•5/29/23, 3:34 AM

Looks like nothing has happened here. I will mark this as stale and this post will be archived until there is new activity.

❔ ✅ Issues With Type Conversion When Validating User Input

Similar Threads

Similar Threads

Similar Threads