c-字符串的动态数组

Question

我必须动态分配单词数组。单词存储在一个文件中，由可变数量的空白字符分隔。我不知道文件中有多少单词，它们可以有可变的长度。

我有这个代码：

void readWord(FILE* stream, char *word, char first_c) {
word[0] = first_c;
char val;
int wlen = 1;
// isWhitespac is my function - tests if char is blank or 'n'
while ((val = fgetc(stream)) != EOF && isWhitespace(val) == 0) {
wlen++;
word = realloc(word, (wlen+1) * sizeof (char));
word[wlen-1] = val;    
}
word[wlen] = '';
}
int readList(const char *file) {
FILE* f;
char **arr;
char val;
int wcount = 0;
arr = malloc(sizeof (char*));
f = fopen(file, "r");
while (fscanf(f, " %c", &val) == 1) {
wcount++;
arr = realloc(arr, wcount * sizeof (char *));
arr[wcount - 1] = malloc(sizeof (char));  
readWord(f, arr[wcount-1], val); 
printf("%sn", arr[wcount-1]);
}

for (int i = 0; i < wcount; ++i) {
free(arr[i]);
}
free(arr);
fclose(f);
return 0;
}

它看起来工作得很好，它读取并打印所有的单词。但当我用Valgrind运行程序时，错误太多了，我找不到。有人能帮我吗？(我知道我必须测试malloc和其他功能是否正常，这只是一个测试函数。)

Valgrind日志很长，我应该也张贴吗？

Answer 1

问题之一是在readWord中执行realloc。如果realloc分配了一个新的缓冲区，而不只是扩展当前的缓冲区时，那么你的代码就会崩溃(你会双倍释放指针)，这就是Valgrind所采用的。为了解决这个问题，我会重写代码，使其返回一个指针而不是void。

char * readWord(FILE* stream, char *word, char first_c) {
word[0] = first_c;
char val;
int wlen = 1;
// isWhitespac is my function - tests if char is blank or 'n'
while ((val = fgetc(stream)) != EOF && isWhitespace(val) == 0) {
wlen++;
word = realloc(word, (wlen+1) * sizeof (char));
word[wlen-1] = val;
}
word[wlen] = '';
return word;
}

然后将readList中的循环更改为：

while (fscanf(f, " %c", &val) == 1) {
wcount++;
arr = realloc(arr, wcount * sizeof (char *));
arr[wcount-1]=malloc(sizeof(char));
arr[wcount - 1] = readWord(f, arr[wcount-1],  val);
printf("%sn", arr[wcount-1]);
}