刚学C，偷鸡不成蚀把米，求指点

MOSO

const uint8_t DisTab[] = {0x3f,0x06,0x5b,0x4f,0x66,0x6d,0x7d,0x07,0x7f,0x6f,0x77,0x7c,0x39,0x5e,0x79,0x71,0x40,0x00,0x6e,0x37,0x50,0x6d,0x1c};

//数码管显示函数
//此函数显示正常
void Display(uint16_t data)
{
  uint8_t i, j;
  uint8_t DisBuff[4];
  DisBuff[3] = data % 10000 / 1000;
  DisBuff[2] = data % 1000 / 100;
  DisBuff[1] = data % 100 / 10;
  DisBuff[0] = data % 10;
  for(i = 4; i > 0; i--)
    SPIWriteByte(DisTab[DisBuff[i-1]]); //送显示数据
}

//改为以下循环模式就异常了

//此函数显示异常
void Display(uint16_t data)
{
  uint8_t i, j;
  for(i = 0; i < 4; i++)
  {
    if(i < 3)
      SPIWriteByte(DisTab[data % (10*(4-i)) / 10*(3-i)]);   //送显示数据
    else
      SPIWriteByte(DisTab[data % (10*(4-i))]);
  }
}

杯酒问月

修改后如下：
void Display(uint16_t data)
{
  uint8_t i, j;
  uint16_t Buf=10000;
  for(i = 0; i < 4; i++)
  {
      SPIWriteByte(DisTab[(data % Buf)/ (Buf/10)]);   //送显示数据
      Buf/=10;
  }
}

LcwSwust

10*n不是10的n次方
10*(4-i)结果就是40、30、20...... 所以结果不对。
我的建议：

1. uint8_t i；
   
2. uint16_t  j=1000;
   
3. for(i = 0; i < 4; i++)
   
4.   {   
   
5.       SPIWriteByte(DisTab[data / j %10]);   //送显示数据
   
6.           j=j/10;          
   
7.   }

复制代码

还在忙

你送显示数据那里算出来跟上面的结果好像不一样

杯酒问月

代码：SPIWriteByte(DisTab[data % (10*(4-i)) / 10*(3-i)]);//送显示数据
这里10*4=40，10*3=30，并不是10的4次方，或者10的3次方。

MOSO

谢谢各位老师的耐心指导，居然犯了这么低级的错误。不过你们的思路确实不错，预设一个10000，然后在处理，太棒了！ 我咋就没想到呢，哈哈