C++函数返回char数组时出现返回局部变量地址警告的原因

根本原因 #

你代码中定义的bin数组是函数内的局部自动变量，这类变量分配在程序的栈内存区域，生命周期仅局限于函数执行期间。当函数执行完成返回后，对应的栈帧会被操作系统回收，这块内存的访问权限会失效，后续如果通过返回的地址访问该内存区域，属于未定义行为，可能会读到脏数据、触发段错误。编译器检测到你返回了栈上局部变量的地址，因此抛出该警告。

现有代码的其他隐含问题 #

• 二进制存储顺序是逆序的：你按n%2取到的是最低位，存在数组下标0的位置，最终输出的二进制是反过来的
• 没有添加字符串结束符\0，后续把返回值当做字符串读取时会出现越界访问

这里提供4种常见的修复思路，按C++工程实践的推荐度排序：

1. 返回std::string（最推荐） #

C++场景下优先使用标准库的字符串容器，无需手动管理内存，也不存在生命周期问题：

#include <string>
#include <algorithm>

std::string decToBin(unsigned long n) {
    if (n == 0) return "0";
    std::string bin;
    while(n) {
        bin += (char)(n%2 + '0');
        n /= 2;
    }
    std::reverse(bin.begin(), bin.end());
    return bin;
}

2. 调用者传入缓冲区 #

由调用方提前分配好存储二进制的内存空间，函数仅做写入逻辑，内存生命周期由调用方控制：

// 参数buf为调用者传入的缓冲区，buf_len为缓冲区大小
void decToBin(unsigned long n, char* buf, int buf_len) {
    if (buf == nullptr || buf_len < sizeof(n)*8 +1) return;
    int i = 0;
    if (n == 0) {
        buf[i++] = '0';
    } else {
        while(n && i < buf_len -1) {
            buf[i] = (char)(n%2 + '0');
            n /= 2;
            i++;
        }
    }
    buf[i] = '\0';
    // 逆序调整
    for(int left=0, right=i-1; left < right; left++, right--) {
        char temp = buf[left];
        buf[left] = buf[right];
        buf[right] = temp;
    }
}

3. 堆上分配内存返回 #

用new在堆区分配内存，生命周期由开发者手动控制，调用者使用完需要调用delete[]释放避免内存泄漏：

char* decToBin(unsigned long n) {
    char* bin = new char[sizeof(n)*8 +1];
    int i=0;
    if (n == 0) {
        bin[i++] = '0';
    } else {
        while(n) {
            bin[i] = (char)(n%2 + '0');
            n /= 2;
            i++;
        }
    }
    bin[i] = '\0';
    // 逆序调整
    for(int left=0, right=i-1; left < right; left++, right--) {
        char temp = bin[left];
        bin[left] = bin[right];
        bin[right] = temp;
    }
    return bin;
}

4. 局部数组加static修饰 #

将bin数组用static修饰，变量会被分配到静态存储区，生命周期和程序一致。缺点是函数不可重入，多线程场景下不安全，多次调用会覆盖上一次的结果：

char *decToBin(unsigned long n) {
    static char bin[sizeof(unsigned long)*8 +1];
    int i=0;
    if (n == 0) {
        bin[i++] = '0';
    } else {
        while(n) {
            bin[i] = (char)(n%2 + '0');
            n /= 2;
            i++;
        }
    }
    bin[i] = '\0';
    // 逆序调整
    for(int left=0, right=i-1; left < right; left++, right--) {
        char temp = bin[left];
        bin[left] = bin[right];
        bin[right] = temp;
    }
    return bin;
}

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