分享BUG-8位PIC的MCU的类型问题:unsigned long cnt=15*60*50;//理论为45000，实际为-20536

倒拔萝卜

最近在做8位单片机的程序，真的是郁闷了，查了半天才查出这个问题，
现分享给大家，也提醒自己一定要注意MCU的位数和编译器和标准C之间的类型问题。

就是这个表达式15*60*50=45000, 0xAFC8

PC机上测试:
unsigned long cnt=15*60*50;
printf("cnt=%d,%#x",cnt,cnt);
//输出 45000，0xAFC8   结果正确没问题


8位单片机PIC16F1509上测试：sizeof(unsigned long)是等于4；
unsigned long cnt=15*60*50;
itoa(cnt, buf, 10);    dputs(buf);//转为10进制字符串
itoa(cnt, buf, 16);    dputs(buf);//转为16进制字符串
//输出结果：-20536, -5038
//-20536(10进制)=-5038(16进制)
//这结果碉堡了，我说我的程序老是不对劲

原因：
表达式结果是45000绝对没错，但是8位单片机居然会对这个数
做溢出处理，从32767溢出到-32768，再向上继续溢出，所以最终
结果就成了-20536。

解决方法，使用标准给每一个数都指明一下类型就OK了：
unsigned long cnt=15UL*60UL*50UL;

char *itoa(long val, char *buf, int radix)
{
    char *high, *low, tmp;
    int digit;
    high = buf;
    if (val < 0)        //判断符号
    {
        *high++ = '-';
        val = -val;
    }
    low = high;         //记录数字的低位所处的位置
    do                  //从个位开始分离，数字是反转的
    {
        digit = val % radix; //个位
        val /= radix;   //val向个位移动
        if (digit > 9)
            *high++ = (char)(digit - 10 + 'A'); //16进制
        else
            *high++ = (char)(digit + '0'); //10进制
    }
    while(val > 0);
    *high-- = '\0';     //添加结束符, high指向数字的高位，开始进行反转
    for (; low < high; low++, high--)
    {
        tmp   = *high;
        *high = *low;
        *low  = tmp;
    }
    return buf;
}

正点原子

谢谢分享....

TinyBoy

很明显的事情，给第一个就可以了，其他自动跟着转。

TinyBoy

如果能理解CPU的硬件运算器如何运算的，在那款单片机都是一样的。

倒拔萝卜

回复【4楼】TinyBoy:
---------------------------------
出现这样的编译器也只能认栽了，哎，谁会想到45000这样的数也会被溢出处理呢？

TinyBoy

回复【5楼】倒拔萝卜:
---------------------------------
你要做到在任何编译器上运算结果都是对了就行了。

ofourme

回复【5楼】倒拔萝卜:
---------------------------------
不是bug，而是C标准就是这样。楼主对C了解还不够透彻。
默认整形是int，至于int是16bit还是32bit，就要看你的硬件平台了。15*60*50的结果如果是16bit的int类型，自然无法存放45000而溢出了。

倒拔萝卜

回复【7楼】ofourme:
---------------------------------
不是不了解C标准，这个编程习惯很难改的，
写的时候谁又会在意这么个小小的表达式还会存在类型问题？