Need help with C++?

Question

Is this the appropiate way to using the switch structure im having problems, sending it to an outfile char User_input;
switch (User_Input)
{
case 'short'
outfile << setw(6) << Band_Name << setw(6)<< Title << endl;
break;

case 'medium'
outfile << setw(6) << Band_Name << setw(6)<< Title << setw(6) << number_Of_Tracks << endl;
break;

case 'long'
outfile << setw(6)<< Band_Name << setw(6) << Title << setw(6) << number_Of_Tracks << setw(6)<< Total_Time
<< setw(6)<< Cost << endl;
break;
}
system ("Pause");
return Exit_SUCESS;
}

Thanks you guys rock! :)

Answer 1

you are missing the colons. None of those statements are being evaluated. it should be case 'short': and so on. The colon after the case is important.

Answer 2

Yes and no. This is an okay way to do it, but there is a better way to do it to simplify the code. Since you have some duplicated code, you can simply do the following.

Is this the appropiate way to using the switch structure im having problems, sending it to an outfile char User_input;
outfile << setw(6) << Band_Name << setw(6)<< Title;
switch (User_Input)
{
case "short"
break;
case "medium":
outfile << setw(6) << number_Of_Tracks;
break;
case "long":
outfile << setw(6) << number_Of_Tracks << setw(6)<< Total_Time << setw(6)<< Cost;
break;
}
outfile << endl;
system ("Pause");
return Exit_SUCESS;
}

Also, notice that your strings need double quotes, not single quotes, as those a characters, and you needed the colons too.

Answer 3

First, you cannot use single quotes with strings.
Second, in C++ switch() you cannot use strings. Use integer values.
Third, every case has to have colon at the end.
switch (User_Input[0])
{
case 's': // instead of "short"
...
case 'm': // instead of "medium"
...
}
will work

Answer 4

You need a colon at the end of the case statements.