switch case failure

[YitzHandel]

The following is an extract from my program, which I seperated out and
compiled as its own program.

The problem is that as soon as a case tests true, all the cases below
it execute as well. So if the user inputs, 4, the statements from
cases 4-9 will execute.

(I replaced function calls with cout for testing.)

I am using Win2k, Open Watcom IDE v 1.3.

This has tested as happening in several places in the program.

(p.s. - it compiles as is, it also compliles fine with all the
commented out libraries - same error either way)


//#include <stdio.h>
#include <stdlib.h>
#include <iostream.h> // instead of using namespace std
//#include <string.hpp>
//#include <fstream.h>
//#include <istream>
//#include "Fcopy.h"
//#include <ctime>
//#include <math.h>
int main ()
{
int menu_select_int =-1;

do
{
system("CLS");
cout <<"************ **********MENU* *************** *******\n";
cout << " Create Or Edit Input File..........1 \n";
cout << " Calc. no. of stages......... .......2\n";
cout << " Data plot........... ............... 3\n";
cout << " RR vs. No. Of stages......... ......4\n";
cout << " McCabe Thiele plot........... ......5\n";
cout << " Printout the input file........... .6\n";
cout << " Printout of the tray compositions..7 \n";
cout << " Printout of the vle data........... 8\n";
cout << " Quit........... ............... .....9\n";
cout << "************** *************** *************** *****\n\n";
cout << "MENU: ";
cin >> menu_select_int ;

switch (menu_select_in t)
{

case 1:
cout << "\ngoing to input data function\n";
cout << "case 1";
cin.ignore();

case 2:
cout << "\ngoing to CNOS function\n";
cout << "case 2";
cin.ignore();

case 3:
cout << "\ngoing to DP function\n";
cout << "case 3";
cin.ignore();

case 4:
cout << "\ngoing to RR vs function\n";
cout << "case 4";
cin.ignore();

case 5:
cout << "\ngoing to McTh function\n";
cout << "case 5";
cin.ignore();

case 6:
cout << "\ngoing to PIF function\n";
cout << "case 6";
cin.ignore();

case 7:
cout << "\ngoing to printout function\n";
cout << "case 7";
cin.ignore();

case 8:
cout << "\ngoing to PO VLE function\n";
cout << "case 8";
cin.ignore();

case 9:
cout << "\ngoing to AYS - end function\n";
cout << "case 9";
cin.ignore();
} //end switch case
} while (menu_select_in t != 9);

cin.ignore();
return(0);

}

[David Resnick]

YitzHandel wrote:

The following is an extract from my program, which I seperated out and
compiled as its own program.

The problem is that as soon as a case tests true, all the cases below
it execute as well. So if the user inputs, 4, the statements from
cases 4-9 will execute.

(I replaced function calls with cout for testing.)

I am using Win2k, Open Watcom IDE v 1.3.

This has tested as happening in several places in the program.

(p.s. - it compiles as is, it also compliles fine with all the
commented out libraries - same error either way)


//#include <stdio.h>
#include <stdlib.h>
#include <iostream.h> // instead of using namespace std
//#include <string.hpp>
//#include <fstream.h>
//#include <istream>
//#include "Fcopy.h"
//#include <ctime>
//#include <math.h>
int main ()
{
int menu_select_int =-1;

do
{
system("CLS");
cout <<"************ **********MENU* *************** *******\n";
cout << " Create Or Edit Input File..........1 \n";
cout << " Calc. no. of stages......... .......2\n";
cout << " Data plot........... ............... 3\n";
cout << " RR vs. No. Of stages......... ......4\n";
cout << " McCabe Thiele plot........... ......5\n";
cout << " Printout the input file........... .6\n";
cout << " Printout of the tray compositions..7 \n";
cout << " Printout of the vle data........... 8\n";
cout << " Quit........... ............... .....9\n";
cout << "************** *************** *************** *****\n\n";
cout << "MENU: ";
cin >> menu_select_int ;

switch (menu_select_in t)
{

case 1:
cout << "\ngoing to input data function\n";
cout << "case 1";
cin.ignore();

Add a "break" statement after every case you don't want
to fall through to the next case...

-David

[YitzHandel]

excellent. that worked. thanks.

[mlimber]

YitzHandel wrote:

The following is an extract from my program, which I seperated out and
compiled as its own program.

The problem is that as soon as a case tests true, all the cases below
it execute as well. So if the user inputs, 4, the statements from
cases 4-9 will execute.

[snip]

You need to insert break statements after each case "body". By default
(unfortunately) , switch statements fall through. Here's an example:

switch( some_int )
{
case 1:
cout << "Case 1" << endl;
break;
case 2:
cout << "Case 2" << endl;
break;
default:
cout << "Default case" << endl;
}

Cheers! --M

[Ian]

YitzHandel wrote:

The following is an extract from my program, which I seperated out and
compiled as its own program.

The problem is that as soon as a case tests true, all the cases below
it execute as well. So if the user inputs, 4, the statements from
cases 4-9 will execute.

Your forgot the "break;" after each case!

Ian

[Default User]

mlimber wrote:

You need to insert break statements after each case "body". By default
(unfortunately) , switch statements fall through. Here's an example:

It's not "unfortunat e". It would be a real pain if it didn't work that
way. That's the only way C (and so C++) can do multiple cases resolving
to the same action. If fall-through weren't there, you couldn't have
this:
switch (n)
{
case 1:
case 2:
case 3:
// do thing A
break;
case 4:
// do thing B
break;
default:
// do thing C
}

Brian

[Wilton Helm]

Whether it is unfortunate or not is a matter of opinion. It probably does
allow some programming techniques which are prone to bugs or aren't as
structured. On the other hand a lot of C programmers take advantage of it
for things like this excerpt from a parser:

switch(c) {

case '0' :
case '1' :
case '2' :
case '3' :
case '4' :
case '5' :
case '6' :
case '7' :
case '8' :
case '9' :
accum = 10 * accum + (c - '0');
break;

case 'S' : // Store value

// other alpha command cases

Or the situation where two cases are identical except for a line or two at
the beginning so one can fall through into the other:

case FIRST:
// some initial code
// NOTE: Falls through

case SECOND:
/// code shared between cases

In which case the wise programmer will ALWAYS place the note there pointing
out the lack of a break statement--otherwise somebody will get bit.

Wilton

[Jon Slaughter]

"Default User" <de***********@ yahoo.com> wrote in message
news:3q******** ****@individual .net...

mlimber wrote:

You need to insert break statements after each case "body". By default
(unfortunately) , switch statements fall through. Here's an example:

It's not "unfortunat e". It would be a real pain if it didn't work that
way. That's the only way C (and so C++) can do multiple cases resolving
to the same action. If fall-through weren't there, you couldn't have
this:
switch (n)
{
case 1:
case 2:
case 3:
// do thing A
break;
case 4:
// do thing B
break;
default:
// do thing C
}

Brian

um, or you could just have an "unbreak" statement that is less error prone
and still do the exact same thing?

[Josce]

>>You need to insert break statements after each case "body". By default

(unfortunatel y), switch statements fall through. Here's an example:

It's not "unfortunat e". It would be a real pain if it didn't work that
way. That's the only way C (and so C++) can do multiple cases resolving
to the same action. If fall-through weren't there, you couldn't have
this:

switch (n)
{
case 1:
case 2:
case 3:
// do thing A
break;
case 4:
// do thing B
break;
default:
// do thing C
}

Talking about switch statements in general:
I would have liked the possibility to write 1..5 meaning 1,2,3,4 or 5 like:

switch (n)
{
case 1..3:
// do thing A
break;
case 4:
// do thing B
break;
default:
// do thing C
}

Josce

[Peter C. Chapin]

On Fri, 7 Oct 2005, Default User wrote:

It's not "unfortunat e". It would be a real pain if it didn't work that
way. That's the only way C (and so C++) can do multiple cases resolving
to the same action. If fall-through weren't there, you couldn't have
this:

One could argue that a better language design would be to allow for syntax
such as

switch (n)
{
case 1..3:
// do thing A

case 8 or 9:
// do some other thing

default:
// etc
}

Peter