剛剛入門C51，大家幫幫看下下面這個相當菜的程序，指點下哪里有問題，拜謝了！

 #include<reg52.h>
 #define uchar unsigned char
 #define uint unsigned int
 uint a,b;
 void delay(uint,uint)//函數聲明
 void main()
 {
  uchar temp;
  uint i,j,x,y;
  temp=0xfe;
  P0=temp;
  for(i=8;i>0;i--)
  {x=temp<<i;
  y=temp>>8-i;
  P0=x|y;
  delay(200,200);//將實參200,200賦值給下面子函數形參a,b
  }
 }
 void delay(uint a,b)
 {
  for(;a>0;a--)
   for(;b>0;b--);//這里不知道寫對沒有？
 }

#include<reg52.h>
#define uchar unsigned char
#define uint unsigned int
 uint a,b;
void delay(uint a,uint b)
 {
  for(;a>0;a--)
   for(;b>0;b--);//這里不知道寫對沒有？
 }

 void main()
 {
  uchar temp;
  uint i,x,y;
  temp=0xfe;
  P0=temp;
  for(i=8;i>0;i--)
  {x=temp<<i;
  y=temp>>8-i;
  P0=x|y;
  delay(200,200);//將實參200,200賦值給下面子函數形參a,b
  }
 }

大哥，你好歹編譯下試試，編譯都沒通過啊

無需定義全局變量a,b

函數聲明那里要加個“;”

main函數里定義的j沒用到，為多余變量

其它沒什么問題

另外有一問題，把temp賦給P0作為初值，有什么作用？

嗯，樓上的回答讓我學到東西了，謝啦哈~

延時寫的這么多啊！用for(i=0;i<某個數；i++)

                                {}

空操作不就可以了，還搞這么多啊！如果是要延時的準確性，那就用中斷吧！我覺得那樣比較好