c - 我正在编写一个程序，输出用户输入的最早日期，但出现了一些问题

Question

我正在编写这个程序，提示用户输入任意数量的日期，并在用户输入 0/0/0 时最早输出。但是我的程序在输入前两个日期后输出结果。当我输入更多日期时，它会不断比较它们，但最后会给出错误的结果。请帮我修一下。这是我的代码

/*
//Input
Enter a date (mm/dd/yy): 3/6/08
Enter a date (mm/dd/yy): 5/17/07
Enter a date (mm/dd/yy): 6/3/07
Enter a date (mm/dd/yy): 0/0/0
//output
5/17/07 is the earliest date*/


#include <stdio.h>
#include <stdlib.h>

int main(int argc, char const *argv[])
{
int month, day, year;
int month2, day2, year2;
printf("Enter a date (mm/dd/yy): ");
scanf("%d/%d/%d", &month2, &day2, &year2);

while(month != 0 && day != 0 && year != 0)
{
    printf("Enter a date (mm/dd/yy): ");
    scanf("%d/%d/%d", &month, &day, &year);


    if(year == year2)
    {
        if(month == month2)
        {
            if(day > day2)
            {
                printf("%d/%d/%d is the earliest date\n", month2, day2, year2);
            }
            else
            {
                printf("%d/%d/%d is the earliest date\n",month, day, year);
            }
        }
        else if (month > month2)
        {
            printf("%d/%d/%d is the earliest date\n", month2, day2, year2);
        }
        else
        {
            printf("%d/%d/%d is the earliest date\n",month, day, year);
        }

    }
    else if(year > year2)
    {
        printf("%d/%d/%d is the earliest date\n", month2, day2, year2);
    }
    else
    {
        printf("%d/%d/%d is the earliest date\n",month, day, year);
    }


} 
return 0;
}

Answer 1

第一次使用year,moth,day，缺少初始化。

// int month, day, year;
int month = -1;
int day = 0;
int year = 0;
// ...
while(month != 0 && day != 0 && year != 0)

否则 OP 已关闭，但需要更改最早日期，因为会出现各种新日期。 @Barmar .在循环之后打印一次最早的日期。

循环退出条件测试需要在数据输入后立即发生。

while (1) {
  printf("Enter a date (mm/dd/yy): ");
  scanf("%d/%d/%d", &month, &day, &year);
  printf("%d/%d/%d is the date entered\n", month, day, year);
  if (!(month != 0 && day != 0 && year != 0)) {
    break;
  }

  int earlier = 0;
  if (year < year2) {
    earlier = 1;
  } else if (year == year2) {
    if (month < month2) {
      earlier = 1;
    } else if (month == month2) {
      if (day < day2) {
        earlier = 1;
      }
    }
  }
  if (earlier) {
    year2 = year; 
    month2 = month; 
    day2 = day;
  }
}

printf("%d/%d/%d is the earliest date\n", month2, day2, year2);

---

其他改进包括:

读取一行输入，然后解析日期。

检查扫描错误。

刷新输出以确保在输入之前出现提示。

printf("Enter a date (mm/dd/yy): ");
fflush(stdout);

使用更好的名称.. 而不是 year2，也许是 early_year？

do 循环而不是 while 循环，因为循环至少迭代一次。然后 month, day, year 可以移动到循环内部。

检查有效日期的范围......