Two Switch Structures

[rynzor6]

If anyone can help me on switches. How can I differentiate the two if S is chosen or M is chosen.

if (in_marital_status == "s")
		case_no= (income <= single_level1) || (income<= single_level2);	
		switch (case_no)
				{
					case 1:	(income <= single_level1); tax = rate1 * income; break;
					case 2: (income <= single_level2); tax = single_tax1 + rate2 * (income- single_level1);break;
					case 3: tax = single_tax2 + rate3 * (income-single_level2);break;
					
				}	
		
if (in_marital_status == "m")
		case_no= (income <= married_level1) || (income <= married_level2);	
		switch (case_no)
					{					
					 case 1: tax = rate1 * income; break;
					 case 2: tax = married_tax1 + rate2 * (income-married_level1);break;
					 case 3: tax = married_tax2 + rate3 * (income - married_level2);break;
					}			
				

Am I correctly using the switch? any feedback gladly appreciated.

[iantac]

are you sure that in_marital_status is a string??
if not. you need to use ' ' not " " .. because
" " << this one is for string
' ' <<this one is for char

oh.. and what's with the case_no

[TheIdeasMan]

Your code is harder to read with multiple statements on 1 line.

The case_no assignment is the result of a conditional so will be 0 or 1.

[Peter87]

To include more than one statement in the if statement you have to use curly brackets.

if (in_marital_status == "s")
{
	case_no= (income <= single_level1) || (income<= single_level2);	
	switch (case_no)
	{
		case 1:	(income <= single_level1); tax = rate1 * income; break;
		case 2: (income <= single_level2); tax = single_tax1 + rate2 * (income- single_level1);break;
		case 3: tax = single_tax2 + rate3 * (income-single_level2);break;
	}
}