[Klasa][PHP7] Cache Opcje

[KsaR]

Źródło:
https://github.com/KsaR99/php-cache

Plik: class.Cache.php, zawiera klasę.
Plik: cache.php, to przykładowe zastosowanie.

Klas cache są miliony jednak chciałem stworzyć swoją własną, prostą. (IMG:style_emoticons/default/biggrin.gif)

Do oceny:

• Użyteczność
• Co można poprawić żeby sobie/innym życia nie skomplikować
• Jakość kodu (ciągle uczę się nią polepszać)

Zastanawiam się nad metodą do kasacji cache,
Jednak cache zazwyczaj się nie kasuje a podmienia.
Warto dodać? Jeśli tak to prosił bym też o przykład - po co.

Wszelkie uwagi mile widziane. (IMG:style_emoticons/default/tongue.gif)

Ten post edytował KsaR 8.06.2016, 17:51:54

[com]

Tylko że w systemie nie masz x instancji tego cache raczej, tylko jedną.

Użyć setterów zamiast parametrów

Nie, parametry są lepsze, settery powinny zginąć. Obiekt cache to immutable, nie można zmieniać w czasie działania tego, bo można sobie narobić więcej problemów niż pożytku.
No chyba, że zrobimy tak jak w psr7 i będziemy zwracać nowy obiekt (IMG:style_emoticons/default/smile.gif)
Dlatego też zrobiłem z tego static, ale nie testowałem i co słusznie zauważył nospor, zrobił się nam singleton.


I jak już to return; null tam jest nadmiarowy (IMG:style_emoticons/default/wink.gif)

Na github twierdzisz ze jesteś za optymalizacja to czemu upierasz się nadal na te piramidę

[PHP] pobierz, plaintext 
if (isset($path)) {
            if (!file_exists($path)) {
                throw new \Exception('Path:'."\r\n".$path."\r\n".'Doesn\'t exists.');
            }
            $this->path = $path;
        } else {
            throw new \Exception('Please specify path for cache.');
        }
[PHP] pobierz, plaintext 

KsaR

Widać brak Ci trochę podstaw, bo twojego konstruktora nie dało się wywołać inaczej niż z 3 parametrami, który w dodatku powinien być zgodny z typem wiec te (IMG:style_emoticons/default/questionmark.gif) nawet nie miało sensu. Bo nawet wstawienie nulla wyrzuca błąd typu, jak już chcesz używać php7 to wypadało by trochę się z nim zaznajomić.

Dlatego chciałem go dodać w tym pull requescie i w końcu po poście Pyton_000 je tam ustawiłem.

Ten post edytował com 9.06.2016, 20:14:41