std::getline 在 for 循环中不起作用

Question

我正在尝试在一个字符串变量中收集用户的输入，该变量在指定的时间内接受空格。

由于通常的 cin >> str 不接受空格，所以我会使用 std::getline from

这是我的代码：

#include <iostream>
#include <vector>
#include <string>
#include <algorithm>
using namespace std;
int main()
{
    int n;
    cin >> n;
    for(int i = 0; i < n; i++)
    {
        string local;
        getline(cin, local); // This simply does not work. Just skipped without a reason.
        //............................
    }

    //............................
    return 0;
}

有什么想法吗？

Answer 1

如果您输出存储在local 中的内容（顺便说一句，这是一个糟糕的变量名：P），您就会明白为什么会失败：

#include <iostream>
#include <vector>
#include <string>
#include <algorithm>
using namespace std;
int main()
{
    int n;
    cin >> n;
    for(int i = 0; i < n; i++)
    {
        string local;
        getline(cin, local);
        std::cout << "> " << local << std::endl;
    }

    //............................
    return 0;
}

您将看到它在输入您的号码后立即在> 之后打印一个换行符。然后继续输入其余部分。

这是因为getline 为您提供了输入号码后剩下的空行。 （它会读取数字，但显然不会删除 \n，因此您会留下一个空白行。）您需要先删除任何剩余的空格：

#include <iostream>
#include <vector>
#include <string>
#include <algorithm>
using namespace std;
int main()
{
    int n;
    cin >> n;
    cin >> ws; // stream out any whitespace
    for(int i = 0; i < n; i++)
    {
        string local;
        getline(cin, local);
        std::cout << "> " << local << std::endl;
    }

    //............................
    return 0;
}

这按预期工作。

题外话，也许它只是为了手头的 sn-p，但如果你没有using namespace std;，代码往往更可读。它违背了命名空间的目的。不过，我怀疑这只是为了在这里发帖。

Answer 2

声明一个字符以在您输入数字后进入马车返回。char ws;int n;cin>>n;ws=cin.get(); 这样就可以解决问题了。

使用cin>>ws而不是ws=cin.get()，将使字符串的第一个字符位于变量ws中，而不是仅仅清除'\n'。

Answer 3

这很简单。 你只需要在循环的末尾放一个 cin.get() 。

Answer 4

你按回车了吗？如果没有得到线将不会返回任何内容，因为它正在等待行尾......

Answer 5

我的猜测是你没有正确阅读n，所以它转换为零。由于 0 不小于 0，因此循环永远不会执行。

我会添加一些工具：

int n;
cin >> n;
std::cerr << "n was read as: " << n << "\n"; // <- added instrumentation
for // ...

Answer 6

• n 是否从输入正确初始化？
• 您似乎没有对 getline 进行任何操作。这是你想要的吗？
• getline 返回一个 istream 引用。你把它丢在地上这件事重要吗？

Answer 7

你在哪个编译器上试过这个？我在 VC2008 上试过，效果很好。如果我在 g++ (GCC) 3.4.2 上编译了相同的代码。它不能正常工作。以下是在两个编译器中工作的版本。我的环境中没有最新的 g++ 编译器。

int n;
cin >> n;
string local;
getline(cin, local); // don't need this on VC2008. But need it on g++ 3.4.2. 
for (int i = 0; i < n; i++)
{
    getline(cin, local);
    cout << local;
}

Answer 8

重要的问题是“你对字符串做了什么，让你觉得输入被跳过了？”或者，更准确地说，“你为什么认为输入被跳过了？”

如果您正在单步调试调试器，您是否在编译时进行了优化（允许重新排序指令）？我不认为这是你的问题，但有可能。

我认为更有可能填充了字符串，但没有正确处理。例如，如果要将输入传递给旧的 C 函数（例如，atoi()），则需要提取 C 样式字符串（local.c_str()）。

Answer 9

你可以直接在使用分隔符的字符串中使用getline函数，如下：

#include <iostream>
using namespace std;
int main()
{
    string str;
    getline(cin,str,'#');
    getline(cin,str,'#');
}

您可以根据需要多次输入 str，但这里适用的一个条件是您需要传递 '#'（第三个参数）作为分隔符，即字符串将接受输入，直到按下 '#' 为止，无论换行符如何。

Answer 10

在getline(cin, local) 之前，只需添加if(i == 0) { cin.ignore(); }。这将从字符串中删除最后一个字符 (\n)，这会导致此问题，并且仅在第一个循环中需要它。否则，它将在每次迭代时从字符串中删除最后一个字符。例如，

i = 0 -> string
i = 1 -> strin
i = 2 -> stri

等等。

Answer 11

只需在循环前使用cin.sync()。

Answer 12

只需在 getline 之前添加 cin.ignore() 即可完成工作

#include <iostream>
#include <vector>
#include <string>
#include <algorithm>
using namespace std;
int main()
{
    int n;
    cin >> n;
    for(int i = 0; i < n; i++)
    {
        string local;
        cin.ignore();
        getline(cin, local);
    }

    return 0;
}