file.open()函数返回值异常：文件读取输出多余内容求助

核心原因 #

你遇到的乱码问题，本质是C风格字符串必须以'\0'（空字符）作为结束标志，但你的代码读取文件内容到char数组后，没有添加这个终止符。cout输出char*时会持续读取内存，直到找到第一个'\0'，所以末尾会出现随机的垃圾数据（乱码）。

另外你的代码还有两个隐患：

• 手动new的char数组没有delete，会造成内存泄漏
• 文件打开失败时，函数没有返回有效指针，属于未定义行为

修复方案1：补全字符串终止符（兼容原char*逻辑） #

修改getDataFromFile函数，在分配数组时多留一个字节存储'\0'，读取完成后手动添加：

#include <iostream>
#include <fstream>

using namespace std;

char *getDataFromFile(char *file_path)
{
    fstream file;
    file.open(file_path, ios::in | ios::binary);

    if (file.is_open())
    {
        file.seekg(0, file.end);
        int length = file.tellg();
        file.seekg(0, file.beg);

        // 多分配1个字节用于存储'\0'
        char *data = new char[length + 1];
        file.read(data, length);
        // 添加字符串终止符
        data[length] = '\0';
        
        file.close();
        return data;
    }
    file.close();
    // 文件打开失败时返回空指针，避免未定义行为
    return nullptr;
}

int main(int argc, char **argv)
{ 
    char* fileData = getDataFromFile("test.txt");
    if (fileData != nullptr) {
        cout << fileData << endl;
        // 手动释放内存，避免泄漏
        delete[] fileData;
    } else {
        cout << "Failed to open file" << endl;
    }
    system("PAUSE");
    return 0;
}

修复方案2：使用std::string（推荐，更安全） #

C++中推荐用std::string管理字符串，它会自动处理内存和终止符，避免手动管理的麻烦：

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

using namespace std;

string getDataFromFile(const string& file_path)
{
    ifstream file(file_path, ios::in | ios::binary);
    if (!file.is_open()) {
        return "";
    }
    // 用stringstream读取整个文件内容
    stringstream buffer;
    buffer << file.rdbuf();
    file.close();
    return buffer.str();
}

int main(int argc, char **argv)
{ 
    string fileData = getDataFromFile("test.txt");
    if (!fileData.empty()) {
        cout << fileData << endl;
    } else {
        cout << "Failed to open file or file is empty" << endl;
    }
    system("PAUSE");
    return 0;
}

验证效果 #

两种方案运行后，控制台都会正确输出：

test_line1
test_line2
test_line3

不会再出现末尾乱码。

内容的提问来源于stack exchange，提问作者elo_melo