Programming Forums > Application Development > C++

jayme · Nov 24th, 2005, 10:46 PM

whenever i use cin >> integer;
and input a letter, i end up having to quit the console and rerun the exe for it to exit the continuous loops of words that are flying down the console screen.. i think ppl know what im talking about. I need a new way for users to input a number, but if the user inputs a letter.. the console will just ask for the number again, instead of having to exit and run the exe again. Someone posted this code before, but i couldnt figure out how it's supposed to work.. this is just one of the ways i would like to know how to get the input, but any other way is still good..

(Toggle Plain Text)

// assumes you have a variable called pointsRemaining that
// is properly initialized
cout << "Please enter your character's strength: ";
strength = getStat(pointsRemaining); // get the strength score
pointsRemaining -= strength;// reduce the number of points left

Kilo · Nov 24th, 2005, 10:54 PM

try putting this directly before your cin >> statment.

(Toggle Plain Text)

cin.sync();

This clears the data buffer. If that doesn't work there are a few other options.

jayme · Nov 24th, 2005, 11:00 PM

Says cin.sync(); is and undeclared function.. any special headers?

Kilo · Nov 24th, 2005, 11:03 PM

#include <iostream>

using namespace std;

jayme · Nov 24th, 2005, 11:06 PM

ok, i found out.. tried using a different cin code, cin.get();.. still no luck, i tried putting the cin.sync(); before, and after.. same problem. Thanks anyways

Kilo · Nov 24th, 2005, 11:12 PM

try not using cin... try including

(Toggle Plain Text)

#include <conio.h>   //console input output

and using

(Toggle Plain Text)

getch();

andro · Nov 25th, 2005, 12:03 AM

Quote:

Originally Posted by Kilo

try not using cin... try including

(Toggle Plain Text)

#include <conio.h>   //console input output

and using

(Toggle Plain Text)

getch();

That's not kosher

Try something like this:

(Toggle Plain Text)

cin >> iInvoiceNumber;
while (cin.fail() || iInvoiceNumber < 1 || iInvoiceNumber > 20000) 
{
	if (cin.fail())
	{
		cin.clear();
		cin.ignore(10, '\n');
	}
	cout << "Must be between 1 and 20000" << endl;
	cout << "Invoice number: ";
	cin >> iInvoiceNumber;
}

Obviously that needs to be slightly modified to fit your needs. I stole it out of some code from a class a few semesters ago.

jayme · Nov 25th, 2005, 12:41 AM

tried your little fix, doesnt work either.. not sure why nothing is working but heres what im trying to make work..

(Toggle Plain Text)

    do{ // This is the strength loop, if you enter a number larger than 20 or less than 1, you will be taken back to this step.
    Points = 20;
    cout << "Enter your strength: "; 
    cin >> Str;
 
          if ( Str < 1 ) { // If the input is less than 1, the string below is executed and you will be asked to input a new integer.
               cout << "You Must spend at least one point!\n\n"; 
               }
          if ( Str > Points - 2 ){ // If the input is greater than the existing points, the string below will be executed and you will be asked to input a new integer.
          cout << "You don't have enough points!\n\n";
          }

} while ( Str > Points - 2 || Str <= 0 );
        cout << "Strength leveled up to " << Str << "!" << endl; // Lets you know that your selected integer was successfully added.
         Points = Points - Str; // Adds the selected points on to strength and reduces that same amount from your remaining skill points.
         cout << "You have " << Points << " skill points remaining.\n\n";

andro · Nov 25th, 2005, 12:51 AM

You forgot to add cin.fail() to your while condition.

bl00dninja · Nov 25th, 2005, 12:55 AM

this should work for anything you throw at it. it gives stupid output if you enter a semicolon or the letter 'g', but any retard that does that deserves what they get.

(Toggle Plain Text)

#include <iostream>
using namespace std;

int main()
{
    int i;
    
    while (1)
    {
      cout<<"number?\n"<<endl;
      cin>>i;
      if (i<0 || i >18)
      {
              continue;
      }      
      break;      
    }
    
    cout<<"\nyou entered "<<i<<"\n"<<endl;
    
    system("pause");
    
    return 0;
}

i used 0 to 18 cuz that's D&D standard.

Nov 24th, 2005, 10:54 PM	#2
Kilo Expert Programmer Join Date: Nov 2005 Location: In Pink Clam? Posts: 542 Rep Power: 0	try putting this directly before your cin >> statment. (Toggle Plain Text) cin.sync(); cin.sync(); This clears the data buffer. If that doesn't work there are a few other options. __________________ "When in Rome, Do as the Romans Do" "Beauty is in the eye of the BEER holder" "Save your breath your going to need it for your blow up doll later" SearchLores.org

Nov 24th, 2005, 11:03 PM	#4
Kilo Expert Programmer Join Date: Nov 2005 Location: In Pink Clam? Posts: 542 Rep Power: 0	#include <iostream> using namespace std; __________________ "When in Rome, Do as the Romans Do" "Beauty is in the eye of the BEER holder" "Save your breath your going to need it for your blow up doll later" SearchLores.org

Nov 24th, 2005, 11:12 PM	#6
Kilo Expert Programmer Join Date: Nov 2005 Location: In Pink Clam? Posts: 542 Rep Power: 0	try not using cin... try including (Toggle Plain Text) #include <conio.h> //console input output #include <conio.h> //console input output and using (Toggle Plain Text) getch(); getch(); __________________ "When in Rome, Do as the Romans Do" "Beauty is in the eye of the BEER holder" "Save your breath your going to need it for your blow up doll later" SearchLores.org

Nov 25th, 2005, 12:55 AM	#10
bl00dninja Programming Guru Join Date: Oct 2004 Location: namespace std Posts: 1,246 Rep Power: 5	this should work for anything you throw at it. it gives stupid output if you enter a semicolon or the letter 'g', but any retard that does that deserves what they get. (Toggle Plain Text) #include <iostream> using namespace std; int main() { int i; while (1) { cout<<"number?\n"<<endl; cin>>i; if (i<0 \|\| i >18) { continue; } break; } cout<<"\nyou entered "<<i<<"\n"<<endl; system("pause"); return 0; } #include <iostream> using namespace std; int main() { int i; while (1) { cout<<"number?\n"<<endl; cin>>i; if (i<0 \|\| i >18) { continue; } break; } cout<<"\nyou entered "<<i<<"\n"<<endl; system("pause"); return 0; } i used 0 to 18 cuz that's D&D standard. __________________ i put on my robe and wizard hat... Have you ever heard of Plato, Aristotle, Socrates?...Morons.

Thread Tools
Show Printable Version Email this Page
Display Modes
Linear Mode Switch to Hybrid Mode Switch to Threaded Mode

Nov 25th, 2005, 12:51 AM	#9
andro Professional Programmer Join Date: Oct 2005 Location: California Posts: 310 Rep Power: 3	You forgot to add cin.fail() to your while condition.

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

Programming Forums > Application Development > C++

User input