时间戳

首页 > 解决方案 > C中的问题与重复元素的字符串数组

c - C中的问题与重复元素的字符串数组

问题描述

我正在尝试获取未知数量的控制台输入行，使用 malloc 将每一行转换为字符串，然后通过每次动态重新分配内存将每个字符串添加到字符串数组中。我的目标是有一个数组，其中每个元素都是控制台输入的不同行（while 循环应该以 EOF 结尾）。

我的代码如下。

char * inputString = (char *)malloc(100);
char * aWord = (char *)malloc(1);
char ** listWords = (char **)malloc(1);
int wordCount = 0;

while (fgets(inputString, 100, stdin) != NULL)
{       
    wordCount++;
    *(inputString + (strlen(inputString) - 1)) = '\0';
    aWord = (char *)realloc(aWord, strlen(inputString) + 1);
    aWord = inputString;

    listWords = realloc(listWords, sizeof(char) * wordCount);
    *(listWords + (wordCount - 1)) = aWord;
}

for (int i = 0; i < wordCount; i++)
    printf("%s\n", listWords[i]);

如果我要在控制台中输入

abc\n
b\n
cad\n
^Z\n

理论上，我希望我的代码打印出来

abc\n
b\n
cad\n

每行控制台输入。相反，它打印

cad\n
cad\n
cad\n

最后一行输入。非常感谢帮助

标签： carrayspointersfgets

解决方案

首先是这样的：

aWord = (char *)realloc(aWord, strlen(inputString) + 1);
aWord = inputString;

在这里您为 分配空间aWord，然后用 中包含的地址覆盖返回的地址inputString。因此，您读取的每个字符串最终都会覆盖前一个字符串。您还应该使用malloc而不是realloc因为您想获得一个新的缓冲区而不是重用现有的缓冲区。

更改对 strcpy 的realloc调用malloc and instead of assigning one pointer to the other, use以复制它。

aWord = malloc(strlen(inputString) + 1);
strcpy(aWord, inputString);

然后在这一行：

listWords = realloc(listWords, sizeof(char) * wordCount);

您只需为字符数组而不是字符指针数组分配足够的内存。这导致写入超过分配内存的末尾，调用undefined beahvior。

正确的分配是：

listWords = realloc(listWords, sizeof(char *) * wordCount);

推荐阅读