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;
}

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

这是我从

获取日期的文件

Answer 1

您似乎试图同时使用date来表示三件事。

int date;
inFile >> date;

在这里你读了一个数字。一。不是三个; 一个

while (!inFile.eof()){

在这里你犯了一个主要的罪; that is not how to loop over input

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;
    }
}

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

1 个答案: