Switch Statement

[T RATHA KRISHNAN]

I wrote the following c++ code. #include float meters, centimeters; char choice; main () { cout<>meters; centimeters = (float) 100 * meters; cout<<"The length in centimeters is"; cout<>centimeters; meters = centimeters/100; cout<<"The Length in Meters is:"; cout<

[prasad_som]

T.RATHA KRISHNAN wrote:

I got this error: "case value 1 already used".

It because of your logic. You are using boolean operation as case input. Which is repetitive. case should be different in switch statement.

Prasad Notifier using ATL | Operator new[],delete[][^]

[Cedric Moonen]

T.RATHA KRISHNAN wrote:

case('m'||'M'):

This is wrong. This will in fact evaluate to a boolean, not a character. What you need to do is this: case('m'): case('M'): // Code here break;

---

Cédric Moonen Software developer
Charting control [v1.1]

[ThatsAlok]

T.RATHA KRISHNAN wrote:

I got this error: "case value 1 already used". What is the cause for this error?

you only have to use constant in case field.. switch case just work like goto statement but with scope

"Opinions are neither right nor wrong. I cannot change your opinion. I can, however, change what influences your opinion." - David Crow

cheers, Alok Gupta Global Interface Table: An Easy Way to Marshal an Interface Pointer[new] VC Forum Q&A :- I/ IV Support CRY- Child Relief and You

[C Mahesh]

Use choice = (char)toupper( choice ) ; switch( choice ) { case 'M' : ---------- ---------- break ; case 'C' : ----------- ---------- break ; }

[ThatsAlok]

C Mahesh wrote:

choice = (char)toupper( choice ) ;

one of most simpler way to solve above problem!

"Opinions are neither right nor wrong. I cannot change your opinion. I can, however, change what influences your opinion." - David Crow

cheers, Alok Gupta Global Interface Table: An Easy Way to Marshal an Interface Pointer[new] VC Forum Q&A :- I/ IV Support CRY- Child Relief and You

[T RATHA KRISHNAN]

Fantastic! It's working. Thank You Mahesh.