(C#) goto and labeled statements

[Mo5]

I'm having a small problem with a code similar to this:

Console.WriteLine("Input number");                string a3 = Console.ReadLine();if (a3 == "1")                {                    goto Label1;                }else if (a3 == "2")                {                    goto Label2;                }            }        Label1:            Console.WriteLine("Label1");        Label2:            Console.WriteLine("Label2");

The problem is that when I input 1 is writes out both "Label1" and "Label2". How do you properly separate labeled statements?

[Wtalk2]

You can put the WriteLine("Label1") and WriteLine("Label2") in seperate functions and just call whatever function you need instead of using goto.

[Nuluvius]

You don't...

 

Agreed. You should endeavour to NEVER use goto at all, it is extremely bad form... there are exceptions to this but they are generally not found in higher level languages.

[Deprecated]

Agreed. You should endeavour to NEVER use goto at all, it is extremely bad form... there are exceptions to this but they are generally not found in higher level languages.

Avoid that spaghetti code

[LePawel]

Avoid that spaghetti code

Agreed. You should endeavour to NEVER use goto at all, it is extremely bad form... there are exceptions to this but they are generally not found in higher level languages.

eh..

As long as code doesn't lose its visual structure, and it's easy to follow it, I don't see a problem.

Use it if you have to, but before you decide on it check if a function, switch, flag (bool or enum) or even an if-else statement can substitute it.

Once you include it in your code, review it with your friendly rubber duck and see if you don't come up with a solution after listening to yourself one final time.

Lastly, if you work for/with someone, prepare good explanation for its use.

 

P.S Same goes for continue;