读取多个文件时出现分段错误

Segmentation fault when reading from multiple files

本文关键字：分段 错误 文件 读取      更新时间：2023-10-16

我有以下程序，它从user获取文件数，然后获取文件名，并将文件的内容读入优先级队列。当我执行程序时，在我输入第一个文件名后，它给出了分割错误。

#include <cstdlib>
#include <ctime>
#include <functional>
#include <iostream>
#include <queue>
#include <fstream>
using namespace std;
int main() {
    char *filename;
    int fnum;
    cout<<"Number of files:"<<endl;
    cin>>fnum;
    int i;
    priority_queue<int, vector<int>, greater<int> > pqi;
    for(i = 0; i<fnum;i++){
        cout <<"Enter Filename:"<<endl;
        cin>>filename;
        ifstream inFile(filename);
        long n;
        while(!inFile.eof()){
            inFile >> n;
            pqi.push(n);
        }
        inFile.close();
        inFile.clear();
    }
    while(!pqi.empty()) {
        cout << pqi.top() << ' ';
        pqi.pop();
    }
}

不知道为什么。

问题在于您的char*定义。你只需要定义一个指针，不给它分配任何内存。您必须使用new关键字为它分配内存:

char *filename = new char[256];
//... rest of your code ...
//When you no longer need filename (usually at the end of the code)
//you have to free the memory used by it manually:
delete[] filename;
在这个简单的例子中，你也可以使用静态数组: 
char filename[256];
//No need to delete[] anything in this way.

上述两种方法都为filename分配了固定数量的内存，这意味着如果用户在上面的示例中输入的文件名超过256字节，我们会遇到缓冲区溢出。您可以使用string类型，它自动为您进行内存管理，并且易于使用:

#include <string>
string filename;
cin >> filename;

在你的代码中有

char *filename;

然后用

cin>>filename;

您根本没有为文件名分配空间，因此输入被写入一些未定义的内存中。要么将filename定义为char数组，要么使用std::string。

相关文章：