Cannot implicitly convert type 'string' to '

Question

2.00/5 (3 votes)

See more:

I am trying to make a simple shotgun game where the user vs the CPU and the both pick shot, shield or reload but In my GetOptionFromUser I keep getting the error Cannot implicitly convert type string to;ShotgunGame.Program.ShotgunOption from the cases in my switch statement.

Any Guidance would be appreciated

The error happens where it says

C#

case "SHOOT":, case "RELAOD":, case "SHIELD":

C#

//Declare Variables
            Console.Title = "Welcome To The Shotgune Game";
            int CPUBullets = 3, userBullets = 3;
            ShotgunOption UserOption;
            int computerChoice, userScore = 0;
            bool QUIT = false;
            double gameCount = 0.0;
            Random computer = new Random();
           
            Console.Clear();
            Console.WriteLine("SHOOT RELOAD SHIELD");

            UserOption = GetOptionFromUser();
            ShotgunOption CPUOption = (ShotgunOption)computer.Next(1, 3); // 1 is Shot, 2 is Reload, 3 is Shield

             do
            {
                

                //if (UserOption == "QUIT")
                //{
                   // break;
                //}
                
                do
                {
                //Console.Write("Please enter choice, or enter QUIT to quit: ");
                
             


                  switch (UserOption)
                {
                    case "SHOOT":
                        if((int)CPUOption == 1)
                        {
                            Console.WriteLine("You chose {0} and the computer chose Shoot. It was a tie!", userChoice);
                            ; userBullets --;CPUBullets --; ++gameCount;
                        }
                        else if ((int)CPUOption == 2)
                        {
                            Console.WriteLine("You chose {0} and the computer chose Reload. You win!", userChoice);
                            ++userScore; ++gameCount;
                        }
                        else if ((int)CPUOption == 3)
                        {
                            Console.WriteLine("You chose {0} and the computer chose Shield. No Damage!", userChoice);
                            ++gameCount;
                        }
                        break;
                    case "RELAOD":
                        if((int)CPUOption == 1)
                        {
                            Console.WriteLine("You chose {0} and the computer chose Shoot. You lose!", userChoice);
                             ++userScore; ++gameCount;
                        }
                        else if ((int)CPUOption == 2)
                        {
                            Console.WriteLine("You chose {0} and the computer chose Reload. You Both Gain A bullet", userChoice);
                            userBullets++; CPUBullets++; ++gameCount;
                        }
                        else if ((int)CPUOption == 3)
                        {
                            Console.WriteLine("You chose {0} and the computer chose Shield. No Damage!", userChoice);
                           
                        }
                        break;
                    case "SHIELD":
                        if((int)CPUOption == 1)
                        {
                            Console.WriteLine("You chose {0} and the computer chose Shoot. You lose!", userChoice);
                            ++gameCount;
                        }
                        else if ((int)CPUOption == 2)
                        {
                            Console.WriteLine("You chose {0} and the computer chose Reload. You win!", userChoice);
                            ++userScore; ++gameCount;
                        }
                        else if ((int)CPUOption == 3)
                        {
                            Console.WriteLine("You chose {0} and the computer chose Shield. No Damage!", userChoice);
                            ++gameCount;
                        }
                        break;                  
                       
                }
              }
                while (UserOption != ShotgunOption.Shield || CPUOption != ShotgunOption.Shield);
            } while (QUIT == false || gameCount == 3);

Posted 24-Jun-15 7:38am

Josh909

Updated 24-Jun-15 7:54am

Suvendu Shekhar Giri

v2

Add a Solution

Comments

[no name] 24-Jun-15 13:58pm

The error is very clear and specific. So..... what exactly is your question?

2 solutions

Add a Solution

Add your solution here

Treat my content as plain text, not as HTML

Preview 0

…

Existing Members

Sign in to your account

...or Join us

Download, Vote, Comment, Publish.

Your Email
Password
Forgot your password?

Your Email
This email is in use. Do you need your password?
Optional Password

I have read and agree to the Terms of Service and Privacy Policy
Please subscribe me to the CodeProject newsletters

When answering a question please:

Read the question carefully.
Understand that English isn't everyone's first language so be lenient of bad spelling and grammar.
If a question is poorly phrased then either ask for clarification, ignore it, or edit the question and fix the problem. Insults are not welcome.
Don't tell someone to read the manual. Chances are they have and don't get it. Provide an answer or move on to the next question.

Let's work to help developers, not make them feel stupid.

This content, along with any associated source code and files, is licensed under The Code Project Open License (CPOL)

OriginalGriff · Accepted Answer · 2015-06-24T08:10:00

OK...
Your switch block uses UserOption which is declared as:

C#

ShotgunOption UserOption;

But your

case

statements all use strings:

C#

case "SHOOT":
    ...
case "RELAOD":
    ...
case "SHIELD"

So unless ShotgunOption is actually a string (which it isn't, string is a sealed class) means that you can't directly compare it's values to strings. It's liek trying to compare integers to strings - would you expect

C#

if (123 == "hello!")
   {
   ...

To compile, or even make any sense in the real world? :laugh:
What you're doing is in fact exactly that, I suspect: ShotgunOption is almost certainly an
enum - which means that it's really an integer, but with added names.

I suspect that what you want to try is this:

C#

case ShotgunOption.Shoot:
    ...
case ShotgunOption.Reload:
    ...
case ShotgunOption.Shield

Or at least something very similar!

Ravi Bhavnani · Accepted Answer · 2015-06-24T08:06:00

Solution 1

You're trying to compare the enum type ShotgunOption with a string. Instead use the different enum values in your case statements.

/ravi

Posted 24-Jun-15 8:06am

Ravi Bhavnani