电小生 发表于 2015-10-17 08:36
#include
sbit led1=P1^0;
sbit key1 = P3^4;

你的程序……错误不止一点点呐……
首先来说一下数据类型，51单片机是8BIT的单片机，所以处理8BIT的数据（即char或unsigned char型的数据）最快，而编译器会把延时程序的for编译成DJNZ命令并以此延时，你用的是int型变量也就是16BIT的有符号变量，而51单片机只能对8BIT的数据进行运算，所以，在这里要把16BIT的数据折合成8BIT运算，会有更长的延时时间。另外，你的i、t、y、z并不需要全部定义为全局变量，其中t可以改成Delay函数的参数，y和z放到Delay里面，放到main里面。而且，你对C语言可能不太熟悉，比如你的for(y;y>100;y++)；，你初始化变量y为0，不满足“y>100”的条件，也就是说，第一个for根本不会循环，而第二个循环“for(z=t;z<0;z--);”你写对了，只是过于麻烦。再说一说变量i，有两个错误：1，在C语言中，任何变量的初始值都为0。即使你不对i进行任何操作，它都是0，不满足i>50的条件，不会循环。2，在程序中，并没有对i进行任何操作，所以i一直都是0。（综上，你的松手检测部分名存实亡）。
所以说，你写的两个并没有差别。
正确的程序如下：
#include <reg52.h>
sbit led1=P1^0;
sbit key1 = P3^4;

void Delay(unsigned char t)
{
   unsigned  char x；
   for(;t;t--)
     for(x=1000;x;x--);

}

void main(void)
{
    unsigned char i;
    led1=0;
   
    while(1)
    {
        if (key1 == 0)
        {
            delay(10);
            if(key1 == 0)
            {
               while((i < 50)&&(key1 == 0))
               {
                   Delay(10);
                   i++;
               }
               led1=1;
               //建议改成 “led1 = ！led1;”
            }
        }               
    }
}