指向文件的指针的 C++ 向量

Question

我目前正在尝试从 vector 访问指向文件的指针。我面临的问题是我无法单独解决的问题是，我正在用最后创建的向量重写已经存储的指针（或者至少看起来如此）。 因此，如果我尝试访问它们，则只有最后一个可用。 也许有一个我看不到的简单解决方案，但我已经花了三天时间试图找出我做错了什么（我已经完全从头开始重写了两次代码）。

int cnt = 0;
vector<AbstractInput*> abstractFiles;
while (cnt < 6) {
    string path = "D:/tempSort_" + to_string(cnt) + ".txt";
    ofstream fileOutput(path);
    if (!fileOutput)
    {
        // todo error 
        exit(1);
    }
    AbstractOutput* tmp = new FileOutput(fileOutput, kernel);
    tmp->WriteLine(to_string(cnt) + " zkouska");
    tmp->WriteLine(to_string(cnt) + " zkouska2");
    tmp->WriteLine(to_string(cnt) + " zkouska3");
    tmp->Close();
    delete(tmp);

    ifstream fileInput(path);
    if (!fileInput)
    {
        exit(1);
    }       

    abstractFiles.push_back(&(FileInput(fileInput, kernel)));

    output->WriteLine("CNT = " + to_string(cnt));
    for (AbstractInput* it : abstractFiles)
    {
        bool succes;
        output->WriteLine((it)->ReadLine(succes));
    }       


    cnt++;

} 

文件输入：

using namespace std;

class FileInput : public AbstractInput
{
    using AbstractInput::AbstractInput;


private:
    ifstream& inputFile;
    bool closed;

public:
    FileInput::FileInput(ifstream& inputFile, Kernel* kernel) : 

    AbstractInput(kernel), inputFile{ inputFile }, closed(false)
    {   

    }

    int FileInput::Close()
    {
        inputFile.close();
        closed = true;
        return 0;
    }

    bool FileInput::HasNext()
    {
        return !closed;
    }
    string FileInput::Read()
    {
        return GetKernel()->ReadFromKeyboard();
    }
    string FileInput::ReadLine(bool& success)
    {
        string line = GetKernel()->ReadLineFromFile(inputFile, success);

        closed = !success;
        return line;    
    }

};

内核函数：

string Kernel::ReadLine(istream& stream, bool& success)
{
    string line;
    if (getline(stream, line))
    {
        success = true;
        return line;
    }
    success = false;
    return "";
}

string Kernel::ReadLineFromFile(ifstream& stream, bool& success)
{   
    return ReadLine(stream, success);
}

编辑：添加整个代码 + FileInput 类

EDIT2：添加内核函数

EDIT3：我尝试使用&(FileInput(inputFile, kernel)) 和new FileInput(inputFile, kernel) 创建新指针。两者都以相同的结果结束 -> 只有最后添加的指针有效，其他指针指向同一个文件。

EDIT4：当前输出 + 预期输出

当前版本的输出：

CNT = 0
0 zkouska
CNT = 1
1 zkouska
1 zkouska2
CNT = 2
2 zkouska
2 zkouska2
2 zkouska3
CNT = 3
3 zkouska
3 zkouska2
3 zkouska3

CNT = 4
4 zkouska
4 zkouska2
4 zkouska3


CNT = 5
5 zkouska
5 zkouska2
5 zkouska3



.

预期结果是：

CNT = 0
0 zkouska
CNT = 1
0 zkouska
1 zkouska
CNT = 2
0 zkouska
1 zkouska
2 zkouska

还有更多……因为我只读取每个文件的第一行，所以 zkouska1、zkouska2、……不应该被写入输出。

Answer 1

&(FileInput(fileInput)) 是保存对变量的引用的临时地址。

1. 用悬空指针填充向量。
2. 您的类包含对在循环结束时超出范围的变量的引用。

您想在堆上创建非临时对象并存储它们的地址。

auto fileInput = new std::ifstream(path);
abstractFiles.push_back(new FileInput(*fileInput));

记得在之后正确删除这些指针（ifstream 和 FileInput）。

注意：for 循环应该做什么？在每次 while 迭代中，您从 abstractFiles 的每个有效条目中读取一行。

我期望的输出是：

CNT = 0
0 zkouska
CNT = 1
0 zkouska2
1 zkouska
CNT = 2
0 zkouska3
1 zkouska2
2 zkouska
CNT = 3

1 zkouska3
2 zkouska2
3 zkouska
CNT = 4


2 zkouska3
3 zkouska2
4 zkouska

Answer 2

替换

ifstream fileInput(path);
if (!fileInput)
{
    exit(1);
}       

与

std::shared_ptr<ifstream> fileInput(new ifstream(path) );
if (*fileInput)
{
    exit(1);
}       

然后将ifstream& 的所有其他实例替换为std::shared_ptr<ifstream>（当实际使用流从文件中读取时，您还必须将stream 引用为*stream）。这将防止您的文件流对象超出范围，并在程序结束时负责销毁和释放您的对象。 （只要您注意销毁 AbstractFiles 向量中的对象；事实上，您应该考虑在那里使用unique_ptr。

关于您的预期输出...好吧，如果您打开一个文件，读取一行，然后在下一个循环中读取具有相同文件描述符的另一行，则必须读取后续行，除非您重置流fileStream->Seek( 0, SeekOrigin::Begin ); 之类的东西，正如 here 所解释的那样。