Autorski framework, czy udostępnić? Opcje

[matrik]

Witajcie
Chciałbym udostępnić autorski system framework, który teoretycznie (hmm, nawet praktycznie) kładzie inne systemy.
Wykonałem prosty test na surówkach dwóch frameworków. Konfig domyślny i bez połączenia z bazą danych.
Zwykłe wyświetlenie tekstu "Hello world!". Bez wykorzystania eAccelerator'a

CodeIgniter: [0.011924028396606] 1 mb
Yii: [0.050626039505005] 5 mb
Autorski: [0.005169153213501] 768 kb

System jest okrojony i pozbawiony śmieci takich jak generatory do tworzenia formularzy czy też tworzenie zapytań SQL.
Tworzony był dla szybkiego stawiania serwisów (wiele stron korzystało i korzysta z tego systemu), dlatego nie ma wzorca projektowego MVC, lecz autorski model CCV (Controller - Component - View) dzięki któremu osiąga takie wyniki (ale czepiając się innych frameworków zasada działania jest bardzo zbliżona).
Ładowarka klas Singleton i Router nieszablonowego wywoływania kontrolerów.

Yii czy CI nie powinien ładować bibliotek, których nie potrzebuje aktualnie więc co o tym sądzicie?

[vokiel]

Ok, skoro jest kod do review, to zrobię pobieżny (uwagi w komentarzach)

[PHP] pobierz, plaintext 
<? // krótka notacja tagów otwierających może przysporzyć problemów na serwerach bez ustawionej flagi short_open_tag w php.ini
class woBase{
        static public $baseClass = array();
 
        function run(){ // brak określenia widoczności metody
                return self::getClass('Application')->start();
        }
 
        static function getClass($name,$run=true){ // po co te $run?
                $strtolower = strtolower($name); // nazywanie zmiennych nazwami funkcji to nie jest dobra praktyka
                //echo WOPATCH.'core/'.$name.'.php<br />';
                if(self::$baseClass[$strtolower]){ // NOTICE przy braku elementu w tablicy, tablica jest public, to co jeśli ktoś podmieni obiekt klasy $name na inny?
                        //echo '- '.$name.'<br />';
                        return self::$baseClass[$strtolower];
                }else{
                        //echo '+ '.$name.'<br />';
                        if(!class_exists($strtolower,false)){
                                if(is_file(WOPATCH.'core/'.$name.'.php')){
                                        require WOPATCH.'core/'.$name.'.php';
                                }else{
                                        throw new ErrorHandler('getClass - Nie odnaleziono pliku klasy o nazwie '.$name); // ErrorHandler - chyba raczej Exception lub WoException, handler byłby tym co przechwyci wyjątek
                                        return false; // zbędne, nie wykona się
                                }
                        }
                        $class = new ReflectionClass($strtolower); // jaki zysk z zastosowania Reflection? Czemu nie łapiesz wyjątku ReflectionException?
                        $object =& $class->newInstance(); // a co jeśli nowa klasa wymaga przekazania parametrów do konstruktora? po co referencja @Pyton_000
                        self::$baseClass[$strtolower] =& $object; // po co znów referencja @Pyton_000
                        return $object; 
                         // skoro nie korzystasz z przekazywania zmiennych do konstruktora, z ReflectionException, to wystarczy takie rozwiązanie
                        return self::$baseClass[$strtolower] = new $strtolower();
                }
        }
 
 
       // w zasadzie autoload od getClass różni się tylko dwiema kwestiami: sprawdzaniem czy jest to smarty oraz tym, że getClass tworzy nową instancję obiektu, duża duplikacja kodu jak na działanie w kierunku optymalizacji. Może jednak pójdź w stronę $run=true i połącz zduplikowany kod.
        public static function autoload($name){
                $strtolower = strtolower($name);
                $dir = WOPATCH.'core/'.$name.'.php'; // $dir to raczej $file, mocno mylące
                if(preg_match('/smarty_/',$strtolower)){ // strpos lub strstr @by_ikar
                        $dir = WOPATCH.'core/Smarty/sysplugins/'.$strtolower.'.php'; // znów $dir wskazujący na plik
                }
                        if(!class_exists($name,false)){
                                if(is_file($dir)){ // c.d. nieścisłości, wiem, że to tylko nazwa zmiennej, jednak za jakiś czas takie nieścisłości mogą spowodować, że głupiego błędu będziesz szukał dużo dłużej
                                        require_once $dir;
                                        return true;
                                }else{
                                        throw new ErrorHandler(get_class($this).' - Not found class '.$name); // Exception nie ErrorHandler
                                        return false; // nie wykona się
                                }
                        }
        }
}
spl_autoload_register(array('woBase','autoload'));
[PHP] pobierz, plaintext 

Ten post edytował vokiel 9.02.2015, 21:17:42