C语言函数仅返回地址而非字符串内容，指针错误排查求助

首先，咱们直接揪出最核心的指针问题：
你函数里定义的char words[20];是栈上的局部变量。C语言中，函数执行时会在栈上为局部变量开辟临时空间，一旦函数执行完毕返回，这块栈空间就会被操作系统回收复用——也就是说，words数组占用的内存已经不属于你的程序了。你返回的是这个数组的首地址，这就得到了一个野指针，后续用这个地址去访问内存，结果完全不可控（可能输出乱码、程序崩溃，甚至触发更严重的未定义行为）。

为什么会这样？ #

栈内存的生命周期和函数调用严格绑定：每个函数调用都会创建一个“栈帧”，用来存放局部变量、函数参数和调用上下文。函数返回时，栈帧被销毁，里面的所有局部变量都会被清理，对应的内存地址不再有效。返回局部栈变量的地址是C语言里非常常见的新手坑，本质是对内存生命周期的理解不到位。

除了指针问题，你的代码还有几个小漏洞（顺便一起指出来）：

• 没检查fopen的返回值，如果text.txt不存在或者打不开，后续操作直接崩溃；
• 统计行数后直接用rand()%10，如果文件行数少于10，会得到比实际行数大的随机数，导致循环逻辑出错；
• 读取目标行的循环逻辑有问题：for(i=ran; i<j; i++)会把从第ran行到最后一行都读一遍，最后返回的是文件最后一行，而不是你想要的第ran行。

修复方案 #

这里给你两种常用的解决思路，按需选择：

方案1：用动态内存分配（malloc）

把局部数组换成堆内存分配，堆内存不会随函数返回自动释放，需要你手动用free回收（避免内存泄漏）。修改后的代码如下：

char* randstring(){
    // 分配20字节的堆内存，记得检查分配是否成功
    char *words = malloc(20 * sizeof(char));
    if (words == NULL) {
        perror("malloc failed");
        return NULL;
    }

    FILE *fp = fopen("text.txt", "r");
    if (fp == NULL) { // 必须检查文件是否打开成功
        perror("fopen failed");
        free(words); // 已经分配的内存要提前释放
        return NULL;
    }

    int line_count = 0;
    // 先统计文件总行数
    while (fgets(words, 20, fp) != NULL) {
        line_count++;
    }

    if (line_count == 0) { // 处理空文件的情况
        fclose(fp);
        *words = '\0';
        return words;
    }

    // 生成0到line_count-1之间的随机行号
    srand(time(NULL));
    int target_line = rand() % line_count;

    // 回到文件开头，跳转到目标行
    rewind(fp);
    for (int i = 0; i < target_line; i++) {
        fgets(words, 20, fp); // 跳过前面的行
    }
    // 读取目标行
    fgets(words, 20, fp);

    fclose(fp);
    return words;
}

使用注意：调用这个函数后，一定要记得用free(返回的指针)来释放内存，否则会造成内存泄漏。比如：

char *result = randstring();
if (result != NULL) {
    // 写入文件等操作
    FILE *out_fp = fopen("output.txt", "w");
    if (out_fp != NULL) {
        fputs(result, out_fp);
        fclose(out_fp);
    }
    free(result); // 务必释放！
}

方案2：让调用者提供缓冲区

这种方式不需要动态分配内存，由调用者负责管理内存空间，更安全也避免了内存泄漏的麻烦。修改后的函数：

// buf：调用者提供的缓冲区；buf_size：缓冲区的大小
char* randstring(char *buf, size_t buf_size){
    if (buf == NULL || buf_size < 1) {
        return NULL; // 参数合法性检查
    }

    FILE *fp = fopen("text.txt", "r");
    if (fp == NULL) {
        perror("fopen failed");
        return NULL;
    }

    int line_count = 0;
    char temp_buf[20];
    // 统计行数，用临时缓冲区不影响调用者的buf
    while (fgets(temp_buf, sizeof(temp_buf), fp) != NULL) {
        line_count++;
    }

    if (line_count == 0) {
        fclose(fp);
        *buf = '\0';
        return buf;
    }

    srand(time(NULL));
    int target_line = rand() % line_count;

    rewind(fp);
    for (int i = 0; i < target_line; i++) {
        fgets(temp_buf, sizeof(temp_buf), fp);
    }
    // 把目标行读取到调用者的缓冲区里
    fgets(buf, buf_size, fp);

    fclose(fp);
    return buf;
}

调用示例：

int main() {
    char result[20];
    if (randstring(result, sizeof(result)) != NULL) {
        printf("随机选取的单词：%s", result);
        // 写入文件
        FILE *out_fp = fopen("output.txt", "w");
        if (out_fp != NULL) {
            fputs(result, out_fp);
            fclose(out_fp);
        }
    }
    return 0;
}

这种方式的好处是内存管理更简单，不需要手动调用free，适合小型程序场景。

内容的提问来源于stack exchange，提问作者HQ189