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]

setXMLPath, to IMO powinno być w konstruktorze, z wymaganą zmienną path.

Jak zdziedziczę to mam metody $o->destroy() i $o->destroyDie(). Bezsensowne nazewnictwo. Poza tym, z tego co widzę to przed odczytem, muszę dać destroyDie(), które przejdzie przez cały system keszowania? Bzdura. Czyli jak mam 10000 plików keszu, to serwer klęka.

To powinno wyglądać tak, że przy odczycie danego elementu keszu, jest sprawdzany tylko jego TTL i usuwany w tym momencie.

[Fifi209]

setXMLPath, to IMO powinno być w konstruktorze, z wymaganą zmienną path.

Jak zdziedziczę to mam metody $o->destroy() i $o->destroyDie(). Bezsensowne nazewnictwo. Poza tym, z tego co widzę to przed odczytem, muszę dać destroyDie(), które przejdzie przez cały system keszowania? Bzdura. Czyli jak mam 10000 plików keszu, to serwer klęka.

To powinno wyglądać tak, że przy odczycie danego elementu keszu, jest sprawdzany tylko jego TTL i usuwany w tym momencie.

Nie wiem co w nazewnictwie jest nie tak. :< Wysłucham Twoich propozycji.

Przecież destroyDie odczyta, które są już nieużyteczne i je wywali - fakt mogłem to inaczej rozwiązać, poprawię to w kolejnej wersji.

Co do konstruktora, niestety odpada bo musiałbym nadpisać konstruktor z klasy głównej - co za tym idzie niepotrzebny kod + zmiany w dostępie do zmiennych w klasie głównej.

Ten post edytował fifi209 14.06.2009, 22:41:01

[kamil_biela]

Nie wiem co w nazewnictwie jest nie tak. :< Wysłucham Twoich propozycji.

destroyDie, jest nieintuicyjne. brzmi trochę jak jakieś combo w mortal combat.
cleanAll, checkAllLifetime, można pewnie jeszcze coś lepszego wymyślić.

Przeciez destroyDie odczyta, które są już nieużyteczne i je wywali - fakt mogłem to inaczej rozwiązać, poprawię to w kolejnej wersji.

Fakt, ale przechodzenie po wszystkich elementach mija się ogólnie z celem. Jako ogólna metoda ok, może się przydać, ale przy odczycie pojedynczego elementu mam sprawdzać wszystkie?

Co do konstruktora, niestety odpada bo musiałbym nadpisać konstruktor z klasy głównej - co za tym idzie niepotrzebny kod + zmiany w dostępie do zmiennych w klasie głównej.

a parent::__construct to już się nie da wywołać?

Ten post edytował kamil_biela 14.06.2009, 22:44:39