提问问题：C编程怎么写

#include<reg52.h>
#define uint unsigned int
#define uchar unsigned char
sbit S1=P3^0;
sbit S2=P3^1;
sbit S3=P3^2;
sbit LED1=P0^0;
sbit LED2=P0^1;
sbit LED3=P0^2;
void main()
{
while(1)
{
LED1=S1;
LED2=LED3=S2;
}
while(P3^2==1)
{
P0=0xff;
Delay();
}
}
我写的这个谁能给我改改

#include<reg52.h>
sbit S1=P3^0;
sbit S2=P3^1;
sbit S3=P3^2;
sbit LED1=P0^0;
sbit LED2=P0^1;
sbit LED3=P0^2;
void main()
{
while(1)
{
LED1=!((S1==0)&&(S3==1));
LED2=!((S2==0)&&(S3==1));
LED3=LED2;
}
}