switch, case stop after one statement

Question

0.00/5 (No votes)

See more:

I have this:

C#

static void Main(string[] args)
       {
           //TimePlanner timePlanner = new TimePlanner();
           //LoginDetails loginDetails = new LoginDetails();

           IBotAction botAction;
           //string value;
           Console.WriteLine("choose the option you want to see");
           Console.WriteLine("\n1.Type timePlanner - here you can see your tasks ");
           Console.WriteLine("\n2.Type  loginDetails - Here you can see the log in data of the client");
           Console.WriteLine("\nType here your choice");
           string input = Console.ReadLine();
           while (true)
           {
               //if (input == "exit")
               //{


                   switch (input)
                   {
                       case "timePlanner":
                           // Console.WriteLine("You have choosen for Time Planner");
                           botAction = new TimePlanner();
                           botAction.ActionName();
                           break;
                       case "loginDetails":
                           //Console.WriteLine("You have choosen for Log in Details");
                           botAction = new LoginDetails();
                           botAction.ActionName();
                           break;


                       default:
                           break;
                   }
               //}
               //break;


           }


           Console.ReadKey();
       }

but the problem is that the output string continued, and doesnt stop after first output

Thank you

Posted 20-Feb-15 1:47am

Member 10225052

Add a Solution

Comments

CHill60 20-Feb-15 7:49am

You are not breaking out of the while loop - the "break" statements that are not commented out are within the switch statement not the loop

[no name] 20-Feb-15 7:55am

Thank you, I posted my my correction

3 solutions

Solution 2

You have a never ending loop

C#

while (true)
...

This will cause the program to repeat the switch statement again and again and again and again and again and again and again...

Posted 20-Feb-15 1:56am

Wendelius

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)

CHill60 · Accepted Answer · 2015-02-20T01:57:00

Solution 3

As per my comment, you are not breaking out of that (infinite) while loop.

Try something like this instead:

C#

Console.WriteLine("\nType here your choice");
Console.WriteLine("\nOr type 'Exit' to end");
while (true)
{
    string input = Console.ReadLine();
    if (input.ToLower() == "exit")
    {
        break;
    }
    switch (input)
etc...

Posted 20-Feb-15 1:57am

CHill60

Comments

[no name] 20-Feb-15 8:24am

thank you, see my update

CHill60 20-Feb-15 8:29am

See my comments to your update

[no name] 20-Feb-15 9:33am

ok, thank you

Member 10225052 · Accepted Answer · 2015-02-20T01:55:00

Like this:

C#

hile (true)
            {
                //if (input == "exit")
                //{


                    switch (input)
                    {
                        case "timePlanner":
                            // Console.WriteLine("You have choosen for Time Planner");
                            botAction = new TimePlanner();
                            botAction.ActionName();
                            break;
                        case "loginDetails":
                            //Console.WriteLine("You have choosen for Log in Details");
                            botAction = new LoginDetails();
                            botAction.ActionName();
                            break;
                        default:
                            break;
                    }
                    break;
                //}
                //break;


            }



            Console.ReadKey();
        }

I have it now like this:

C#

while (true)
           {
               //if (input == "exit")
               //{
               string input = Console.ReadLine();
               if (input.ToLower() == "exit")
               {
                   break;
               }

                   switch (input)
                   {
                       case "timePlanner":
                           // Console.WriteLine("You have choosen for Time Planner");
                           botAction = new TimePlanner();
                           botAction.ActionName();
                           break;
                       case "loginDetails":
                           //Console.WriteLine("You have choosen for Log in Details");
                           botAction = new LoginDetails();
                           botAction.ActionName();
                           continue;
                       case "Q":
                           break;
                       default:
                           break;
                   }
                   break;
               //}
               //break;


           }



           Console.ReadKey();
       }

but the problem is that I only can type one option