总觉得什么地方不对，我这是不是民科代码？

Tang123

新手，没人教，就看视频和资料下面是我写的一个按键控制1个灯和马达开关，另一个按键控制马达和另一个灯定时60秒（或更久）的代码。
) V4 k: n7 Q- h1 }& m虽然在板子上能工作，定时也准，总觉得代码不规范。
# ^1 Y" L: m  d6 q( j- U当然还有按键1开始工作后，再按按键2是有问题的，这个还在自我研究，哈哈
不多说，发下代码，有没有高人帮忙看看
3 E6 T1 E7 G% D0 y' y0 b8 C1 E#include<reg52.h>

" Q3 L$ r  ]; ~* ?typedef unsigned int u16;
typedef unsigned char u8;
0 a8 q5 t9 c; r* w* H
  F7 h/ R' y8 {1 k4 {6 A, `sbit led1=P0^0;
sbit led2=P0^7;
sbit k1=P1^0;
  a2 @% M; @( D" a/ B- Ysbit k2=P1^7;
sbit moto=P1^4;

u16 num;
6 j4 _# q5 f6 [% ?/ Tu16 TIME1;

2 M2 ^, G" S. b0 j" }. c2 `. a# r
/**************************************************************
; F: y1 m! o8 X                                        延时 Xms
**************************************************************/
void delayxms(u8 x)
: u# C: a2 v& p; s- M2 k{
) l6 _+ A, e* h8 j! T3 z9 |        u8 i,j;
+ W8 ]2 Z; `, w0 B8 G8 ]        for(i=x;i>0;i--)
                for(j=124;j>0;j--);
}
+ m' a: J8 e/ R1 T* Y

: `( a: N6 A# U% |) s. P! \

! D# n, K! m  R2 i. ]
! S2 _! s; t2 a- L. b/**************************************************************
7 u! F& G! p0 `5 n                                        初始化
$ Y' `' C7 B) |8 S% ^1 d**************************************************************/
9 i' p$ h" E9 L9 n8 Z& ^8 lvoid init()
{
        moto=0;
. Y. |" g" b5 ], ?8 A: K5 o5 X        led1=1;
* D* G9 J4 c3 c& D% D0 a5 B        led2=1;
        TMOD=0X01;
        TH0=(65536-50000)/256;                 //50ms发送中断请求一次
        TL0=(65536-50000)%256;
. a6 `0 \* s/ C2 j1 Y. t        EA=1;                                                //打开总中断
        ET0=1;                                                //打开定时计数器T0
8 C. ]: ?( |" h$ ?& A    //TR0=1;                                //启动定时器T0，进入中断程序
9 m) K0 g7 s0 T+ m: m}
- K9 V1 W4 y; n$ J8 V/**************************************************************
                                        按键扫描
**************************************************************/
3 R8 ?* d! m' h: u9 J+ s  \void kscan()
{       
0 X4 v9 R( H( t+ N0 L" G" s, F        if(k1==0)
        {       
; e$ R6 n8 i+ M' x                delayxms(10);
                if(k1==0)
8 ^# ]3 G  ~0 t" y! K8 o                {               
                        moto=~moto;
2 G1 F& S+ v" u  N7 Y                        led1=~led1;       
, g* `# u8 R: M* p( h5 F                }       
                while(!k1);
' J5 z) l2 [( t. T  t6 k2 P        }
4 \) T* P& ^. V/ z& c5 n        if(k2==0)
        {       
" ]9 ~; C! e8 g, C" H3 @6 |                delayxms(10);
                if(k2==0)
                {
                        TR0=1;                                //打开定时器T0
                        moto=~moto;
7 k9 T& t0 d; n6 n6 P( I" [6 N                        led2=~led2;       
  J: b* j: Z1 T1 t1 f3 t0 ~                }       
6 Q6 D- L* m" `' V. w& p( m& `  L1 R                while(!k2);
5 M! t* }* @8 ^8 l        }
0 a, c+ b4 `& ]! x% b        if(num==20)                                         //1S定时
                {
: I( ?. r. a! k5 V  A  f, j                        num=0;
                        TIME1++;
4 U8 J4 E/ w% s! V. G2 O                }
, e9 O1 b/ l( I7 w; ?: Y4 A+ F        if(TIME1>=60)                                 //60s定时
% `  S) r$ s6 H: j        {
7 v1 \, K+ @/ k( A1 k. E9 \% K                TIME1=0;
                moto=0;
: `' V9 @! a5 v" m8 }( m( u                led2=1;       
                TR0=0;                                                //关闭定时器T0
: l9 l; ^7 v3 Q) e+ x. `% Y9 d        }
4 l" n0 P. v# B* W0 i' f- [}
: c8 L; k0 X) _% T9 G: }, E9 D
/**************************************************************
4 w0 y  Z5 E- B2 C                                        主程序
4 Y% U/ ?( B. ^: |/ B**************************************************************/
# Z$ ^: d4 a: W0 x$ y1 Z8 l2 D: ]- kvoid main()
3 @2 k: B, ]% ]7 P! \{

% C: }+ {. c4 m# A: m/ M1 w        init();
        while(1)
$ e) T4 _0 q3 y7 E+ y$ r                kscan();

% z, M' I) o, _# {8 e8 @; Q}

" K" v+ G: d# D* h1 g5 ?/**************************************************************
6 G# S+ l7 `/ W                                        中断
**************************************************************/
void T0_time()interrupt 1
{
        TMOD=0X01;
        TH0=(65536-50000)/256;                 //50MS发送中断请求一次
) ^1 l7 W  K, I% ]! V$ Y        TL0=(65536-50000)%256;
, ?/ p' C# V. m        num++;           
}


) V6 y$ {0 i6 |7 C8 b8 A0 C; I
& g  T3 }; K% X

pulbieup

Heaven_1

代码没有问题，我们也是从新手过来得，
多练习，肯定就会了

nightinmy

程序一定要模块化

行者～ABC

可以把按键这里用CASE语句，