c - 数组被覆盖时遇到麻烦

我试图将拼字字典加载到内存中，但是当我打印出字典的内容时，它似乎用buf看到的最后一个字符串重写了数据数组中的每个字符串，例如，当我打印出最后一个数据数组时，数组中的每个条目都是'zzz'，因为它是scrabble.txt中的最后一个条目。我不明白为什么要重写？我做错什么了？

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

#define LEN 100

struct data_t {
    int nval;       /* current number of values in array */
    int max;        /* allocated number of values */
    char **data;        /* the data array */
};

enum {INIT = 1, GROW = 2};

int main(void)
{
    FILE *fp = fopen("scrabble.txt", "r");
    char buf[LEN];
    int i = 0;
    struct data_t *data = malloc(sizeof(struct data_t));
    data->nval = INIT;
    data->max = INIT;
    data->data = NULL;
    while (fgets(buf, LEN, fp)) {
            if (data->data == NULL) {
                    data->data = malloc(LEN);
                    assert(data->data);
            }
            else if (data->nval > data->max) {
                    data->data = realloc(data->data, GROW * data->max * LEN);
                    assert(data->data);
                    data->max = GROW * data->max;
            }
            //printf("%s\n",buf);
            data->data[i] = buf;
            //printf("data->data[%d] = %s ", i, data->data[i]);
            i++;
            data->nval++;
    }
    /* overcounted */
    data->nval--;

    for(i = 0; i < data->nval; i++)
            printf("data->data[%d] = %s", i, data->data[i]);

    fclose(fp);
    free(data->data);
    return 0;
}

最佳答案

你写道：

data->data[i] = buf;

这使得指针指向缓冲区。每次在循环中都会这样做，因此所有指针都指向data->data[i]。因此，当您输出每个指针指向的内容时，它会多次输出buf的内容。
您可能想为每一行使用不同的缓冲区。您必须使用buf或相关来分配这些缓冲区。例如，将buf替换为：

data->data[i] = strdup(buf);

（malloc不是标准的C函数，但如果您的系统没有它，您可以data->data[i] = buf;然后strdup）。

关于c - 数组被覆盖时遇到麻烦，我们在Stack Overflow上找到一个类似的问题：https://stackoverflow.com/questions/29335910/