Thread: Problems calling functions

Code:

switch(a)
{
   case 1: f1();
      break;
   case 2: f2();
      break;
   case 3: f1(); f2();
      break;
}

Sorry, i was just trying to figure the best way of doing this. Its a college assignment so i removed the code. I was going to PM it to you but there is no PM option available.

Have you enough information in the following or is there any other way i can send the complete code to you? I dont want to posit it on an open forum as its a college assignment.

Code:

void menu2(void)
{
void PrintVariances(void) ;
void OutputVariances(void);
void BothVariances(void);
int iChoice;
puts("\n\n\n\t\t\t\tMENU\n"); 
puts("\t\t\t1.\tView Variances on the Screen");
puts("\t\t\t2.\tOutput Variances to Text File");
puts("\t\t\t3.\tBoth 1 and 2");
puts("\t\t\t4.\tMain Menu");
puts("\t\t\t5.\tExit");
printf("Enter your choice: ");
scanf_s("%d", &iChoice);

switch (iChoice)
{
case 1:
	PrintVariances();
	break;
case 2:
	OutputVariances();
	break;
case 3:
	PrintVariances();
                OutputVariances();
	break;
case 4:
	menu1();
	break;
case 5:
	exit(0);
default:
	puts("\nPlease select either 1, 2, 3, 4 or 5");
	menu2();
} 
}


void PrintVariances(void)
{

	Code to print the numbers to the screen is here

puts("ENTER to continue");
	 _getch();
menu2();
} 


void OutputVariances(void)
{

Code to open the file and print to it is here

puts("ENTER to continue");
	 _getch();
menu2();
}

I can pretty much see what the eproblem is but i dont know how to get around it. When i pick option 3 it opens the print variances function, when it gets to the bottom it opens the menu 2 function and then ignores the output variances function.

I need the program to go back to menu 2 if option 1 is selected and same for option 2 but like i said if i pick option 3 i want it to run both functions.

Any ideas?

Thanks

If you want the OutputVariances() to be called after PrintVariances() has been called while choosing option 3, you simply need to remove the menu2() call from the PrintVariances() function. I mean, just remove the menu2() from the PrintVariances() function, it'll take you to the immediate next point from where you called PrintVariances().

Originally Posted by barryr

I only want it to run both Output Variances and Print Variances together when option 3 is selected.

When i remove the menu2() function like you suggested from the print variances function and select option 1, it prints the variances on the screen, asks me to press enter to continue and then comes up with press any key to continue, and then closes out the program.

I want the program to run Print Variances() and return to menu 2() if option 1 is selected


I want the program to run Output Variances() and return to menu 2() if option 2 is selected

But i want it to run both Print Variances() and Output Variances() and then return to menu 2() if option 3 is selected.

If I understood you correctly, then this is what you should do. Call menu2() before break statement in option 1, 2 and 3 as well.

You can do one more thing, simply run a loop before printing the menu. This will save you from calling menu2() recursively which will be more efficient.

But I would advise you to take my last post into consideration too. It'll make your program more efficient and the hardwork of calling menu2() again and again will also get eliminated.