C语言函数返回本地字符串buffer出现乱码的解决方法

问题根因 #

你遇到的乱码问题本质是栈内存生命周期问题：

• 你代码里定义的char buffer[(digits+1)*rows];是函数内部的局部变量，分配在栈内存空间上
• 函数执行结束返回后，对应的栈帧会被操作系统回收，这块内存的内容会被后续的函数调用、局部变量操作覆盖
• 你返回的buffer指针本质是野指针，访问它的内容是未定义行为，所以会出现乱码、空内容等随机结果

另外你的代码还有一个隐藏bug：buffer没有初始化为空字符串，第一次调用strcat时目标字符串没有以\0结尾，之前能正常输出完全是巧合。

解决方案 #

方案1：用malloc在堆上分配内存（最方便适配你原有函数原型） #

堆内存的生命周期由你自己控制，只要不主动调用free就会一直有效，修改后的代码如下：

#include <stdlib.h>
#include <string.h>

const char *members(char strings[][TCAD], int rows) {
    int digits = 0;
    char *ndigits = strings[1];
    int ncommas = 0;
    while (ncommas == 0) {
        if (ndigits[digits++] == ',') ncommas++;
    }
    // 堆上分配内存，足够容纳所有卡号
    char *buffer = malloc((digits + 1) * rows);
    if (buffer == NULL) { // 内存分配失败处理
        return NULL;
    }
    buffer[0] = '\0'; // 必须初始化空字符串，否则strcat行为异常

    for (int i = 1; i < rows; i++) {
        char *member = strings[i];
        char licence[digits+1];
        strncpy(licence, &member[0], digits);
        licence[digits] = '\0';
        strcat(buffer, licence);
    }
    printf("%s", buffer);
    return buffer;
}

调用方式不变，注意main函数使用完返回的字符串后，必须调用free释放内存避免泄漏：

const char *member = members(strings, rows);
if (member != NULL) {
    printf("%s", member);
    free((void*)member); // 释放堆内存
}

方案2：调用方提前传入缓冲区（更安全，无需手动管理内存） #

把函数原型修改为接收调用方提前分配好的缓冲区，内存分配的责任交给调用方，不需要手动释放：

// 修改后的函数原型
void members(char strings[][TCAD], int rows, char *out_buffer, int buffer_size) {
    int digits = 0;
    char *ndigits = strings[1];
    int ncommas = 0;
    while (ncommas == 0) {
        if (ndigits[digits++] == ',') ncommas++;
    }
    // 检查缓冲区大小是否足够
    if (buffer_size < (digits + 1) * rows) {
        out_buffer[0] = '\0';
        return;
    }
    out_buffer[0] = '\0';

    for (int i = 1; i < rows; i++) {
        char *member = strings[i];
        char licence[digits+1];
        strncpy(licence, &member[0], digits);
        licence[digits] = '\0';
        strcat(out_buffer, licence);
    }
    printf("%s", out_buffer);
}

调用方式：

// main里提前分配足够大的缓冲区，比如用定长数组
char buffer[10240]; // 大小根据实际最大卡号数量调整
members(strings, rows, buffer, sizeof(buffer));
printf("%s", buffer);

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