C语言中GSList存储文件行数据重复指向最后一行的问题求助

你的思路完全正确！咱们先搞清楚为什么原来的代码会出问题：getline函数在第一次调用时会给s分配内存，之后的每次循环都会重用这块内存空间，每次读取新行时都会把新内容覆盖到同一个地址上。所以你往GSList里添加的其实都是同一个指针，最后自然所有元素都指向最后一行的内容了。

正确的实现方向：为每行单独分配内存 #

你想的“为每行分配新的char*内存再添加到列表”是完全正确的思路，这里有两种常见的实现方式：

1. 使用strdup（最简单快捷）

strdup会自动帮你分配足够的内存，然后把输入字符串复制进去，非常适配这种场景：

GSList* getWordList() {
    GSList *list = NULL;
    FILE *fh = fopen("words.txt", "r");
    
    // 先检查文件是否成功打开
    if (!fh) {
        g_warning("Failed to open words.txt");
        return NULL;
    }

    char *s = NULL;
    size_t len = 0;
    ssize_t read;
    
    while ((read = getline(&s, &len, fh)) != -1) {
        // 可选：去掉行尾的换行符（根据你的需求调整）
        if (read > 0 && s[read-1] == '\n') {
            s[read-1] = '\0';
        }
        // 复制字符串到新内存，再添加到列表
        char *dup_str = strdup(s);
        if (dup_str) { // 检查内存分配是否成功
            list = g_slist_append(list, dup_str);
        } else {
            g_warning("Failed to allocate memory for string");
            // 内存分配失败时，清理已分配的资源再退出
            g_slist_free_full(list, free);
            free(s);
            fclose(fh);
            return NULL;
        }
    }

    free(s); // 释放getline分配的最后一块内存
    fclose(fh);
    return list;
}

2. 手动malloc+strcpy（更灵活，适配无strdup的环境）

如果你不想依赖strdup（部分嵌入式环境可能没有这个函数），可以手动分配内存：

// 替换循环里的strdup部分：
char *dup_str = malloc(read + 1); // +1是给字符串终止符留位置
if (dup_str) {
    strncpy(dup_str, s, read);
    dup_str[read] = '\0'; // 确保字符串以终止符结尾
    // 可选：去掉换行符
    if (s[read-1] == '\n') {
        dup_str[read-1] = '\0';
    }
    list = g_slist_append(list, dup_str);
}

额外的优化建议 #

• 错误处理不能少：原代码没有检查fopen是否成功、内存分配是否失败，这些细节在实际项目里很容易导致程序崩溃，上面的示例补充了这些检查。
• 内存清理要到位：当你不再需要这个GSList时，记得用g_slist_free_full(list, free)来同时释放列表节点和每个字符串的内存，避免内存泄漏。
• 按需处理换行符：getline会把行尾的换行符也读入，如果你不需要这个换行符，记得像示例里那样处理掉。

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