在C ++中使用getline后,必须按“Enter”两次?

时间:2014-10-27 05:46:40

标签: c++ visual-studio-2012

我已经用疯狂搜索了这个,并找到了许多“解决方案”,以及2001-2007这样的论坛帖子,说它是Visual Studio中的“众所周知的错误”...那么为什么不是现在修好了?我正在使用VisualStudio 2013(版本12)。

这是我的代码:

#include <iostream>
#include <iomanip>
#include <string>
#include <Windows.h>
#include <vector>
#include <conio.h>
#include <string>

#undef max

using namespace std;

/*
TEST CASES:

*/



// START FUNCTION - Add Restaurant
void resto(vector<string> &restaurant)
{
    std::string resto_name;
    cin.ignore();
    //cin.clear();
    //cin.sync();
    std::getline (std::cin, resto_name,'\n');
    restaurant.push_back(resto_name);
}
// END FUNCTION - Add Restaurant

// START FUNCTION - Print Vector
void print_vector(vector<string> &restaurant)
{
    cout << endl << "Your Current Restaurants:" << endl;
    for (int i = 0; i < restaurant.size(); i++)
    {
        cout << i + 1 << ". " << restaurant[i] << endl;
    }
}
// END FUNCTION - Print Vector



int main(void)
{
    srand(time(0));

    int menu;

    vector<string> restaurant; //Initialize Vector

    do {
        system("cls");
        cout << " -----------------------------------------------------------" << endl;
        cout << "|     -:|:- Welcome to Restaurant Battle Royale! -:|:-      |" << endl;
        cout << "|-----------------------------------------------------------|" << endl;
        cout << "|  Please select an option by pressing 1, 2, 3, 4, 5, or 6  |" << endl;
        cout << "|\t\t\t then press ENTER.\t\t    |" << endl;
        cout << "|-----------------------------------------------------------|" << endl;
        cout << "| 1 - Display all restaurants\t\t\t\t    |" << endl;
        cout << "| 2 - Add a restaurant\t\t\t\t\t    |" << endl;
        cout << "| 3 - Remove a restaurant\t\t\t\t    |" << endl;
        cout << "| 4 - Shuffle the vector\t\t\t\t    |" << endl;
        cout << "| 5 - Begin the tournament\t\t\t\t    |" << endl;
        cout << "| 6 - Quit the program\t\t\t\t\t    |" << endl;
        cout << " -----------------------------------------------------------" << endl;

        cout << "Option: ";
        cin >> menu;

        //OPTION 1
        switch (menu)
        {
        case 1:

            print_vector(restaurant);

            cout << endl;

            printf("Press any key to return to the main menu...");
            system("pause > nul");
            break;

            //OPTION 2
        case 2:
            cout << "\nPlease enter the name of the restaurant you'd like to add." << endl;

            resto(restaurant);

            break;

            //OPTION 3
        case 3:

            break;

        case 4:
            return 0;
        }
        cin.clear();
        cin.ignore(numeric_limits<streamsize>::max(), '\n');
    } while (menu != 6);


    return 0;

}

我遇到的问题是“添加餐厅”功能。在为getline输入后,您必须按两次Enter键。我尝试过添加“\ n”和“endl”等功能无济于事。我很想认为这是编译器的一个错误,但是这个错误在经过这么多年的开发后持续这么长时间才是可悲的,对吗?

所以它必须是我的代码。你能帮我发现什么是错的吗?

我一直被困在这个小时,我不能继续前进,因为它让我烦恼不已。

=============================================== =====

参考文献:

微软的解决方案来自谁知道何时: http://support2.microsoft.com/default.aspx?scid=KB;EN-US;q240015&ID=KB;EN-US;q240015 (查看String文件后,看起来他们已将此建议的更改应用于当前版本)

我尝试过的解决方案适用于其他人,但不适用于我: http://www.dreamincode.net/forums/topic/35262-visual-c-getline-bug/

1 个答案:

答案 0 :(得分:0)

@sajas是正确的。你在resto中需要cin.ignore()的原因是因为在cin >> menu之后,流中会留下一个迷路换行符。随后的getline提取换行并立即终止。 cin.ignore()将提取并丢弃杂散换行符,以使getline正常工作。

实际上,导致问题的一行是:

cin.ignore(numeric_limits<streamsize>::max(), '\n');

所有这一切都是提取并丢弃任何字符到换行符,所以完全没必要。


为了捕获错误的输入,你可以这样做:

    while (!(cin >> menu))
    {
        cin.clear();
        cin.ignore(numeric_limits<streamsize>::max(), '\n');
        std::cout << "Please enter a number.\nOption: ";
    }

如果提取失败(因为用户没有输入数字),则清除错误标志,丢弃流中的任何错误字符,然后再次请求输入。它会循环,直到用户输入一个数字。


@The Paramagnetic Croissant's commentusing namespace std; is considered bad practice.我注意到您与std::的位置不一致。通常,如果您选择使用using指令,请将其放在函数中,这样它就不会污染全局命名空间。养成在任何地方输入std::的习惯会更好,因为它不是那么多打字。

您包括非标准Windows标头,Windows.h和conio.h。由于您没有利用任何API功能,因此您不需要它们。

删除printf。你不需要它。

system()非常特定于平台并且不赞成(虽然我没有任何消息来支持这一点。)首选cin.get()system("pause")并删除{{1}因为这可能会刺激用户。

system("cls")内,当您使用print_vector或最好int时,您正在使用std::size_t。您的编译器可能会警告您:

std::vector<string>::size_type