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