C语言求助,这个函数能有简单点的写法吗?想蒙住了. 求指点.

2020-02-02 09:19发布

目的是用在单片机数码管显示,高位清零,本来的代码能用,但是一个数如果是980,转换后是00000980,就想把高位是零的消掉.这是很笨的办法.所以请教高手能否简化写法.
uint32 定义为 无符号32位整数.


void BCDDataConversion(uint32 Bign)
{


    if (0<=Bign<10){//
    ledcache[7]= leddatacl[Bign%10];   
    ledcache[6]= 0x00;
    ledcache[5]= 0x00;
    ledcache[4]= 0x00;
    ledcache[3]= 0x00;
    ledcache[2]= 0x00;
    ledcache[1]= 0x00;
    ledcache[0]= 0x00;
    }
    else if (10<=Bign<100){
    ledcache[7]= leddatacl[Bign%10];   
    ledcache[6]= leddatacl[Bign/10%10];
    ledcache[5]= 0x00;
    ledcache[4]= 0x00;
    ledcache[3]= 0x00;
    ledcache[2]= 0x00;
    ledcache[1]= 0x00;
    ledcache[0]= 0x00;
    } else if (100<=Bign<1000){
    ledcache[7]= leddatacl[Bign%10];   
    ledcache[6]= leddatacl[Bign/10%10];
    ledcache[5]= leddatacl[Bign/100%10];
    ledcache[4]= 0x00;
    ledcache[3]= 0x00;
    ledcache[2]= 0x00;
    ledcache[1]= 0x00;
    ledcache[0]= 0x00;
    }else if (1000<=Bign<10000){
    ledcache[7]= leddatacl[Bign%10];        
    ledcache[6]= leddatacl[Bign/10%10];
    ledcache[5]= leddatacl[Bign/100%10];
    ledcache[4]= leddatacl[Bign/1000%10];
    ledcache[3]= 0x00;
    ledcache[2]= 0x00;
    ledcache[1]= 0x00;
    ledcache[0]= 0x00;
    }else if (10000<=Bign<100000){
    ledcache[7]= leddatacl[Bign%10];        
    ledcache[6]= leddatacl[Bign/10%10];
    ledcache[5]= leddatacl[Bign/100%10];
    ledcache[4]= leddatacl[Bign/1000%10];
    ledcache[3]= leddatacl[Bign/10000%10];
    ledcache[2]= 0x00;
    ledcache[1]= 0x00;
    ledcache[0]= 0x00;
    }else if (100000<=Bign<1000000){
    ledcache[7]= leddatacl[Bign%10];   
    ledcache[6]= leddatacl[Bign/10%10];
    ledcache[5]= leddatacl[Bign/100%10];
    ledcache[4]= leddatacl[Bign/1000%10];
    ledcache[3]= leddatacl[Bign/10000%10];
    ledcache[2]= leddatacl[Bign/100000%10];
    ledcache[1]= 0x00;
    ledcache[0]= 0x00;
    }else if (1000000<=Bign<10000000){
    ledcache[7]= leddatacl[Bign%10];   
    ledcache[6]= leddatacl[Bign/10%10];
    ledcache[5]= leddatacl[Bign/100%10];
    ledcache[4]= leddatacl[Bign/1000%10];
    ledcache[3]= leddatacl[Bign/10000%10];
    ledcache[2]= leddatacl[Bign/100000%10];
    ledcache[1]= leddatacl[Bign/1000000%10];
    ledcache[0]= leddatacl[Bign/10000000%10];
    ledcache[0]= 0x00;
    }else if (10000000<=Bign<100000000){
    ledcache[7]= leddatacl[Bign%10];        
    ledcache[6]= leddatacl[Bign/10%10];
    ledcache[5]= leddatacl[Bign/100%10];
    ledcache[4]= leddatacl[Bign/1000%10];
    ledcache[3]= leddatacl[Bign/10000%10];
    ledcache[2]= leddatacl[Bign/100000%10];
    ledcache[1]= leddatacl[Bign/1000000%10];
    ledcache[0]= leddatacl[Bign/10000000%10];
    }   
    else Bign=0;
}


这个函数能有简单点的写法吗?想蒙住了.  求指点.
补上开发板图片
IMG_20121107_004248_0.jpg (740.3 KB, 下载次数: 0) 下载附件 2012-11-7 00:50 上传
友情提示: 此问题已得到解决,问题已经关闭,关闭后问题禁止继续编辑,回答。
29条回答
eduhf_123
2020-02-03 23:33
millwood0 发表于 2012-11-9 02:37
You can simplify it further by combining the two loops into one:


您的代码在参数Bign值为0时无法正常工作——本应该显示“       0”而您的代码将显示“        ”。

基于运行时效率/Cache/流水线/编译器优化等因素,我个人习惯于不在短小的循环中使用分支。

空闲时想过这段代码的进一步优化:
  1. void BCDDataConversion(uint32 Bign)
  2. {
  3.         unsigned char i;

  4.         //从右端开始填充每个数位上的数值
  5.         i = 8;
  6.         do
  7.         {
  8.                 ledcache[--i] = leddatacl[Bign%10];
  9.                 Bign /= 10;
  10.         }while(Bign && i);

  11.         //如果有必要,继续填充左边的“空白”
  12.         while(i)
  13.         {
  14.                 ledcache[--i] = leddatacl[_BLANK_];           //这里表示“空白”字符,即消隐控制码
  15.         };
  16. }
复制代码

一周热门 更多>