为何数组内的所有指针指向同一字符串?求解决方法
问题描述
通过getline函数读取文件行,将包含字符'X'的行存入char *array_slave[128]数组中。经array_length函数确认数组存有32行数据,但后续打印数组元素时,所有输出均为同一行。误以为是循环中i++操作导致指针偏移,需明确问题根源及修正方法。
相关代码如下:
char *array_slave[128]; int i = 0; while ((read = getline(&line, &len, fp)) != -1) { if (strstr(line, "X") != NULL) { array_slave[i] = line; printf("%s\n",array_slave[i]); i++; } }
确认数组长度的代码:
size_t array_length(char *ptr[]) { size_t i=0; while(ptr[i]!=NULL){ //printf("%d\n", i); //printf("%s\n",ptr[i]); i++; } return i; }
打印代码:
for(int i=0; i<10;i++){ printf("%s\n", array_slave[i]); }
问题根源
核心是指针复用错误:
getline每次调用时,会复用或重新分配line指向的内存空间,将新读取的行内容覆盖到这块内存里。- 代码中
array_slave[i] = line只是把line的内存地址存入数组,并没有复制字符串内容。 - 循环结束后,数组所有元素都指向同一块内存,这块内存里存的是最后一次
getline读取的行,所以打印时全是同一行。
解决方案
需要为每个符合条件的行单独分配内存、复制字符串内容,而非直接存储指针:
方法1:用strdup快速实现
strdup会自动分配足够内存并复制字符串,返回新指针:
char *array_slave[128]; int i = 0; while ((read = getline(&line, &len, fp)) != -1) { if (strstr(line, "X") != NULL) { array_slave[i] = strdup(line); // 检查内存分配是否成功 if (array_slave[i] == NULL) { perror("strdup failed"); // 释放已分配内存并退出 for (int j = 0; j < i; j++) free(array_slave[j]); fclose(fp); exit(EXIT_FAILURE); } printf("%s\n",array_slave[i]); i++; // 避免数组越界 if (i >= 128) break; } } // 添加NULL终止符,让array_length能正确计算长度 array_slave[i] = NULL;
方法2:手动malloc+strcpy(兼容无strdup的系统)
// 循环内替换为以下代码: array_slave[i] = malloc(strlen(line) + 1); // +1用于存储字符串结束符'\0' if (array_slave[i] == NULL) { // 错误处理同方法1 } strcpy(array_slave[i], line);
额外注意
- 必须给
array_slave添加NULL终止符,否则array_length会因找不到终止条件越界访问,引发未定义行为。 - 程序结束前,要逐个
free数组元素指向的内存,避免内存泄漏。
内容的提问来源于stack exchange,提问作者RDP1998
相关产品推荐
相关产品推荐

