Cześć. Wziąłem się za pisanie systemu szablonów (nie mówić mi tu o Smarty bo i tak go nie użyję bo jest on jak dla mnie zbyt dużą aplikacją a po drugie do nauki postanowiłem napisać własny .)

Ok do rzeczy mam taki kod class.temp.php:

[PHP] pobierz, plaintext 
<?php
class new_template
{
private $sTemplate;  // Szablon
private $sReplacement; // Zamienniki
 
	public function __construct($rFile_name, $z, $na)
	{	
		$this->file_name = $rFile_name ;
		$this->template = $sTemplate ;
		$this->replacement = $sReplacement ;
		$this->z = $z ;
		$this->na = $na ;
	}
 
	public function read()
	{
		if(!empty($this->file_name)) ;
		if(!file_exists($this->file_name))
		{
		trigger_error(' Błąd. Nie ma takiego pliku. ') ; 
		return true ;
		}
 
		$this->template = @file($this->file_name) ;
		$this->template = imlode("", $this->template);
		}
 
	public function add_replacement()
	{
		if(!empty($this->z))
		{
		$this->replacement[$this->z] = $this->na ;
		}
	}
 
	public function display()
	{
		$this->out = $this->templates ;
		foreach($this->replacement as $a => $b)
		{
		$this->out = str_replace("{$a}", $b, $this->out) ;
		}
 
		return $this->out ;
	}
}
 
?>
 
[PHP] pobierz, plaintext 

Oraz w index.php:

[PHP] pobierz, plaintext 
$szab = new new_template('templates/index.tpl', '{a}', $cos) ;
echo($szab->display());
[PHP] pobierz, plaintext 

I wywala mi błąd związany z pętlą for:

Forum nie parser, a angielski przy programowaniu wypada znać.

To nie ma nic wspólnego z OOP. przenoszę.

A co to jest jak nie OOP ? Może strukturalne ?

Problemem nie jest organizacja obiektów ani coś z rodziny wzorców projektowych tylko strasznie niedbalstwo autora i brak umiejętności myślenia. Dlatego = przedszkole.

Błąd oznacza, że $this->replacement nie jest tablicą. Ogólnie to sporo jest do poprawienia: konstruktor i funkcja display() na pewno, poza tym kilkakrotnie niepotrzebnie dodatkowo przepisujesz zmienne, nigdzie nie używasz zmiennej prywatnej $sReplacement (poza bezsensownym przepisaniem w konstruktorze, w dodatku źle napisanym) i funkcji add_replacement().