c ++处理来自输入文件的数据会创建无限循环

Question

我是 C++ 的初学者，正在做一个（据说很简单的）作业，但这让我头疼...

我熟悉 C# 和 Java，其中允许使用动态数组，并且存在诸如 foreach 循环之类的其他简洁事物...

该程序包括从如下所示的文件中读取数据：

5 约翰逊 5000 磨坊主 4000 达菲 6000 罗宾逊 2500 阿什顿 1800

第一行始终是“候选人”的数量，然后是该候选人的重复姓名/票数（分别为以下每一行）。 该程序有 20 个候选者的限制——我认为这是因为不允许使用动态数组，所以我们必须创建固定大小的数组来保存每种类型的数据。

我基本上希望读取第一行，将其存储在一个变量中，该变量将用作我的循环迭代限制，并将所有名称存储在一个数组中，然后将所有数字存储在第二个数组中以处理它们。

这是我在尝试读取文件时所做的一个小测试 sn-p。它可以工作，但会生成一个无限循环:(

int main() {
    string candidateNames[21];
    double votesReceivedPerCandidate[21];
    ifstream infile("votedata.txt");

    while (!infile.eof()) {
        string firstLine;
        int numberofCandidates;
        getline(infile, firstLine);
        numberofCandidates = atoi(firstLine.c_str());

        for (int x = 0; x < numberofCandidates; x++) {
            cout << "hello!";
        }
    }
}

我现在睡眠不足，甚至不确定我是否以正确的方式思考这个问题。

我的问题是：

1. 使用 C++，您将如何只读取第一行，然后将该行中的数字用作for 循环中的条件？ （另外 - for 循环是最好的选择吗？你将如何设置它？）

2. 您将如何设置上述循环来读取文件并将一个条目存储到一个数组中，将下一个条目存储到第二个数组中，然后成对重复该过程？ （就像你给两个人发牌一样）？

Answer 1

C++ 有动态数组（通过std::vector）和foreach 循环（通过std::for_each()、range-based for loops 等）。

试试这样的：

#include <vector>
#include <string>
#include <fstream>
#include <limits>

struct candidate
{
    std::string name;
    double votes;
};

int main()
{
    std::vector<candidate> candidates;

    std::ifstream infile("votedata.txt");

    int numberofCandidates;
    if (infile >> numberofCandidates)
    {
        infile.ignore(std::numeric_limits<std::streamsize>::max(), '\n');

        for (int x = 0; x < numberofCandidates; x++)
        {
            candidate c;
            if (!std::getline(infile, c.name)) break;
            if (!(infile >> c.votes)) break;
            infile.ignore(std::numeric_limits<std::streamsize>::max(), '\n');
            candidates.push_back(c);
        }
    }

    // use candidates as needed...

    return 0;
}

---

如果你真的想变得棘手，你可以让 STL 为你循环文件，通过使用 std::copy() 或 std::copy_n() 和 std::istream_iterator 和 std::back_inserter 作为输入/输出迭代器：

#include <vector>
#include <string>
#include <fstream>
#include <algorithm>

struct candidate
{
    std::string name;
    double votes;
};

std::istream& operator>>(std::istream &in, candidate &c)
{
    if (std::getline(in, c.name))
    {
        if (in >> c.votes)
            in.ignore(std::numeric_limits<std::streamsize>::max(), '\n');
    }
    return in;
}

int main()
{
    std::vector<candidate> candidates;

    std::ifstream infile("votedata.txt");

    int numberofCandidates;
    if (infile >> numberofCandidates)
    {
        infile.ignore(std::numeric_limits<std::streamsize>::max(), '\n');

        // using std::copy()...

        // ignoring numberofCandidates, read until EOF...
        std::copy(std::istream_iterator<candidate>(infile),
                 std::istream_iterator<candidate>(),
                 std::back_inserter(candidates));

        // using std::copy_n()...

        std::copy_n(std::istream_iterator<candidate>(infile),
                 numberofCandidates,
                 std::back_inserter(candidates));
    }

    // use candidates as needed...

    return 0;
}

---

考虑到您还没有涵盖std::vector，您可能还没有涵盖STL算法，所以上面看起来更像是使用手动循环和2个固定数组而不是std::vector：

#include <string>
#include <fstream>
#include <limits>

const int cMaxCandidates = 20;

int main()
{
    std::string candidateNames[cMaxCandidates];
    double candidateVotes[cMaxCandidates];
    int numberofCandidates = 0;

    std::ifstream infile("votedata.txt");

    int numberofCandidatesInFile;
    if (infile >> numberofCandidatesInFile)
    {
        infile.ignore(std::numeric_limits<std::streamsize>::max(), '\n');

        for (int x = 0; (x < numberofCandidatesInFile) && (x < cMaxCandidates); x++)
        {
            if (!std::getline(infile, candidateNames[x])) break;
            if (!(infile >> candidateVotes[x])) break;
            infile.ignore(std::numeric_limits<std::streamsize>::max(), '\n');
            ++numberofCandidates;
        }
    }

    // use candidateNames[] and candidateVotes[] up to max
    // numberofCandidates of elements as needed...

    return 0;
}

或者，使用动态分配的数组：

#include <string>
#include <fstream>
#include <limits>

int main()
{
    std::string *candidateNames = NULL;
    double *candidateVotes = NULL;
    int numberofCandidates = 0;

    std::ifstream infile("votedata.txt");

    int numberofCandidatesInFile;
    if (infile >> numberofCandidatesInFile)
    {
        infile.ignore(std::numeric_limits<std::streamsize>::max(), '\n');

        candidateNames = new std::string[numberofCandidatesInFile];
        candidateVotes = new double[numberofCandidatesInFile];

        for (int x = 0; x < numberofCandidatesInFile; x++)
        {
            if (!std::getline(infile, candidateNames[x])) break;
            if (!(infile >> candidateVotes[x])) break;
            infile.ignore(std::numeric_limits<std::streamsize>::max(), '\n');
            ++numberofCandidates;
        }
    }

    // use candidateNames[] and candidateVotes[] up to max
    // numberofCandidates of elements as needed...

    delete[] candidateNames;
    delete[] candidateVotes;

    return 0;
}

Answer 2

#include <bits/stdc++.h>
using namespace std;

int main() {

        string candidateNames[21];
        double votesReceivedPerCandidate[21];
        ifstream infile("in.txt");

        int numberofCandidates;
        infile >> numberofCandidates;
        infile.get();

        for(int i = 0; i < numberofCandidates; ++i) {

            getline(infile, candidateNames[i]);
            infile >> votesReceivedPerCandidate[i];

            for (int x = 0; x < numberofCandidates; x++) {
                cout << "hello!" << endl;
            }
            cout << endl;
        }
        return 0;
 }

首先获取候选者的数量，然后使用 for 循环运行到此数量的候选者。对于每个数量的候选人，以这种方式取名字和选票。为了确保一切正常，我打印了 "hello" numberofcandidates * numberofcandidates 次。 希望这会有所帮助。