动态数组输出问题

Question

我正在做一个家庭作业，我必须使用指针将程序中的所有静态数组转换为动态数组。我很确定我正在理解这个概念，我已经进行了更改并且我的程序可以运行。问题出在我的输出结果上。我怀疑我从正在使用的文件中输入的数据不正确。这是我的问题的图片以及相关代码：

此点之后的 9 月 21 日编辑：

输出 &数据文件：

主要

#include "Ch9_Ex7.h"

int main()
{
    int numCandidates;
    string *allCandidates;
    int *votes;
    int index, totalVotes;
    ifstream infile;

    allCandidates = new string[1];
    votes = new int[1];


    infile.open("./Ch9_Ex7Data.txt");
    if (!infile)
    {
        cerr << "Cannot open input file. Program terminates!" << endl;
        return 1;
    }

// read number of candidates

    readVotes (infile, votes, allCandidates, numCandidates);

    //delete [] votes;
    //delete [] allCandidates;

输入功能：

#include "Ch9_Ex7.h"

void readVotes (ifstream & infile, int *&votes,
                string *&allCandidates, int & numCandidates)
{

//    read number of candidates
    infile >> numCandidates;
    infile.ignore();  // carriage return

    //delete [] votes;
    //delete [] allCandidates;

    allCandidates = new string[numCandidates];
    votes = new int[numCandidates];

    for (int index = 0; index < numCandidates; index++)
    {
        infile >> votes[index];
        infile.ignore();  // space
        getline(infile, allCandidates[index]);
    }

}

Answer 1

您正在使用以下代码创建一个由一个 char 和一个 int 组成的数组：

allCandidates = new char[1];
votes = new int[1];

我相信您的意思是：

allCandidates = new char[numCandidates];
votes = new int[numCandidates];

它创建大小为 numCandidates 的动态数组。

另外，当您输入候选人的姓名时，您可能想使用 std::string ，如下所示：（

string *allCandidates;
allCandidates = new string[numCandidates];

感谢 Ben Voigt 指出这一点）既然您输入了他们的全名，您将需要以不同的方式输入。也许使用 getline() ：

getline(cin, allCandidates[i]);

响应您的编辑：

您必须将指针作为引用传递，如下所示：

void readVotes (ifstream & infile, int *&votes, string *&allCandidates, int & numCandidates)

并在 main() 中释放它们

delete[] votes;
delete[] allCandidates;

Answer 2

首先，这绝对是糟糕的设计，所以请不要在本练习的范围之外这样做。

现在，回答问题。如果你想在某处创建一个动态对象（或数组）并将指针传递回它，你应该通过引用获取指针。您还必须将名称读入字符串，而不是单个字符。

void readVotes (std::ifstream & infile, int * & votes, std::string * & allCandidates, int & numCandidates)
{
  // read numCandidates

  votes = new int[numCandidates];
  allCandidates = new std::string[numCandidates];

  // populate
}

调用者必须记住清理：（

int main()
{
  int n;
  int * votes;
  std::string * names;

  readVotes(std::cin, votes, names, n);

  // ...

  delete[] votes;
  delete[] names;
}

在现实情况下，我会让函数返回一个 std::vector> .)