[PHP/OOP] Prosta klasa szablonów - co do poprawy? Opcje

[l0ud]

Witam Od jakiegoś czasu opracowuję sobie prostą klasę templatów, która wykorzystuję na swoich stronach. Na smarty jest dla mnie jeszcze za wcześnie, zresztą nie lubię takich wielkich 'krów' w małych projektach . Niestety w programowaniu obiektowym nie mam jeszcze żadnego doświadczenia, więc zapewne ta klasa wiele wspólngo z OOP nie ma... Tak więc proszę o ogólną koncepcję - jak to powinno być zbudowane właściwie, oraz ewentualne uwagi co poprawić/dorobić w klasie, bo aktualnie nie ma ona prawie żadnych możliwości...

Kod nie mieścił się w poście:
http://77.88.178.156/priv/

Jakby ktoś chciał wypróbować, przykład zastosowania:

[PHP] pobierz, plaintext 
<?php
include('templates_class.php');
$tpl = new templates('templates/','.tpl',1,2,0);
 
$data = array(array('Pierwszy element', 'drugie pole', 'cos innego'), array('Drugi element', 'cos innego', 'cos innego'));
 
$tpl->setvar('title','Spis elementów z tablicy');
 
for ($i=0; $i<count($data); $i++) {
	$tpl->setvar('data',$data[$i]);
	$tpl->to_buffer('rows','test','row');
}
 
$tpl->display('test','index');
 
?>
[PHP] pobierz, plaintext 

templates/test.tpl

[HTML] pobierz, plaintext 
<!-- BEGIN index -->
{#INC:#this/header}
<table>
{#BUF:rows}
</table>
<!-- END index -->
 
<!-- BEGIN header -->
<h1>{#VAR:title}</h1>
<!-- END header -->
 
<!-- BEGIN row -->
<tr>
<td>{#ARR:data.0}</td>
<td>{#ARR:data.1}</td>
<td>{#ARR:data.2}</td>
</tr>
<!-- END row -->
[HTML] pobierz, plaintext 

...wbrew pozorom tutaj nie ma żadnego cache, ta tablica tak nazwana jest tylko po to, żeby nie wczytywać jednego pliku 10 razy do takiego samego bloku Brak instrukcji warunkowych i pętli bo... nie wiem jak to ma wyglądać? Przecież w końcu po to są szablony, żeby nie umieszczać logiki pomiędzy styl?

Tak więc proszę o uwagi i pozdrawiam

Dodałem link do kodu ze swojego serwera, bo ten wyżej jak widzę padł albo usunął kod...
http://77.88.178.156/priv/

btw, nie oczekuję nie wiem jakiej odpowiedzi, chociaż wskazówkę jak to zrobić poprawnie w OOP

Ten post edytował l0ud 10.03.2008, 15:56:57

[carbolymer]

1. Przydało by się trochę komentarzy.
2. Bardziej skapsułkować kod, pojedyńcza operacja dla pojedyńczej metody pojedyńczej klasy. (w granicach rozsądku ofc)

Dokładniej jak to zrobić to ci nie powiem, to jest tylko i wyłącznie twoja decyzja, to ma być wygodne i funkcjonalne dla ciebie. Jeżeli ta postać ci wystarcza to pozostań przy niej.

Co do ładowania templatki, to lepiej by to było zrobić w odddzielnej metodzie niż w konstruktorze.

Ten post edytował carbolymer 11.03.2008, 22:13:22

[l0ud]

Dzięki za odpowiedź

Co do ładowania templatki, to lepiej by to było zrobić w odddzielnej metodzie niż w konstruktorze.

Hm... Zauważ że sam plik tpl jest ładowany przy metodzie display i gencode. W konstruktorze przekazuję tylko nazwę folderu i rozszerzenie dla wszystkich templatów, co raczej nie powinno się zmienić podczas działania skryptu. Jeżeli jednak jest to zrobić oddzielną metodą to napisz (bo teraz nie wiem czy nie doczytałeś, czy masz inne zdanie).

Co do komentarzy masz zupełną rację, wystawiam kod do oceny a nawet nie przygotowałem go, by inni zrozumieli. Ok, w najbliższym czasie to poprawię.

Pozdrawiam

Komentarze dodane Mam nadzieję, że to bardziej skłoni co niektórych do odpowiedzi

Pozdrawiam