C++文件处理，is_open 返回错误

Question

如果我在代码中包含 if 测试，则会返回错误消息，但我不确定为什么。 当不使用它时，我的程序会陷入循环，永远不会到达文件末尾。我不明白出了什么问题。

int countlines()
{
    fstream myfile;
    myfile.open("questions.txt", ios::in);
    string contents;
    int linenumber = 0;

    //if (myfile.is_open())  
    // {
    while (!myfile.eof())  
    {
        getline( myfile, contents );
        if (contents != "")
        {
            linenumber++;
        }
    }
    cout << "there are " << linenumber << " lines.\n";
    //}else {cout<<"Unable to get file.\n";}

    myfile.close();
    return(linenumber);
}

Answer 1

发生的情况是您的文件没有被打开。这就是 is_open 失败的原因。

然后，当您注释掉检查时，您就会破坏循环，因为您迭代不正确（请参阅我的评论）并且没有检测到流故障（.eof() 将永远 在该流上为 true）。

确保该文件位于正确的位置并且可以访问。

Answer 2

在 C++ 中逐行读取文件的正确习惯用法是使用如下循环：

for (std::string line; std::getline(file,line);)
{
    // process line.
}

在示例中插入此内容（+修复缩进和变量名称）给出如下内容：

int countlines(const std::string& path)
{
    // Open the file.
    std::ifstream file(path.c_str());
    if (!file.is_open()) {
        return -1; // or better, throw exception.
    }

    // Count the lines.
    int count = 0;
    for (std::string line; std::getline(file,line);)
    {
        if (!line.empty()) {
            ++count;
        }
    }

    return count;
}

请注意，如果您不打算处理行内容，您实际上可以使用 std::streambuf_iterator，这可以让你的代码看起来像这样：

int countlines(const std::string& path)
{
    // Open the file.
    std::ifstream file(path.c_str());
    if (!file.is_open()) {
        return -1; // or better, throw exception.
    }

    // Refer to the beginning and end of the file with
    // iterators that process the file character by character.
    std::istreambuf_iterator<char> current(file);
    const std::istreambuf_iterator<char> end;

    // Count the number of newline characters.
    return std::count(current, end, '\n');
}

第二个版本将完全绕过复制文件内容，并避免为长行分配大块内存。

Answer 3

当使用std::istream和std::ostream（其std::fstream实现）时，推荐的用法是直接使用流在 bool 上下文中，而不是调用 eof() 函数，因为当您设法读取文件的最后一个字节时，它仅返回 true 。如果在此之前发生任何错误，该函数仍将返回true。

因此，您应该将代码编写为：

int countlines() {
    ifstream myfile;
    int linenumber = 0;
    string linecontent;
    myfile.open("question.txt", ios::in);
    while (getline(myfile, linecontent)) {
        if (!linecontent.empty()) {
            ++linenumber;
        }
    }
    return linenumber;
}

Answer 4

尝试以下代码。它还（希望）让您了解为什么文件打开失败......

int countlines()
{
    ifstream myfile;
    myfile.open("questions.txt");
    string contents;
    int linenumber = 0;

    if (myfile.is_open())  
    {
        while (getline(myfile, contents))  
        {
            if (contents != "")
                linenumber++;
        }
        cout << "there are " << linenumber << " lines." << endl;        
        myfile.close();
    }
    else
        cout << "Unable to get file (reason: " << strerror(errno) << ")." << endl;

    return linenumber;
}