C++ Why does this work

Question

I am using Microsoft Visual studio 2013

I currently have this switch statement:

switch (option)
{
        case 1:
            getline(cin, newname);
            cout << "What would you like your new username to be?\nName: ";
            getline(cin, newname);
            name = newname;
            cout << "\nYour username is now '" << name << "' with your balance being $" << balance << "\n";
            options();
            break;
        case 2:
            cout << "\nOpenning cheat menu\n";
            cheats();
            break;
        default:
            cout << "\nExitting to main title...\n";
            title();
            break;
}

you can just ignore the case 2 and default, just showing so you know that those cases work. Case 2 and default work perfectly fine but case 1 is acting weird. As you can see on case 1 I have two getline(cin, newname); You may think I should only have one, but for some reason I need two for this code to function correctly.

If I only have one of these the code jumps past the input the user should give. If I have two it works normally.

I have asked someone this and they do not understand either, so therefore: Why does this work.

Please post a [Minimal, Complete, and Verifiable Example](http://www.stackoverflow.com/help/mcve). — Barry, Jun 16 '15 at 23:22
Add `cin.ignore();` before your `getline()`. That should fix you up. `getline()` leaves the new line character in the input stream. — Icemanind, Jun 16 '15 at 23:24
@Icemanind: No, `getline` does not "leave the new line character in the input stream." The advice to use `ignore` is good though. — Cheers and hth. - Alf, Jun 16 '15 at 23:26
@Icemanind it reads it but doesn't add it to the string, instead discards it — Ediac, Jun 16 '15 at 23:29

score 8 · Accepted Answer · answered Jun 16 '15 at 23:22

Apparently you have leftover input including a newline, in the input buffer of cin. That happens when you use other input operations than getline. They generally don't consume everything in the input buffer.

Instead of the extra getline you can use ignore, but better, do that at the place where you earlier do the inputting.

score 0 · Answer 2 · edited May 23 '17 at 11:43

0

I think a similar problem has been posted here.

You need to flush the newline with cin.ignore() or cin.sync().

The first call also seems misplaced...

edited May 23 '17 at 11:43

Community

1
1

answered Jun 16 '15 at 23:31

Fyubar

3
2

C++ Why does this work

2 Answers2