C中strcat()的分段错误

Question

// Processes the array, starting from the second element
int j, k;
char *doubled;
for (j = 1; j < 500; j++) {
strcpy(doubled, output[j]);
strcat(doubled, doubled);
for (k = 0; k < j; k++) {
if (strcmp(output[k], output[j]) == 0) {
output[j] = doubled;
}
if (strcmp(output[k], doubled) == 0) {
output[j] = '';
}
}
}

尝试处理一个字符串数组，每当某个特定字符串出现两次时，第二次出现会打印该字符串的副本(例如dog-dog---->dog-dog(，如果一个字符串出现两个以上，则删除该字符串(例如dog dog-dog--->dog dog(。

我试过调试，发现问题出在这段代码中，我一直在那里得到分段故障报告。

我该怎么办才能解决这个问题？我已经研究了strcat((创建分段错误的几种解决方案，但似乎都不起作用。

Answer 1

编辑：根据注释，如果cstrings的原始数组没有为每个字符串分配足够的内存，那么最好的解决方案可能是构造另一个空数组，每个字符串的内存是分配内存的两倍，并将旧数组复制到新的空数组中，如下所示：

int count = 500;
int temp_size = 0;
char** new_array = malloc(count * sizeof(char*));
for(int i = 0; i < count; i++) {
temp_size = strlen(output[i]);
new_array[i] = malloc(((temp_size * 2) + 2) * sizeof(char)));
strcpy(new_array[i], output[i]);
}

使用malloc 动态创建字符串数组

通过在for循环中声明变量，可以避免很多麻烦，然后它们的生存期和范围仅限于for循环。在你的情况下，我认为这是有效的(见评论和编辑(：

int size = 0;
for (j = 1; j < count; j++) {
size = strlen(output[j]);        //get the size of output[j]
char doubled[(size * 2) + 1];    //make cstring twice size + space for ''
strcpy(doubled, output[j]);      //put one copy in the temporary buffer
strcat(doubled, output[j]);      //concatenate next copy like this
for (k = 0; k < j; k++) {
if (strcmp(output[k], output[j]) == 0) {
strcpy(new_array[j], doubled);     //make changes to new_array
}
if (strcmp(new_array[k], doubled) == 0) {
char deleted[1] = "";            //needed for free to work 
strcpy(new_array[j], deleted);     //otherwise runtime error  
}
}
}

在循环中声明变量，好的做法还是坏的做法？

当使用new_array时，也需要删除：

for (int i = 0; i < count, i++) {
free(new_array[i]);
}
free(new_array);
new_array = NULL;