search

opengbu / gbuonline

GBU Online
5 stars 6 forks source link

redundant work. #10

Closed bhaweshchandola closed 9 years ago

bhaweshchandola commented 9 years ago

i noticed in most of the pages that you've written the code more than it is required like: capture (i am not pointing out this one in particular). now the thing is rather than writing this part again and again. you should construct a loop which will work until there is data available in database.

Varun-garg commented 9 years ago

Agreed, placing the same code 3-4 times makes it difficult to spot and erase the duplicate work with all the html formatting, and change it back to single with loops. The loops should have been constructed the first time and run like 3-4 times depending upon intended purpose.

bhaweshchandola commented 9 years ago

yes varun is absolutely right. so to explain to how it will be done i have worked a prototype. capture1 just take a look at this and work accordingly.