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 |
|
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.
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 .
|