strcpy函数无法正常工作：字符串数组拷贝后全数组被覆盖问题咨询

嘿，这个坑我之前踩过好几次！你遇到的问题本质是所有MsgList[i].ga_data都指向了同一块内存地址，每次调用strcpy都会把新内容覆盖到这块内存里，最后整个数组自然看起来全是最后拷贝的那个字符串。

为什么会出现这个问题？ #

大概率是你的MsgList结构体中ga_data是一个char*指针，但你没有为每个MsgList[i]的ga_data分配独立的内存空间。举个典型的错误场景：

// 错误示例：所有元素共享同一块缓冲区
typedef struct {
    char* ga_data;
} MsgItem;

MsgItem MsgList[4];
char shared_buf[256]; // 全局/共享的缓冲区

// 初始化时让所有ga_data都指向同一个缓冲区
for (int i = 0; i < 4; i++) {
    MsgList[i].ga_data = shared_buf;
}

每次strcpy都会把内容写入shared_buf，之前的内容被直接覆盖，最后所有MsgList[i].ga_data自然都显示最后一次拷贝的结果。

怎么解决？ #

核心是给每个MsgList[i].ga_data分配独立的内存，再拷贝字符串：

方案1：用malloc+strcpy手动分配

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

// 假设你的数组长度是4，根据实际情况调整
for (int i = 0; i < 4; i++) {
    // 计算需要的内存：源字符串长度 + 1（存储字符串终止符'\0'）
    size_t str_len = strlen(a_Database[i].ga_data);
    MsgList[i].ga_data = (char*)malloc(str_len + 1);
    
    // 一定要检查内存分配是否成功，避免空指针问题
    if (MsgList[i].ga_data == NULL) {
        perror("malloc failed");
        // 这里可以根据业务逻辑处理错误，比如退出程序
        exit(EXIT_FAILURE);
    }
    
    // 拷贝字符串到新分配的独立内存
    strcpy(MsgList[i].ga_data, a_Database[i].ga_data);
}

方案2：用strdup简化操作（推荐）

strdup会自动完成内存分配+字符串拷贝的操作（注意：strdup是POSIX标准，不是C标准，但绝大多数编译器都支持）：

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

for (int i = 0; i < 4; i++) {
    MsgList[i].ga_data = strdup(a_Database[i].ga_data);
    
    if (MsgList[i].ga_data == NULL) {
        perror("strdup failed");
        exit(EXIT_FAILURE);
    }
}

额外注意事项 #

• 使用完MsgList后，记得遍历每个元素释放内存，避免内存泄漏：

for (int i = 0; i < 4; i++) {
    free(MsgList[i].ga_data);
    MsgList[i].ga_data = NULL; // 避免野指针
}

• 如果MsgList的ga_data是固定大小的字符数组（比如char ga_data[256];），要确保数组大小足够容纳源字符串，否则会出现缓冲区溢出问题，这时候可以用strncpy代替strcpy来保证安全。

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