Could you please share exactly what kind of abnormality you have in the output?

I’ve tried to modify your code. Please try this also and update the result.

void main()
{
    TRISA=0b01101;             // Configure RA0,RA2,and RA3 as input sensor.
    PORTA=0b00000;
    TRISB=0b00000000;          //Configure ALL PORTB as output
    PORTB=0b00000000;
    
    while(1)                   //Loop forever
    {
        if(PORTA.F2==1)            //Analogue Distance Sensor
        {
            PORTB.F3=1;                //Buzzer ON
            delay_ms(400);             //time delay wait for 0.4s
        }
        else
        {
             PORTB.F2=0;                //Buzzer OFF
             delay_ms(50);              //time delay wait for 0.05s
        }

        if(PORTA.F3==1)            //Water sensor
        {
            PORTB.F2=1;                //Vibrator ON
            delay_ms(400);             //time delay wait for 0.4s
        }
        else
        {
             PORTB.F3=0;                //Vibrator OFF
            delay_ms(50);              //time delay wait for 0.05s
        }
    
                       
        if(PORTA.F0==0)             //Dark sensor
        {
            PORTB.F4=1;                 //Blue LED ON
            PORTB.F5=0;                 //RED LED OFF
            PORTB.F6=0;                 //WHITE LED OFF
            delay_ms(200);              // time delay wait for 0.2s
            
            PORTB.F4=0;                 //Blue LED OFF
            PORTB.F5=1;                 //RED LED ON
            PORTB.F6=0;                 //WHITE LED OFF
            delay_ms(200);
            
            PORTB.F4=0;                 //Blue LED OF
            PORTB.F5=0;                 //RED LED OFF
            PORTB.F6=1;                 //WHITE LED ON
            delay_ms(200);
        }    
        else
        {
             PORTB.F4=0;                //Blue LED OFF
             PORTB.F5=0;                //RED LED OFF
             PORTB.F6=0;                //WHITE LED OFF
             delay_ms(50);              //time delay wait for 0.05s
        }
    }
}

When the sensor detect obstacle, the buzzer will ON, and when the LDR detect dark condition, The 3 LED will running, when both of this sensor function at the same time. The Buzzer always ON, and LED running and then stop. the buzzer still ON.

The buffer remains ON because the code for turning the buffer OFF is included in the ‘else’ of  dark sense check. So when the dark sensing is TRUE, the statements inside the else won’t get executed.

Have you tried the code modified by me?

yes, i’ve already try your modified code, but it still won’t function as well.

Hello Chelah,

I think i have a solution to your problem. Will post you soon.

Thank you !!!

Hello Chelah,

I think i have a solution to your problem. Will post you soon.

Thank you !!!

Try This & let me know the result

void main()
{
    TRISA=0b01101;             // Configure RA0,RA2,and RA3 as input sensor.
    PORTA=0b00000;
    TRISB=0b00000000;          //Configure ALL PORTB as output
    PORTB=0b00000000;
    
    while(1)                   //Loop forever
    {
        if(PORTA.F2==1)            //Analogue Distance Sensor
        {
            PORTB |= 0b00001000;                //Buzzer ON
            delay_ms(400);             //time delay wait for 0.4s
        }
        else
        {
             PORTB &= 0b11110111;                //Buzzer OFF

             delay_ms(50);              //time delay wait for 0.05s
        }

        if(PORTA.F3==1)            //Water sensor
        {
            PORTB |=0b00000100;                //Vibrator ON
            delay_ms(400);             //time delay wait for 0.4s
        }
        else
        {
             PORTB &= 0b11111011;                //Vibrator OFF
            delay_ms(50);              //time delay wait for 0.05s
        }
    
                       
        if(PORTA.F0==0)             //Dark sensor
        {
            PORTB |= 0b00010000;                 //Blue LED ON
            PORTB &= 0b11011111;                 //RED LED OFF

            PORTB &= 0b10111111;                //WHITE LED OFF
            delay_ms(200);              // time delay wait for 0.2s
            
            PORTB &= 0b11101111;                 //Blue LED OFF
            PORTB |= 0b00100000;                 //RED LED ON
            PORTB &= 0b10111111;                  //WHITE LED OFF
            delay_ms(200);
            
            PORTB &= 0b11101111;   //Blue LED OF
             PORTB &= 0b11011111;                 //RED LED OFF
             PORTB |= 0b01000000;               //WHITE LED ON
            delay_ms(200);
        }    
        else
        {
               PORTB &= 0b00000000;
             delay_ms(50);              //time delay wait for 0.05s
        }
    }
}

Try This & let me know the result

void main()
{
    TRISA=0b01101;             // Configure RA0,RA2,and RA3 as input sensor.
    PORTA=0b00000;
    TRISB=0b00000000;          //Configure ALL PORTB as output
    PORTB=0b00000000;
    
    while(1)                   //Loop forever
    {
        if(PORTA.F2==1)            //Analogue Distance Sensor
        {
            PORTB |= 0b00001000;                //Buzzer ON
            delay_ms(400);             //time delay wait for 0.4s
        }
        else
        {
             PORTB &= 0b11110111;                //Buzzer OFF

             delay_ms(50);              //time delay wait for 0.05s
        }

        if(PORTA.F3==1)            //Water sensor
        {
            PORTB |=0b00000100;                //Vibrator ON
            delay_ms(400);             //time delay wait for 0.4s
        }
        else
        {
             PORTB &= 0b11111011;                //Vibrator OFF
            delay_ms(50);              //time delay wait for 0.05s
        }
    
                       
        if(PORTA.F0==0)             //Dark sensor
        {
            PORTB |= 0b00010000;                 //Blue LED ON
            PORTB &= 0b11011111;                 //RED LED OFF

            PORTB &= 0b10111111;                //WHITE LED OFF
            delay_ms(200);              // time delay wait for 0.2s
            
            PORTB &= 0b11101111;                 //Blue LED OFF
            PORTB |= 0b00100000;                 //RED LED ON
            PORTB &= 0b10111111;                  //WHITE LED OFF
            delay_ms(200);
            
            PORTB &= 0b11101111;   //Blue LED OF
             PORTB &= 0b11011111;                 //RED LED OFF
             PORTB |= 0b01000000;               //WHITE LED ON
            delay_ms(200);
        }    
        else
        {
               PORTB &= 0b00000000;
             delay_ms(50);              //time delay wait for 0.05s
        }
    }
}

hi chelah,

                You need some changes in you coding. use connection carefully . your obstacle sensor is connected with RA3, water sensor with RA2, dark sensor with RA0, buzzer with RB2 & vibrator with RB3 as per your circuit diagram. use this connection all where. you have used RB3 for your buzzer while switching on & RB2 for buzzer while switching of. again use code if and else for all the condition. try the and tell me what happen.

amit jha