C语言中结构体数组无法显示正确值问题求助

嘿，看起来你在处理动态内存分配、指针和结构体的时候碰到了异常结果，我先帮你梳理下现有代码里可能的问题点，再给你一些排查方向：

你提供的代码片段： #

#include <stdio.h>
#include <stdlib.h>
#include <time.h>
#include <string.h>
#define NEW_POINT(x) (POINT *)malloc(x*sizeof(POINT))
#define EXTEND(ptr,size) (POINT *)realloc(ptr,size*sizeof(POINT))
typedef struct p{ int x; int y; }POINT;
POINT *g;
void show(POINT *s){ printf("x = %d\ty = %d\n",s->x,s->y );
}
POINT* getPoint() { POINT *t = NEW_POINT(1); t->...

可能导致异常的几个关键点： #

• 宏定义的不必要强制转换：在C语言中，malloc和realloc返回的void*可以隐式转换为任意指针类型，强制转换反而可能掩盖潜在的编译警告（虽然你已经包含了stdlib.h，但这个习惯并不推荐）。建议修改宏定义，去掉强制转换：

  #define NEW_POINT(x) malloc(x*sizeof(POINT))
  #define EXTEND(ptr,size) realloc(ptr,size*sizeof(POINT))
  

• 未初始化的结构体成员：NEW_POINT(1)只是分配了内存，但并没有初始化结构体的x和y成员，此时它们的值是随机的垃圾数据。如果getPoint函数直接返回这个未初始化的指针，后续调用show打印时就会出现异常结果。你需要在getPoint里给t->x和t->y赋值，比如：

  POINT* getPoint() { 
      POINT *t = NEW_POINT(1); 
      if (t == NULL) { // 一定要检查内存分配是否成功！
          perror("malloc failed");
          return NULL;
      }
      t->x = 10; // 或者用rand()生成随机值，记得先调用srand(time(NULL))初始化随机数种子
      t->y = 20;
      return t;
  }
  

• realloc的安全问题：EXTEND宏直接返回realloc的结果，但如果realloc失败会返回NULL，这时候原来的指针会被覆盖成NULL，导致原有内存丢失。正确的做法是先用临时指针接收realloc的结果，检查成功后再赋值：

  // 相比宏，写成函数更安全易维护
  POINT* extend_points(POINT *ptr, size_t new_size) {
      POINT *temp = realloc(ptr, new_size * sizeof(POINT));
      if (temp == NULL) {
          perror("realloc failed");
          return ptr; // 返回原指针，避免内存丢失
      }
      return temp;
  }
  

• 全局指针的内存管理：全局指针g如果被多次赋值，记得先释放之前分配的内存，否则会造成内存泄漏。比如：

  if (g != NULL) {
      free(g);
      g = NULL;
  }
  g = getPoint();
  

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