[jądro]System aukcyjny, Właśnie zaczynam pisać jajko dla syste.. Opcje

[splatch]

Właśnie zaczynam pisać jajko dla systemu aukcyjnego. Jestem świadom tego, że serwis będzie obciążony i dlatego chcę możliwie najlepiej rozwiązać kwestię jądra. Przy wcześniejszych projektach używałem silnika napisanego strukturalnie podpartego smarty.

Chcę stworzyć główną klasę 'VenteSystem', która będzie odpowiedzialna za połączenie z bazą danych oraz załadowanie error handlera i smarty.
Jej kod wygląda tak:

[PHP] pobierz, plaintext 
<?php
 
define( 'LIB_DIR', 'F:/www/localhost/libs/' );
 
class VenteSystem
{
	function VenteSystem( )
	{
		echo &#092;"[system]Laduje potrzebne biblioteki<br>\";
 
		require_once LIB_DIR .'smarty/Smarty.class.php';
		require_once LIB_DIR .'driver/mysql.php'; // w przyszlosci dodanie wyboru mysql/pgsql
		require_once LIB_DIR .'error_handler.php'; //sam ustawia sie jako domyslny
 
		$this->smarty = & new Smarty;
		$this->db = & new sql_db('localhost', 'splatch', 'splatch', 'splatch');
	}
 
	function VenteOutput( )
	{
		echo &#092;"[system]Pokazuje gotowa strone<br>\";
		$this->VenteDie( );
		//return $this->smarty ? $this->smarty->display( 'index.tpl' ) : '';
	}
 
	function VenteDie( )
	{
		$this->db->close( );
		echo &#092;"[system]Zamykam polaczenia z baza<br>\";
	}
}
 
?>
[PHP] pobierz, plaintext 

Teraz w kodzie strony głównej mam

[PHP] pobierz, plaintext 
<?php
$system = new VenteSystem( );
 
echo &#092;"..sprawdzam url..<br>\";
switch( $_GET['action'] )
{
	case 'auction':
		echo &#092;"..laduje dane o auckji..<br>\";
		include 'modules/auction.php'; //wczytuje klase auction
		$object = new auction( $system );
		$object->parse(); //dodanie do szablonu
	break;
	default:
		echo &#092;"<b>nieznana akcja<b>\";
	break;
}
echo &#092;"..zamykanie polaczenia z baza danych..<br>\";
$system->db->close( );
$system->VenteOutput( );
?>
[PHP] pobierz, plaintext 

Kod klasy auction wygląda tak:

[PHP] pobierz, plaintext 
<?php
 
class auction
{
	function auction( &$object )
	{
		$this->system = $object;
 
		echo &#092;"[auction]Do konstruktora przekazana zostala kopia obiektu VenteSytem<br>\";
		if( !empty( $_GET['id'] ) )
		{
			echo &#092;"[auction]Pobierz dane o aukcji nr \". $_GET['id'] .\"<br>\";
		}
		else
		{
			echo &#092;"[auction]Pobierz liste auckji<br>\";
		}
	}
 
	function parse( )
	{
		echo &#092;"[auction]Przygotuj do wyswietlenia<br>\";
		//$this->system->smarty->assign( 'mainpage', 'tresc' );
	}
}
 
?>
[PHP] pobierz, plaintext 

Całość (oprócz administracji) będzie obsługiwana przez stronę główną przy użyciu mod_rewrite. Serwer jest tak skonfigurowany, że url powinen być w postaci: vente.pl/wyraz/liczba[/dodatkowe parametry].
Nie jestem pewien czy takie rozwiązanie jest poprawne. Oczywiście nie byłoby problemu z przerobieniem switch na if( file_exists() ), dodania. Czy wszystko ma realizować jądro.

Czekam na Wasze opinie i rady, za każdą będę wdzięczny.

[DeyV]

spokojnie - nie stresuj się - szkoda serca. (a o zdrowie trzeba dbać..)

Co do twojego pomysłu - uważam że idziesz w dobrym kierunku, co zaowocuje mechanizmem znacznie elastyczniejszym i przenośnym, niż switch.
Proponowałbym tylko, by odrazu zaimplementować w nim 2 dodatkowe mechanizmy.
1. to wyciąganie odpowiedniego pliku z klasą akcji, i dopiero wtedy uruchamianie jej.

Oznacza to, że w kodzie

[PHP] pobierz, plaintext 
<?
 function action( $what )
	{
		out( 'tworze obiekt auction' );
		return new auction();
	}?>
[PHP] pobierz, plaintext 

nazwa uruchamianej akcji będzie musiała być podawana dynamicznie, po uprzednim przekonaniu się, czy odpowiedni plik i wybrana klasa istnieją.


2. dopisanie tzw. routera, czyli mechanizmu potrafiącego rozpoznać i zwrócić informacje o tym, jaką akcje chce użytkownik.

Dodatkowo - pisząc w oparciu o MVC warto zachować się dokładnie do zaleceń, i np. nie pozwalać sobie nigdy na pracę z bazą danych poza modelem, a do Requesta odwoływac się tylko w akcjach.
Zapewnia to znacznie bezpieczniejszy i łatwiejszy do utrzymania kod.