Question

Can someone find why this loop isn’t working? I’m new to C#.

while (move == "r" || move == "s" || move == "f")
            {
                Console.Write("\nEnter your move: ");
                move = Console.ReadLine();


                switch (move)
                {
                    case "r":
                        Console.Write("\nYou have reloaded, press enter for Genius");
                        Console.ReadLine();
                        break;
                    case "s":
                        Console.Write("\nYou have shielded, press enter for Genius");
                        Console.ReadLine();
                        break;
                    case "f":
                        Console.Write("\nYou have fired, press enter for Genius");
                        Console.ReadLine();
                        break;
                    default:
                        Console.Write("\nInvalid move, try again\n\n");
                        break;
                }


            }

Answer 1

Probably because move is initialized inside the loop and is probably null or an empty string since I can’t see the code before the loop I’m assuming its not initialized.

My suggestion is to use a boolean flag that is set as such

bool done = false;
while (!done)
{
    // do work
    if (move == finalMove) // or whatever your finish condition is
        done = true; // you could also put this as a case inside your switch
}