如何修改结构体数组中单个变量且不影响其他结构体实例？

Hey，我一眼就看出问题出在哪了——你修改单个结构体对象时其他也跟着变，核心原因是结构体里的字符串指针（char *S、char *N、char *Name这些）指向了同一块内存区域！

问题根源 #

你的结构体里的字符串成员都是指针类型，从你贴的代码来看，你应该是把同一个缓冲区（比如循环里的name变量）的地址直接赋值给了每个结构体的Name（以及S/N/W/E）指针。这就导致所有结构体对象的这些指针都指向同一块内存：

• 每次用fscanf更新name的内容时，所有结构体的Name都会同步变成新值
• 你修改任意一个结构体的字符串成员时，本质是在修改共享的那块内存，自然会影响所有指向它的结构体

解决办法 #

1. 为每个字符串成员分配独立内存

不要直接赋值指针，而是复制字符串的内容到新分配的内存里，让每个结构体的字符串成员拥有自己独立的空间。有两种常用方式：

• 用strdup()（POSIX标准函数，自动分配内存并复制字符串）：

  // 假设读取到的name是有效字符串
  rooms[numOfRooms].Name = strdup(name);
  // 同理处理S、N、W、E
  rooms[numOfRooms].S = strdup(south_input);
  

• 如果是Windows环境（strdup可能不可用），手动用malloc+strcpy：

  // 计算需要的内存长度（+1是为了存储字符串结束符'\0'）
  size_t name_len = strlen(name) + 1;
  char *name_copy = malloc(name_len);
  if (name_copy != NULL) {
      strcpy(name_copy, name);
      rooms[numOfRooms].Name = name_copy;
  }
  

2. 修正你的赋值逻辑

检查循环里的代码，绝对不能把循环复用的缓冲区（比如name）的地址直接赋值给结构体的指针成员。必须像上面那样，每次都复制一份独立的内容。

3. 记得释放内存

因为手动分配了内存，程序结束或不再使用结构体数组时，要逐个释放每个字符串成员的内存，最后再释放结构体数组本身，避免内存泄漏：

for (int i = 0; i < numOfRooms; i++) {
    free(rooms[i].S);
    free(rooms[i].N);
    free(rooms[i].W);
    free(rooms[i].E);
    free(rooms[i].Name);
}
free(rooms);

额外提示 #

你的realloc逻辑没问题，但要记得检查返回值——如果realloc返回NULL，说明内存分配失败，这时候原来的rooms指针还是有效的，别直接覆盖它，不然会造成内存泄漏：

struct room *temp_rooms = realloc(rooms, sizeof(*rooms)*(numOfRooms+1));
if (temp_rooms != NULL) {
    rooms = temp_rooms;
} else {
    // 处理内存分配失败的情况，比如打印错误信息
    fprintf(stderr, "Failed to allocate memory for rooms\n");
    // 可以选择释放已分配的内存后退出
    // ...
}

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