将重载的 << 运算符与动态数组一起使用

Question

我的问题如下：

我想使用重载的 >> 运算符将文件中的值读取到名为 Voter 的特定类的动态数组中。 然后，我想将我的结果导出到屏幕上，用另一个重载的

使用普通数组，我得到了即时结果，但在动态化方面遇到了困难。 我在论坛上读到向量可能是一种更好的方法，但我想了解我在这方面做错了什么。

谢谢！

详情：

该文件有 3 个用空格分隔的“值”： id(string) nr(int) (1 or 0)bool

代码摘录：

string RollFileName = "VotersRoll.dat";
ifstream inFile_VotersRoll;
inFile_VotersRoll.open(RollFileName);

if(inFile_VotersRoll.fail())
{
    printf("The VotersRoll.dat file failed to open.\n");
    exit(1);

}
//connect to in file



int numberOfEntries = 0;
Voter testVoter;
while(inFile_VotersRoll>>testVoter)
{
    numberOfEntries++;
}
//get number of voter objects from file

typedef Voter* VoterArrayPntr;
VoterArrayPntr rollArray;
rollArray = new Voter[numberOfEntries];
//create dynamic arrray for voter objects


while (inFile_VotersRoll>>*rollArray) {
    cout << *rollArray;

}

类（摘录）：

class Voter
{
public:
    friend istream& operator>>(istream &inP, Voter &voterP);
    friend ostream& operator<<(ostream &outP,const Voter &voterP);
private:
    string id;
    int nr_times_voted;
    bool voted;
}

好友功能：

istream& operator>>(istream &inP,Voter &voterP)
{  
   inP >> voterP.id >> voterP.nr_times_voted >> voterP.voted;
   return inP; 
}
ostream& operator<<(ostream &outP,const Voter &voterP)
{ 
   outP << voterP.id << voterP.nr_times_voted << voterP.voted;   
   return outP;
}

Answer 1

在 C++ 中，std::vector 实现了动态数组的概念。这不仅仅是“也许”更好的方法。 不要使用new[] 和delete[]！

（实际上，您的示例代码中完全缺少delete[]。如果您不在任何地方释放内存，并且只要程序运行，该对象在概念上并不意味着存在，那么这就是内存泄漏。）

至于为什么这里的代码不起作用，我们先来看看这个循环：

int numberOfEntries = 0;
Voter testVoter;
while(inFile_VotersRoll>>testVoter)
{
    numberOfEntries++;
}

这很奇怪；您第一次读取整个文件只是为了获取元素的数量，然后会再次读取它以获取实际元素？

这是多余的。您应该在一个循环中获得所有内容。

但是这里也有两个严重的bug：

1. 在第二个循环开始之前，您已到达文件末尾。
2. 您只使用数组的第一个元素。

让我们先解决第一个问题。在下一个循环中，

typedef Voter* VoterArrayPntr;
VoterArrayPntr rollArray;
rollArray = new Voter[numberOfEntries];
//create dynamic arrray for voter objects

while (inFile_VotersRoll>>*rollArray) {
    cout << *rollArray;
}

循环条件永远不会为真。您的 operator>> 被调用一次。流从运算符返回。最后，它被用在循环条件中，导致false——因为你已经在文件的末尾了。

一个非常快速的解决方法是为同一个文件打开一个新流来解决这个问题：

typedef Voter* VoterArrayPntr;
VoterArrayPntr rollArray;
rollArray = new Voter[numberOfEntries];
//create dynamic arrray for voter objects

ifstream inFile_VotersRoll2; // ugly workaround
inFile_VotersRoll2.open(RollFileName);
while (inFile_VotersRoll2>>*rollArray) {
    cout << *rollArray;
}

这将读取并打印整个文件。但是，所有内容都将仅从数组的第一个元素读取和写入。 *rollArray 指的是第一个元素，rollArray 永远不会改变。

第二个错误是这样修复的：

for (int index = 0; (index < numberOfEntries) && (inFile_VotersRoll2>>rollArray[index]); ++index) { 
    cout << rollArray[index];
}

但正如我之前所说，您的目标应该是一次阅读整本书。这意味着您可以随时增加数组的大小。幸运的是，有了std::vector，这很简单，因为std::vector 不仅会自动增长，in a clever way 也会自动增长。并且内存也会自动释放。我们开始吧，您的完整示例已修改为使用std::vector：

#include <string>
#include <iostream>
#include <fstream>
#include <vector>
#include <stddef.h>

using namespace std; // just for quick'n'dirty test code

class Voter
{
public:
    friend istream& operator>>(istream &inP, Voter &voterP);
    friend ostream& operator<<(ostream &outP,const Voter &voterP);
private:
    string id;
    int nr_times_voted;
    bool voted;
};

istream& operator>>(istream &inP,Voter &voterP)
{  
   inP >> voterP.id >> voterP.nr_times_voted >> voterP.voted;
   return inP; 
}
ostream& operator<<(ostream &outP,const Voter &voterP)
{ 
   outP << voterP.id << voterP.nr_times_voted << voterP.voted;   
   return outP;
}

int main() {

    string RollFileName = "VotersRoll.dat";
    ifstream inFile_VotersRoll;
    inFile_VotersRoll.open(RollFileName);

    if(inFile_VotersRoll.fail())
    {
        std::cerr << "The VotersRoll.dat file failed to open.\n";
        return EXIT_FAILURE;
    }

    std::vector<Voter> voters;

    Voter input;
    while (inFile_VotersRoll >> input) {
        voters.push_back(input);
    }

    for (auto const &voter : voters) {
        std::cout << voter << "\n";
    }
}

Answer 2

“我想了解我在这个问题上做错了什么。”

你的循环在这里

while (inFile_VotersRoll>>*rollArray) {
    cout << *rollArray;
}

只会写入数组中的第一个元素。读入后需要增加指向下一个元素的指针：

inFile_VotersRoll.seekp(0); // Rewind to the beginning of inFile_VotersRoll

VoterArrayPntr curItem = rollArray;
while (inFile_VotersRoll>>*curItem) {
    cout << *curItem;
    curItem++; // <<<<<<
}