请帮忙看一下这个程序问题出在那里？

2019-03-24 09:50发布

站内问答 / TI MCU
8290 11 1536
本帖最后由 inazuma 于 2015-8-26 17:48 编辑
  1. #include <msp430x14x.h>

  3. #define wei_h P5OUT |= BIT5

  5. #define wei_l P5OUT&= ~BIT5

  7. #define duan_l P6OUT&= ~BIT6

  9. #define duan_h P6OUT |= BIT6

  11. #define keyin (P1IN & 0x0f)

  13. typedef unsigned char uchar;
  14. uchar duan[6]={0x3f,0x06,0x5b,0x4f,0x66,0x00};
  15. uchar wei [9]={0x00,0xfe,0xfd,0xfb,0xf7,
  16.                0xef,0xdf,0xbf,0x7f};
  17. void delay(void);
  18. void shumaguan(void);
  19. void main(void)
  20. {
  21.   WDTCTL = WDTPW + WDTHOLD;
  22.   uchar temp,k,x=0,a=5,b=5,c=5,d=5,e=5,f=5,g=5,h=5;
  23.   P6DIR |= BIT2;
  24.   P6OUT |= BIT2;
  25.   P1DIR = 0;
  26.   P1OUT = 0;
  27.   P4DIR = 0xff;
  28.   P4OUT = 0xff;
  29.   P5DIR = 0xff;
  30.   P6DIR = 0xff;
  31.   while (1)
  32.   {
  33.   loop:if (keyin !=0x0f)
  34.     {
  35.       delay();
  36.       if (keyin !=0x0f)
  37.      {
  38.        temp=keyin;
  39.        while (keyin !=0x0f);
  40.        switch (temp)
  41.         {
  42.           case 0x0e: k = 1;  break;
  43.           case 0x0d: k = 2;  break;      
  44.           case 0x0b: k = 3;  break;
  45.           case 0x07: k = 4;  break;
  46.           default:   k = 0;  break;         
  47.         }
  48.         x++;
  49.         if (x>8)x=1;
  50.         switch (x)
  51.           {
  52.             case 1:a=k;break;
  53.             case 2:b=k;break;
  54.             case 3:c=k;break;
  55.             case 4:d=k;break;  
  56.             case 5:e=k;break;
  57.             case 6:f=k;break;      
  58.             case 7:g=k;break;
  59.             case 8:h=k;break;
  60.             default:break;
  61.           }
  62.         while (1)
  63.         {
  64.           shumaguan();
  65.           P4OUT=duan[a];
  66.           duan_h;
  67.           duan_l;
  68.           P4OUT=wei[1];
  69.           wei_h;
  70.           wei_l;
  71.           P4OUT=duan[b];
  72.           duan_h;
  73.           duan_l;
  74.           P4OUT=wei[2];
  75.           wei_h;
  76.           wei_l;
  77.           P4OUT=duan[c];
  78.           duan_h;
  79.           duan_l;
  80.           P4OUT=wei[3];
  81.           wei_h;
  82.           wei_l;
  83.           P4OUT=duan[d];
  84.           duan_h;
  85.           duan_l;
  86.           P4OUT=wei[4];
  87.           wei_h;
  88.           wei_l;
  89.           P4OUT=duan[e];
  90.           duan_h;
  91.           duan_l;
  92.           P4OUT=wei[5];
  93.           wei_h;
  94.           wei_l;
  95.           P4OUT=duan[f];
  96.           duan_h;
  97.           duan_l;
  98.           P4OUT=wei[6];
  99.           wei_h;
  100.           wei_l;
  101.           P4OUT=duan[g];
  102.           duan_h;
  103.           duan_l;
  104.           P4OUT=wei[7];
  105.           wei_h;
  106.           wei_l;
  107.           P4OUT=duan[h];
  108.           duan_h;
  109.           duan_l;
  110.           P4OUT=wei[8];
  111.           wei_h;
  112.           wei_l;
  113.           if(keyin !=0x0f)  
  114.           {   
  115.               goto loop;
  116.            }
  117.         }
  118.       }
  119.     }
  120.   }
  121. }
  122. void delay(void)
  123.     {
  124.   unsigned int x;
  125.    for(x = 12000;x > 0;x--);
  126.     }
  127. void shumaguan(void)
  128. {
  129. unsigned int w;
  130. for(w=3000;w>0;w--);
  131. }
复制代码
最近买了一个学习板，想做一个按键控制数码管显示的功能，但是不知道哪里有错一直显示不正常。希望给位能帮帮我谢谢！
功能如下：
1：4个按键按下后能分别显示1~4的数字在数码管上。
2：数码管由两个4位八段数码管组成，并使用两个74HC573控制。
360反馈意见截图16511003101110138.png

3：当第一次按下任意一个按键时，数码管第一位点亮相应的数字，
4：当第二次按下任意一个按键时，数码管第二位点亮相应的数字，同时第一位不变，····
5：当第九次按下任意一个按键时，数码管第一位点亮相应的数字，其他位不变。



但是显示有问题
1：第一次按下按键1时
数码管第一位和第八位同时点亮
2：第二次按下按键2时
数码管第3位显示2，第二位显示的是1和2的合体。

 IMG_20150826_150117.jpg

IMG_20150826_150130.jpg
IMG_20150826_150140.jpg
IMG_20150826_150146.jpg
希望各位能指正一下程序的错误出在哪里谢谢！！




此帖出自小平头技术问答
友情提示: 此问题已得到解决，问题已经关闭,关闭后问题禁止继续编辑，回答。
11条回答
qiushenghua
2019-03-25 17:36
第三个要吐槽的内容是程序的可读性太差，这是近期见到第二个用多个while(1)语句的网友了。

goto语句明明可以用break语句代替的，非得用强行跳转，这样程序结构会显得混乱，别人看不懂程序会在哪循环，很少人会跟着你的程序去跳转的。

第二个就是变量名称不明确，我到现在没看明白里面的x是用来干嘛的，程序也几乎看不下去，是通过现象来分析故障原因的。

第三个，程序几乎没有注释，加上变量、函数命名混乱，几乎是连蒙带猜才搞明白程序的大概运行流程。想细查都没查，实在查不下去了。

一周热门 更多>

相关问题

    相关文章