[PHP]eval do szablonów - czy dobry pomysł? Opcje

[Evinek]

Przeglądałem różne skrypty for w poszukiwaniu dobrych rozwiązań. Padło na eval() z MyBB.
Zrobiłem taki kod:

#template.php (proszę nie krytykować, że to nie nadaje się na OOP ponieważ później tą klasę będę rozwijał)

[PHP] pobierz, plaintext 
<?php
class template{
	function parse($file){
		return file_get_contents('./html/'.$file.'.php');
	}
}
?>
[PHP] pobierz, plaintext 

#index.php

[PHP] pobierz, plaintext 
<?php
include('./template.php');
$template = new template;
eval("\$test = \"".$template->parse('index')."\";");
echo $test;
?>
[PHP] pobierz, plaintext 

Czy takie rozwiązanie jest pod względem optymalności i bezpieczeństwa dobre?
Jeśli są jakieś zagrożenia to jakie?

Jak testowałem 51 szablonów w jednym skrypcie, szablony:

[HTML] pobierz, plaintext 
Test: {$ttt}<br />
<ul>
{$linki}
</ul>
[HTML] pobierz, plaintext 

Użyte jeden raz

[HTML] pobierz, plaintext 
<li>$links</li>
[HTML] pobierz, plaintext 

Użyte 50 razy

, to czas na localhost (z pendrive) dawał: około 0.05s.
a na ugu: 0.0009s.

Przy większych plikach pewnie czas się zwiększy dlatego pytam was w tej sprawie.
Pewnie będę musiał też cachować te szablony, tylko pytanie jak?
Np. szablon header będzie posiadał zmienną {$login} więc tak czy siak będzie musiało przejść przez eval. Macie na to rady?


Z góry dziękuje i pozdrawiam, Paweł.

[Mephistofeles]

Zły. Eval to zło samo w sobie.
Dlaczego nie użyjesz gotowego silnika, np. Twiga?

[Evinek]

Po prostu nie chce następnego języka w CMSie. Chce prostote. Masz inne rady jak zrobić taki system szablonów?
Wystarczy mi aby były zmienne w szablonie. Potrzebuje właśnie coś ala MyBB.

[Mephistofeles]

Użyj Twiga. Skoro potrzebujesz tylko zmiennych, to nie ma problemu, nowej składni nie będzie zbyt wiele: {{ zmienna }}.
Można oczywiście bawić się np. z preg_replace, ale po co wynajdywać koło na nowo? Kiedyś stwierdzisz, że przyda się jeszcze np. pętla, i będzie trzeba znowu dopisywać, a w gotowym silniku masz to gotowe.

[Pilsener]

Chce prostote. Masz inne rady jak zrobić taki system szablonów?

Ja mam. Używa się systemu znanego lub prostego - zamiast tworzyć własny pseudo-język lepiej użyć coś znanego (np. SMARTY), albo po prostu via PHP (prostota):

[HTML] pobierz, plaintext 
<p>
<?php echo $this->variable ?>
</p>
[HTML] pobierz, plaintext 

Dla kogoś, kto babrze się w HTMLu nie ma większego znaczenia język szablonu, byle HTML był oddzielony.

[by_ikar]

@Mephistofeles - twig korzysta z evala, ale tylko w sytuacji kiedy nie korzysta się z cache: https://github.com/fabpot/Twig/blob/master/...onment.php#L327 fakt jest to złe, ale jego użycie, czasami jest uzasadnione, bo inaczej danego zadania nie da się zrealizować. Próbowałem kiedyś jakoś to obejść, i nie użyć evala, ale zwyczajnie się nie da, oczywiście mówię o sytuacji kiedy tego cache nie ma.

Dla kogoś, kto babrze się w HTMLu nie ma większego znaczenia język szablonu, byle HTML był oddzielony.

Dokładnie tak! Niektórzy uważają że to jest dla frontendowca łatwiejsze do zapamiętania, zrozumienia ITP, a jest to tak na prawdę gówno prawda

[Mephistofeles]

Wiem, że eval czasami jest nieunikniony, ale jednak zastosowanie evala w gotowych systemach jest nieco inne, niż proponuje to autor tematu. Tam jest wykorzystywany do wykonania skompilowanego kodu szablonu, więc niewiele się to różni od wykonania zapisanego pliku.

[Evinek]

No to już nie wiem jak to zrobić.
Chce robić coś takiego:

[PHP] pobierz, plaintext 
<?php
<?php
$template = new template;
 
$template->load('link'); //ładowanie pliku do template
$links = '';
for($i = 0; $i < 10; $i++){
	$link['link'] = 'http://google.pl';
	$link['title'] = 'Poszukaj w google '.$i;
	$links .= $template->parse('link'); //Parsowanie szablonu który wcześniej został załadowany - dodatkowo przypisanie go do zmiennej
	/*
	#link <--szablon
	<li><a href="$link['link']">$link['title']</a></li>
	//Czyli stworzy po prostu link z szablonu
	*/
}
 $template->load('menu'); //ładowanie menu
$menu = $template->parse('menu'); //parsowanie
/*
#menu <--szablon
<ul>
$links
</ul>
//Doda do menu linki
*/
 
$template->display('index');
/*
#index <--szablon
Bla bla bla
$menu
Bla bla bla
//Wyświetli stronę z dołączonym menu
*/
?>
[PHP] pobierz, plaintext 

Zależy mi na tym aby wszystkie zmienne mógłbym używać w szablonie. Czyli replace odpada bo jak będzie tablica to za dużo zabawy.
Wolałbym swój kod który mógłbym łatwo edytować. Chce własne klasy tylko do swoich projektów. Jeśli coś będę robił komercyjnego to nie będę się bawił w tworzenie własnego kodu tylko będę brał gotowce.
Ważne jest abym mógł właśnie z szablonu tworzyć takie linki po czym dodawać je do szablonu innego (połączyć to) po czym w końcu ten cały szablon dać do głównego szablonu (przykład powyżej). Chce ominąć rzeczy w stylu:
$links .= '<a href="...">...</a>';
ponieważ chce maksymalnie oddzielić HTML od PHP.

Co mi możecie polecić? Szukam na ten temat już z ponad dwa miesiące. Wiele stron przeszukałem i nadal nic.

Ten post edytował Evinek 24.06.2012, 16:54:48