Autorski framework, czy udostępnić? Opcje

[!*!]

dlatego nie ma wzorca projektowego MVC, lecz autorski model CCV (Controller - Component - View) więc co o tym sądzicie?

Jak coś nie ma MVC(lub pochodnych), to jest do dupy. I co to jest to CCV i czym to się różni od MVP czy MVA?

[by_ikar]

Wszystko co jest za throw nie zostanie wykonane, tak samo jak wszystko co jest za exit, die, return i podobnymi, więc ten return false tam jest zbędny. is_file zwraca tylko czy plik istnieje, a plik może istnieć ale możesz nie mieć do niego uprawnień. Warunek:

Kod

if(self::$baseClass[$strtolower])
{
    // some code
}

będzie wyświetlać notice jeżeli włączysz raportowanie błędów na E_ALL. require_once jest zbędny (i wolniejszy) jeżeli wcześniej i tak sprawdzasz czy klasa już istnieje.

Kiepsko trochę, sporo błędów logicznych. Nie zrozum mnie źle, ale śmieszne trochę jest kiedy mówisz o super optymalizacji, jednocześnie używając wyrażenia regularnego do wyszukiwania "stałego" ciągu. O czym nawet jest napisane w dokumentacji preg_match, że jak masz go użyć do wyszukania ciągu w podciągu, użyj strpos lub strstr. Niby to nie jest jakaś kolosalna różnica, ale to już widać na pierwszy rzut oka że nie bardzo wiesz jak dane funkcje/elementy języka działają. Tak samo mnie zastanawia, bo napisałeś że jest bezpieczny jak ktoś nie korzysta bezpośrednio z post, get etc. Czyli co, ty te dane traktujesz htmlspecialchar w jakiejś pętli, czy jak?

Konkluzja jest taka, że twój "framework" jest szybszy od innych, bo nie ma wielu innych ficzerów. Jakbyś miał z użyciem 2 frameworków napisać 2 takie same aplikacje, i użył byś swojego fw, jak i powiedzmy laravela; to jest wysokie prawdopodobieństwo że w obu przypadkach te prędkości czy użycie danych będą bardzo zbliżone.

Ten post edytował by_ikar 9.02.2015, 15:53:01

[Pyton_000]

@up Ba, ten kod który został pokazany można by było spokojnie odchudzić do ok 35 linijek i to nie dla tego że wszystko w jednej, ale wywalając część i zamieniając bardziej odpowiednimi elementami.

Poza tym te 2 metody autoloadu są podobne. Zmienia się w sumie na dobrą sprawę sprawdzenie czy klasa jest smarty i zmiana ścieżki.
Razi mnie też użycie referencji. Blee. Nie lubimy tego bo potem kwiatki wychodzą.

[PHP] pobierz, plaintext 
$object =& $class->newInstance();
                        self::$baseClass[$strtolower] =& $object;
                        return $object;
[PHP] pobierz, plaintext 

Na prawdę nie uważasz że to jest bez sensu?

[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

[solificati]

Witajcie
Chciałbym udostępnić autorski system framework, który teoretycznie (hmm, nawet praktycznie) kładzie inne systemy.

Minimum benchmarków: https://github.com/TechEmpower/FrameworkBenchmarks
Możesz zrobić pull requesta ze swoim frameworkiem.

[Neymar11]

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

Ja też tak mogę mówić:

O a ja robiłem testy na surówkach dwóch frameworków bez połączenia z bazą danych na "Hello world":

CodeIgniter: [0.011924028396606] 1 mb
Yii: [0.050626039505005] 5 mb
Autorski matrika: [0.005169153213501] 768 kb
Autorski Neymara11: [0.000000000000001] 1 b

No bo ja mam najlepsiejszy framework i nie mam tam zbędnych pierdouff co by tylko spowalniały ... [bla bla] [bla bla]
Daj dowód, udostępnij frameworka no i już. Każdy sobie oblooka skrytykuje i wszyscy będą "happy" (IMG:style_emoticons/default/smile.gif)

[peter13135]

Skoro kładziesz na lekkość, to po co w Twoim FW smarty ? PHP sam w sobie jest systemem szablonów.