Re: [PEPr] +1 for Math::Math_Finance
| From: | Daniel Convissor | Date: | Wed, 25 May 2005 13:44:49 +0000 |
| Subject: | Re: [PEPr] +1 for Math::Math_Finance | ||
| References: | 1 2 | Groups: | php.pear.dev |
| Request: | Send a blank email to pear-dev+get-37822@lists.php.net to get a copy of this message | ||
Hi Folks:
> > left the TBill functions starting with an uppercase because TBill
> > stands for Treasury Bill and that's how everybody writes it.
While that makes sense, the method names don't. For example, "TBillYield"
should be something like "getTBillYield". The same thing applies to all
of your method names.
Also, please completely read the Sample File in the Coding Standards:
http://pear.php.net/manual/en/standards.sample.php
There are loads of
coding standards issues your source code is overlooking.
Thanks,
--Dan
--
T H E A N A L Y S I S A N D S O L U T I O N S C O M P A N Y
data intensive web and database programming
http://www.AnalysisAndSolutions.com/
4015 7th Ave #4, Brooklyn NY 11232 v: 718-854-0335 f: 718-854-0409