Multiple cases in switch statement
Asked Answered
O

26

754

Is there a way to fall through multiple case statements without stating case value: repeatedly?

I know this works:

switch (value)
{
   case 1:
   case 2:
   case 3:
      // Do some stuff
      break;
   case 4:
   case 5:
   case 6:
      // Do some different stuff
      break;
   default:
       // Default stuff
      break;
}

but I'd like to do something like this:

switch (value)
{
   case 1,2,3:
      // Do something
      break;
   case 4,5,6:
      // Do something
      break;
   default:
      // Do the Default
      break;
}

Is this syntax I'm thinking of from a different language, or am I missing something?

Obliquity answered 16/9, 2008 at 1:34 Comment(6)
Is there a reason you don't just use an IF statement (if you are checking a range of ints)?Leatherwood
yes charlse, the first way works fine, I've used it in numerous places. It's dirtier than I'd like, but it is useful. I just used those integers as an example. The real data was more varied. An if (1 || 2 || 3 ) {...} else if (4 || 5 || 6) {...} would have worked too, but it's harder to read.Obliquity
why do you consider the latter dirtier than the former. The latter adds yet another meaning to , and one that isn't shared with any other c-style language. That would seem much dirtier to me.Chesterchesterfield
You're probably thinking of Delphi/Pascal, which allows the case i when 1...3: begin end; 4, 5, 7: begin end; 6, 8..10: begin end; else // handle default end; type syntax.Podophyllin
Important Note. Ranges are supported in switch case starting C# v7 - Please see Steve G.'s answerReadytowear
The top example here is actually a lot shorter and easier to read than all the "this is how you would do it in a fancy range based or enumerable range way today" answers that don't fit my super wide screen. I mean. You look at it and it's clear what it does.Agony
F
371

There is no syntax in C++ nor C# for the second method you mentioned.

There's nothing wrong with your first method. If however you have very big ranges, just use a series of if statements.

Featured answered 16/9, 2008 at 1:40 Comment(4)
As an addition I wanted to add a link to the C# language specification available on MSDN at msdn.microsoft.com/en-us/vcsharp/aa336809.aspxLaicize
User could use some if's (or a table lookup) to reduce the input to a set of enums and switch on the enum.Uvea
I believe this is no longer correct. See #20148379 . Also on this very question there's an answer https://mcmap.net/q/54032/-multiple-cases-in-switch-statementIlldisposed
Using that many "if" statements is just downright cumbersome. See Misha's answer below for a better method.Tottering
O
878

I guess this has been already answered. However, I think that you can still mix both options in a syntactically better way by doing:

switch (value)
{
    case 1: case 2: case 3:          
        // Do Something
        break;
    case 4: case 5: case 6: 
        // Do Something
        break;
    default:
        // Do Something
        break;
}
Olivine answered 1/8, 2010 at 13:29 Comment(8)
The collapsed code gets lengthened to the first example in the question. May as well just do it the way it is in the question.Doorframe
Why bother? The auto indenter in Visual Studio 2013 will revert this to the format in the original question anyway.Cherian
@JeremyChild Maybe because this answer is merely a disguised copy of the question. One of the rare moments I'm down-voting an answer. For real, how did this get so much support?Striped
@Striped it's getting support because it actually answers the question. The OP said, am I missing something... Carlos answered with what he was missing. Seems pretty cut and dried to me. Don't hate that he's got 422 upvotes.Pattison
@MikeDevenney Then you interpreted the question differently, as far as I see the correct answer would be "no, c# doesn't have any syntax for that". If someone asks "is it possible to pour liquid in a glass I'm holding upside down?" the answer should be "no" and not "you can pour liquid up if you look at it upside down and use your imagination", because this answer is all about using imagination. If you use the regular syntax but format it badly it looks like other syntax, with some imagination. Hopefully you get my point... :PStriped
@MikeDevenney For example I have down voted this answer because it is the same answer that Allan Wind gave over two years before this answer! with a bit of unnecessary added fluff.... oh and I upvoted Allans answer ;)Lugworm
This will also get long lines if you are using enums, which is mostly the case for switch case statements....Naples
I like this because it reflects better what is happening than stacking empty cases above the one that does stuff. I.e. Any of these, do stuff; rather than perhaps a bunch of cases were set up and and the programmer missed doing something in one or more of them. This shows intent better.Manvell
F
371

There is no syntax in C++ nor C# for the second method you mentioned.

There's nothing wrong with your first method. If however you have very big ranges, just use a series of if statements.

Featured answered 16/9, 2008 at 1:40 Comment(4)
As an addition I wanted to add a link to the C# language specification available on MSDN at msdn.microsoft.com/en-us/vcsharp/aa336809.aspxLaicize
User could use some if's (or a table lookup) to reduce the input to a set of enums and switch on the enum.Uvea
I believe this is no longer correct. See #20148379 . Also on this very question there's an answer https://mcmap.net/q/54032/-multiple-cases-in-switch-statementIlldisposed
Using that many "if" statements is just downright cumbersome. See Misha's answer below for a better method.Tottering
W
152

Original Answer for C# 7

In C# 7 (available by default in Visual Studio 2017/.NET Framework 4.6.2), range-based switching is now possible with the switch statement and would help with the OP's problem.

Example:

int i = 5;

switch (i)
{
    case int n when (n >= 7):
        Console.WriteLine($"I am 7 or above: {n}");
        break;

    case int n when (n >= 4 && n <= 6 ):
        Console.WriteLine($"I am between 4 and 6: {n}");
        break;

    case int n when (n <= 3):
        Console.WriteLine($"I am 3 or less: {n}");
        break;
}

// Output: I am between 4 and 6: 5

Notes:

  • The parentheses ( and ) are not required in the when condition, but are used in this example to highlight the comparison(s).
  • var may also be used in lieu of int. For example: case var n when n >= 7:.

Updated examples for C# 9

switch(myValue)
{
    case <= 0:
        Console.WriteLine("Less than or equal to 0");
        break;
    case > 0 and <= 10:
        Console.WriteLine("More than 0 but less than or equal to 10");
        break;
    default:
        Console.WriteLine("More than 10");
        break;
}

or

var message = myValue switch
{
    <= 0 => "Less than or equal to 0",
    > 0 and <= 10 => "More than 0 but less than or equal to 10",
    _ => "More than 10"
};
Console.WriteLine(message);
Wicklow answered 30/6, 2017 at 14:21 Comment(2)
This (pattern matching) should generally be best practice when you can use C# 7.x or above, as it is much clearer than the other answers.Lauraine
Is there a way to achieve this with a list of Enums? Where the Enums map to int?Fragonard
G
74

This syntax is from the Visual Basic Select...Case Statement:

Dim number As Integer = 8
Select Case number
    Case 1 To 5
        Debug.WriteLine("Between 1 and 5, inclusive")
        ' The following is the only Case clause that evaluates to True.
    Case 6, 7, 8
        Debug.WriteLine("Between 6 and 8, inclusive")
    Case Is < 1
        Debug.WriteLine("Equal to 9 or 10")
    Case Else
        Debug.WriteLine("Not between 1 and 10, inclusive")
End Select

You cannot use this syntax in C#. Instead, you must use the syntax from your first example.

Goolsby answered 16/9, 2008 at 1:41 Comment(1)
How is this an answer? "You cannot use the following code"Hephaestus
R
64

With C#9 came the Relational Pattern Matching. This allows us to do:

switch (value)
{
    case 1 or 2 or 3:
      // Do stuff
      break;
    case 4 or 5 or 6:
      // Do stuff
      break;
    default:
        // Do stuff
        break;
}

In deep tutorial of Relational Patter in C#9

Pattern-matching changes for C# 9.0

Relational patterns permit the programmer to express that an input value must satisfy a relational constraint when compared to a constant value

Roselynroseman answered 23/1, 2021 at 20:51 Comment(0)
M
40

You can leave out the newline which gives you:

case 1: case 2: case 3:
   break;

but I consider that bad style.

Mattos answered 16/9, 2008 at 3:11 Comment(1)
Bad style is subjective. I prefer this as it clearly shows intent.Manvell
R
22

.NET Framework 3.5 has got ranges:

Enumerable.Range from MSDN

you can use it with "contains" and the IF statement, since like someone said the SWITCH statement uses the "==" operator.

Here an example:

int c = 2;
if(Enumerable.Range(0,10).Contains(c))
    DoThing();
else if(Enumerable.Range(11,20).Contains(c))
    DoAnotherThing();

But I think we can have more fun: since you won't need the return values and this action doesn't take parameters, you can easily use actions!

public static void MySwitchWithEnumerable(int switchcase, int startNumber, int endNumber, Action action)
{
    if(Enumerable.Range(startNumber, endNumber).Contains(switchcase))
        action();
}

The old example with this new method:

MySwitchWithEnumerable(c, 0, 10, DoThing);
MySwitchWithEnumerable(c, 10, 20, DoAnotherThing);

Since you are passing actions, not values, you should omit the parenthesis, it's very important. If you need function with arguments, just change the type of Action to Action<ParameterType>. If you need return values, use Func<ParameterType, ReturnType>.

In C# 3.0 there is no easy Partial Application to encapsulate the fact the the case parameter is the same, but you create a little helper method (a bit verbose, tho).

public static void MySwitchWithEnumerable(int startNumber, int endNumber, Action action){ 
    MySwitchWithEnumerable(3, startNumber, endNumber, action); 
}

Here an example of how new functional imported statement are IMHO more powerful and elegant than the old imperative one.

Reggi answered 16/9, 2008 at 10:27 Comment(4)
Good choice. One thing to note, though - Enumerable.Range has arguments int start and int count. Your examples wont work right the way they were written. You write it as if the second argument is int end. For example - Enumerable.Range(11,20) would result in 20 numbers starting with 11, and not numbers from 11 to 20.Chevron
although, if working with an Enum, why not something like? if(Enumerable.Range(MyEnum.A, MyEnum.M){ DoThing(); } else if(Enumerable.Range(MyEnum.N, MyEnum.Z){ DoAnotherThing(); }Interconnect
Note that Enumerable.Range(11,20).Contains(c) is equivalent to for(int i = 11; i < 21; ++i){ if (i == c) return true; } return false; If you had a large range it would take a long time, while just using > and < would be quick and constant-time.Chesterchesterfield
An improvement: Having MySwitchWithEnumerable return void is weak design for this situation. REASON: You've converted an if-else to a series of independent statements - that hides the intent, which is that they are mutually-exclusive - only one action is executed. Instead return bool, with body if (..) { action(); return true; } else return false; The calling site then shows the intention: if (MySwitchWithEnumerable(..)) else (MySwitchWithEnumerable(..));. This is preferable. However, it is also no longer a significant improvement over your original version, for this simple case.Mayworm
C
19

Here is the complete C# 7 solution...

switch (value)
{
   case var s when new[] { 1,2,3 }.Contains(s):
      // Do something
      break;
   case var s when new[] { 4,5,6 }.Contains(s):
      // Do something
      break;
   default:
      // Do the default
      break;
}

It works with strings too...

switch (mystring)
{
   case var s when new[] { "Alpha","Beta","Gamma" }.Contains(s):
      // Do something
      break;
...
}
Corelli answered 27/12, 2018 at 23:23 Comment(3)
This would mean that you allocate the arrays with each switch statement, right? Wouldn't it be better if we had them as constant variables?Dorsiventral
Elegant, but it would indeed be good to know if the compiler optimizes this scenario so that repeated invocations don't incur the overhead of the array construction every time; defining the arrays ahead of time is an option, but takes away much of the elegance.Lentamente
Create an array and search it just to check if it is a or b or c is very in-efficient!Lunitidal
B
14

The code below won't work:

case 1 | 3 | 5:
// Not working do something

The only way to do this is:

case 1: case 2: case 3:
// Do something
break;

The code you are looking for works in Visual Basic where you easily can put in ranges... in the none option of the switch statement or if else blocks convenient, I'd suggest to, at very extreme point, make .dll with Visual Basic and import back to your C# project.

Note: the switch equivalent in Visual Basic is Select Case.

Barsky answered 4/11, 2012 at 0:3 Comment(0)
M
8

Another option would be to use a routine. If cases 1-3 all execute the same logic then wrap that logic in a routine and call it for each case. I know this doesn't actually get rid of the case statements, but it does implement good style and keep maintenance to a minimum.....

[Edit] Added alternate implementation to match original question...[/Edit]

switch (x)
{
   case 1:
      DoSomething();
      break;
   case 2:
      DoSomething();
      break;
   case 3:
      DoSomething();
      break;
   ...
}

private void DoSomething()
{
   ...
}

Alt

switch (x)
{
   case 1:
   case 2:
   case 3:
      DoSomething();
      break;
   ...
}

private void DoSomething()
{
   ...
}
Magnanimous answered 16/9, 2008 at 3:30 Comment(0)
S
6

In C# 7 we now have Pattern Matching so you can do something like:

switch (age)
{
  case 50:
    ageBlock = "the big five-oh";
    break;
  case var testAge when (new List<int>()
      { 80, 81, 82, 83, 84, 85, 86, 87, 88, 89 }).Contains(testAge):
    ageBlock = "octogenarian";
    break;
  case var testAge when ((testAge >= 90) & (testAge <= 99)):
    ageBlock = "nonagenarian";
    break;
  case var testAge when (testAge >= 100):
    ageBlock = "centenarian";
    break;
  default:
    ageBlock = "just old";
    break;
}
Shiller answered 5/9, 2019 at 9:2 Comment(0)
L
5

One lesser known facet of switch in C# is that it relies on the operator= and since it can be overriden you could have something like this:


string s = foo();

switch (s) {
  case "abc": /*...*/ break;
  case "def": /*...*/ break;
}
Lavine answered 16/9, 2008 at 2:16 Comment(1)
this could become a big gotcha later on for someone else trying to read the codeRorke
A
5

gcc implements an extension to the C language to support sequential ranges:

switch (value)
{
   case 1...3:
      //Do Something
      break;
   case 4...6:
      //Do Something
      break;
   default:
      //Do the Default
      break;
}

Edit: Just noticed the C# tag on the question, so presumably a gcc answer doesn't help.

Agitation answered 16/9, 2008 at 2:59 Comment(0)
V
5

I think this one is better in C# 7 or above.

switch (value)
{
    case var s when new[] { 1,2 }.Contains(s):
    // Do something
     break;
   
    default:
    // Do the default
    break;
 }

You can also check Range in C# switch case: Switch case: can I use a range instead of a one number

OR

 int i = 3;

        switch (i)
        {
            case int n when (n >= 7):
                Console.WriteLine($"I am 7 or above: {n}");
                break;

            case int n when (n >= 4 && n <= 6):
                Console.WriteLine($"I am between 4 and 6: {n}");
                break;

            case int n when (n <= 3):
                Console.WriteLine($"I am 3 or less: {n}");
                break;
        }

Switch case multiple conditions in C#

Or if you want to understand basics of C# switch case

Voile answered 24/4, 2020 at 14:40 Comment(0)
S
5

In C# 8.0 you can use the new switch expression syntax which is ideal for your case.

var someOutput = value switch
{
    >= 1 and <= 3 => <Do some stuff>,
    >= 4 and <= 6 => <Do some different stuff>,
    _ => <Default stuff>
};
Sapper answered 22/6, 2021 at 23:0 Comment(0)
R
5

C# 8 way to do it:

var value = 5;
var output = value switch
{
    1 or 2 or 3 => value + 4,// Do some stuff
    4 => value * 3,// Do some different stuff
    5 => value / 3,// Do some different stuff
    6 => value - 3,// Do some different stuff
    _ => value  // Default stuff
};
Rahn answered 8/6, 2023 at 19:54 Comment(3)
Sorry mate, but this example seems unnecessarily confusing.Emilia
@Emilia This is basic pattern matching that is part of functional languagesPinole
You're correct. I lacked understanding when I made the comment.Emilia
S
4

Actually I don't like the GOTO command too, but it's in official Microsoft materials, and here are all allowed syntaxes.

If the end point of the statement list of a switch section is reachable, a compile-time error occurs. This is known as the "no fall through" rule. The example

switch (i) {
case 0:
   CaseZero();
   break;
case 1:
   CaseOne();
   break;
default:
   CaseOthers();
   break;
}

is valid because no switch section has a reachable end point. Unlike C and C++, execution of a switch section is not permitted to "fall through" to the next switch section, and the example

switch (i) {
case 0:
   CaseZero();
case 1:
   CaseZeroOrOne();
default:
   CaseAny();
}

results in a compile-time error. When execution of a switch section is to be followed by execution of another switch section, an explicit goto case or goto default statement must be used:

switch (i) {
case 0:
   CaseZero();
   goto case 1;
case 1:
   CaseZeroOrOne();
   goto default;
default:
   CaseAny();
   break;
}

Multiple labels are permitted in a switch-section. The example

switch (i) {
case 0:
   CaseZero();
   break;
case 1:
   CaseOne();
   break;
case 2:
default:
   CaseTwo();
   break;
}

I believe in this particular case, the GOTO can be used, and it's actually the only way to fallthrough.

Source

Surgery answered 8/9, 2011 at 8:21 Comment(1)
Note that in practice, the goto can almost always be avoided (though I don't consider it "terrible" here - it is filling a specific, structured, role). In your example, because you have wrapped the case bodies in functions (a good thing), case 0 can become CaseZero(); CaseZeroOrOne(); break;. No goto required.Mayworm
M
4

You can also have conditions that are completely different

            bool isTrue = true;

            switch (isTrue)
            {
                case bool ifTrue when (ex.Message.Contains("not found")):
                case bool ifTrue when (thing.number = 123):
                case bool ifTrue when (thing.othernumber != 456):
                    response.respCode = 5010;
                    break;
                case bool ifTrue when (otherthing.text = "something else"):
                    response.respCode = 5020;
                    break;
                default:
                    response.respCode = 5000;
                    break;
            }
Memorialize answered 10/2, 2021 at 14:13 Comment(0)
C
3

If you have a very big amount of strings (or any other type) case all doing the same thing, I recommend the use of a string list combined with the string.Contains property.

So if you have a big switch statement like so:

switch (stringValue)
{
    case "cat":
    case "dog":
    case "string3":
    ...
    case "+1000 more string": // Too many string to write a case for all!
        // Do something;
    case "a lonely case"
        // Do something else;
    .
    .
    .
}

You might want to replace it with an if statement like this:

// Define all the similar "case" string in a List
List<string> listString = new List<string>(){ "cat", "dog", "string3", "+1000 more string"};
// Use string.Contains to find what you are looking for
if (listString.Contains(stringValue))
{
    // Do something;
}
else
{
    // Then go back to a switch statement inside the else for the remaining cases if you really need to
}

This scale well for any number of string cases.

Contravention answered 29/3, 2018 at 16:0 Comment(0)
C
2

A more beautiful way to handle that

if ([4, 5, 6, 7].indexOf(value) > -1)
    //Do something

You can do that for multiple values with the same result

Curst answered 7/9, 2021 at 13:15 Comment(0)
H
1

An awful lot of work seems to have been put into finding ways to get one of C# least used syntaxes to somehow look better or work better. Personally I find the switch statement is seldom worth using. I would strongly suggest analyzing what data you are testing and the end results you are wanting.

Let us say for example you want to quickly test values in a known range to see if they are prime numbers. You want to avoid having your code do the wasteful calculations and you can find a list of primes in the range you want online. You could use a massive switch statement to compare each value to known prime numbers.

Or you could just create an array map of primes and get immediate results:

    bool[] Primes = new bool[] {
        false, false, true, true, false, true, false,    
        true, false, false, false, true, false, true,
        false,false,false,true,false,true,false};
    private void button1_Click(object sender, EventArgs e) {
        int Value = Convert.ToInt32(textBox1.Text);
        if ((Value >= 0) && (Value < Primes.Length)) {
            bool IsPrime = Primes[Value];
            textBox2.Text = IsPrime.ToString();
        }
    }

Maybe you want to see if a character in a string is hexadecimal. You could use an ungly and somewhat large switch statement.

Or you could use either regular expressions to test the char or use the IndexOf function to search for the char in a string of known hexadecimal letters:

        private void textBox2_TextChanged(object sender, EventArgs e) {
        try {
            textBox1.Text = ("0123456789ABCDEFGabcdefg".IndexOf(textBox2.Text[0]) >= 0).ToString();
        } catch {
        }
    }

Let us say you want to do one of 3 different actions depending on a value that will be the range of 1 to 24. I would suggest using a set of IF statements. And if that became too complex (Or the numbers were larger such as 5 different actions depending on a value in the range of 1 to 90) then use an enum to define the actions and create an array map of the enums. The value would then be used to index into the array map and get the enum of the action you want. Then use either a small set of IF statements or a very simple switch statement to process the resulting enum value.

Also, the nice thing about an array map that converts a range of values into actions is that it can be easily changed by code. With hard wired code you can't easily change behaviour at runtime but with an array map it is easy.

Handy answered 23/2, 2012 at 6:25 Comment(2)
You could also map to lambda expression or a delegateSomber
Good points. One minor comment: I usually find it easier to maintain a list of the values that match a given case, than an array map. The problem with array map is that it is easy to make a mistake. For example, instead of the primes array map of true/falses, simply have a list of primes, and load them into a HashSet for lookup performance. Even if there are more than two cases, usually all but one case is a small list, so build either a HashSet of enums (if sparse) or an array map, in code, from lists of the other cases.Mayworm
H
1

Just to add to the conversation, using .NET 4.6.2 I was also able to do the following. I tested the code and it did work for me.

You can also do multiple "OR" statements, like below:

            switch (value)
            {
                case string a when a.Contains("text1"):
                    // Do Something
                    break;
                case string b when b.Contains("text3") || b.Contains("text4") || b.Contains("text5"):
                    // Do Something else
                    break;
                default:
                    // Or do this by default
                    break;
            }

You can also check if it matches a value in an array:

            string[] statuses = { "text3", "text4", "text5"};

            switch (value)
            {
                case string a when a.Contains("text1"):
                    // Do Something
                    break;
                case string b when statuses.Contains(value):                        
                    // Do Something else
                    break;
                default:
                    // Or do this by default
                    break;
            }
Harass answered 3/7, 2019 at 0:32 Comment(1)
Isn't this dependent on the C# version, not the .NET version?Adnate
U
0

We can also use this approach to achieve Multiple cases in switch statement... You can use as many conditions as you want using this approach..

    int i = 209;
    int a = 0;
    switch (a = (i>=1 && i<=100) ? 1 : a){    
    case 1:
        System.out.println ("The Number is Between 1 to 100 ==> " + i);
        break;
    default:
        switch (a = (i>100 && i<=200) ? 2 : a) {
            case 2:
                System.out.println("This Number is Between 101 to 200 ==> " + i);
                break;
        
            default:
                switch (a = (i>200 && i<=300) ? 3 : a) {
                    case 3:
                        System.out.println("This Number is Between 201 to 300 ==> " + i);
                        break;
                
                    default:
                        // You can make as many conditions as you want;
                        break;
                }
        }
        
    }
Ulna answered 12/12, 2021 at 18:6 Comment(0)
H
0

Using new version of C# I have done in this way

public string GetValue(string name)
            {
                return name switch
                {
                    var x when name is "test1" || name is "test2" => "finch",
                    "test2" => somevalue,
                    _ => name
                };
            }
Haldane answered 17/10, 2022 at 19:54 Comment(0)
T
0

The C#9 example is preferable as it is easiest to read for most developers regardless of their background.

If you are capitated to using an earlier framework that doesn't allow for this syntax then use the format that got over 800 votes.

The prime number example while good academically speaking is an unrealistic business case, I seriously doubt that the over 800 respondents have had a need to find prime number ranges (yes I know it's just an example) and switch between them for their day to day programming.

Lambda expressions and LINQ syntax (less so) examples, while cool, are difficult to read for the uninitiated.

Who knows, maybe Microsoft will add a 'between' keyword in future versions of C# to make ranges easier to read -- that I'd like to see!

Until such time, the C#7 and updated C#9 examples below for ranges work and look best for readability.

Tory answered 7/4, 2023 at 14:16 Comment(0)
C
-6

For this, you would use a goto statement. Such as:

    switch(value){
    case 1:
        goto case 3;
    case 2:
        goto case 3;
    case 3:
        DoCase123();
    //This would work too, but I'm not sure if it's slower
    case 4:
        goto case 5;
    case 5:
        goto case 6;
    case 6:
        goto case 7;
    case 7:
        DoCase4567();
    }
Crucifixion answered 9/7, 2011 at 22:41 Comment(5)
@Crucifixion goto breaks a fundamental principal of procedural programming (of which c++ and c# are still rooted in; theyre not pure OO languages (thank God)). Procedural programming has a well defined flow of logic determined by language constructs and method calling conventions (how the runtime stack grows and shrinks). The goto statement circumvents this flow by allowing arbitrary jumping around, basically.Pleader
I'm not saying it's good style, persay, but it does what the original question was asking for.Crucifixion
No, it doesn't "do what the original question was asking for". The original question had code that worked as is. They didn't need it fixed. And even if they did, this is a horrible suggestion. Its less concise, and uses goto. Worse, it is a completely unnecessary use of goto, as the original syntax stated by OP does work. The question was whether there was a more concise way to give the alternate cases. As people answered years before you did, yes there is - if you are willing to put the several cases on one line case 1: case 2:, and if the auto-style of editor allows.Mayworm
The only reason goto's are determined to be bad is because some people find it hard to follow logic flow. .Net MSIL (assembled object code) uses goto all over because it is fast, but if .Net code can be written and be just as performant without them it's better to not use them and so you don't get flamed by people like @ToolmakerSteve's condescending reply.Coiffure
@wchoward - Please read my reply more carefully. My complaint isn't just about the use of goto. I objected because the question showed code that already works as is, and this answer a) takes that working code and makes it more verbose, and less-well-structured, to no benefit, b) doesn't answer the question.Mayworm

© 2022 - 2024 — McMap. All rights reserved.