Sprintf函数将unsigned long转为单个字符而非字符串的问题

问题描述 #

我尝试将值为10000000000的unsigned long类型变量converted_binary转换为字符串，但sprintf却将其转换为单个字符'1'。通过VSCode调试器可确认此问题。原本以为是malloc的问题，但即使手动创建足够大的字符数组，问题仍存在。尝试将sprintf替换为printf，发现converted_binary变量可正常在标准输出打印出10000000000。

代码片段：

int get_bit(unsigned long int n, unsigned int index)
{
        unsigned long int converted_binary, arg_int_len, int_len = 0;
        char *converted_string;
        int bit;

        /*convert n to binary*/
        converted_binary = convert(n);
        /*convert binary to string*/
        arg_int_len = converted_binary;
        do
        {
                arg_int_len = arg_int_len / 10;
                ++int_len;
        }
        while (arg_int_len != 0);
        converted_string = malloc(sizeof(char *) * int_len);
        if (converted_string == NULL)
                return (-1);
        sprintf(converted_string, "%lu", converted_binary);
        /*Loop through string to binary at index*/
        bit = (int)converted_string[index];
                /*pass that into a variable*/

        /*Return the variable*/
        return bit;
}

核心原因 #

问题出在用unsigned long存储二进制数的十进制表示的思路上：

1. 10000000000已经超过了32位unsigned long的最大值（4294967295），如果你的系统中unsigned long是32位，convert(n)返回的数值会被溢出截断，实际存储的是一个错误值（比如调试中看到的1）。
2. 即使unsigned long是64位，当二进制数的位数足够多时，对应的十进制数也会超出unsigned long的存储上限，最终还是会溢出。

你看到printf能正常输出10000000000，大概率是调试器或printf的临时处理掩盖了溢出问题，实际变量converted_binary已经是溢出后的错误值。

解决方案 #

最优方案：直接用位运算获取指定位 #

完全不需要将数值转成二进制字符串，用位运算可以直接高效获取指定位的值，避免溢出和字符串转换的问题：

int get_bit(unsigned long int n, unsigned int index)
{
    // 先判断索引是否越界（unsigned long的总位数是 sizeof(unsigned long)*8）
    if (index >= sizeof(unsigned long int) * 8) {
        return -1; // 索引超出范围，返回错误
    }
    // 假设index从0开始，0对应最低位（最右边）
    return (n >> index) & 1;
}

如果你的需求是index从最高位（最左边）开始计数，调整代码如下：

int get_bit(unsigned long int n, unsigned int index)
{
    unsigned int total_bits = sizeof(unsigned long int) * 8;
    if (index >= total_bits) {
        return -1;
    }
    // 移位到最低位后取与
    return (n >> (total_bits - 1 - index)) & 1;
}

修复原代码的问题（不推荐，仅作参考） #

如果一定要保留转字符串的逻辑，需要：

1. 修改convert函数，直接返回二进制字符串，而不是用数值存储。
2. 修复malloc的错误：需要为字符串结束符\0分配额外空间，正确写法是malloc(int_len + 1)，原代码中sizeof(char *)是错误的，应该用sizeof(char)（虽然char是1字节，结果一致，但逻辑错误）。

额外说明 #

原代码中sprintf写入的字符串没有结束符\0，会导致后续操作（比如访问字符串）出现未定义行为，这也是需要修复的潜在问题。

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