Transpose information

[fr0zen]

this is another question from my review... i need change an if.. else to a switch case and a while loop to a for keeping the program logic.

this is the code givin

do{
  choice = menu();
    if(choice == 1){
	addtogether(one, two);
     }else 
        if(choice == 2){
	  multiply(one, two);
	 }else 
          if(choice == 3){
	    listall(one, two);
	   }
}while(choice != 4);

this is what i have ...

int choice(one,two,choice){switch(choice){
 case 1:addtogether(one,two);break;
 case 2:multiply(one,two);break;
 case 3:listall(one,two);break;}

would i have to incorporate a for loop? and if so how?

I'm just not really confident with my programming at this point so I'm a bit worried

[Brad0407]

Your switch statement is good but why did you make up a choice function in the second version?

[Adak]

this is another question from my review... i need change an if.. else to a switch case and a while loop to a for keeping the program logic.

this is the code givin

do{
  choice = menu();
    if(choice == 1){
	addtogether(one, two);
     }else 
        if(choice == 2){
	  multiply(one, two);
	 }else 
          if(choice == 3){
	    listall(one, two);
	   }
}while(choice != 4);

this is what i have ...

int choice(one,two,choice){switch(choice){
 case 1:addtogether(one,two);break;
 case 2:multiply(one,two);break;
 case 3:listall(one,two);break;}

would i have to incorporate a for loop? and if so how?

I'm just not really confident with my programming at this point so I'm a bit worried

What happened to choice = menu() ? And I would not start a switch statement on the same line of code as choice = menu(), either. Follow the pattern they gave you!

The do loop might be changed to something like this:

for(choice = 0; choice != 4; ;)  { /*don't forget 3rd semi-colon! */
   choice = menu();
    /* your switch statement here */

}

Last, but not least, I would not advise putting a closing curly brace at the end of a line of code. That is not a format that they are likely to want to see.

[whiteflags]

> for(choice = 0; choice != 4; ; ) { /*don't forget 3rd semi-colon! */
When's the last time you wrote a for loop, Adak? There is no third semi-colon.

[Elysia]

Indeed, a 3rd one will give a compile error.