switch with repeating cases?

[acppdummy]

I have some cases in a switch statement sharing common code. Would this work?

                switch(mycase){
                    case 0: { code for case 0; } break;
                    case 1:
                    case 2: {
                        common code for case 1 and 2;
                    }
                    break;
                    case 2: {
                        code only for case 2;
                    }
                    break;
                }

[Lindley]

Yes, that will work fine.

[acppdummy]

Thank you!

[Paul McKenzie]

I have some cases in a switch statement sharing common code. Would this work?

                switch(mycase){
                    case 0: { code for case 0; } break;
                    case 1:
                    case 2: {
                        common code for case 1 and 2;
                    }
                    break;
                    case 2: {
                        code only for case 2;
                    }
                    break;
                }

void foo()
{
   int mycase = 0;
   switch(mycase)
   {
      case 0: 
         break;
      case 1:
      case 2: 
      {
      }
      break;
      case 2: 
      {
      }
      break;
   }
}

Thank you for testing your code with Comeau C/C++!
Tell others about http://www.comeaucomputing.com/tryitout !

Your Comeau C/C++ test results are as follows:

Comeau C/C++ 4.3.10.1 (Oct 6 2008 11:28:09) for ONLINE_EVALUATION_BETA2
Copyright 1988-2008 Comeau Computing. All rights reserved.
MODE:strict errors C++ C++0x_extensions

"ComeauTest.c", line 13: error: case label value has already appeared in this
switch at line 9
case 2:
^

1 error detected in the compilation of "ComeauTest.c".

In strict mode, with -tused, Compile failed

Regards,

Paul McKenzie

[acppdummy]

Well g++ didn't like it either:

error: duplicate case value

I suppose I could break it up by inserting some more switches like this?

switch(mycase){
                    case 0: { code for case 0; } break;
                    case 1:
                    case 2: {
                        common code for case 1 and 2;
                    }
                    break;
            }
            switch(mycase){
                    case 2: {
                        code only for case 2;
                    }
                    break;
             }

Or is there a way to turn off the "strict" mode in g++?

[laserlight]

I wonder if it would not just be simpler to write:

switch(mycase){
    case 0: { /* code for case 0; */ } break;
    case 1:
    case 2: {
        /* common code for case 1 and 2; */
        if (mycase == 2) {
            /* code only for case 2; */
        }
    }
    break;
}

[nuzzle]

I would prefer to first separate the cases and then use functions for common code, like

switch (mycase){
    case 0:  
          // specific to case 0
       break;
    case 1:
       do12(); // common to both 1 and 2
          // specific to case 1
       break;
    case 2: {
       do12(); // common to both 1 and 2
          // specific to case 2
       break;
}

[acppdummy]

Thanks everyone!

[Lindley]

I missed that case 2 was in there twice, sorry.