Witam wszystkich na forum,

Przerzucilem sie niedawno na php5 i przy okazji budowy nowej strony postanowilem przepisac od nowa moje jadro i zadbac by bylo ono calkowicie obiektowe. No i tak zaczalem kodowac wg. tego co mialem dawniej, az natrafilem na problem komunikacji wewnatrz calego systemu. Okazalo sie ze nie jest to takie proste (przynajmniej dla mnie, slabego z OO), dlatego chcialbym prosic Was o pomoc.

W chwili obecnej wyglada u mnie to tak:

[PHP] pobierz, plaintext 
<?php
// file kernel/class_engine.php
 
class engine extends Smarty {
	const PATH = 'c:/strony/jadro/'; // path to the main directory with ending slash
	const URL = 'http://127.0.0.1/jadro/'; // path to the main http directory with ending slash
	public $parameters;
	public $moduleName;
	public $moduleFile;
 
	function __construct()
	{
		$this->moduleName = 'news'; // pobieramy go z URL
		$this->moduleFile = 'moduly/news/news.php' // nazwa pliku glownego z modulem
		$this->parameters = array('display','2'); // parametry GET
 
		$this->Smarty();
		$this->template_dir = engine::PATH . 'templates/';
		$this->compile_dir = engine::PATH . 'smarty/templates_c/';
		$this->config_dir = engine::PATH . 'smarty/configs/';
		$this->cache_dir = engine::PATH . 'smarty/cache/';
		$this->caching = false;
	} 
 
// (...) kilka mniejszych funkcji
}
?>
[PHP] pobierz, plaintext 

[PHP] pobierz, plaintext 
<?php
// file kernel/class_session.php
 
class session {
	public $id;
	private $name = 'ses';
 
	function __construct()
	{
		ini_set('arg_separator.output', '&amp;');
		session_name($this->name);
		session_start();
		$this->id = session_id();
	} 
}
 
// file kernel/class_db.php
 
class db {
// tutaj metody connect, query etc.
}
?>
[PHP] pobierz, plaintext 

[PHP] pobierz, plaintext 
<?php
// przykladowy modul (modules/news/news.php)
 
class news {
	public $templateFile;
 
	function __construct() {
	// w zaleznosci od wlasciwosci $parameters klasy engine podejmij wybrana akcje (np. display)
	// czyli np. if ($parameters[0]=='display' && is_integer($parameters[1])) {
	// $this->display($parameters[1]);
	 // }
	}
 
// kilka metod jak display, comments etc
 
}
?>
[PHP] pobierz, plaintext 

Teraz uzycie:

[PHP] pobierz, plaintext 
<?php
require('kernel/class_engine.php');
$site=new engine;
$session=new session; // dzieki autoload automatycznie zaladuje potrzebny plik z klasa
$db=new db; // j.w.
$db->connect();
include($site->moduleFile); // ladujemy plik z klasa modułu news
$module = new $site->moduleName; // uruchamiamy klase z modulem, czyli np. $module = new news;
 
// teraz wykonuje sie metoda __construct, ktora powinna uruchamiac kolejna metode 
lasy news, w zaleznosci od parametrow $_GET
 
 
$db->close();
$site->display($module->templateFile);
 
?>
[PHP] pobierz, plaintext 

Mam nadzieje ze jest to dosyc jasne, do takiego rozwiazanie teraz doszedlem, jednak uwazam iz cos robie zle - zapewne da sie to inaczej (lepiej) rozwiazac. Chcialbym bowiem w klasie news miec dostep do metod klasy $db, nie dziala mi takze $site->assign('zmienna','wartosc') (klasa site jest extends Smarty) - jednym slowem: nie wiem jak rozwiazac ta komunikacje. Czyli aby w klasie news (i w klasach innych modulow) bez problemu operowac na $db czy na $site.. Jak to rozwiazac? Chcialbym np. za pomoca metody klasy news , zmienic wartosc wlasciwosci klasy site (engine)..

Bede wdzieczny za pomoc.. Nie wiem czy jasno napisalem, jesli nie to prosze rowniez pisac, postaram sie jasniej..

Poczytaj sobie o czymś takim jak Singleton.

Dzięki za odpowiedź, poczytałem na tymże forum o singletonie no i zaczęło mi działać, ale mam jeszcze kilka pytań, gdyż nie wiem czy robię dobrze..

Dopisałem do klasy engine i db metodę getInstance, która zwraca instancję tejże klasy. I teraz w klasie news robię tak:

[PHP] pobierz, plaintext 
<?php
 
class news {
	public $site; 
	private $db;
 
	function __construct()
	{
		$this->site = engine::getInstance();
		$this->db = sqlite::getInstance();		
		$this->site->templateFile = engine::PATH . 'modules/' . $this->site->moduleName . '/' . $this->site->moduleName . '.tpl';
		$this->getNews();
	} 
// (...)
}
?>
[PHP] pobierz, plaintext 

Jak widać - pobieram instancje klas engine i sqlite, i przypisuje je do wlasciwosci klasy news ($site i $db). Teraz gdy w metodzie getNews() chce odwolac sie np. do metody klasy $db musze robic tak:

[PHP] pobierz, plaintext 
<?php
$this->db->query('select costam from costam');
?>
[PHP] pobierz, plaintext 

Da sie to jakos ominac, tzn. abym mógł sobie wywoływac $db->query(), a nie $this->db->query() ? Wydaje mi się, że jedyną taką możliwością jest tworzenie w każdej metodzie klasy news instancji do engine/sqlite. Czyli w ten sposob:

[PHP] pobierz, plaintext 
<?php
class news {
	// (...)
	private function getNews()
	{
		$site = engine::getInstance();
		$db = sqlite::getInstance()
		$db->query('costam');
	} 
 
	private function addNews()
	{
		$site = engine::getInstance();
		$db = sqlite::getInstance()
		$db->query('costam');
	} 
}
?>
[PHP] pobierz, plaintext 

Co jednak nie jest zbyt wygodne...

Mam też taki problem:

Gdy robie cos takiego:

[PHP] pobierz, plaintext 
<?php
class news {
	// (...)
	private function getNews()
	{
		$this->site->templateFile = 'szablonik.tpl';
	} 
}
?>
[PHP] pobierz, plaintext 

A nastepnie w pliku index.php dam:

[PHP] pobierz, plaintext 
<?php
$site = new engine;
$session = new session;
$db = new db;
$db->connect();
$module = new news;
 
// tutaj zostanie uruchomiona funkcja getNews, ktora przypisze wartosc wlasciwosci
$this->site->templateFile;
 
echo $site->templateFile;
// powyzsza linijka nic nie wyswietli.. - a chcialbym aby bylo inaczej;-)
?>
[PHP] pobierz, plaintext 

Chodzi o to, że w klasie news tworzę instancję klasy engine, następnie przypisuję ją właściwości klasy news, potem robię $this->site->templateFile='szablonik.tpl' i wlasciwosc templateFile zmienia sie tylko gdy wywolam ja z wewnatrz, tzn. $this->site->TemplateFile - dziala, a $site->templateFile (wywolane z zewnatrz) - nie dziala.

Kurcze.. chyba troche niezrozumiale napisalem.. Moze jednak po krotkiej analizie kodu stanie sie to jasne - jesli nie, postaram sie napisac jasniej.

Z gory dzieki za pomoc i przepraszam ze tak zamieszalem..