Prosta klasa styli, templates - oddzielenie wyglądu od logiki Opcje

[Jifer]

Jako początkujący programista-pasjonat PHP prosiłbym bardziej doświadczonych użytkowników o skrytykowanie mojej pracy, pokazanie mi błędów lub wyjaśnienie jak poprawić poniższy skrypt (inne metody robienia tego samego, szybkosć i funkcjonalność)

Co chciałem osiągnąć:
-całkowicie oddzielić PHP od (X)HTML.
-by uzupełniane były fragmenty pomiędzy {}

[HTML] pobierz, plaintext 
Nick: {MY_NICK}
[HTML] pobierz, plaintext 

-by sekcje kodu mogły się powtarzać na podstawie elementów tablicy ( również zagnieżdżone )

[HTML] pobierz, plaintext 
<!--BEGIN REPEAT_SECTION-->
   {REPEAT_ELEMENT}
<!--END REPEAT_SECTION-->
[HTML] pobierz, plaintext 

-by w zależnosci od parametru można było włączać/wyłączać pewne sekcje (np. w zależności od tego czy zalogowany jest admin)

[HTML] pobierz, plaintext 
<!--ON BLOCK-->
   Jakiś fragment strony.
<!--OFF BLOCK-->
[HTML] pobierz, plaintext 

Dlaczego nie użyłem 'gotowych' rozwiązań:
-nie chciałem mieć w plikach stylów żadnych dodatkowych form sterujących (nie licząc 2 wymienionych) jak to np. jest w smartach i większości podobnych klas
-chciałem żeby kod mógł być zaimplementowany na serverach w których nie mam dostępu do plików konfiguracyjnych

Założenia:
-klasa wykorzystywana przez jedną osobę, nie stosowana w wielkich projektach, w których uczestniczą zespoły.
-łatwa w obsłudze
-w miarę szybka
-do obsługi raczej małych stron

Klasa:

[PHP] pobierz, plaintext 
<?php
class temp{
    private $t_name;
    private $temp_name;
    private $temp_array;
    //$t_name - nazwa pliku ze stylem lub gotowy styl (string)
    //$t_array - tablica z danymi
    //$t_type - 0=plik 1=string
    public function __construct($t_name,$t_array,$t_type){        
        if($t_type == 0){
            $this->t_name = $t_name;
            $this->temp_name = $this->GetFile();
        } else {
            $this->temp_name = $t_name;
        }        
        $this->temp_array = $t_array;    
    }
    //pobieranie pliku (domyœlnie zakładam 'templates/*.tpl'
    public function GetFile(){
        $fr = fopen('templates/'.$this->t_name.'.tpl', "r");
            if(!$fr){
                $content = "Nie wczytano pliku";
            } else {
                $content = fread($fr, filesize('templates/'.$this->t_name.'.tpl'));
            }
            @fclose($fr);
        return $content;
    }
    
    private function repl($str){    
        $output = preg_replace_callback("/({)([A-Z_]*)(})/",array($this,'check_key'),$str);
        return $output;
    }
    private function repl_ar($str){
        $output = preg_replace_callback('/<!--BEGIN ([A-Z_]*)-->([dDs]*)<!--END [A-Z_]*-->/',array($this,'check_key_array'),$str);
        return $output;
    }
    private function repl_on_off($str){    
        $output = preg_replace_callback('/<!--ON ([A-Z_]*)-->([dDs]*)<!--OFF [A-Z_]*-->/',array($this,'check_key_on_off'),$str);
        return $output;
    }    
    
    private function check_key_on_off($param){
        //TRUE - włšcza
        //FALSE lub brak - Wyłšcza
        if(isset($this->temp_array[$param[1]]) && $this->temp_array[$param[1]]) $send = $param[2];
        else $send = '';
        return $send;
    }
    private function check_key($param){
        if(isset($this->temp_array[$param[2]])) $send = $this->temp_array[$param[2]];
        else $send = '';
        return $send;
    }
    private function check_key_array($param){
        if(isset($this->temp_array[$param[1]])){            
            foreach($this->temp_array[$param[1]] as $lp=>$v_arr){
                $repe = new temp($param[2],$v_arr,1);
                $send = $send.$repe->ret();
            }
            return $send;
        }
    }
    
    private function change(){
        $first = $this->temp_name;
        while(preg_match('/<!--ON ([A-Z_]*)-->([dDs]*)<!--OFF [A-Z_]*-->/',$first)){
            $first = $this->repl_on_off($first);
        }        
        while(preg_match('/<!--BEGIN ([A-Z_]*)-->([dDs]*)<!--END [A-Z_]*-->/',$first)){
            $first = $this->repl_ar($first);
        }        
        while(preg_match("/({)([A-Z_]*)(})/",$first)){
            $first = $this->repl($first);
        }
        return $first;
    }
    //zwracanie zamienionego stylu
    public function ret(){
        $out = $this->change();
        return $out;        
    }    
}
?>
[PHP] pobierz, plaintext 

Kod wykonujący:

[PHP] pobierz, plaintext 
<?php
include("class/mytpl.php");
$y_repeat_tpl[1] = array(
    'OPT' => 'test_1'
    );
$y_repeat_tpl[2] = array(
    'OPT' => 'test_2'
    );
$z_repeat_tpl[1] = array(
    'OPT' => 'Test_01'
    );
$z_repeat_tpl[2] = array(
    'OPT' => 'Test_02'
    );
$z_repeat_tpl[3] = array(
    'OPT' => 'Test_03'
    );
        
$x_repeat_tpl[2] = array(
    'CL' => 'Pierwszy',
    'OPTION' => $z_repeat_tpl
    );
$x_repeat_tpl[3] = array(
    'CL' => 'Drugi',
    'OPTION' => $y_repeat_tpl
    );
 
$ar['SECTION'] = 'Sekcja';    
$ar['START_REPEAT'] = $x_repeat_tpl;
$ar['ALLOW'] = TRUE; //FALSE wylacza
 
//Utworzenie klasy
$test = new temp('test_template',$ar,0);
//wyświetlenie wygenerowanego kodu
echo $test->ret();
?>
[PHP] pobierz, plaintext 

Plik 'test_template.tpl'

[PHP] pobierz, plaintext 
<?php
Poczatek<br><br>
<!--ON ALLOW-->
    {SECTION} on/off<br><br>
<!--OFF ALLOW-->
 
<!--BEGIN START_REPEAT-->
Nazwa testu: {CL}<br>    
    <!--BEGIN OPTION-->
        -{OPT}<br>
    <!--END OPTION-->    
<!--END START_REPEAT-->
<br>
Koniec
?>
[PHP] pobierz, plaintext 

Tekst wyświetlany:

[HTML] pobierz, plaintext 
Poczatek
 
Sekcja on/off
 
Nazwa testu: Pierwszy	
-Test_01
-Test_02
-Test_03
Nazwa testu: Drugi
-Test_1
-Test_2
 
Koniec
[HTML] pobierz, plaintext 

Cierpliwym, którzy przeczytali całość i są gotowi ocenić moją pracę i pomóc w poprawieniu ewentualnych niedociągnięć lub udoskonalić tę klasę,
bardzo dziękuję.

[Jifer]

Dobra, załapałem. Chodzi o wielokrotne wywoływanie funkcji przeczesującej plik preg_ która przy większych szablonach będzie wolna.

Smarty przeglądałem i... no własnie, to co napisałem wyżej.
Smarty to nowy język szablonów. Wymaga wpakowania do szablonu tony dodatkowych znaczników. Taka nowa odsłona PHP.
Nie oddzielają kodu zupełnie. Wolę podać na szablon tablicę z wynikami, niż w szablonie pisać funkcje np. powtarzające fragment skryptu.
Dokładnie mi o to chodzi.

Dodatkowy problem jest taki, że jak mam zainstalować skrypt smartów na serverze w którym nie mam dostępu do katalogów konfiguracyjnych?
Chociażby '/usr/local/lib/php/Smarty/Smarty.class.php'.

Dlatego stworzyłem taką klasę. Stąd pytam: czy nie ma czegoś adekwatnego do smart, które będzie działało podobnie jak napisana przeze mnie klasa, tyle że szybciej.

Poza tym, o jakiej my szybkości mówimy. Często słyszę, że to jest szybsze od tamtego.
Przy jakich projektach i ilu odwiedzających zaczyna mieć to znaczenie? Ile potrzeba, żeby to 'wolniej' zacząć odczuwać?
(nie pracowałem na masowych projektach, więc nie wiem)

Ten post edytował Jifer 2.04.2009, 11:45:09