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ę.

A zastanów się, co jest wydajniejsze w celach produkcyjnych - prekompilowanie szablonów, czy obrabianie wszystkiego wyrażeniami regularnymi?

Zapewne prekompilowanie szablonów, bo w innym przypadku nie zadawał byś mi tego pytania :]
Będzie mi jednak miło, jeśli wyjaśnisz mi to pierwsze pojęcie. Być może wtedy zrozumiem na czym polega mój błąd.

Przeglądałem sporą ilość różnych klas służących do tego celu 1 w 95% przypadków opierają się one na rekurencji i właśnie wyrażeniach regularnych.

To chyba jakieś pamięciożerne klasy widziałeś.

Przejrzyj sobie źródła np. Smarty.

Kompilacja to przetwarzanie szablonow w 2ch etapach:
1. z ladniutkiego kodu szablonu z tymi wszystkimi {wow} i {ah} i {oh} zamieniasz na kod php - to moze byc dowolnie powolne bo robisz to raz (dopoki szablon sie nie zmieni)
2. majac juz prosty kod php - podstawiasz wartosci pod zmienne i includujesz ten plik - wszystko sie ladnie wypelni, masz petelki itp - to juz ma byc szybkie/bedzie

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)

A kto ci kaze tutaj trzymac smartiego? trzymasz go w swoim katalogu + ew. poprawiasz set_include_path().

No dobra... minęło trochę czasu od ostatniego posta.
Przekonaliście mnie do tego bym usiadł i trochę więcej popracował na smartach.

....i niestety, ale powoli zaczynam się przekonywać dlaczego tak mnie do nich wszyscy nakłaniają. Może nawet je polubię.