无法重新迭代指针数组

Question

我有一个从文件中读取的项目指针数组。这些项目是typedef structs。

我的数组可以调整大小，因为文件包含很多项目。

while (fscanf(fp, "%X %[^\n]", &code, string) != EOF) {
    if (list->length >= list->capacity) {
        list->capacity *= 2;
        list->items = (Item**)realloc(list->items,
            list->capacity * sizeof(Item*));
    }
    item.code = code;
    strcpy(item.string, string);
    list->items[list->length++] = &item;
}

假设 list 和 item 已经声明。空间已分配给列表。

static List * list; & Item item = { 0, { 0 } };

在代码中较早地声明和初始化。 list 是必要的全局变量。

如果我尝试通过尝试获取第一个值来迭代这个已完成的列表：

printf("line %d: %X\t%-89s\n", 1, list->items[0]->code, list->items[0]->string);

我将从这里获取文件中的最后一个元素。但是，如果我将这个printf 包含在上述while 循环中，我将获得所需的控制台输出（当索引为list->length 时）。

为什么我会遇到这种行为？关于遍历指针有什么我忘记了吗？

Answer 1

您似乎将item 用作自动Item 对象，因此用指向同一Item 对象的指针填充list->items 数组。

您可能希望在每个循环中都有一个新的Item。喜欢：

Item* newItem;
while (fscanf(fp, "%X %[^\n]", &code, string) != EOF) {
    newItem = malloc(sizeof *newItem);
    if (!newItem) exit(1);

    if (list->length >= list->capacity) {
        list->capacity *= 2;
        list->items = (Item**)realloc(list->items,   // BTW DONT DO THIS - SEE BELOW
            list->capacity * sizeof(Item*));
    }
    newItem->code = code;
    strcpy(newItem->string, string);
    list->items[list->length++] = newItem;
}

顺便说一句：总是 realloc 指向一个 tmp 指针，例如：

tmp = realloc(....);
if (tmp == NULL)
{
    // error handling
}
else
{
    list = tmp;
}

进一步：在 C 语言中，通常不建议使用 cast malloc/ realloc / calloc。使用sizeof *pointer 而不是sizeof(type) 也很常见

所以而不是：

list->items = (Item**)realloc(list->items, list->capacity * sizeof(Item*));

你可以这样做：

tmp = realloc(list->items, list->capacity * sizeof *tmp);

其中tmp 与list->items 的类型相同