求大神幫忙看下單片機代碼哪里錯了 按鍵控制數碼管加減

想實現的功能是按鍵控制數碼管加減，對應位數碼值變化 其他位常亮 ，現在是可以實現加減，但是每次按鍵按下數碼管都不亮

單片機源程序如下:
#include <REGX51.H>

sbit KEY2 = P3^4;
sbit KEY1 = P3^5;

void Display(unsigned int num);
unsigned int displaynumber = 0;
void Delay(unsigned char ms);

unsigned char code smg_data[]={0x3F,0x06,0x5B,0x4F,0x66,0x6D,0x7D,0x07,0x7F,0x6F};

unsigned char KeyScan()
{
        if(KEY1 == 0)
        {
                Delay(50);
                        if(KEY1 == 0)
                        {
                                while(KEY1 == 0);
                                return 1;
                        }

        }
        if(KEY2 == 0)
        {
                Delay(50);
                        if(KEY2 == 0)
                        {
                                while(KEY2 == 0);
                                return 2;
                        }
        }
                return 0;
}

void Delay(unsigned char ms)                //@11.0592MHz
{
        unsigned char i;
        while(ms--)
        {
        i = 2;
        while (--i);
        }

}
void Display(unsigned int num)
{
        P2 = 0x18;
        
        P0 = smg_data[num %10];
        P2 = 0X88;
        Delay(5);
        P2 = 0X08;
        
        P0 = smg_data[(num /10)%10];
        P2 = 0X48;
        Delay(5);
        P2 = 0X08;
        
        P0 = smg_data[(num /100)%10];
        P2 = 0X28;
        Delay(5);
        P2 = 0X08;
        
        P0 = smg_data[(num /1000)%10];
        P2 = 0X18;
        Delay(5);
        P2 = 0X08;
        
}
void main()
{
        unsigned char key;
        while(1)
        {
                key = KeyScan();
                 if(key == 1)
                 {
                        if(displaynumber<9999)
                                displaynumber++;
                 }
                else if(key == 2)
                {
                        if(displaynumber>0)
                                displaynumber--;
                }
               
                Display(displaynumber);

        }
}   

keyscan和display都在一個大循環里面阻塞運行，執行一個整個流程就阻塞了，另一個根本執行不了。把這兩者中間的一個挪到定時中斷里面去。

Hephaestus 發表于 2023-9-4 18:05
keyscan和display都在一個大循環里面阻塞運行，執行一個整個流程就阻塞了，另一個根本執行不了。把這兩者中 ...

你這有點誤人子弟了啊，哪有教人把代碼都往中斷里寫的。這種應該把數碼管刷新和按鍵檢測都做成非阻塞的方式，在主循環里不停執行才好

非阻塞式示例

1. #include <REGX51.H>
   
2. sbit KEY2 = P3^4;
   
3. sbit KEY1 = P3^5;
   
4. void Display(unsigned int num);
   
5. unsigned int displaynumber = 0;
   
6. unsigned char code smg_data[]= {0x3F,0x06,0x5B,0x4F,0x66,0x6D,0x7D,0x07,0x7F,0x6F};
   
7. unsigned char KeyScan()
   
8. {
   
9.         unsigned char Temp = 0;
   
10.         static bit Lock1 = 0, Lock2 = 0;
    
11.         if(KEY1 == 0)
    
12.         {
    
13.                 if(Lock1 == 0)
    
14.                 {
    
15.                         Temp = 1;
    
16.                         Lock1 = 1;
    
17.                 }
    
18.         }
    
19.         else
    
20.                 Lock1 = 0;
    
21.         if(KEY2 == 0)
    
22.         {
    
23.                 if(Lock2 == 0)
    
24.                 {
    
25.                         Temp = 2;
    
26.                         Lock2 = 1;
    
27.                 }
    
28.         }
    
29.         else
    
30.                 Lock2 = 0;
    
31.         return Temp;
    
32. }
    
33. void Display(unsigned int num)
    
34. {
    
35.         static unsigned char i = 0;
    
36.         P2 = 0X08;
    
37.         switch (i)
    
38.         {
    
39.         case 0:
    
40.                 P0 = smg_data[num %10];
    
41.                 P2 = 0X88;
    
42.                 break;
    
43.         case 1:
    
44.                 P0 = smg_data[(num /10)%10];
    
45.                 P2 = 0X48;
    
46.                 break;
    
47.         case 2:
    
48.                 P0 = smg_data[(num /100)%10];
    
49.                 P2 = 0X28;
    
50.                 break;
    
51.         case 3:
    
52.                 P0 = smg_data[(num /1000)%10];
    
53.                 P2 = 0X18;
    
54.                 break;
    
55.         default:
    
56.                 break;
    
57.         }
    
58.         i++;
    
59.         if(i>=4)i=0;
    
60. }
    
61. unsigned int Delay = 0;
    
62. void main()
    
63. {
    
64.         unsigned char key;
    
65.         while(1)
    
66.         {
    
67.                 if(Delay++>=350)
    
68.                 {
    
69.                         Delay = 0;
    
70.                         key = KeyScan();
    
71.                         if(key == 1)
    
72.                         {
    
73.                                 if(displaynumber<9999)
    
74.                                         displaynumber++;
    
75.                         }
    
76.                         else if(key == 2)
    
77.                         {
    
78.                                 if(displaynumber>0)
    
79.                                         displaynumber--;
    
80.                         }
    
81.                         Display(displaynumber);
    
82.                 }
    
83.         }
    
84. }

復制代碼

lkc8210 發表于 2023-9-5 09:43
非阻塞式示例

十分感謝

lkc8210 發表于 2023-9-5 09:43
非阻塞式示例

按鍵還是需要加去抖

123156fsadf 發表于 2023-9-5 08:36
你這有點誤人子弟了啊，哪有教人把代碼都往中斷里寫的。這種應該把數碼管刷新和按鍵檢測都做成非阻塞的方 ...

中斷是給你用的，不是讓你供起來上香用的。