Electronic – PIC program LED on off with same push button

Your problem is probably caused by the Read-Modify-Write effect, see page 31 of the data sheet. You should use a shadow register, set and reset the bits in that, and write it to the port. Delays usually help, so I find it surprising that it caused you problems. The fact that you could write bytes to the port and it works as expected indicates that it probably is an R-M-W effect.

Here is a good description of the problem.

The 18F PICs have latch registers for the ports, and don't suffer from the R-M-W problem.

1. Firstly, you always need series resistors with the LEDs when driven from a voltage source (e.g. battery, DC supply, etc)
   This is because LEDs have a non-linear I-V curve, which looks like a high impedance up to the threshold voltage of the LED then rises very sharply, so it means with a very slight change of voltage the current changes a lot, making it almost impossible to set the current to a stable value in this manner.
   By using the correct value series resistors, you ensure the current cannot rise enough to damage the LED.
   To calculate the resistor value you need to know the LED forward voltage (Vf) then subtract Vf from supply voltage and divide by desired current, e.g for a 5V supply, a 2V Vf and 15mA:

   (5V - 2V) / 0.020A = 200Ω (The standard value of 220Ω will do - if you don't have this then aim for anything between 150Ω and 600Ω for a 20mA to 5mA range)
   This is assuming a typical 5mm or 3mm LED of 20mA max operating current.

2. Although it says "3 x 1.5V Battery in Series" in your schematic, the batteries look like they may be actually connected in parallel. T confirm, the batteries need to be connected end to end like the bottom diagram in this image:

   

3. You need decoupling capacitors present between the microcontroller Vdd and ground. I won't go into detail (search on here, there are many good answers on this subject) but they are basically to provide the microcontroller with a low impedance local energy reserve for high frequency current demand, which the power supply cannot respond to quickly enough.
   Ideally you should place one (100nF or 1uF ceramic is pretty standard) across the power and ground pins, as near to the pins as possible.

4. Make sure you are using the internal oscillator if you have no crystal present. Your code does not show the config bit settings, if you have left them out you need to add them to ensure the microcontroller is setup correctly. The CCS manual should tell you how to do this. Also in the config bits, make sure the Watchdog Timer is set to off, otherwise your micro will continually reset (unless you call the WDT clear command regularly)

5. Make sure that you have the LEDs the right way round.

6. Make sure you have the MCLR pin tied high, otherwise your PIC will be held in reset (unless you turn MCLR off in the config bits) This is usually done with a resistor to Vdd, value around 10kΩ. The datasheet will have an example of this in the reset section. (thanks to ajs410 for mentioning this)