按键释放问题，请过来看看

只看该作者 · 2007-6-27 08:03

我写了一个程序8个按键控制8个led，也通过了，这个是4X4矩阵键盘，P1口接键盘，当按一下P1_0时，led1闪烁，按一下P1.1时，led2闪烁。。。。。
但是现在却是要按键不放才能闪烁，找不出原因，请各位给看看，问题出在哪？

#include <AT89X51.H>

unsigned char keyscan()
{ unsigned char row,col,i;

  P1=0xf0;
  row=P1&0xf0;
  //row=P1&0xf0;   （这一行可以不要，为什么有的教程上要多写一行？？）
  P1=0x0f;
  col=P1&0x0f;
  //col=P1&0x0f;
  i=~(col|row);
// while((P1&0xf0)!=0xf0    );
  return  i;}
void delay10ms(void)
{   unsigned char i,j;
    for(i=20;i>0;i--)
     for(j=248;j>0;j--){;;}
     }

void delay02s(void)

{ unsigned char i;

  for(i=20;i>0;i--)

    {delay10ms();

    }

}

void main(void)
{
  unsigned char key;
   while(1)
   {
          key=keyscan();
      switch(key)

        {
          case 0x18:

            P0_0=~P0_0;

            delay02s();

            break;

          case 0x14:

            P0_1=~P0_1;

            delay02s();

            break;

          case 0x12:

            P0_2=~P0_2;

            delay02s();     break;
    case 0x28:
            P0_3=~P0_3;
            delay02s();     break;
    case 0x24:
            P0_4=~P0_4;
            delay02s();     break;
    case 0x22:
            P0_5=~P0_5;
            delay02s();    break;
    case 0x48:
            P0_6=~P0_6;
            delay02s();    break;
    case 0x44:
            P0_7=~P0_7;
            delay02s();    break;
    case 0x42:
            P0=0xff;
            delay02s();    break;
    case 0x84:
            P0=0x0;
            delay02s();   break;
        }


        }
              }

只看该作者 · 2007-6-27 08:18

因为你的程序是在不停的进行按键扫描，当按键抬起的时候还在执行key=keyscan();那个值当然就不对了
看看下面的该法好不好用，呵呵！
void main(void)
{
  unsigned char key;
   while(1)
   {
         if( keyscan()!=0)
      {
        key=keyscan();
      }
      switch(key)

        {
          case 0x18:

            P0_0=~P0_0;

            delay02s();

            break;

          case 0x14:

            P0_1=~P0_1;

            delay02s();

            break;

          case 0x12:

            P0_2=~P0_2;

            delay02s();     break;
    case 0x28:
            P0_3=~P0_3;
            delay02s();     break;
    case 0x24:
            P0_4=~P0_4;
            delay02s();     break;
    case 0x22:
            P0_5=~P0_5;
            delay02s();    break;
    case 0x48:
            P0_6=~P0_6;
            delay02s();    break;
    case 0x44:
            P0_7=~P0_7;
            delay02s();    break;
    case 0x42:
            P0=0xff;
            delay02s();    break;
    case 0x84:
            P0=0x0;
            delay02s();   break;
        }


        }
              }

  return  i;}

只看该作者 · 2007-6-27 08:23

你的程序中每一次循环都重新扫描按键，你松手当然就不闪烁了。

你可以定义一个变量来保存按键值，每次循环开始都过行判断，如果没有键按下，就让key等于上一次的按键值

只看该作者 · 2007-6-27 08:43

unsigned char keyscan()
{ unsigned char row,col,i;

  P1=0xf0;
  row=P1&0xf0;
  //row=P1&0xf0;   （这一行可以不要，为什么有的教程上要多写一行？？）
  P1=0x0f;
  col=P1&0x0f;
  //col=P1&0x0f;
  i=~(col|row);
// while((P1&0xf0)!=0xf0    );
  return  i;}

老兄，我这个键盘扫描程序，为什么有的教程要判断两次啊，我用一次也可以啊，就像我括号里说的？

还有你的程序最后一行 return i,为什么放在最后？？？ keyscan() 中不用了吗？？

只看该作者 · 2007-6-27 08:49

只要把key=keyscan();换成下面的语句就行了
         if( keyscan()!=0)
      {
        key=keyscan();
      }
   用重复的两行，我个人感觉是为了产生一定的延时，为了让外围的硬件有个反应的时间。

只看该作者 · 2007-6-27 09:11

void delay10ms(void)
{   unsigned char i,j;
    for(i=20;i>0;i--)
     for(j=248;j>0;j--){;;}
     }

void delay02s(void)

{ unsigned char i;

  for(i=20;i>0;i--)

    {delay10ms();

    }

还有就是，能不能用定时器来做代替以上的延时呢？放在呢比较恰当？