8
cout << "How many questions are there going to be on this exam?" << endl;
cout << ">>";
getline(cin, totalquestions);

This small piece of code comes from a function in a class that I have created and I need totalquestions to be an int so that it can run through a for loop and keep asking the total amount of questions that I have asked.

question q;
for(int i = 0; i < totalquestions; i++)
{
    q.inputdata();
    questions.push_back(q);
}

Where does this piece of code comes to play? Does anyone have any idea to make this work?

Michael Petrotta
  • 59,888
  • 27
  • 145
  • 179
Scott
  • 81
  • 1
  • 1
  • 2

5 Answers5

12

Use

cin >> totalquestions;

Check the errors too

if (!(cin >> totalquestions))
{
    // handle error
}
sehe
  • 374,641
  • 47
  • 450
  • 633
  • @OlayemiIbrahim https://stackoverflow.com/a/4533102/85371 Thanks for [contributing](https://stackoverflow.com/review/suggested-edits/19527999) anyways! – sehe Apr 24 '18 at 07:36
5

getline reads an entire line as a string. You'll still have to convert it into an int:

std::string line;
if ( !std::getline( std::cin, line ) ) {
//  Error reading number of questions...
}
std::istringstream tmp( line );
tmp >> totalquestions >> std::ws;
if ( !tmp ) {
//  Error: input not an int...
} else if ( tmp.get() != EOF ) {
//  Error: unexpected garbage at end of line...
}

Note that just inputting std::cin directly into totalquestions does not work; it will leave the trailing '\n' character in the buffer, which will desynchronize all of the following input. It's possible to avoid this by adding a call to std::cin.ignore, but this would still miss the error due to trailing garbage. If you're doing line oriented input, stick with getline, and use std::istringstream for any necessary conversions.

James Kanze
  • 150,581
  • 18
  • 184
  • 329
3

Do this:

int totalquestions;
cout << "How many questions are there going to be on this exam?" << endl;
cout << ">>";
cin >> totalquestions;

Getline is meant for grabbing chars. It can be done with getline(), but cin is much easier.

riwalk
  • 14,033
  • 6
  • 51
  • 68
1

One of the better ways of getting an int from user :-

#include<iostream>
#include<sstream>

int main(){
    std::stringstream ss;

    ss.clear();
    ss.str("");

    std::string input = "";

    int n;

    while (true){
        if (!getline(cin, input))
            return -1;

        ss.str(input);

        if (ss >> n)
            break;

        std::cout << "Invalid number, please try again" << std::endl;

        ss.clear();
        ss.str("");
        input.clear();
}

Why is it better than using cin >> n ?

Actual article explaining why

As for your question, use the above code to get the int value and then use it in the loop.

Tin Svagelj
  • 124
  • 1
  • 13
dark_shade
  • 139
  • 1
  • 8
0

Don't use getline:

int totalquestions;
cin >> totalquestions;
Fred Foo
  • 355,277
  • 75
  • 744
  • 836