My Super Program - Page 3 - Overclock.net

G|F.E.A.D|Killa · 04-10-08

ahhh reminds me of my old program i made along time ago. it calculated how may times a ball will bounce and the feet between each bounce with givin variables. nice job man.

DeRock131 · 04-10-08

I think case 7 can be shortened.

I dont remember exactly anymore because its been 2-3 years, but i think there is a function that compresses those.

rubendodge · 04-10-08

I updated it will be coding in it for alittle while longer (hour or 2) then I will be done for the night but in the mean time check and see if I add anythign new or change anythign on my first post.

Scriptorum · 04-10-08

Quote:

Originally Posted by rubendodge

Ok I will work on adding some of these ideas. The only real problem is the whole thing about making it be able to restart the app after a operation. I've never done it before so I'm still newbie lol.

To make an app "restart," use a loop. One example:

Code:

int menuchoice = -1;
while(menuchoice != 14)
{
  cin >> menuchoice;
  performOperation(menuchoice);
}

dankoni · 04-10-08

I didn't have time at work to look through the whole thing, but I can make a couple quick suggestions for the calculator.

1. For subtraction, why are you making the user enter the larger number first? Can you not figure out a way to handle negative numbers?

2. For division, you should have a guard against dividing by 0, otherwise it will crash the program.

And like I saw someone else say, you should also check to make sure the input is actually a number.

You say you are in your first month of programming? Looking good

.dp-BarackObama · 04-10-08

looks pretty exciting.

redsunx · 04-10-08

I hate programming.

.dp-BarackObama · 04-10-08

haha booo!

rubendodge · 05-27-08

Ok I added some suggestions you guys stated I am still having errors and problems with getting the "loop" idea to work right lol.

I added like 5-6 new options and made the program ALOT more organised enjoy guys

.

Stormwolf · 05-28-08

You don't have to 'restart' the program - you need to wrap the innards of the program in a loop. C++ has 3 loop structures - for this style you would probably want to use a do..while loop because you want to present options and get input at least once then test the loop condition at the 'end': i.e.

Code:

{
 //snippet
  const int ExitCode = -1;
  const int NumMainMenuOpts = 6;

  do
  {
     display_base_menu_opts();
     // limit possible input values from 1..6
     menuchoice = read_valid_key_input(NumMainMenuOpts); 

     switch(menuchoice)
     {
      case 1: do_stuff();
                 break;

     case 2: do_more_stuff();
                break;  

     case 3..5: // etc.
     case 6: menuchoice = ExitCode; // assign 'impossible to reach' value
                 break;

      default: cout << "unhandled menu option error" << endl;
                  break;

     };

  }while (menuchoice != ExitCode);

  //clean up and exit
}

You can then structure your submenus within your worker functions similarly. Each task should be contained within the scope of its own function... they do teach you functions in your programming class right?

'do_stuff()' and 'do_more_stuff()' are not good names btw - they're just placeholders

05-27-08	#29 (permalink)
rubendodge New to Overclock.net Join Date: Apr 2008 Posts: 6 Rep: 0 Unique Rep: 0 Trader Rating: 0	Ok I added some suggestions you guys stated I am still having errors and problems with getting the "loop" idea to work right lol. I added like 5-6 new options and made the program ALOT more organised enjoy guys . Last edited by rubendodge : 05-27-08 at 03:09 PM.

05-28-08	#30 (permalink)
Stormwolf New to Overclock.net Join Date: May 2008 Location: Nottingham, UK Posts: 24 Rep: 7 Unique Rep: 5 Trader Rating: 0	You don't have to 'restart' the program - you need to wrap the innards of the program in a loop. C++ has 3 loop structures - for this style you would probably want to use a do..while loop because you want to present options and get input at least once then test the loop condition at the 'end': i.e. Code: { //snippet const int ExitCode = -1; const int NumMainMenuOpts = 6; do { display_base_menu_opts(); // limit possible input values from 1..6 menuchoice = read_valid_key_input(NumMainMenuOpts); switch(menuchoice) { case 1: do_stuff(); break; case 2: do_more_stuff(); break; case 3..5: // etc. case 6: menuchoice = ExitCode; // assign 'impossible to reach' value break; default: cout << "unhandled menu option error" << endl; break; }; }while (menuchoice != ExitCode); //clean up and exit } You can then structure your submenus within your worker functions similarly. Each task should be contained within the scope of its own function... they do teach you functions in your programming class right? 'do_stuff()' and 'do_more_stuff()' are not good names btw - they're just placeholders

		Overclock.net - Overclocking.net > Software, Programming and Coding > Coding and Programming > Application Programming
My Super Program

04-10-08	#23 (permalink)
rubendodge New to Overclock.net Join Date: Apr 2008 Posts: 6 Rep: 0 Unique Rep: 0 Trader Rating: 0	I updated it will be coding in it for alittle while longer (hour or 2) then I will be done for the night but in the mean time check and see if I add anythign new or change anythign on my first post.

Currently Active Users Viewing This Thread: 1 (0 members and 1 guests)