使用unsigned char实现十进制转16位二进制出错，请求技术协助

你的代码出现问题主要有两个核心原因，咱们一步步拆解清楚：

1. 类型不匹配导致的输入值截断与未定义行为 #

当你把operand1定义为unsigned char时，它的取值范围是0~255（仅占1字节的无符号整数）。但你用scanf("%d", &operand1)来读取输入：

• %d格式符要求接收一个int*类型的指针，但你传的是unsigned char*，这属于类型不匹配，会触发C语言中的未定义行为；
• 即便忽略未定义行为，像4096、65536这类远大于255的数值，会被截断为低8位的值——比如4096的二进制是1000000000000，低8位全是0，所以operand1最终会被赋值为0，自然输出全0的二进制结果。

而当operand1是int类型时，它的存储空间通常是4字节，能轻松容纳这些大数值，所以输入不会被截断，代码能正常运行。

2. 二进制转换循环的索引错误 #

你的循环写的是for (pos = 16; pos >= 0; --pos)，但bin16_1数组定义为"0000000000000000"，它的实际长度是17（16个字符加末尾的\0字符串结束符）。当pos=16时，你修改的是数组的结束符\0，这会破坏字符串的结构，可能导致后续输出异常。正确的索引应该从15到0（对应16位二进制的每一位）。

修正后的代码 #

针对这两个问题，调整后的代码如下：

#include <stdio.h>

int ALU() {
    // 要处理16位数值，用unsigned short更合适（2字节，范围0~65535）
    unsigned short operand1, operand2; 
    
    printf("Enter Operand 1(in decimal): ");
    // 用%hu匹配unsigned short类型，避免类型不匹配
    scanf("%hu", &operand1); 
    
    printf("\nEnter Operand 2(in decimal): ");
    scanf("%hu", &operand2);
    
    char bin16_1[] = "0000000000000000";
    int pos;
    // 从索引15开始遍历到0，对应16位二进制的每一位
    for (pos = 15; pos >= 0; --pos) { 
        if (operand1 % 2) {
            bin16_1[pos] = '1';
        }
        operand1 /= 2;
    }
    
    printf("\n\nBinary Equivalence of Operand 1: %s", bin16_1);
    return 0;
}

int main() {
    ALU();
    return 0;
}

关键调整说明： #

• 替换类型：把unsigned char换成unsigned short，刚好能容纳0~65535的16位无符号整数；
• 匹配格式符：用%hu作为scanf的格式符，完美对应unsigned short类型，消除类型不匹配的隐患；
• 修正循环索引：从15到0遍历，确保只修改16位二进制字符，不触碰字符串结束符。

如果坚持要用unsigned char，那只能处理0~255的输入，同时要把scanf的格式符改成%hhu（对应unsigned char），但这样就无法处理4096这类超过255的数值了。

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