At 11:48 AM +0800 11/20/09, gaochong wrote:
Thanks .
But the code is from cpan.org ,and is crappy ,where I will go ?

You figure out what is wrong with it and improve it. You have already been given some hints from people on this list. Here are some more:

1. Don't call subroutines with the '&' character.
2. Don't quote variables unnecessarily.
3. Don't capture in regular expressions if you do not use the captured strings.
4. Don't use string comparison operators to compare numbers.
5. Use a little more horizontal whitespace and a little less vertical whitespace.

Does the program do what you want it to do? If not, then let us know how it differs from your expectations.

--
To unsubscribe, e-mail: beginners-unsubscr...@perl.org
For additional commands, e-mail: beginners-h...@perl.org
http://learn.perl.org/


Reply via email to