what's wrong with this code?(microC & proteus)

[mailus]

sbit led1 at rb0_bit;
sbit led2 at rb1_bit;

sbit sw at ra0_bit;
sbit sw2 at ra1_bit;
void main() 
{
adcon1=0x07;
cmcon=0x07;
trisa=0xff;
trisb=0x00;
portb=0x00;

do{
 if(Button(&porta,0,10,1))
 {
 led2=1;
 led1=1;
 delay_ms(10);

 }while(sw=~sw);
 if(button(&porta,1,10,1))
 {
 led1=0;
 led2=0;
 }while(sw2=~sw2);
 delay_ms(10);
 }while(1);
}

when i use this code output assign function did not work correctly;
it sets the last assigned values only;
for example:

 if(Button(&porta,0,10,1))
 {
 led2=1;
 led1=1;
 delay_ms(10);

 }while(sw=~sw);

after executing this code output assigned only for led1 only;
here led2 is missed in this step;

please clarify my problem..............

 

[jayanth.devarayanadurga]

while(sw=~sw); should be while(sw==~sw); You have used = instead of ==

 

[mailus]

Thank you, but same problem exist.

 

[zuisti]

First:

while(sw=~sw); //??? sw is an input, so not writable

//correctly, as jayanth wrote:
while (sw == ~sw );

But this is meaningless (generally), can be omit (1 is never equal to 0)

2nd:
Your problem is a tipical RMW issue. Try this:

 if( Button(&porta,0,10,1) )
 {
   led2 = 1;
   /*  led1 = 1;   moved after the delay */
   delay_ms(10);
   led1 = 1;

 } 
// while ( sw == ~sw );

And look at the MikroC help, what says about the "Read Modify Write Problem"