[crazybeans]

Ok, my problem is a rather simple one (I think) but can't seem to find what I need online. I am writing a fitness program. The function I am having trouble with asks the user to enter 1 or 2 for male or female. I have it set up so if they enter >2 or <1 that it will simply loop, clear screen and ask question again. This part works fine, the problem is if the user enters "m" accidently the program freaks out and continues looping. How can I prevent the user from entering anything but 1 or 2. Is there a work-around for this problem?


Code:


// Assess your expenditure
// Ask user if male or female and calculate RMR dependant on given formula
// Store variables to be used in Step4
int Step2()
{
cout << "Enter your sex:\n\n";
cout << " [1] Male\n";
cout << " [2] Female\n\n";
cout << "Enter selection: ";
cin >> Sex;
if (Sex == 1 || Sex == 2)
{
if (Sex == 1)
{
system("CLS");
cout << "Enter your weight: ";
cin >> Weight;
cout << "\nEnter your Height (in inches): ";
cin >> Height;
cout << "\nEnter your age: ";
cin >> Age;
RMR = 66 + (13.7 * (Weight / 2.2)) + (5 * (Height * 2.54)) - (6.8 * Age);
return 0;
}
if (Sex == 2)
{
system("CLS");
cout << "Enter your weight: ";
cin >> Weight;
cout << "\nEnter your Height (in inches): ";
cin >> Height;
cout << "\nEnter your age: ";
cin >> Age;
RMR = 655 + (9.6 * (Weight / 2.2)) + (1.8 * (Height * 2.54)) - (4.7 * Age);
return 0;
}
}
else
{
system("CLS");
Step2();
return 0;
}
}

[sypher04]

if( sex != 1 && sex != 2 )
//return to the beginning of the loop

[ever_thus]

declare Sex as a char array and use the following code.

cin<<Sex;
if (atoi (Sex) == 1){
//etc...
}
else if (atoi (Sex) == 2){
//etc...
}else
//etc...

atoi converts a char array into an integer. Since cin is expecting characters it won't go crazy when it receives them.

Btw, you should put the your code in a loop while (Sex != 1 && Sex != 2). Avoid having functions call themselves (recursion) whenever you can.

[Cache]

It would actually be safer and easier to use std::string, as opposed to a char array.

if (Elstingo == "1")

[crazybeans]

Thanks alot for the help, I looked into this and this is what I found:

double atof(const char* p); // convert p[] to double ("alpha to floating")
double strtod(const char* p, char** end); // convert p[] to double ("string to double")
int atoi(const char* p); // convert p[] to int, assuming base 10
long atol(const char* p); // convert p[] to long, assuming base 10
long strtol(const char* p, char** end, int b); // convert p[] to long, assuming base b

I'm currently incorporating this new information into my code, I have also removed references to recursion and replaced them with while statements.