using all analogue inputs of pic16f877a for adc

[duke.sir]

hello!
I am designing a system which uses a keypad and an lcd plus pic16f877a MCU
It is to work as such:
If i press say key one on my keypad An0 should be active and is displayed on the lcd such that when i vary its potentiometer it should vary live on lcd.
When i press two, it should leave the loop and enter into the second case and vary live on lcd also.
The problem is when it enters into any case it won't leave the loop until i rerun the simulation.
Please help me with the C statement which can make my circuit work live without restarting my simulation.

I used

[Ttelmah]

After you have read the key, you never read it again, so it's value will remain the same. So while(key==1) will always be true, and loop for ever. Same applies with every other value you use...

Also learn to use the code buttons, and read the data sheet about when you can use ADC_CLOCK_INTERNAL. You don't tell us your clock rate, but it is almost certainly not legal....

Best Wishes

[Mike Walne]

1) Are you using REAL hardware or computer SIMULATION?
2) Please use the code buttons to retain formatting.

Mike

[duke.sir]

i am using proteus for the simulation and for this purpose i use 20mhz as my internal clock frequency.

[duke.sir]

thanks you all for your support atleast am happy.
but really what i need is if some one can suggest a particular c statement that can make me live the loop of each case.
thanks and God bless you!!!

[Ttelmah]

You are missing the point.

There is no simple C statement, you need to change your code, so it checks the key again. This will depend on how the key data is actually being 'done' (you are receiving it via RS232, but what happens when the key is held down - does the source keep sending characters?).

You can loop for ever, exit immediately, etc. etc.. These are all decisions made by _your_ code. Currently though the code keeps looping, and never tests the key again, so 'of course' it'll loop for ever. Just remove the while statements, and it'll respond to the next character, but will only send the reply once for each key character. If your source sends has 'key repeat', so will keep sending the key value, for as long as the key is held down, this is all you need to do....

[rikotech8]

In my opinion you don't need driver for such a simple algorithm. You are using quite complicated code for implementation of a simple actions. I would suggest you to examine this https://drive.google.com/?tab=wo&authuser=0&urp=https://www.google.bg/?gws_rd%3Dcr%26ei%3DJwbdUriqFoHl4w#folders/0B_EYtdS7J5eKNGVqOVM5VlBFWW8 book. Refer to if - else conditions.
_________________
A person who never made a mistake never tried anything new.

[Mike Walne]

[duke.sir]

Thanks to you all but the are some security measures on the following link you gave
https://drive.google.com/?tab=wo&authuser=0&urp=https://www.google.bg/?gws_rd%3Dcr%26ei%3DJwbdUriqFoHl4w#folders/0B_EYtdS7J5eKNGVqOVM5VlBFWW8 book. Refer to if - else conditions.

can you make it available in order means?

[rikotech8]

Try this: https://drive.google.com/file/d/0B_EYtdS7J5eKZ1RaeEt4a2hlQnM/edit?usp=sharing

Also consider what Mike Walne says. Isis is not the most reliable tool for debugging.
_________________
A person who never made a mistake never tried anything new.