C - typedef，在两个函数中找不到错误

Question

我试图用C语言制作一个程序，让我可以存储一些关于学生的信息。这个想法是创建一个新的数据类型，然后用它创建一些函数来处理数据。特别是有一次，我必须找到一个小组中年龄最大的学生，然后使用另一个函数打印出其详细信息。我做的这个函数应该是用来查找组中的最大年龄，但它不起作用。有人能找出错误吗？特别是我认为不起作用的函数是calcmax和find。没有错误，但当我在main中调用calcmax并试图打印结果时，会出现一些随机数，仅此而已。我认为其中一个循环中的逻辑有问题，但我找不到。非常感谢。

typedef struct{
char f[MAXIM];
char s[MAXIM];
char id[MAXIM];
char dep[MAXIM];
char g;
int age;
}student_t;
student_t mkstud(char first[],char sec[], char idn [],char de[], char sex, int eta){
student_t person;
strcpy(person.f,first);
strcpy(person.s,sec);
strcpy(person.id,idn);
strcpy(person.dep,de);
person.g = sex;
person.g = eta;
return person;
}
student_t scstud(){
char first[MAXIM],sec[MAXIM],idn[MAXIM],de[MAXIM];
char sex;
int eta;
student_t person;
printf("name ");
scanf("%s",first);
printf("surname ");
scanf("%s",sec);
printf("id ");
scanf("%s", idn);
printf("depart ");
scanf("%s",de);
printf("sex ");
scanf(" %c",&sex);
printf("age ");
scanf(" %d", &eta);
person= mkstud(first,sec,idn,de,sex,eta);
return person;
}

int calcmax(student_t *class, int length){
int diff,i;
int max[MAXIM];
diff=class[0].age-class[1].age;
if(diff>=0){
    max[0]=class[0].age;
}
else if(diff<0){
    max[0]=class[1].age;
}
max[1]=max[0];
for (i=0;i<length;i++){
    if (max[i]>=class[i+1].age){
        max[i+1]=max[i];
    }
    else if(max[i]<class[i+1].age){
        max[i+1]=class[i+1].age;
    }
}
    return max[length];
}
void printdetails(student_t std){
printf("name: %s",std.f);
printf("surname: %s",std.s);
printf("ID: %s",std.id);
printf("department: %s",std.dep);
printf("gender: %c",std.g);
printf("age: %d",std.age);
}
/*void find(student_t *class, int length){
int std,i;
std = calcmax(class, length);
for(i=0;i<=length-1;i++){
    if (std==class[i].age) {
         printdetails(class[i]);
    }
}
}*/
int main(void){
int i,j;
student_t class[6];
class[0]= mkstud("John","Bishop","s1234","inf", 'm',18);
class[1]=mkstud("Lady","Cook","s2345","Eng",'f',21);
class[2]=mkstud("James","Jackson","s3456","Eng",'m',17);
for(i=4;i<=6;i++){
    printf(" student %d n", i);
    class[i-1]=scstud();
}
j=calcmax(class, 6);
printf("%d", j);

/*find(class, 3);*/
return EXIT_SUCCESS;}

Answer 1

我看到的问题：

1. mkstud中的小错误。您正在为结构的g成员指定性别和年龄。

   person.g = sex;
   person.g = eta;
   

   应该是：

   person.g = sex;
   person.age = eta;
   

2. 您尚未在calcmax中初始化数组max。

3. 你在calcmax中的逻辑有缺陷。如果在正确的位置添加printf语句，您将能够看到max[length]被分配了一个不正确的值。

   int calcmax(student_t *class, int length){
      int diff,i;
      int max[MAXIM] = {0};
      diff=class[0].age-class[1].age;
      if(diff>=0){
         max[0]=class[0].age;
      }
      else if(diff<0){
         max[0]=class[1].age;
      }
      max[1]=max[0];
      // ADDED
      printf("max[0]: %dn", max[0]);
      printf("max[1]: %dn", max[1]);
      for (i=0;i<length;i++){
         if (max[i]>=class[i+1].age){
            max[i+1]=max[i];
            // ADDED
            printf("max[%d]: %dn", i+1, max[i+1]);
         }
         else if(max[i]<class[i+1].age){
            max[i+1]=class[i+1].age;
            // ADDED
            printf("max[%d]: %dn", i+1, max[i+1]);
         }
      }
      // ADDED
      printf("max[%d]: %dn", length, max[length]);
      return max[length];
   }
   

   你可以尝试修复这个相当复杂的函数，但它可以被简化。

   int calcmax(student_t *class, int length){
      if ( length <= 0 )
      {
         return -1;
      }
      int max = class[0].age;
      for (int i = 1; i < length; ++i )
      {
         if ( max < class[i].age )
         {
            max = class[i].age;
         }
      }
      return max;
   }