CacheSystem, Chciałbym abyście ocenili mój Cache System Opcje

[Fifi209]

[PHP] pobierz, plaintext 
<?php
# autor: fifi209 (Fast)
    # class: CacheSystem
    # version: 1.02
 
    class CacheSystem {
    
        const COMPRESSION_NONE=0;
        const COMPRESSION_BZIP2=1;
        protected $dir;
        private $compression;
        
        public function __construct($dir, $compression) {
            if (@opendir($dir)) {
                $this->dir = $dir;
                switch ($compression) {
                    case self::COMPRESSION_NONE:
                        $this->compression = false;
                        break;
                    case self::COMPRESSION_BZIP2:
                        $this->compression = true;
                        break;
                }
            }else{
                echo 'Podany katalog nie istnieje lub nie masz praw do jego otworzenia.';
            }
        }
        
        public function __get($cacheFile) {
            if (file_exists($this->dir.'/'.$cacheFile.'.cache')) {
                if ($this->compression == false) {
                    return file_get_contents($this->dir.'/'.$cacheFile.'.cache');
                }else{
                    $handle = bzopen($this->dir.'/'.$cacheFile.'.cache', 'r');
                    $text = bzread($handle, filesize($this->dir.'/'.$cacheFile.'.cache'));
                    bzclose($handle);
                    return $text;
                }
            }else{
                return false;
            }
        }
        
        public function __set($cacheFile, $value) {
            if ($this->compression == false) {
                file_put_contents($this->dir.'/'.$cacheFile.'.cache', $value);
                return true;
            }else{
                $handle = bzopen($this->dir.'/'.$cacheFile.'.cache', 'w');
                bzwrite($handle, $value);
                bzclose($handle);
                return true;
            }
        }
        
        public function destroy($cacheFile) {
            if ($this->$cacheFile != false) {
                unlink($this->dir.'/'.$cacheFile.'.cache');
                return true;
            }else{
                return false;
            }
        }
        
    }
?>
[PHP] pobierz, plaintext 

Napisałem ją dla siebie, ale może się komuś przyda. A teraz prosiłbym o jakieś uwagi dot. kodu, tj. co by można zmienić, zrobić inaczej.
Wydaje mi się, że jak na tą prostotę funkcjonalność jest wystarczająca. (przynajmniej dla mnie)

Użycie przykładowe:

Najpierw konstruktor:

[PHP] pobierz, plaintext 
<?php
$cache = new CacheSystem('cache', CacheSystem::COMPRESSION_NONE);
?>
[PHP] pobierz, plaintext 

Parametr pierwszy - ścieżka do folderu cache
Parametr drugi - używana kompresja (w tym wypadku brak, lecz "NONE" można zastąpić "BZIP2")

a) Dodawanie cache/Zmienianie jego wartości (konstruktor pomijam)

[PHP] pobierz, plaintext 
<?php
$cache->podstrona = 'Jakas wartosc';
?>
[PHP] pobierz, plaintext 

(IMG:http://forum.php.pl/style_emoticons/default/cool.gif) Odczyt cache

[PHP] pobierz, plaintext 
<?php
echo $cache->podstrona;
?>
[PHP] pobierz, plaintext 

c) Usuwanie cache

[PHP] pobierz, plaintext 
<?php
$cache->destroy('podstrona');
?>
[PHP] pobierz, plaintext 

podstrona -> nazwa pliku cache (IMG:http://forum.php.pl/style_emoticons/default/smile.gif)

Ten post edytował fifi209 14.06.2009, 22:54:36

[kamil_biela]

osobiście wolalbym pobierać/ustawiać zawartość keszu przez settera/gettera - wtedy w podpowiadaniu składni wyskakują odpowiednie metody.

Przydałoby się też keszowanie nagłówków strony.

I nie pogardziłbym możliwością ustawiania czasu życia.

edit:

hmm... i nigdzie nie widzę phpdoc'a

Ten post edytował kamil_biela 14.06.2009, 00:52:28

[Fifi209]

osobiście wolalbym pobierać/ustawiać zawartość keszu przez settera/gettera - wtedy w podpowiadaniu składni wyskakują odpowiednie metody.

Przydałoby się też keszowanie nagłówków strony.

I nie pogardziłbym możliwością ustawiania czasu życia.

edit:

hmm... i nigdzie nie widzę phpdoc'a

Co za problem zrobić cache nagłówków i zapisać np. jako headers ?
Czas życia? Przecież jak np. aktualizujesz coś w bazie to tylko zmieniasz zawartość cache, a jeżeli usuwasz coś z bazy to i wykonujesz destroy() dla danego pliku cache.

Czas życia - możesz dorobić przez cron'a (sprawdź kiedy plik został utworzony i po x czasie możesz go usuwać) (IMG:http://forum.php.pl/style_emoticons/default/haha.gif) [bez sensu]

[kamil_biela]

Co za problem zrobić cache nagłówków i zapisać np. jako headers ?

Taa... żaden. Cache_Lite lepsze. Tylko mi w nim brakuje keszowania nagłówków.

Czas życia? Przecież jak np. aktualizujesz coś w bazie to tylko zmieniasz zawartość cache, a jeżeli usuwasz coś z bazy to i wykonujesz destroy() dla danego pliku cache.

Czas życia - możesz dorobić przez cron'a (sprawdź kiedy plik został utworzony i po x czasie możesz go usuwać) (IMG:http://forum.php.pl/style_emoticons/default/haha.gif) [bez sensu]

Heh, tak powiedzmy że zmieniam coś 20 razy na sekundę w bazie danych i za każdym razę robię destroy?
Z tym cronem to nie przesadzaj, z natury człowiek leniwy jestem - mam sobie jeszcze zaprzątać głowę cronem, jakbym mógł jedną opcję ustawić?

Bez obrazy, ale czy nie popadłeś w samozachwyt nad własnym kodem? Chciałeś propozycje, to masz.

Aha, ja bym zwracał null'a zamiast false. Ale to tylko takie już moje bajdurzenie (IMG:http://forum.php.pl/style_emoticons/default/winksmiley.jpg)