Self Proclaimed King
Feedback on PHP Framework
I've been building projects on a home brewed PHP framework I built called Nobel (aka NO Bull***) framework.
I've gotten quite happy with it but wanted other peoples opinion on what sucks about it or is missing.
Its open sourced on github found here https://github.com/82rules/noble
thanks in advance
NogDog no cheating!
High Energy Magic Dept.
I'm telling the TPM's you must have too much free time.
OK! I can start
I have scanned your lib directory and 26 of 73 class methods is defined static. That indicates code smell and anti-pattern.
I dont understand why you are building your autoloader that way...
You are using namespaces so why not let them do the job for you?
public function __construct()
What about using a standard psr-0 autoload function that is implemented in 15 lines and supports nerly all new components found on the interwebz this days? I guess you want others to use your framework, and maybe expand it, as you have posted it on github?
Why are you defining constants of superglobals that already is available everywhere?
In the file 'noble':
Why not use some basic coding standards(psr-1) as uppercase first letter in both class names(you already do) and class files and just one file for each class(you have two classes in the file route.php)? In your autoloader you lowercase your class name so it fits your filename...
/// web request
Why not init your autoloader(Autoload::init()) in your nobel-file where you include it? You initialize it after the implementation in the autoloader class file and who other than you have a clue about that? Its also a bad mix of OO and procedural programming.
psr-1: 2.3. Side Effects
That was a start. If you want more just tell me
A file SHOULD declare new symbols (classes, functions, constants, etc.) and cause no other side effects, or it SHOULD execute logic with side effects, but SHOULD NOT do both.
Last edited by gammaster; 05-04-2014 at 09:20 AM.
have a nice OO day!
Users Browsing this Thread
There are currently 1 users browsing this thread. (0 members and 1 guests)