C代码消息转二进制位输出异常：输出32位而非预期8位

问题根源 #

1. 函数声明与定义不匹配：代码声明了int printbulbs(int ascii_input);，但实际实现的函数是printb，属于笔误，会导致编译警告甚至运行异常。
2. 二进制位数处理错误：printb函数用sizeof(int)*8计算总位数，而多数系统中int是4字节（32位），但ASCII字符仅需8位，因此输出了大量多余的前导零。
3. 冗余内存分配：没必要用malloc创建int数组存储字符的ASCII值，直接遍历原字符串即可，完全是多余的内存开销。

修复后的代码 #

#include <stdio.h>
#include <cs50.h>
#include <string.h>

// 统一函数声明与定义的名称
void printbulbs(int ascii_input);

int main(void)
{
    char *Mesaj = get_string("Message: ");
    int size = strlen(Mesaj);

    // 直接遍历字符串处理每个字符，无需额外数组
    for (int i = 0; i < size; i++)
    {
        printbulbs((int)Mesaj[i]);
    }

    return 0;
}

// 仅处理8位二进制，输出ASCII字符对应的8位串
void printbulbs(int ascii_input)
{
    const int bits = 8;
    char bits_message[bits + 1];
    bits_message[bits] = '\0';

    // 从最高位到最低位填充二进制字符
    for (int i = bits - 1; i >= 0; i--)
    {
        bits_message[i] = (ascii_input & 1) + '0';
        ascii_input >>= 1;
    }

    printf("%s\n", bits_message);
}

修复说明 #

• 统一函数名：将声明和实现的函数名统一为printbulbs，解决编译问题。
• 限定8位输出：固定二进制位数为8，只处理ASCII字符的有效8位，彻底消除前导零。
• 优化内存使用：用栈上的字符数组存储8位二进制串，无需动态分配；同时移除冗余的int数组，直接遍历原字符串处理每个字符。
• 调整返回值：printbulbs无需返回值，改为void类型更符合逻辑。

测试输入"Hi!"时，输出将完全符合预期：

01001000
01001001
00100001

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