Program terminating issue

[BaronInara]

I seem to be having trouble with my program terminating when it should. I have a summation loop and I want it to kick out of that and end the entire program should the iterations reach 1000. This is my code and it seems like it should work but it's not. Any tips would be appreciated.

//Steve Osborne, Using the Taylor Series

#include<iostream>
#include<iomanip>
#include<cmath>
using namespace std;

void Header();
void Input(double& prec,int& code);
int Factorial(int number);
void Summation(double prec,double& e,int& iter,bool& escape);
int SetPrecision(double prec);
void OutResults(double e,double iter,int decimals);
void LoopEscape();

int main(){
	double prec;
	int code=1;
	bool escape=1;
	int iter;
	int decimals=1;
	Header();
	Input(prec,code);
	while(code==1){
		if(escape!=0){
			double e=1;
			Summation(prec,e,iter,escape);
			if(escape!=0){
				decimals=SetPrecision(prec);
				OutResults(e,iter,decimals);
				Input(prec,code);
			}	
		}
	}
	return 0;
}
void Header(){
	cout<<"Steve Osborne, Program 12"
		<<"\nEuler's Number by Maclaurin Series"
		<<"\nPlease enter the precision of e in this format: 1E-9";
	return;
}
void Input(double& prec,int& code){
	cout<<"\n\nEnter the precision of e or 0 to quit: ";
	cin>>prec;
	if(prec==0){
		cout<<"\n\nGoodbye\n\n";
		code=0;
	}
	else{
		code=1;
	}
	return;
}
int Factorial(int number){
	int fact=1;
	for(int n=number;n>1;n--){
		fact*=n;
	}
	return fact;
}
void Summation(double prec,double& e,int& iter,bool& escape){
	int n=1;
	double term=1;
	iter=0;
	while(term>prec&&iter<1000){
		if(iter<=999){
			e+=term;
			n++;
			term=1./Factorial(n);
			iter++;
			escape=1;
		}
		if(iter>999){
			LoopEscape();
                        escape=0; //This should terminate the program in my eyes, 
                                         //changes escape to a 0 which kicks out of the loop in main 
                                        //and should go to my return 0; statement, it's not doing it though.
		}
	}
	return;
}
int SetPrecision(double prec){
	int precision=1;
	precision=-(int)log10(prec);
	return precision;
}
void OutResults(double e,double iter,int decimals){
	cout<<"e = "<<fixed<<setprecision(decimals)<<e
		<<" (with "<<fixed<<setprecision(0)<<iter
		<<" iterations required)";
	return;
}
void LoopEscape(){
	cout<<"\n\nThe maximum number of iterations has been reached,"
		<<"\n(1000+) this will terminate the program."
		<<"\n\nGoodbye.\n\n";
	return;
}

[Albatross]

*content deleted, screen needs cleaning*

If you want to kill the whole program early from any function, use exit(int), available in stdlib.h

-Albatross

[BaronInara]

I want to avoid using the exit function, professor considers it bad style.

[Albatross]

My screen is quite dirty, I thought I saw a variable name near that return statement, and didn't recheck. Need... to get... coffee...

Your professor considers using a useful part of the C standard library bad style? I'd hate to have your professor then... :)

break; terminates loops. You don't need your LoopEscape() function, which does nothing except print stuff.

-Albatross

[S6k9GNh0]

Yeah, its bad style and can lead to memory leaks with things still needing to be allocated. For conventional reasons though, and in smaller programs that are easier to manage, there is absolutely nothing wrong with exit.

Why don't you stick the check in a while loop and check it each iteration?

int main()
{
   bool bRun = true;
   while (bRun)
   {
      /* logical statements */
      if (/*something bWrong*/)
         bRun = false; //quit
   }
}