C语言结构体存取时save函数写文件留空不写入有效数据问题排查

核心问题定位 #

所有导致输出文件空白的bug均可独立复现，逐一拆解如下：

• 结构体数组定义失效
  原代码中struct Bird birds;仅定义了单个结构体变量，并非可存储多条记录的数组。后续load、save函数中使用birds[count]、birds[i]的下标访问属于越界操作，读取到的全是未初始化的零值内存，所有字符串成员都是空串，写入文件时没有有效字符内容，这是输出文件空白的最核心原因。
• load函数返回值逻辑错位
  原逻辑中文件打开失败时返回1，正常加载完成返回实际记录数count，导致main函数中if (count == 0)判断加载失败的逻辑完全失效——哪怕文件根本打不开，返回的1也不会触发空加载提示，错误状态被完全隐藏。
• save函数硬编码循环边界
  原save函数中写死循环执行3次，没有接收实际加载完成的记录数参数：如果实际加载的记录数少于3，循环会访问未初始化的空内存；如果加载记录数超过3，又会漏写有效数据。
• 冗余无效参数设计
  main中传递给save的oput是未初始化的野指针，虽然save函数内部自行重新打开了输出文件，但这个多余的参数很容易引发空指针访问问题。
• 语法书写错误
  main函数中else分支使用圆括号包裹代码块，不符合C语言语法要求，正常编译会触发报错。
• 函数缺失返回值
  原save函数声明返回int类型，但函数末尾没有return语句，编译时会触发返回值逻辑告警，部分平台下可能出现文件写入缓冲区未正常刷新，直接导致文件内容为空。

修正后可运行代码 #

#include <stdio.h>
#define MAX_BIRD_RECORDS 100

struct BirdHome{
  char area[500];
  char heightcm[100];
  char feederquantity[10];
  char hasNest[5];
};
struct Bird{
  char isRinged[5];
  char nameSpecies[50];
  char birdAgeMonths[500];
  struct BirdHome hom;
  char gender[6];
};
// 定义可存储MAX_BIRD_RECORDS条记录的结构体数组
struct Bird birds[MAX_BIRD_RECORDS];

// 前置声明函数，避免编译告警
int load(char *filename, struct Bird *birds, int max);
int save(struct Bird *birds, int validCount);

int main(void){
  int max = MAX_BIRD_RECORDS;
  int count = 0;
  char filename[100];
  printf("file name? : ");
  scanf("%s", filename);
  count = load(filename, birds, max);
  if (count == 0) {
    printf("No structures loaded\n");
  } else {
    printf("Data loaded, total %d records\n", count);
  }
  // 传入实际加载的有效记录数，移除无效野指针参数
  save(birds, count);
  return 0;
}

int load(char *filename, struct Bird *birds, int max){
  int count = 0;
  FILE *fp = fopen(filename, "r");
  char line[100 * 4];
  // 修正返回逻辑：文件打开失败返回0，触发空加载提示
  if (fp == NULL) {
    return 0;
  }
  while (count < max && fgets(line, sizeof(line), fp) != NULL){
    sscanf(line, "%s %s %s %s %s %s %s %s", 
      birds[count].isRinged, 
      birds[count].nameSpecies,
      birds[count].birdAgeMonths, 
      birds[count].hom.area,
      birds[count].hom.heightcm, 
      birds[count].hom.feederquantity,
      birds[count].hom.hasNest,
      birds[count].gender);
    count++;
  }
  fclose(fp);
  return count;
}

int save (struct Bird *birds, int validCount){
  int i;
  FILE *oput = fopen("birdssave.txt","w");
  if (oput == NULL) {
    return -1;
  }
  // 按照实际有效记录数循环，不再硬编码次数
  for (i = 0; i < validCount; i++){
    fprintf(oput,"%s %s %s %s %s %s %s %s\n",
      birds[i].isRinged, 
      birds[i].nameSpecies,
      birds[i].birdAgeMonths, 
      birds[i].hom.area,
      birds[i].hom.heightcm, 
      birds[i].hom.feederquantity,
      birds[i].hom.hasNest,
      birds[i].gender);
  }
  fclose(oput);
  return 0;
}

验证说明 #

使用提供的3行测试文本作为输入源文件运行修正后的程序，生成的birdssave.txt会完整写入3条匹配的鸟类监测记录，不会出现空白占位的问题。

内容的提问来源于stack exchange，提问作者user15199447