C:Realloc的行为方式我不知道为什么

Question

#include <stdio.h>
#include <stdlib.h>
int main(int argc, char* argv[]){
    char buffer[103];
    char **words = malloc(1 * sizeof(*words));
    size_t counter = 0;
    size_t array_size = 2;

    for(int i = 0; i < 5; i++){
        if(!fgets(buffer, 103, stdin)){
            fputs("fgets failed", stderr);
        }
        words[counter] = buffer;
        char **more_words = realloc(words, array_size * sizeof(*more_words));
        words = more_words;
        array_size++;
        counter ++;
    }
    printf("********************************************************");

    for(int i = 0; i < 5; i++){
        printf("%sn", words[i]);
    }

}

现在这是我正在处理的简化代码。我知道我不会处理很多可能出现的错误。

关键是，当你执行这个时，单词数组似乎有 5 个"最后一个"条目的条目。

假设你给fgets ：

1
2
3
4
5

然后

words[0] = 5;
words[1] = 5;
words[2] = 5;
words[3] = 5;
words[4] = 5;

为什么不是：

words[0] = 1;
words[1] = 2;
words[2] = 3;
words[3] = 4;
words[4] = 5;

？

Answer 1

问题不在于realloc，而在于您分配给分配的指针的内容：

words[counter] = buffer;

buffer始终是同一个指针，因此您最终会将最后一个字符串读入缓冲区。

您需要malloc并复制每一行的缓冲区：

words[counter] = malloc(strlen(buffer)+1);
strcpy(words[counter], buffer);

不用说，您应该NULL - 在将其分配回 words 之前检查 realloc 返回的值。

Answer 2

if(!fgets(buffer, 103, stdin)){
        fputs("fgets failed", stderr);
}
words[counter] = buffer;

你有一个缓冲区，每次调用fgets时都会被覆盖，以便words中的所有字符串有效地指向同一个 char 数组。试试这个：

if(!fgets(buffer, 103, stdin)){
        fputs("fgets failed", stderr);
}
// here make a new buffer and copy the string just read into it.
char *new_buffer = malloc(strlen(buffer) + 1);
strcpy(new_buffer, buffer);
words[counter] = new_buffer;