On Friday 12 November 2010 11:00:16 Uri Guttman wrote: > >>>>> "CY" == Chaitanya Yanamadala <dr.virus.in...@gmail.com> writes: > i don't know the excell stuff but your code needs to be improved a bit. > > > CY> my $oExcel = new Spreadsheet::ParseExcel; > > don't use camelcase names. if the leading 'o' is for object, don't do > that. hungarian notation is also a bad idea. >
One should also do "Spreadsheet::ParseExcel->new;" instead of "new Spreadsheet::ParseExcel" (which is so-called indirect-object notation). See: http://perl-begin.org/tutorials/bad-elements/ Regards, Shlomi Fish -- ----------------------------------------------------------------- Shlomi Fish http://www.shlomifish.org/ My Public Domain Photos - http://www.flickr.com/photos/shlomif/ <rindolf> She's a hot chick. But she smokes. <go|dfish> She can smoke as long as she's smokin'. Please reply to list if it's a mailing list post - http://shlom.in/reply . -- To unsubscribe, e-mail: beginners-unsubscr...@perl.org For additional commands, e-mail: beginners-h...@perl.org http://learn.perl.org/
