[php]Problem z klasą systemu template'ów Opcje

[qbejs]

Dopiero zaczynam OOP w php i mam do Was prośbę. Piszę sobie system skórek jednak mam problem z metodą build(). Mianowicie jak widać w listningu mam dwa wywołania metody add() która przekazuje tag oraz zawartosc na ktora ma byc zamieniony tag. Jednak u mnie skrypt bierze pod uwagę jedynie wartosci przekazane przez drugą deklaracje metody add(). Co należy poprawić aby brał pod uwagę wszystkie deklaracje metody add() przy przetwarzaniu metody build()

[PHP] pobierz, plaintext 
<?php
class Template{
		function Template()
	{
 
	}
 
	function setPath($path)
	{
		$this->tpl_path = $path;
	}
 
	function setTemplate($template)
	{
		$this->tpl = $template;
	}
 
	function add($text,$code)
	{
		$this->tag = array($text => $code);
	}
 
	function build()
	{
 
		if (count($this->tag) != 0)
		{
			foreach ($this->tag as $text => $code) 
			{
				$tpl = file_get_contents($this->tpl);
				$output = eregi_replace("{". $text ."}",$code,$tpl);
				echo $output;
			}
		}
		else
		die("Brak zdefiniowanych tagów");
	}
 
	function benchTimeStart()
	{
		list($usec, $sec) = explode(" ",microtime()); 
		$this->time = ((float)$usec + (float)$sec); 
	}
 
	function benchTimeStop()
	{
		list($usec, $sec) = explode(" ",microtime()); 
		$time = ((float)$usec + (float)$sec);
		$time_end = $time - $this->time;
		echo number_format($time_end,6,'.','');
	}
}
 
 
$tpl = new Template();
$tpl->setTemplate("test.tpl");
$tpl->add("IMIE","Jan");
$tpl->add("NAZWISKO","Kowalski");
$tpl->build();
}
?>
[PHP] pobierz, plaintext 

Ten post edytował qbejs 17.10.2007, 16:18:45

[matix]

Przede wszystkim za każdy razem gdy wywołasz metode add(), usuwasz sobie wcześniej zapisane dane z zmiennej. Ta metoda powinna wyglądać tak:

[PHP] pobierz, plaintext 
<?php
$this->tag[] = array($text => $code);
?>
[PHP] pobierz, plaintext 

Myślę, że to jest ten twój problem (IMG:http://forum.php.pl/style_emoticons/default/winksmiley.jpg)

Pozdro

[qbejs]

Ok, a teraz doradzcie jak zrobić żeby metoda build odrazu pokazywala calosc wraz z tagami zmienionymi na kod bo aktualnie wywala mi tyle razy plik tpl ile razy uzywalem metody add()

[Cysiaczek]

Zobacz, że dla każdego tagu ($this->tag) wywołujesz wczytanie pliku od nowa. Przenieś to przed pętlę.

Pozdrawiam.

Aha.
Popraw proszę tytuł topiku na zgodny z zasadami forum Przedszkole

[qbejs]

problem rozwiązany. Problem leżał także w metodzie add() gdzie powinno być $this->tag[$text] = $code;

Jeszcze jedna sprawa, dopisałem sobie implementacje pętl. Jednak wogole mi ich nie parsuje. Może Wy znajdziecie błąd bo mnie juz brakło pomysłów

[PHP] pobierz, plaintext 
<?php
function build()
{
		if (count($this->tag) != 0)
		{
			$handle = fopen($this->tpl,"r");
			$tpl = fread($handle,filesize($this->tpl));
 
			foreach ($this->tag as $text => $code) 
			{
					if (is_array($code))
					  {
					   if (preg_match("/{loop:".$name."}(.*?){endloop:".$name."}/is", $tpl, $loop))
					   {
 
						   $temp_string = '';
 
						   for ($i = 0; $i < count($code); $i++)
						   {
							   $k = '';
							   $v = '';
							   $counter = 0;
 
							   foreach($code[$i] as $key => $val)
							   {
								   $k[$counter] = "/{".$key."}/";
								   $v[$counter] = $val;
								   $counter++;
							   }
 
							   $temp_string .= preg_replace($k, $v, $loop[1]);
						   }
 
							$tpl = preg_replace("/{loop:".$name."}(.*?){endloop:".$name."}/is", $temp_string, $tpl);
						}
						else
						{
						  echo "Error !";
						}
					}
					else
					{
					  $tpl = str_replace('{'.$text.'}',$code,$tpl);
					}
			}
				echo $tpl;
		}
		else
		die("Brak zdefiniowanych tagów");
	}
?>
[PHP] pobierz, plaintext 

wywołuje to tak

[PHP] pobierz, plaintext 
<?php
$tablica[] = array("imie" => "jan","nazwisko" => "kowalski");
add("pracownicy",$tablica[]);
build();
?>
[PHP] pobierz, plaintext 

a efekt tego jest taki

[HTML] pobierz, plaintext 
{loop:pracownicy} {endloop:pracownicy}
{imie} 	{nazwisko}
[HTML] pobierz, plaintext 

Czyli krótko mówiąć nie ma żadnego efektu

Ten post edytował qbejs 17.10.2007, 19:31:16