51单片机大神来看看，程序很不稳定，尤其是6，A，E，喜欢闪，变成一。是共阴数码管，按键P3口

Question

#include<reg52.h>
#define uint unsigned int
#define uchar unsigned char
sbit key1=P3^4;
sbit key2=P3^5;
sbit key3=P3^6;
sbit key4=P3^7;
void delay(uint a)
{
uint b,c,d;
for(b=0;b<a;b++)
for(c=0;c<3;c++)
for(d=0;d<220;d++);
}
void m(uchar n)
{
uchar code m[]={0x3f,0x06,0x5b,0x4f,0x66,0x6d,0x7d,0x07,0x7f,0x6f,0x77,0x7c,0x39,0x5e,0x79,0x71};
P0=m[n];
delay(2);
}
uchar key_press()
{
uchar key;
P3=0xf0;
key=P3;
if(key!=0xf0)
{
return 1;
delay(20);
P3=0xf0;
if(key!=0xf0)
return 1;
else
return 0;
}
else
{
return 0;
}
}
uchar key_scan()
{
uchar key_num;
if(key_press())
{
P3=0xfe;
key_num=P3;
switch(key_num)
{
case 0xee:
return 0;
break;
case 0xde:
return 1;
break;
case 0xbe:
return 2;
break;
case 0x7e:
return 3;
break;
}

P3=0xfd;
key_num=P3;
switch(key_num)
{
case 0xed:
return 4;
break;
case 0xdd:
return 5;
break;
case 0xbd:
return 6;
break;
case 0x7d:
return 7;
break;
}
P3=0xfb;
key_num=P3;
switch(key_num)
{
case 0xeb:
return 8;
break;
case 0xdb:
return 9;
break;
case 0xbb:
return 10;
break;
case 0x7b:
return 11;
break;
}
P3=0xf7;
key_num=P3;
switch(key_num)
{
case 0xe7:
return 12;
break;
case 0xd7:
return 13;
break;
case 0xb7:
return 14;
break;
case 0x77:
return 15;
break;
}
}
}
void main()
{
uchar i;
while(1)
{
i=key_press();
if(i==1)
m(key_scan());
}
}

Answer 1

这样写的代码一般都是不稳定的，因为调用函数过多，忙延时太长等。显示不稳是由于显示的时间一直在变动，虽然每一次显示的时间一样，但在一定时间内显示的时间却不定，所以时暗时亮是必然的。从代码看出楼主对C掌握也不好
如： if(key!=0xf0)
{
return 1; ／／返回后下面的代码不可能被执行
delay(20);
P3=0xf0;
if(key!=0xf0)
return 1;
else
return 0;
}
case 0xee:
return 0;
break;
case 0xde:
return 1;
break;
有return后break是不用的
像这种程序应该把他们分时扫描，确宝不管程序怎么增加代码，显示的时间都不变，这样才会稳定

Answer 2

LZ在判断按键的地方的程序看不懂啊……为什么先要赋给P3寄存器一个值再让判断的变量（比如key和key_num），这样变量真的能获得P3接口的值么……