Re: [PEPr] +1 for System::System_Folders
| From: | Stefano F. Rausch | Date: | Sun, 05 Mar 2006 10:13:50 +0000 |
| Subject: | Re: [PEPr] +1 for System::System_Folders | ||
| References: | 1 | Groups: | php.pear.dev |
| Request: | Send a blank email to pear-dev+get-41661@lists.php.net to get a copy of this message | ||
Hi Michal,
Michal Felski wrote:
2. Each method (like getAppData(), getDesktop(), etc.) looks similar to others. In each of these methods you have defined an array and each method contains foreach. Im sure it can be refactored and some repeated code can be moved to private methods. After that you should have shorter code :) 3. you should move array with language data to begin of file or to separate config file. It could not be mixed with php codeThat's what I asked Christian for initially. However, he doesn't see the necessity for it. He argues that the language data is "static". But I fully agree with you that the refactoring should be looked at. -- Stefano