Wytykajcie błedy

[PHP] pobierz, plaintext 
<?php
###############################################################
# Adrian Staniszewski
# adrian_s@toya.net.pl
# GG: 5513236
#
#	File:		  Class.Template.php5
#	Version:	   0.2
#	Modif:		 7.11.2006
#	Description:   Kubu Template Class
#
##############################################################
 
class TT_Template {
 
  // _base id directory to template folder
  public $_base = 'template/';
  // _dir is directory to currently used template
  private $_dir = '';
  // _var is library with varibles and array
  public $_var = array();
  // _file is file library
  public $_file = array();
  // _files is currently used file
  private $_files = '';	
 
  public function __construct($Temp = 'base') {
	$this -> _dir = ''.$this -> _base.''.$Temp.'';
  }
 
  // function add is adding new file to _file library  
  public function add($file) {
	if(file_exists($this -> _dir.'/'.$file) && is_readable($this -> _dir.'/'.$file)) {
		$this -> _file[$file] = array('var' => array(), 'array' => array());
		// set currently used file
		$this -> _files = $file; 
	} else {
		echo 'Error !! Invalidy File.';
	}
  }
 
  // function write is adding new varibles or arrays to definited file varibles libr
ary 
  public function write($key, $value, $point='') {
   if($point=='') $point = $this -> _files;
	// checks is $value is array or varible
	if(is_array($value)) {
	  $this -> _var[$point]['array'][$key] = $value;
	} else {
	  $this -> _var[$point]['var']['{'.$key.'}'] = $value;
	}
  }
 
  // function getFile is getting sources of $point file
  private function getFile($point='') {
	if($point=='') $point = $this -> _files;
	  $this -> _file[$point]['source'] = @file_get_contents($this -> _dir.'/'.$point);
  }
 
  // function parseVar is replace flags for added varibles defined file
  private function parseVar($point='') {
	if($point=='') $point = $this -> _files;
	  $this -> _file[$point]['result'] = $this -> _file[$point]['source'];
 
			foreach ($this -> _var[$point]['var'] as $Key => $Value) {
			  $this -> _file[$point]['result'] = str_replace($Key, $Value, $this -> _file[$point]['result']);		
			}
  }
 
  // function parseWhile is adding loop While and replace flags for added varibles d
efined file
  private function parseWhile($point='') {
	  if($point=='') $point = $this -> _files;
		if($this -> _file[$point]['result']=='') $this -> _file[$point]['result'] = $this -> _file[$point]['source'];
		  // searching for While loop
		  preg_match_all('/{while@(.*?)}(.*?){while@end}/sie', $this -> _file[$point]['result'], $doWhile);
	  $c = 0;
		foreach($doWhile['1'] as $while) {
		  $Value = $doWhile['2'][$c];
		  $row = $this -> _var[$point]['array'][$while];
		  $B = count($row);
		  $b = 0; 
			while($b < $B) {
			  $Result = preg_replace('/{@(.*?)}/sie', '$row[$b][$1]', $Value);
			  $_Result .= $Result;
			  $b++;
			}
		  // saves results  
		  $this -> _file[$point]['result'] = str_replace($doWhile['0'][$c], $_Result, $this -> _file[$point]['result']);
		 $c++;
		}
  }
 
  public function Parse($point='') {
	if($point == '') $point = $this -> _files;
	  $this -> getFile($point);
	  $this -> parseVar($point);
	  $this -> parseWhile($point);
	echo $this -> _file[$point]['result'];
  }
 
}
?>
[PHP] pobierz, plaintext 

Przykładzik

[PHP] pobierz, plaintext 
<?php
$t = new TT_Template('standard');
$t -> add('index2.tpl');
$news[] = array('id' => '2', 'name' => 'gruszka');
$news[] = array('id' => '3', 'name' => 'pomarancz');
$tablica = array('jablko', 'gruszka', 'pomarancz');
$t -> write('title', 'to jest moja super strona !! musisz to zobaczyc');
$t -> write('meta', 'ISO-8859-2');
$t -> write('news', $tablica);
echo $t -> Parse();
?>
[PHP] pobierz, plaintext 

zródło :

[HTML] pobierz, plaintext 
<table border="1" align="center">
<tr>
<td>
<b>{meta}</b>
</td>
</tr>
<tr>
<td>
{while@news}<b>{@id}</b> - {@name}<br />{while@end}
</td>
</tr>
</table>
[HTML] pobierz, plaintext 

po parsowaniu:

[HTML] pobierz, plaintext 
<table border="1" align="center">
<tr>
<td>
<b>ISO-8859-2</b>
</td>
</tr>
<tr>
<td>
<b>2</b> - gruszka<br /><b>3</b> - pomarancz<br />
</td>
</tr>
</table>
[HTML] pobierz, plaintext 

[PHP] pobierz, plaintext 
<?php
$news[] = array('id' => '2', 'name' => 'gruszka');
$news[] = array('id' => '3', 'name' => 'pomarancz');
$tablica = array('jablko', 'gruszka', 'pomarancz');
$t -> write('title', 'to jest moja super strona !! musisz to zobaczyc');
$t -> write('meta', 'ISO-8859-2');
$t -> write('news', $tablica);
?>
[PHP] pobierz, plaintext 

Nie powinno być:

[PHP] pobierz, plaintext 
<?php
$t -> write('news', $news);
?>
[PHP] pobierz, plaintext 

[PHP] pobierz, plaintext 
<?php
public $_var = array();
?>
[PHP] pobierz, plaintext 

Czemu nie dasz co najmniej protected

[PHP] pobierz, plaintext 
<?php
} else {
		echo 'Error !! Invalidy File.';
	}
?>
[PHP] pobierz, plaintext 

Używasz PHP5 więc użyj wyjątków

[PHP] pobierz, plaintext 
<?php
// _base id directory to template folder
  public $_base = 'template/';
?>
[PHP] pobierz, plaintext 

Jak dajesz komentarze dawaj z PHPDoc'owe
+ Mala ilość kodu
- brak cache'u
- niespójne nazewnictwo np ($this -> _dir = ''.$this -> _base.''.$Temp.'';) - małe czy duże zdecyduj się
Do prostych zastosowań fajne, do bardziej zaawansowanych nie zdatne, choć pewnie nie takie było przeznaczenie.

Edit: Chyba lepsze byłoby forum Oceny

a jezeli tam gdzie mam przykładowo "{tytul}" chciał zrobic include jakiegos pliku php to jak?

Pobaw się ob_start" title="Zobacz w manualu php" target="_manual, ob_ get_ clean" title="Zobacz w manualu php" target="_manual, ob_ get_ contents" title="Zobacz w manualu php" target="_manual, bo ta klasa nie wspiera includowania

Ubogo w opcje. Ot lekki parser szablonów ;] Tylko po co? Skoro inny duży kombajn wygeneruje raz cache i bedzie dzialal szybciej. Nie wiem czy dobrze widze że brakuje instrukcji warunkowych(to taki totalne minimum), przydaly by sie jeszcze modyfikatory i tablice(jeśli są to przykłady moze jakies?)

Spokojnie, spokojnie
Pisałem to godzinke z kawałkiem

Nie uzywam php'Doc bo nigdy z niego nie korzystalem - stad teraz tez.. Na szybko napisalem wlasne komentarze, bo chcialem zobaczyc co swiat na to co napisalem. Dotychczas nie zajmowalem sie Szablonami - jest to moje pierwsze starcie z pisaniem wlasnej klasy - stad wersja 0.2.

Brak includowania jest tez dla mnie problemem dlatego postaram sie go dodac.

Klasa jest narazie malutka, ale nie kazdy potrzebuje duzy kombajn ;] A czy cach jest szybszy ? Moze, ale wyniki na poziomie 0.006sec mnie nie zalamuja ;] Cache dojdzie, ale dopiero po napisaniu kilku opcji.

no to przeniesiemy to jeszcze na wlasciwe forum i wszystko bedzie cacy

uzytkownik zagląda do tego dzialu z myśla ze znajdzie jakis gotowy, sprawdzony kod o jakiejs konkretnej funkcjonalnosci