Subject: | The ideia is good, but the... |
Summary: | Package rating comment |
Messages: | 2 |
Author: | Raul |
Date: | 2014-07-18 13:37:34 |
Update: | 2014-07-18 15:23:03 |
|
|
|
Raul rated this package as follows:
Utility: | Sufficient |
Consistency: | Bad |
Documentation: | Good |
Examples: | Sufficient |
|
![Picture of Raul Picture of Raul](/picture/user/811626.jpg) Raul - 2014-07-18 13:37:34
The ideia is good, but the consistency of the code may cause some problem...
It's ugly to see, even the ideia of one file framework, it's always better to organize all class in it's file. You can use an autoload function to load the classes dynamically and a bootstrap to load the requires classes.
![Picture of Mohammed Al Ashaal Picture of Mohammed Al Ashaal](/picture/user/1204878.jpg) Mohammed Al Ashaal - 2014-07-18 15:23:03 - In reply to message 1 from Raul
* till now the file is organized .
* horus already has it's own autoloader .
* i like it as a one file , Horus < 5 , was multiple files .
* i just want to make new thing also keep the code better .
|