Programm für einen Behälter

Hallo

Habe das Programm zu der Aufgabe im anhang geschrieben , nun wollte ich 
wissen ob das so richtig ist oder ob viellt noch fehler drin stecken

hoffe ihr könnt mir bei helfen

MFG

#include <at89c51ed2.h>


//Taster

sbit at P2_0 Taster;   //Taster P2.0
sbit at P2_1 Taster1;   //Taster P2.1



//Behälter
sbit at P1_0 V1;  //v1   Füllen des Behälters
sbit at P1_1 V2;  //v2   Leeren des Behälters
sbit at P1_2 D2;  //D2 Behälter A
sbit at P1_3 D3;  //D3 Behälter B
sbit at P1_4 D4;  //D4 Behälter C

//LEDS
sbit at P2_4 LED4; // LED 4
sbit at P2_5 LED5; // LED 5
sbit at P2_5 LED6; // LED 6



void main (void)
  {


P1_0=0;    // Ventil V2 schließen

        while(1);
{


if (P2_0=1)   //Taster 2.0

{
P1_0=1; //V1 geöffnet
}
else
{
P1_0=0; //V1 geschlossen
}

if (P1_2=1)
{
 P2_4=1;  //LED 2.4
}

else
{
P2_4=0;
}


if (P1_3=1)
{
 P2_5=1;    //LED 2.5
}

else
{
P2_5=0;
}

if (P1_4=1)
{
 P2_6=1; //LED 2.6
}

else
{
P2_6=0;
}

P1_0;  // Volle Behälter Ventil V1 schließen


if(P2_1=1) // Taster 2.1
{
P1_1=1;  //V2 Ventil geöffnet
}
else
{
P1_1=0;  //V2 Ventil geschlossen
}