c++ - 控制台输出不断崩溃

Question

尝试解决此Problem。
我编写了如下代码：

#include <iostream>
#include <string>
#include <vector>

using namespace std;

int i,j,n,t;

int main()
{
cin>>t;

while(t--)
{
    vector <string> dr,rd;
    string a,b;
    cin>>n;
    cin.ignore();

    for(i=0;i<n;i++)
    {
        a.clear(),b.clear();
        getline(cin,a);
        j=a.find(" on ");
        b=a.substr(j,a.size()-1);
        a.resize(j);
        dr.push_back(a);
        rd.push_back(b);
    }

    for(i=0,j=rd.size()-1;i<rd.size();i++,j--)
    {
        cout<<dr[i]<<rd[j]<<endl;
    }
    cout<<endl;
}

return 0;
}

它运行良好，至少在给定的示例中，当我使用文件I / O（fstream）运行它时，但是当我使用控制台I / O时，给定的代码在读取测试用例的第二个数字后便崩溃了。

像这样：
2
4
在路A开始
在B路右
在C路右
留在D路（读取并打印在此之前应打印的内容）
6（阅读并转到新行）
（崩溃）从老马德拉斯路开始
左转到Domlur天桥
左转至100英尺路
就在Sarjapur Road
就在Hosur路上
就在Ganapathi Temple Road上

我的错误在哪里？

Answer 1

欢迎来到SO。

如果您说程序“崩溃”，则还应该说它是如何崩溃的
并尽可能引用显示崩溃的确切输出。许多
有能力提供帮助的人不会打扰他们是否必须工作
找出问题的实质并加以解决。

如果您的问题是关于编码的问题，例如您的问题，那也很重要
用您正在使用的编程语言的名称标记您的问题
编码-在您的情况下为C ++。如果您不这样做，那么大多数C ++程序员
可以帮助的人永远不会注意到您的问题。 （我现在标记了您的
问题C ++）。

如果您使用的是C ++之类的编译语言进行编程，
有助于说明您使用的是哪个编译器以及该编译器的哪个版本，
-例如“ GCC 4.7.2”，“ MS VC ++ 2012”-因为这样程序员可以尝试
重现使用相同的问题。编码很常见
在某种程度上取决于编译器或编译器版本的问题
用过的。

我已经在Linux上使用GCC 4.7.2和clang 3.2构建了您的程序，并执行了测试
当我输入来自
控制台。

这使我可以推测您说“崩溃”时的意思，这
我推测是：

我认为您的意思可能是输入“ 6”后程序会终止
错误诊断，例如：

terminate called after throwing an instance of 'std::out_of_range'
  what():  basic_string::substr
Aborted

如果我对此有误，请立即停止阅读。

如果我说的没错，那么诊断会告诉您
调用a.substr(j,a.size()-1)，索引j超出范围。

如果j当时超出范围，则它必须超出范围，因为
前一行j=a.find(" on ")的结果。那只能意味着
a.find(" on ")在" on "中的任何位置都找不到a的原因。

查找std::string::find的文档，例如
here
并了解返回值：


  如果未找到匹配项，则该函数返回string :: npos。


(size_t)-1，并且肯定超出范围。

接下来，如果a.find(" on ")失败，则仅表示
前面的行getline(cin,a)未能从cin读取行
包含" on "。我们知道这是真的！因为你说程序
在读取“ 6”并打印所需的换行符后立即崩溃。

因此getline(cin,a)必须在控制台之后从控制台读取一行
“ 6”，但在“ Old Madras Road开始”之前。空行。那就是
如果您碰巧在短时间内按下[Enter]太久了
输入“ 6”，以便键盘缓冲区包含“ 6 \ n \ n”，或者
“ 6 \ n \ n \ n”，而不仅仅是“ 6 \ n”。阅读后致电cin.ignore()
整数，将只使用一个以下字符，因为您是
接受以下默认参数：

std::istream& std::istream::ignore(std::streamsize n = 1, int delim = EOF)  

正如我所说，您的程序对我来说很好用。但是我可以让它崩溃
输入测试时过长按[Enter]来描述的方式
案例行数-第一次或第二次-或通过按
[Enter]再次执行。无论哪种方式，我都输入一个空行
到getline(cin,a)。

如果到目前为止我是对的，那么您的代码中的一个严重错误就是您不是
检查std::string::find()是否在输入字符串上成功，并且
假设是这样。即使我不正确，那也是严重的错误。

当用户的手指也停留时，可以防止程序崩溃
确保[Enter]忽略所有可以换行的字符
在cin之后阅读。您必须查看下一个字符（如果有），
不提取它；检查它是否为换行符，然后将其解压缩
并重复。将cin>>n替换为：

for (   ;cin.peek() == '\n'; cin.ignore()){}

但是，这不会阻止您的程序以相同的方式崩溃，如果
用户碰巧错误地输入了不包含cin.ignore()的路线：
尝试“在Domlur天桥上左”。

要修复此错误，您必须检查" on "是否成功并且
处理不是的情况。这是我认为是使用GCC 4.7.2构建的问题的完整解决方案
和clang 3.2：

#include <iostream>
#include <string>
#include <vector>

using namespace std;

int main()
{

int t;

cin>>t;

while(t--)
{
    vector <string> dr,rd;
    string a,b;
    size_t i,j,n;
    cin>>n;
    for (   ;cin.peek() == '\n'; cin.ignore()){}

    for(i=0;i<n;i++)
    {
        a.clear(),b.clear();
        getline(cin,a);
        j=a.find(" on ");
        if (j == string::npos) {
            cout << "Invalid direction. Try again" << endl;
            --i;
            continue;
        }
        b=a.substr(j,a.size()-1);
        a.resize(j);
        dr.push_back(a);
        rd.push_back(b);
    }

    for(i=0,j=rd.size()-1;i<rd.size();i++,j--)
    {
        cout<<dr[i]<<rd[j]<<endl;
    }
    cout<<endl;
}

return 0;
}

注意，我还纠正了一些小缺陷：


不需要任何全局变量。避免全局变量是个好习惯
如果可能，并在尽可能小的范围内声明变量。
我将j=a.find(" on ")的类型从i,j,n更改为int。变数
用于保存size_t或
返回值some_class::some_method()，或者与该返回值进行比较，
最好使用与该返回值相同的类型声明，以避免错误和
编译器警告：some_function()和string::find()返回vector::size()，
与size_t不同。


附言使用原始程序或
我的，如果用户错误地输入整数输入之一，例如
“ y”或“ 6y”而不是“ 6”。您该如何解决？