在此C程序中返回结构时出错的原因是什么？

Question

我的程序打算实现这一目标

（A）编写一个名为larger()的C函数，该函数返回传递给它的任何两个日期中的较晚的日期。 例如，如果将日期10/9/2001和11/3/2001传递给large larger() ，则将返回第二个日期。

（B）在一个完整的单元中创建为（A）编写的large（）函数。 将由large（）返回的日期结构存储在单独的日期结构中，并显示返回的数据结构的成员值。

我正在为我的C语言课程解决这个问题。 我一切顺利（我认为），不过无论我输入什么，我都会不断得到“较大的日期是：0/0/0”。 所以我开始修修补补，现在我无法摆脱语法错误，也无法弄清楚我的0/0/0问题。 显然日期还没过去。 我在这方面还是很新的（在structs上还是很新的），所以任何帮助都会很棒！

给我语法错误的那行在main（）的底部附近：

DATES result[NUM] = larger(DATES user[NUM]);

完整的代码：

#include <stdio.h>
#define NUM 2
struct Dates
{
       int month;
       int day;
       int year;
};
typedef struct Dates DATES;
DATES larger(DATES[NUM]);
DATES more;
int main()
{
    DATES user[NUM];

    printf("You will enter two dates, and the program will return the larger.\n");
    while (user[0].month < 1 || user[0].month > 12)
         {printf("\nPlease enter the first month, 1-12: ");
         scanf("%d", &user[0].month);}
    while (user[0].day < 1 || user[0].day > 31)
          {printf("\nPlease enter the first day, 1-31: ");
          scanf("%d", &user[0].day);}
    while (user[0].year < 1)
          {printf("\nPlease enter the first year: ");
          scanf("%d)", &user[0].year);}
    printf("\nDate entered: %d/%d/%d.\n", user[0].month, user[0].day, user[0].year);

    while (user[1].month < 1 || user[1].month > 12)
         {printf("\nPlease enter the first month, 1-12: ");
         scanf("%d", &user[1].month);}
    while (user[1].day < 1 || user[1].day > 31)
          {printf("\nPlease enter the first day, 1-31: ");
          scanf("%d", &user[1].day);}
    while (user[1].year < 1)
          {printf("\nPlease enter the first year: ");
          scanf("%d)", &user[1].year);}
    printf("\nDate entered: %d/%d/%d.\n\n", user[1].month, user[1].day, user[1].year);

    DATES result[NUM] = larger(DATES user[NUM]);  /* Problem here */

    printf("The larger date is %d/%d/%d.\n\n", result[0].month, result[0].day, result[0].year);
    system("pause");
    return 0;
}

DATES larger(DATES more[NUM])
{        
      int days0;
      int days1;

      days0 = (more[0].month*31)+(more[0].day)+(more[0].year*365);
      days1 = (more[1].month*31)+(more[1].day)+(more[1].year*365);

      if (days1 > days0)
         {more[0] = more[1];}

return (more[0]);
}

Answer 1

DATES result[NUM] = larger(DATES user[NUM]);

这打算做什么？ DATES result[NUM]声明一个数组DATES 。 （但是每个DATES只包含一个日期，这很令人困惑。）但是，尽管它是一个数组，但它是用单个对象初始化的，返回值来自larger 。 larger的参数DATES user[NUM]似乎在声明user是一个已经存在的变量。 您似乎在向编译器说明user是一个日期数组，但是DATES不在那儿， [NUM]似乎正在索引到数组中，这是您不想要的。

可能你想要的是

DATES result = larger( user );

另外，还有一些严重的样式问题，这些问题以后会引起麻烦：

• 函数参数的数组类型，如DATES larger(DATES[NUM]); 被转换为指针。 在那一行， NUM不执行任何操作，并且声明与DATES larger( DATES * );相同DATES larger( DATES * ); 。 尽管有些老师会说些什么，但指针和数组却不是一回事。 当区别很重要时，这种风格会引起混乱。 请改用DATES larger( DATES * ) 。

• 大写DATES是没有意义的。 大写字母通常表示预处理器宏。

• 预处理器宏就像大锤。 将它们用于简单的数字常量要比const int num = 2;这样的构造const int num = 2; 或enum { num = 2 };

• 描述性变量名称（例如input_limit和user_input将比num和user更好。

• 不要依赖未初始化的数据。 定义但未初始化user后， user[0].month可能为6。 始终在读取前写一个值。

• 大括号应保持可见。 将花括号与循环的开头和最后的单词紧密地分组可以将它们隐藏起来，如果您觉得它们很丑，这很好，但是很容易将错误地添加一行到循环中，从而产生难以调试的流控制错误。

Answer 2

该程序需要进行一些小的更正，但是您的主要问题是while和do while循环之间的区别。

while(<condition>)
{
  //Something
}

和

do
{
  //Something
}
while(<condition>)

这样做的区别在于，while至少一次进入循环然后检查要满足的条件。 但是另一方面，如果不满足条件，则永远不会进入循环。 简而言之，您不会进入while循环，因此不会从用户那里读取任何内容。

Answer 3

您的程序中存在许多逻辑和语法错误。我更正了所有语法部分，必须指出while循环逻辑中的主要缺陷。在以下情况下while您似乎混淆了>和< ：

while (user[0].month < 1 || user[0].month > 12);  //Wrong

while (user[0].month > 1 || user[0].month < 12);  //Correct

仅在火星中，月小于1或大于12 ，但在地球上，由于污染，它们在1到12之间。我已将其更正并在注释中指出。并且您应该使用for循环，而不要重复数组DATES每个元素都相同

AAAAAGHHH到处都是错误。以下是有效的版本。它可以实现您在问题中提到的2个目标-向用户询问2个日期，发现哪个日期更大/更晚。我不包括BC ，仅包括AD年。并不重要，除非你想知道一些尼安德特人比任何潜伏，功能较早或较晚出生的larger回报的指针答案，这是存储在一个新的数据结构largerdate并打印出来。

这是修改后的代码的简要说明：

外部的for循环自动执行每个日期的输入。无需为每个日期重复相同的代码。如果日期数增加，则将很繁琐。此外， do-while循环会要求用户输入日期在指定范围内。如果该混蛋出错，则会大吼大叫，并且该循环的条件确保必须再次输入该混蛋。这两个日期存储在数组user[] ，类型为基地址DATE*作为参数传递给函数big larger() ，该函数比较两个日期并返回指向较大DATE*类型的指针，该指针用于将较大的日期存储在称为largerdate的新数据结构中，然后打印出来。

#include <stdio.h>
#define NUM 2
struct Dates
{
       int month;
       int day;
       int year;
};
typedef struct Dates DATES;
DATES  *larger(DATES*);
//DATES more;     //Not needed as you are doing the same in function definition

int main(void)
{
    DATES user[NUM],largerdate;
    int i;
     printf("You will enter two dates, and the program will return the larger.\n");
    for(i=0;i<NUM;i++)   //for loop is handy when NUM increases.
       {
           do
           {
        printf("\nPlease enter the month number %d, 1-12:\n ",i);
         scanf("%d", &user[i].month);
         if(user[i].month<1||user[i].month>12)
         printf("I told you enter a number between 1-12 JERK!!\n");
           }while(user[i].month<1||user[i].month>12);
          do
           {
        printf("\nPlease enter the day number %d, 1-31:\n ",i);
         scanf("%d", &user[i].day);
         if(user[i].day<1||user[i].day>31)
         printf("I told you enter a number between 1-31 JERK!!\n");
           }while(user[i].day<1||user[i].day>31);
           do
           {
        printf("\nPlease enter the year number %d,greater than one:\n ",i);
         scanf("%d", &user[i].year);
         if(user[i].year<1)
         printf("I told you enter a number greater than 1 JERK!!\n");
           }while(user[i].year<1);
    printf("\nDate number %d entered is: %d/%d/%d.\n", i+1,user[i].month, user[i].day, user[i].year);

   } //for loop ends.It avoids repeating the same thing for DATES[1]


   largerdate=*larger(user);
   printf("\n\nThe larger/later date is %d/%d/%d\n",largerdate.month,largerdate.day,\
   largerdate.year);


    system("pause");
    return 0;
}

DATES *larger(DATES *more)
{
      int days0;
      int days1;

      days0 = (more[0].month*31)+(more[0].day)+(more[0].year*365);
      days1 = (more[1].month*31)+(more[1].day)+(more[1].year*365);

      if (days1 > days0)
     return more+1;
      else
      return more;
}

Answer 4

@Rüppell'sVulture您突出显示了他的错误，但是代码过于错误。 我已经研究了一个更简单的解决方案。 看一看。

@ iMPose27请参阅以下代码，如果您遇到某些困难，请告诉我

// includes
#include <stdio.h>
#include <string.h>

// macros
#define NUM 2

// structure Definitions
struct Dates
{
       int month;
       int day;
       int year;
};

// typedefs
typedef struct Dates DATES;

// function declarations
DATES* larger(DATES[NUM]);

// function definitions
int main(int argc, char* argv[])
{
    DATES user[NUM];    // array of NUM DATES
    DATES *result=NULL;
    int i=0;
    printf("\nPlease Enter Two Dates, The program will evaluate and return the later date of the two dates passed to it\n\n");
    for(;i<NUM;i++)   
    {    
        printf("For Date %d\n",i+1);

        do{
             printf("Please enter the month, 1-12:\t");
             scanf("%d", &user[i].month);
        }while (user[i].month < 1 || user[i].month > 12);

        do{
              printf("Please enter the day, 1-31:\t");
              scanf("%d", &user[i].day);
        }while (user[i].day < 1 || user[i].day > 31);

        do{           
              printf("Please enter the year: \t");
              scanf("%d)", &user[i].year);
        }while (user[i].year < 1);

        printf("\nDate %d entered: %d/%d/%d.\n\n", i+1, user[i].month, user[i].day, user[i].year);

    } 

    if((result=larger(user))==NULL)
        printf("The two dates passed, date1: %d/%d/%d and date2: %d/%d/%d are the same.\n",user[0].month, user[0].day, user[0].year, user[1].month, user[1].day, user[1].year); 
    else
        printf("%d/%d/%d is the later date of the two dates passed\n",result->month, result->day, result->year); 

    return 0;
}

DATES* larger(DATES more[NUM])
{        
      int days0, days1;

      days0 = (more[0].month*31)+(more[0].day)+(more[0].year*365);
      days1 = (more[1].month*31)+(more[1].day)+(more[1].year*365);

      if (days0 > days1)
        return more;        
      else if (days1 > days0)
        return more+1;
      else
        return 0;       
}

Answer 5

半信半疑，这是对Rüppell的Vulture 答案的修订。 该代码很难避免其他答案中的重复-与问题中的代码相比已经减少了。

有多个更改。 由于类型仅存储单个日期，因此将其重命名为Date 。 有一个通用函数read_validate_number() ，用于处理日期的每个组成部分的输入（带有单独的提示）。 该函数返回错误/非错误状态，并通过指针参数返回值。 它在提示用户输入数字的次数上设置了上限。 它避免侮辱用户，但会报告错误的值。 该代码使用while循环，而不是do ... while循环； 通常，应避免后者。 读取数字失败后，可以读取换行符，而不仅仅是返回错误。

使用此函数，编写read_date()变得很简单。 有了read_date() ， main()函数得以简化。

我仍然不喜欢函数larger()的接口； 总的来说，我更喜欢later_date()显示的接口。 但是，该代码显示了larger()接口的一个优点。 它可以按数组中的位置标识两个日期中较大的一个，而later_date()值进行later_date() 。

main()以外的函数是静态的，因为它们不在此文件之外使用； 我使用的编译器选项需要static函数或extern声明。

总体而言，我希望输入日期时界面不太冗长。 我也更喜欢ISO 8601格式的日期，因为它们是明确的。 但是，这将是完全国际化（和本地化）代码中的用户首选项。

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

#define NUM 2
typedef struct Date
{
       int month;
       int day;
       int year;
} Date;

enum { MAX_ERRORS = 3 };

static int read_validate_number(const char *tag, int min, int max, int *value)
{
    int errors = 0;
    assert(min <= max);
    while (errors++ < MAX_ERRORS)
    {
        printf("Please enter the %s number (%d-%d): ", tag, min, max);
        if (scanf("%d", value) != 1)
        {
            printf("Failed to read number\n");
            return EOF;
        }
        if (*value >= min && *value <= max)
            return 0;
        printf("The value entered (%d) is outside the range %d-%d\n", *value, min, max);
    }
    printf("Too many errors entering %s\n", tag);
    return EOF;
}

static int read_date(Date *date)
{
    if (read_validate_number("month", 1,   12, &date->month) != 0 ||
        read_validate_number("day",   1,   31, &date->day  ) != 0 ||
        read_validate_number("year",  1, 9999, &date->year ) != 0)
        return EOF;
    return 0;
}

static Date *larger(Date *more)
{
    int days0 = (more[0].month*31)+(more[0].day)+(more[0].year*365);
    int days1 = (more[1].month*31)+(more[1].day)+(more[1].year*365);

    // Resist the temptation to write: return more + (days1 > days0);
    if (days1 > days0)
        return more+1;
    else
        return more+0;
}

static Date later_date(Date d1, Date d2)
{
    int days1 = d1.day + d1.month * 31 + d1.year * 365;
    int days2 = d2.day + d2.month * 31 + d2.year * 365;

    if (days1 > days2)
        return d1;
    else
        return d2;
}

int main(void)
{
    Date user[NUM];

    printf("Enter two dates, and the program will return the larger.\n");
    if (read_date(&user[0]) == 0 && read_date(&user[1]) == 0)
    {
        putchar('\n');
        printf("Date 1: %.4d-%.2d-%.2d\n", user[0].year, user[0].month, user[0].day);
        printf("Date 2: %.4d-%.2d-%.2d\n", user[1].year, user[1].month, user[1].day);
        Date *p_later = larger(user);
        Date  v_later = later_date(user[0], user[1]);
        printf("\nThe later date is the %s (%d/%d/%d)\n",
               (p_later == &user[0]) ? "first" : "second",
               p_later->month, p_later->day, p_later->year);
        printf("Later Date: %.4d-%.2d-%.2d\n", v_later.year, v_later.month, v_later.day);
    }

    return 0;
}

样本输出：

Enter two dates, and the program will return the larger.
Please enter the month number (1-12): 12
Please enter the day number (1-31): 25
Please enter the year number (1-9999): 2013
Please enter the month number (1-12): 1
Please enter the day number (1-31): 1
Please enter the year number (1-9999): 2012

Date 1: 2013-12-25
Date 2: 2012-01-01

The later date is the first (12/25/2013)
Later Date: 2013-12-25

我注意到，您可以通过将unsigned char （或uint8_t ）用于day和month部分，将unsigned short （或uint16_t ）用于year部分来减小Date结构的大小。 但是，您必须稍微修改一下read_date()函数才能这样做：

#include <stdint.h>

typedef struct Date
{
       uint8_t  month;
       uint8_t  day;
       uint16_t year;
} Date;

static int read_date(Date *date)
{
    int mm, dd, yyyy;
    if (read_validate_number("month", 1,   12, &mm  ) != 0 ||
        read_validate_number("day",   1,   31, &dd  ) != 0 ||
        read_validate_number("year",  1, 9999, &yyyy) != 0)
        return EOF;
    date->month = mm;
    date->day   = dd;
    date->year  = yyyy;
    return 0;
}

在某个时候，您可能想阻止某人进入2月31日（但是请记住，曾经是2月30日 -在1712年在瑞典；或者再次出现，也许您不需要记住这一点）。