--- Quote Start ---
Thats because you built a lot of latches. In an asynchronous process (most of your code) must assign all signals in ALL brancches (and remember that means you need an else on ALL if branches).
I dont understand why you have a clock input, but nowhere in your code is it used as a clock.
You are also missing signals from at least 1 sensitivity list.
--- Quote End ---
i need clock signal to toggle the output diodes. will u help me by indicating which part of the needed "else" coz i dont have time.. within 1 week i have to design these all stuff in PCB.