使用 ifstream 解析文本的惯用方法是什么？

Question

我正在尝试解析文本文件以查找模式，然后获取子字符串。该代码片段工作正常，但是我可以改进吗？我可以尽量减少这里的复制吗？即我得到一行并将其存储在buf中然后构造一个字符串，可以消除这种复制吗？

简而言之，实现这一目标的惯用方法是什么？

    std::ifstream f("/file/on/disk");
    while (!f.eof()) {
        char buf[256];
        f.getline(buf, sizeof(buf));
        std::string str(buf);
        if (str.find(pattern) != std::string::npos)
        {
            // further processing, then break out of the while loop and return.
        }
    }

Answer 1

这是一种可能的重写：

std::ifstream f("/file/on/disk");
char buffer[256];
while (f.getline(buffer, sizeof(buf))) { // Use the read operation as the test in the loop.
    if (strstr(buffer, pattern) != NULL) { // Don't cast to string; costs time
        // further processing, then break out of the while loop and return.
    }
}

主要更改被标记为内联，但总结一下：

1. 使用读取操作作为 while 循环中的测试。这使得代码变得更短、更清晰。
2. 不要将 C 样式字符串转换为 std::string；只需使用 strstr 进行扫描即可。

进一步说明，您可能不想在此处使用 C 样式字符串，除非您确定这就是您想要的。 C++ string 可能更好：

std::ifstream f("/file/on/disk");
std::string buffer;
while (std::getline(f, buffer)) { // Use the read operation as the test in the loop.
    if (buffer.find(pattern) != std::string::npos) {
        // further processing, then break out of the while loop and return.
    }
}

Answer 2

在代码中，首先将文件中的字符复制到 char 数组中。这应该是所有必要的复制。如果您需要阅读每个字符一次，那么甚至不需要该副本。

接下来，从您填充的数组构造一个 std::string 。再说一遍，没有必要。如果您想要一个字符串，请直接从流中复制到字符串中。

std::ifstream f("/file/on/disk");
for( std::string line; std::getline(f, line); ) {
    if (str.find(pattern) != std::string::npos) {
        // further processing, then break out of the while loop and return.
    }
}

Answer 3

您根本不需要那个 char[] 。

string line;
std::getline(f, line);
if (line.find(pattern) != std::string::npos)
{
    ....
}