EOF循环在最后结束之前重复行3次

Question

我的作业是给我一个包含编号格式日期的文件，我必须让程序读取该文件并为每个给定的日期吐出一周中的几天。 文件的每一行都是单独的日期。 我的首选方法是eof循环。 但是，我的文件有10个日期，输出有30个日期。这是上下文的主要内容。

int main()
{
    ifstream inFile;
    inFile.open("/Users/holdentatlow/Desktop/date 2");

    const char *Names[] = {"Sunday","Monday", "Tuesday","Wednesday", "Thursday", "Friday", "Saturday"};

    int day = 0;

    int date;

    inFile >> date;
    cout << "Day : " << Names[day] << endl;

    while (!inFile.eof()){

        inFile >> date;
        day = getWeekDay(date, date, date);
        cout<< "Day : "<< Names[day] <<endl;

    }

    inFile.close();

    return 0;
}

另外，我的印象是它报告的日期并不完全准确。 我没有决心单独检查每一个，但是结果似乎很不稳定，不能简单地重复。

这是我要提取日期的文件

0 10  1900
2 16 1900
1 2 1944
0 1 2004
1 29 2004
11 24 2002
6 2 2050
2 23 2004
0 6 1812
1 3 1800

Answer 1

您似乎正在尝试使用date来一次表示三件事。

int date;
inFile >> date;

在这里您读到一个数字。 一。 不是三个 一 。

while (!inFile.eof()){

在这里，你犯了一个主要的罪过。 那不是如何循环输入 。

inFile >> date;
day = getWeekDay(date, date, date);

在这里，你看另外一个号码，并把它传递给getWeekDay三次。

您的循环将继续进行，直到没有更多的数字可读取为止（排序：请参见上面的有关eof用法的注释），在您的情况下，该过程将花费您预期的三倍时间（即30而不是10），因为您读了三倍的数字您所想的数字（再次参见上文）。

您需要对要读取的每个数字进行>>操作。

这是您程序的改进版本：

#include <fstream>
#include <istream>
#include <iostream>

// Assumed defined elsewhere
int getWeekDay(int date, int month, int year);

int main()
{
    std::ifstream inFile("/Users/holdentatlow/Desktop/date 2");

    static const char* Names[] = {
       "Sunday",
       "Monday",
       "Tuesday",
       "Wednesday",
       "Thursday",
       "Friday",
       "Saturday"
    };

    int date, month, year;
    while (inFile >> date >> month >> year) {
       const int day = getWeekDay(date, month, year);
       std::cout << "Day : " << Names[day] << std::endl;
    }
}