为什么C++ stl字符串的find函数有时会出错有时会正确？

Question

我正在尝试在 Ubuntu 16.04（GCC&G++ 5.4 和 CMake 3.5.1）中使用 C++ 读取文件。 测试文件（名为 123.txt）只有一行字，就像这样：

Reprojection error: avg = 0.110258   max = 0.491361

我只想得到avg 错误和max 错误。我的方法是获取一条线并将它们放入std::string 并使用string::find。我的代码很简单，就像这样：

#include <iostream>
#include <string>
#include <stdio.h>

using namespace std;

int main()
{
    FILE *fp = fopen("123.txt", "r");
    char tmp[60];
    string str;
    fgets(tmp, size_t(tmp), fp);
    fclose(fp);
    cout << tmp << endl;
    str = tmp;
    cout << str.size() << endl;
    size_t avg = str.find("avg");
    size_t max = str.find("max");
    cout << avg << endl;
    cout << max << endl;
}

我可以使用g++ 编译成功。但我遇到了一个奇怪的问题。

当我第一次在命令中运行它时，它会得到正确的结果：

Reprojection error: avg = 0.110258   max = 0.491361

52
20
37

如果我再次运行代码，有时会出错，就像这样：

p
2
18446744073709551615
18446744073709551615

“p”是乱码，不能在命令中正确显示。我不擅长 C++，对此感到困惑。有没有人可以说点什么？谢谢！

Answer 1

表达式

fgets(tmp, size_t(tmp), fp);

格式不正确，size_t(tmp) 将无法按预期工作，您需要sizeof(tmp)。

您获得的52 值是因为fgets 消耗了\n 字符，这也被计算在内，实际上字符串中有51 字符以空格计数。

也就是说，在这种情况下，您可以使用更好的 C++ 工具来替换您正在使用的 C 工具，fopen 可以使用 fstream 替换库，fgets 可以替换为getline。

类似：

#include <iostream>
#include <string>
#include <fstream>

int main()
{
    std::ifstream fp("123.txt"); //C++ filestream

    if (fp.is_open()) {//check for file opening errors

        std::string str;
        std::getline(fp, str); //C++ read from file
        fp.close();
        std::cout << str << std::endl;
        std::cout << str.size() << std::endl;
        size_t avg = str.find("avg");
        size_t max = str.find("max");
        std::cout << avg << std::endl;
        std::cout << max << std::endl;
    }
    else{
        std::cerr << "Couldn't open file";
    }
}

请注意，我不使用using namespace std;，这是有原因的，这不是一个好习惯，您可以查看this thread了解更多详细信息。