We Keep Coding

C# switch statement with curly braces for each case/default block within the switch statement?

Typically, a switch statement in C# looks like this
switch (sh)
{
case 1:
DoThis();
DoThat();
break;
case 2:
DoThis();
DoThat();
break;
default:
DoThis();
DoThat();
break;
}
But for the first time ever, I saw someone using curly braces for each case statement within a switch statement like below:
switch (sh)
{
case 1:
{
DoThis();
DoThat();
break;
}
case 2:
{
DoThis();
DoThat();
break;
}
default:
{
DoThis();
DoThat();
break;
}
}
Why are these curly braces {, } being used for each case and default blocks of the case statement above?
Why are they needed?
What is the difference?

They are not required, but they are useful if you declare variables with the same name in multiple branches:
switch (sh)
{
case 1:
var test = "test";
Console.WriteLine(test);
break;
case 2:
var test = "test";
Console.WriteLine(test);
break;
}
This will not compile, complaining about conflicting variable names. But if you add braces
switch (sh)
{
case 1:
{
var test = "";
Console.WriteLine(test);
break;
}
case 2:
{
var test = "";
Console.WriteLine(test);
break;
}
}
that will create its own scope for each branch and it will compile fine.
Some people get used to this and always add bracers, even when not defining any variables.

The issue you sometimes face in a switch statement is that all the cases are in the same scope. That means (for example) that you can not use the same variable name in two cases. Consider this:
switch (sh)
{
case 1:
int result = 1;
return result;
case 2:
int result = 2;
return result;
}
This will lead to a compiler error, since you are declaring result twice in the same scope. You remove the error by introducing new scopes in the cases:
switch (sh)
{
case 1:
{
int result = 1;
return result;
}
case 2:
{
int result = 2;
return result;
}
}
Considering that switch cases are somewhat controversial, because they introduce additional complexity, adding extra scopes adds to the confusion. I prefer to use switch blocks in a way that does not lead to the described problem (by keeping the amount of code in the cases low and try to avoid large switch blocks).
In your example the scopes are not needed and I would not use them on a general basis, since this is "unusual" code. And unusual usually means confusing. Your question is a proof to that opinion (and it is an opinion, just to avoid a religious war here), since this construct confused you enough to ask the question.

I belive this is remains of old code. I remember in C when I didnt put the break I would just fall in the next case...
This is no longer relevant and is just code style.
Note you can also do
switch (sh)
{
case 1:
DoThis();
DoThat();
break;
case 2:
{
DoThis();
DoThat();
}break;
case 3:
case 4:
{
DoThis();
DoThat();
}
case 5:
goto case 1;
default:
{
DoThis();
DoThat();
}
}
The break will just leave the Switch but which will be left after leaving the bracket.

Write a switch statement with one case containing the other

I would like to write a switch statement with three cases: A, B and C.
However, C is going to contain everything covered in A and then will add some other statements after that.
So if case A writes to a file, case C is supposed to write to a file and then write it to a table, too. How can we achieve this behavior without repeating the code for A in C?

put A's code in a method and then call it from A and C and then add the extra code before the break of C
public void Foo (){}
switch(variable)
{
case a: Foo(); break;
case b: otherMethod(); break;
case c: Foo(); ExtraWork(); break;
}

Another alternative : C# have a goto statement which might come in handy [Less codes]
switch (CASE){
case A:
JUMP:
// Code for A
break;
case B:
// code for B
break;
case C:
// Code for C
goto JUMP; // Will jump to A
}

You can create a method for writing to file.
public void Write(){
//your code here.
}
Now from switch statement case A and C call Write(). For case C after calling Write(), you can execute other codes.

In your case I think about two options:
The first is having a distinct case for each value and extract the code in a private method that is called in both case:
switch (caseSwitch)
{
case a:
YourPrivateMethod();
break;
case b:
// What ever you want to do
break;
case c:
YourPrivateMethod();
break;
default:
Console.WriteLine("Default case");
break;
}
private void YourPrivateMethod()
{
//some stuff
}
Second one is having a shared case for a and c :
switch (caseSwitch)
{
case a:
case c:
// Do what A should
if(caseSwitch == c)
{
// Do what C should
}
break;
case b:
// What ever you want to do
break;
default:
Console.WriteLine("Default case");
break;
}
Regards.

C# Change If to switch, how to handle "and"

So I have a task from a worksheet at work:
I have to change these Ifs to a switch, but how do i do it? Heres the Ifs:
if (i<0 || i>15) Console.WriteLine ("A");
else if (i%5<2 && i/3>3) Console.WriteLine ("B");
else if (3<i && i<10) Console.WriteLine ("C");
else if (i&2==2) Console.WriteLine ("D");
else Console.WriteLine ("E");
and here is the switch i made, but that one is bad, but i dont know how to make a good one out of it, i hope you can help me with this.
switch (i)
case (i<0):
case (i>15):
Console.WriteLine ("A“)
Break;
Case (i%5<2 && i/3>3) :
Console.WriteLine ("B“)
Break;
case (3<i && i<10) :
Console.WriteLine ("C");
Break;
Case (i&2==2) :
Console.WriteLine ("D");
Break;
Default
Console.WriteLine ("E");
Break;
It doesnt have to run in a programm, it's just a task from a worksheet

I haven't figured out all the rules completely, but I think what you're after is something like this:
The question wants you to realise that you've got a limited number of results, since the first rule is (i<0 || i>15). This can be the default for a switch statement.
Then you need to work out what would happen for all integers 0-15 and write them into the rest of the switch.
So you're after something along the lines of (although this doesn't have your logic - you can figure that out so you understand what's going on):
switch (i)
{
case 0:
case 2:
case 5:
Console.Write("Something")
break;
case 1:
case 7:
Console.Write("Something Else")
break;
default
Console.WriteLine ("A“)
Break;
}

I think it's:
switch (i)
{
case 4:
case 5:
case 6:
case 7:
case 8:
case 9:
Console.WriteLine("C");
break;
case 0:
case 1:
case 3:
case 10:
case 11:
case 12:
case 13:
case 14:
Console.WriteLine("E");
break;
case 2:
Console.WriteLine("D");
break;
case 15:
Console.WriteLine("B");
break;
default:
Console.WriteLine("A");
break;
}

Use a separate variable, in which you have to embed the segmentation logic. Here is a sample:
//using enum
public enum MySwitchableVariable {
NotDefined, //optional
LessThanZero,
MoreThanFifteen,
RestOfFiveLessThanTwoAndFactorMoreThanThree, //its too complex to describe
BetweenThreeAndTen,
RestOfTwoIsTwo,
}
Then you have to make a function which takes an integer and spit out a case of this enum:
public MySwitchableVariable calculate(int i) {
var result = MySwitchableVariable.NotDefined;
// your implementation here
return result;
}
finally you may switch that annoying variable like this:
var Variable = calculate(i);
switch(Variable) {
case MySwitchableVariable.LessThanZero:
// you know the rest
is that what you (or your instructor) want?

Simply you can't do this. Using the switch statement entails that the variable in switch can take a number of discrete constant values.
From the documentation:
The switch statement is a control statement that selects a switch
section to execute from a list of candidates.
Furthermore,
Each case label specifies a constant value. The switch statement transfers control to the switch section whose case label matches the
value of the switch expression (caseSwitch in the example). If no case
label contains a matching value, control is transferred to the default
section, if there is one. If there is no default section, no action is
taken and control is transferred outside the switch statement. In the
previous example, the statements in the first switch section are
executed because case 1 matches the value of caseSwitch.

how to check for multiple switch statement in c#

i need to check more than one statement in switch statement to evalute
like
int a=5;
switch(a)
{
case 4,5:
console.write("its from 4 to 5);
break;
}

You want to do:
case 4:
case 5:
//Code goes here.
break;
Remember though C# doesn't allow fall through so you can't do:
case 4:
//Do some stuff here
//fall through to 5
case 5:
//Code goes here.
break;

In c# you stack cases to do this:
case 4:
case 5:
//do something
break;
case 6:
//do something
etc.

This allows you to execute multiple cases for 1 value.
int a=5;
switch(a)
{
case 4:
// Do work here
goto case 5;
case 5:
console.write("its from 4 to 5);
break;
}
or
This is giving a case two labels.
switch(a)
{
case 4:
case 5:
console.write("its from 4 to 5);
break;
}

This is how..
int a=5;
switch(a)
{
case 4:
case 5:
console.write("its from 4 to 5);
break;
}

Switch Statement in C# [duplicate]

This question already has answers here:
Multiple cases in switch statement
(24 answers)
Switch case: can I use a range instead of a one number [duplicate]
(16 answers)
Closed 1 year ago.
Does anyone know if it's possible to include a range in a switch statement (and if so, how)?
For example:
switch (x)
{
case 1:
//do something
break;
case 2..8:
//do something else
break;
default:
break;
}
The compiler doesn't seem to like this kind of syntax - neither does it like:
case <= 8:

No, this isn't possible. There are a few ways I've done this in the past:
Fixed coding:
switch (x)
{
case 1:
//do something
break;
case 2:
case 3:
case 4:
case 5:
case 6:
case 7:
case 8:
//do something else
break;
default:
break;
}
In combination with an if {} statement:
switch (x)
{
case 1:
//do something
break;
default:
if (x <= 8)
{
// do something
}
else
{
// throw exception
}
break;
}

No, but you can write this, so you at least avoid writing the // do something else part multiple times.
switch (x)
{
case 1:
//do something
break;
case 2: case 3: case 4: case 5: case 6: case 7: case 8:
//do something else
break;
default:
break;
}

Whilst this wasn't possible when I originally asked this question, through the miracle of C# Pattern Matching, it now is (in C# 7):
switch (i)
{
case var test when test <= 2:
Console.WriteLine("Less than 2");
break;
case var test when test > 2 && test < 10:
Console.WriteLine("Between 2 and 10");
break;
case var test when test >= 10:
Console.WriteLine("10 or more");
break;
}
A blog post on the subject

Short answer : no. It would be possible to write all of the cases there but such a range notation is not supported.
I think you have to use if statement here or switch to a language where there is a better support for case descrimination.

One possibility is to convert your ranges into integers. For example:
//assuming x>=9 or if (x <= 0) return;
switch((x+12)/7)
{ case 1:Console.WriteLine("one");
break;
case 2:Console.WriteLine("2 through 8 inclusive");
break;
case 3:Console.WriteLine("9 through 15 inclusive");
break;
default: Console.WriteLine("16 or more");
break;
}

If you have so few cases, if would be much preferred.

You could, handle the explicit cases case by case, and if you only have one range, deal with it in the default case.

you can do
case 2:
case 3:
case 4:
...
case 8:
// code here
break

You can use case fall through:
switch (x)
{
case 1:
//do something
break;
case 2:
case 3:
case 4:
case 5:
case 6:
case 7:
case 8:
//do something else
break;
default:
break;
}
But I'd just use if for this.

You cannot use any conditional statements in a switch case.
If you want to execute the same lines of code for different options then you can do one thing:
switch (i)
{
case 0:
case 1:
case 2:
case 3:
//do something here.
break;
default:
//do something here.
break;
}