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.