-
C++ (
http://www.daniweb.com/forums/forum8.html)
- -
Input error (
http://www.daniweb.com/forums/thread96260.html)
| manzoor | Nov 11th, 2007 3:02 am | |
| Input error I coded a program its for temperature conversion its still not developed yet and I'm having a problem.
Here's the code:
/* Software Name = Temperature Converter Calculator...
Made by Manzoor Ahmed.*/
// Program that helps converting Temperature degrees.
// Formulas obtained from wikipedia.org.
#include <cstdlib>
#include <cctype>
#include <iostream>
using namespace std;
int CelFunc() ;
void CnvrtTemp()
{
char Cnvrt_Temp_Option ;
cout << "\n==================== Convert Temperature =======================" << endl ;
do
{
cout << "\n\n[C] Celsius Converter" << endl ;
cout << "[F] Fahrenheit Converter" << endl ;
cout << "[K] Kelvin Converter" << endl ;
cout << "[R] Rankine Converter" << endl ;
cout << "[M] Back to Main Menu." << endl ;
cout << "\n(Characters enclosed by the square brackets are the options.)" << endl ;
cout << "\nEnter your option: " ;
cin >> Cnvrt_Temp_Option ;
if ( Cnvrt_Temp_Option == 'C' || Cnvrt_Temp_Option == 'c' )
{
CelFunc();
}
if ( Cnvrt_Temp_Option == 'F' || Cnvrt_Temp_Option == 'f' )
{
cout << "FahrFunc()" << endl ;
}
if ( Cnvrt_Temp_Option == 'K' || Cnvrt_Temp_Option == 'k' )
{
cout << "KelFunc()" << endl;
}
if ( Cnvrt_Temp_Option == 'R' || Cnvrt_Temp_Option == 'r' )
{
cout << "RankFunc()" << endl ;
}
} while (!( Cnvrt_Temp_Option == 'M' || Cnvrt_Temp_Option == 'm' ));
}
int main()
{
char Main_Menu_Option;
// program name output
cout << "################################################################################";
cout << "# #";
cout << "# #";
cout << "# #";
cout << "# Temperature Converter Calculator #";
cout << "# #";
cout << "# #";
cout << "# #";
cout << "# #";
cout << "################################################################################\n\n";
do
{ // Main Menu
cout << "\n================================ Main Menu =====================================\n\n\n";
// Menu
cout << "[C] Convert Temperature" << endl ;
cout << "[H] Help" << endl ;
cout << "[E] Exit" << endl ;
cout << "\n(Characters enclosed by the square brackets are the options.)\n" << endl ;
cout << "\nEnter your option : ";
cin >> Main_Menu_Option ;
if (!( Main_Menu_Option == 'C' || Main_Menu_Option == 'c' || Main_Menu_Option == 'H' || Main_Menu_Option == 'h' || Main_Menu_Option == 'E' || Main_Menu_Option == 'e' ))
{
cout << "\nWrong option entered\n";
}
if ( Main_Menu_Option == 67 || Main_Menu_Option == 99) // 67 == C , 99 == c.
{
CnvrtTemp();
}
if ( Main_Menu_Option == 69 || Main_Menu_Option == 104 )
{
cout << "Help section is under construction.\n";
}
} while (!( Main_Menu_Option == 69 || Main_Menu_Option == 101 ));
system ("pause");
return 0;
}
int CelFunc()
{ char again;
cout << "\n=============================== Celsius Converter ==============================\n\n" ;
do {
cout << "Enter °C (Celsius) degree to convert :";
double cTemp2Conv;
cin >> cTemp2Conv;
if (isdigit(cTemp2Conv)) {
double cFahrenheit = (cTemp2Conv * 1.8) + 32;
double cKelvin = cTemp2Conv + 273.15;
double cRankine = (cTemp2Conv + 273.15) * 1.8;
cout << "\nAnswer: " << cTemp2Conv << " Celsius in Fahnrenheit scale = " << cFahrenheit;
cout << "\nAnswer: " << cTemp2Conv << " Celsius in Kelvin scale = " << cKelvin;
cout << "\nAnswer: " << cTemp2Conv << " Celsius in Rankine scale = " << cRankine;
cout << "\n\n";
}
else {
cout << "\nINVALID INPUT !";
cout << "\nPlease enter numerical values.\n";
}
cout << "\nDo you want to convert temperature again ?";
cout << "\nEnter Y for yes, N for no.";
cout << "\nEntering wrong option other than Y/N will result in getting back to \nConvert Temprature menu.";
cout << "\n[y]/[n]:";
cin >> again;
} while (again == 'y'|| again == 'Y');
}
Well the problem is when you go to Celsius Converter and their input a character then its starts looping infinitely. How to avoid looping ?
I used isdigit() here to check whether the input is a numerical value or its an alphabet.
I also used the cin.good() function but I'm still getting the same error.
Sorry if my code is too big but I couldn't lessen it because I'm a beginner and don't know where the real problem is. |
| Ancient Dragon | Nov 11th, 2007 7:45 am | |
| Re: Input error I think the problem is that after cin >> Cnvrt_Temp_Option ; you need to strip the '\n' (Enter key) from the keyboard buffer. One way to do that is cin.ignore() |
| manzoor | Nov 11th, 2007 11:46 am | |
| Re: Input error But I have problem in the CelFunc()
Anyway I'm gonna check it out
The Problem is in the CelFunc(), there when it asks for a number, but instead of inputting a number if you input an alphabet then it stars looping infinitely.
In CelFunc(), if you see the if statement its if (!(isdigit(cTemp2Conv))) not if (isdigit(cTemp2Conv)) |
| hgedek | Nov 11th, 2007 12:47 pm | |
| Re: Input error You are using isdigit for double or int.but it wont return true.you may use for char.and if char is digit it will return true.so you may change your method for getting number from user. |
| manzoor | Nov 11th, 2007 12:54 pm | |
| Re: Input error Can any one tell me which method should I use ???
Just tell me the function I'm not saying that do it for me |
| hgedek | Nov 11th, 2007 3:10 pm | |
| Re: Input error string s;
cin>>s;
for(int i=0;i<s.length();i++)
{
if(isalpha(s[i]))
{
cout<<"error!!!!!!";
return -1;
}
}
int a=atoi(s.c_str()); This is a basic test function.User cant input a alpha or alphanumeric string. |
| Ancient Dragon | Nov 11th, 2007 3:51 pm | |
| Re: Input error isdigit only works with a single character, not a double or int. doubles are always guarenteed to be all digits plus the decimal point so there's no point using isdigit with it. If you want to check that someone did not type a digit then use cin's error method if( cin.fail() ) |
| manzoor | Nov 12th, 2007 7:42 am | |
| Re: Input error Do i need to pass arguments in the cin.fail() function to check my input ? |
| Ali_110 | Nov 12th, 2007 8:11 am | |
| Re: Input error manzoorr plzz add me i also wanted a little help from u
my id is
<snipped> |
| Ancient Dragon | Nov 12th, 2007 8:24 am | |
| Re: Input error Quote: Originally Posted by manzoor (Post 468250) Do i need to pass arguments in the cin.fail() function to check my input ? | No -- you need to start reading the documentation |
| All times are GMT -4. The time now is 4:57 am. | |
Forum system based on vBulletin Copyright ©2000 - 2009, Jelsoft Enterprises Ltd.
©2003 - 2009 DaniWeb® LLC