in reply to Improving the efficiency of this piece of code
Two comments on your hard-coded HTML:
1. Stop It. You have already used CGI and imported all its methods? Use them. The functional approach is very easy to use, and will make this code a lot easier to read. Also, it is infinitely better to rely on a computer to properly close tags and other mechanical stuff, than it is to have to remember to do that stuff yourself. I know, I've wasted countless hours mucking around with simple HTML issues because it's hard to see what's what when it's scattered all over a perl script.
2. While some older standard of HTML made it possible to omit the quotes from your tag attributes, doing so just makes it that much harder to read. As a further note on #1, if you use the functional CGI approach for more than the headers, the module will take care of this for you.
|