在 C++ 中从文件读取行的首选模式是什么？

Question

我在 C++ 教程中至少看到了两种从文件中读取行的方法：

std::ifstream fs("myfile.txt");
if (fs.is_open()) {
  while (fs.good()) {
    std::string line;
    std::getline(fs, line);
    // ...

和：

std::ifstream fs("myfile.txt");
std::string line;
while (std::getline(fs, line)) {
  // ...

当然，我可以添加一些检查以确保文件存在并已打开。除了异常处理之外，还有理由选择更详细的第一个模式吗？你的标准做法是什么？

Answer 1

while (std::getline(fs, line))
{}

这不仅是正确的，而且也是首选 因为它是惯用的。

我假设在第一种情况下，您在 fs 之后没有检查 fs >std::getline() 为 if(!fs) break; 或类似的东西。因为如果你不这样做，那么第一种情况就完全错误了。或者如果你这样做，那么第二个仍然是更好的选择，因为它在逻辑上更加简洁和清晰。

函数good()应该在您尝试从流中读取之后使用；它用于检查尝试是否成功。在第一种情况下，你不这样做。在 std::getline() 之后，您假设读取成功，甚至不检查 fs.good() 返回的内容。另外，您似乎假设如果 fs.good() 返回 true，std::getline 将成功从流中读取一行。您正朝着相反的方向前进：事实是，如果 std::getline 成功从流中读取一行，则 fs.good() 将返回真。

cplusplus 的文档介绍了 good()那个，

如果没有设置流的错误标志（eofbit、failbit 和 badbit），则该函数返回 true。

也就是说，当您尝试从输入流读取数据时，如果尝试失败，则仅设置失败标志，并且 good() 返回 false 作为失败的指示。

如果您想将 line 变量的范围限制在循环内部，那么您可以编写一个 for 循环：

for(std::string line; std::getline(fs, line); )
{
   //use 'line'
}

注意：在阅读@后我想到了这个解决方案约翰的解决方案，但我认为它比他的版本更好。

---

请阅读此处的详细解释，为什么第二个更可取且惯用：

• Linux | C++ 中的分段错误 - 由于函数 ifstream

或者阅读 @Jerry Coffin 撰写的这篇写得很好的博客：

• 读取文件

Answer 2

将此视为对纳瓦兹已经非常出色的答案的扩展评论。

关于您的第一个选择，

while (fs.good()) {
  std::string line;
  std::getline(fs, line);
  ...

这有多个问题。问题 1 是 while 条件位于错误的位置并且是多余的。它位于错误的位置，因为 fs.good() 指示对文件执行的最新操作是否正常。 while 条件应该与即将发生的操作有关，而不是与之前的操作有关。无法知道接下来对该文件执行的操作是否正常。即将采取什么行动？ fs.good() 不会读取您的代码来查看即将执行的操作是什么。

第二个问题是您忽略了 std::getline() 的返回状态。如果您立即使用 fs.good() 检查状态，那就没问题了。因此，稍微修复一下这个问题，

while (true) {
  std::string line;
  if (std::getline(fs, line)) {
    ...
  }
  else {
     break;
  }
}

或者，您可以执行 if (! std::getline(fs, line)) { break; } } 但现在循环中间有一个break。耶赫。如果可能的话，最好将退出条件作为循环语句本身的一部分。

将其与

std::string line;
while (std::getline(fs, line)) {
  ...
}

“这是从文件中读取行的标准习惯用法”进行比较。 C 中存在一个非常相似的习惯用法。这个习惯用法非常古老，使用非常广泛，并且被广泛视为从文件中读取行的正确方法。

如果您来自一家禁止有副作用的条件句的商店怎么办？ （有很多很多编程标准就是这样做的。）有一种方法可以解决这个问题，而无需诉诸循环中间的中断方法：

std::string line;
for (std::getline(fs, line); fs.good(); std::getline(fs, line)) {
  ...
}

不像中断方法那么丑陋，但大多数人都会同意这不是“它几乎和标准习语一样漂亮。

我的建议是使用标准习惯用法，除非某些标准白痴禁止使用它。

附录
关于 for (std::getline(fs, line); fs.good(); std::getline(fs, line)) ：这很丑陋，原因有两个。一是明显的重复代码块。

不太明显的是，调用 getline 然后调用 good 会破坏原子性。如果其他线程也在读取该文件怎么办？现在这还不太重要，因为 C++ I/O 当前不是线程安全的。它将出现在即将发布的 C++11 中。仅仅为了让标准的执行者满意而破坏原子性只会导致灾难。

Answer 3

实际上，我更喜欢另一种方式，

for (;;)
{
  std::string line;
  if (!getline(myFile, line))
    break;
  ...
}

对我来说，它读起来更好，并且字符串的范围正确（即在使用它的循环内，而不是在循环外），

但是在您编写的两种方式中，第二个是正确的。

Answer 4

第一个在每个循环中释放并重新分配字符串，浪费时间。
第二次将字符串写入已经存在的空间，消除了释放和重新分配，使其实际上比第一次更快（更好）。

Answer 5

试试这个=>

// reading a text file
#include <iostream>
#include <fstream>
#include <string>
using namespace std;

int main () {
  string line;
  ifstream myfile ("example.txt");
  if (myfile.is_open())
  {
    while ( myfile.good() )
    {
      getline (myfile,line);
      cout << line << endl;
    }
    myfile.close();
  }

  else cout << "Unable to open file"; 

  return 0;
}