在 istream 错误上添加空终止符

Question

我正在尝试将包含字符串集合的文本文件读取到对象数组中，并且输入出现问题。我收到一个转到 istream 的错误，

*_Str = _Elem();    // add terminating null character

我不太了解如何在 C++ 中使用字符串，因此我们将不胜感激。

我的代码：

char bird_name[MAX_LINE_LENGTH];
char* description =new char [MAX_LINE_LENGTH];
char* sound=new char [MAX_LINE_LENGTH];
int num_states= 0;
char* states[10];
bool valid = true;
char* state_name = new char [MAX_LINE_LENGTH];

for (int j =0; j<10; j++)
states[j]=new char [MAX_LINE_LENGTH];

char *input_filename = argv[1];

ifstream input(input_filename);
if (!input.is_open())
{
cerr << "Invalid filename: " << input_filename << endl;
system("pause");
return 1;
}



input.getline(bird_name, MAX_LINE_LENGTH);

char* state_num = new char [MAX_LINE_LENGTH];
while (strcmp(bird_name, "END") != 0) 
{
    input.getline(description, MAX_LINE_LENGTH);
    consume_newline(input);
    input.getline(sound, MAX_LINE_LENGTH);
    consume_newline(input);
    input.getline(state_num, MAX_LINE_LENGTH);
    num_states = int(state_num);
    consume_newline(input);
    for (int k = 0; k<num_states; k++)

        input.getline(states[k], MAX_LINE_LENGTH);

    consume_newline(input);
    consume_newline(input);

    birds[num_birds++] = new Bird(bird_name, description, sound, num_states, states);

    //birds[num_birds]->display();
    input.getline(bird_name, MAX_LINE_LENGTH);
}

Answer 1

您提到的有问题的代码……

    *_Str = _Elem();    // add terminating null character

可能来自某些标准库源代码文件。

请注意，在您自己的代码中，不应使用以下划线后跟大写字母开头的标识符，因为它们是为实现保留的（例如上面的代码）。

该注释表明，当标准库代码将完整的输入行读入缓冲区并尝试添加终止空字节时，事情就会出错。

这反过来表明缓冲区太小，或者传递给标准库代码的缓冲区指针甚至无效。

我无法在您显示的代码中找到它。我怀疑您所显示的代码不是出现问题的代码。请注意，对于未来：如果可能的话，发布您在一毫秒前测试过的完整代码……

无论如何，没有必要准确地知道哪里出了问题以及出了什么问题（详细）来修复问题。您可以采用“亚历山大解决方案”。这个表达指的是亚历山大大帝，当他找不到任何绳头来开始解开一个真正的坏结™时，他只是用剑将其切成两半。

所以考虑一下你的声明……

char* description =new char [MAX_LINE_LENGTH];

现在第一个明显的错误是使用了全部大写标识符，这让我们眼前一亮。为宏保留它。然后它就变成了……

char* description =new char [max_line_length];

其次，使用原始指针和原始new通常只是Bad™。所以摆脱它吧。然后它看起来像……

char description[max_line_length];

第三，使用像这样的原始数组通常是一个很好的解决方案，但事实证明这个数组用于可变长度字符串。对于这种用法，它只是 Bad™。相反，使用某个字符串类的对象，例如标准库的 std::string：

std::string description;

您需要为此包含 [string] 标头，即 #include代码>.

第四，该变量仅在循环内使用，因此将声明移至循环内！

第五，使用std::string，需要更改getline调用，目前...

input.getline(description, MAX_LINE_LENGTH);

使用 [string] 标头中独立的 getline 函数，即……

std::getline( input, description );

第六，对输入操作没有错误检查。您需要添加错误检查和错误处理。假设 input 作为 std::istream，那么您可以检查 input.fail();如果某些输入操作失败，则为 true。

七……哦，从逻辑上讲，这里应该是第七点，因为七是一个比六更令人愉快的数字。然而，对于第七点，我没有什么可说的。

干杯&呵呵，