[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]

nbie chce mi sę teraz wiecej pisać - dlatego powiem tylko:

wielkim switchom mówimy NIE.

odpowiedzią jest MVC

[splatch]

Pisałem przecież, że można zastąpić switch warunkiem

[PHP] pobierz, plaintext 
<?php
$fname = 'modules/'. $_GET['action'] .'.php';
$cname = $_GET['action'];
if( file_exists( $fname ) )
{
  include_once( $fname );
  $object = new $cname();
  $object->parse();
}
else
{
  //nieznana akcja
}
?>
[PHP] pobierz, plaintext 

[dag]

Pisałem przecież, że można zastąpić switch warunkiem

A nie lepiej zrobić wszystko według wzorca MVC tak jak pisał DeyV?? Będzie łatwiej tym zarządzać, bo podejrzewam, że system będzie sporawy.

[splatch]

A co w tym projekcie jest niezgodne z MVC? Jest kontroler, jest akcja i jest widok (smarty). Teraz zostaje tylko kwestia odpowiedniego połączenia modułów, o tym niestety nie napisaliście. Pisanie, żeby skorzystać z wzorca bez dania przykładu nic mi nie daje. Znam tylko podstawy MVC i staram się to na tej zasadzie zrobić, chociaż mogę mylić pojęcia..

[dag]

Wprowadzenie do MVC - php.pl

Przeczytaj dokładnie.

[splatch]

Czytałem ten tekst, szukałem na forum i ostatecznie uciekłem się do pomocy starego znajomego (dzięki bora (IMG:http://forum.php.pl/style_emoticons/default/smile.gif) ). Wyjaśnił mi nieco zasady MVC i przy jego pomocy napisałem nowy szkielet, co prawda nie jest to nic specjalnego, ale jest. Oto nowy kod, jeśli widzicie w nim jakieś błędy poza złym użyciem :: to piszcie (IMG:http://forum.php.pl/style_emoticons/default/winksmiley.jpg) .

[PHP] pobierz, plaintext 
<?php
 
class controll
{
	var $access = array(
					'home page' => 'all',
					'view auction' => 'all',
					'auction' => 'users',
					'account' => 'user',
				 );
 
	function controll( $page = null )
	{
		$this->page = $page;
		out( &#092;"uruchamianie kontrolera\" );
		if( $page === null )
		{
			return true;
		}
		else
		{
			return $this->access( $page );
		}
	}
 
	function access( $page )
	{
		out( &#092;"sprawdzam prawa uzytkownika\" );
		$status = $_SESSION['status'];
 
		if( $this->acess[ $page ] == 'all' )
		{
			$this->authorized = true;
		}
		else
		{
			if( $status == 'u' && $this->access[ $page ] == 'users' ) 
			{
				$this->authorized = true;
			}
			else
			{
				//sprawdz dokladne uprawnienia usera
				$this->authorized = true;
			}
		}
	}
 
	function run( )
	{
		out( &#092;"teraz uruchamiam akcje\" );
		if( $this->authorized )
		{
			$action = init::action( $this->page );
			$view = $action->run( );
			$action->_die( );
		}
		out( &#092;"mam widok!\" );
	}
 
	function display( )
	{
		out( &#092;"wyswietlam widok\" );
	}
 
	function _die( )
	{
		out( &#092;"niszcze obiekt\" );
		//zamkniecie polaczen z baza itp.
	}
}
 
class init
{
	function action( $what )
	{
		out( 'tworze obiekt auction' );
		return new auction();
	}
}
 
class auction
{
	function auction( )
	{
		out( &#092;"uruchamiam akcje 'auction'\" );
	}
 
	function run( )
	{
		out( &#092;"pobieram dane poprzez model i przekazuje do widoku\" );
	}
 
	function _die( )
	{
		out( &#092;"niszcze obiekt\" );
	}
}
 
class model
{
//tutaj ogolne metody dla modeli
}
 
class auction_model extends model
{
// model tylko do obslugi aukcji
}
 
class view
{
//ogolne metody dla widokow
}
 
class auction_view extends view
{
//metody unikalne, tylko dla aukcji
}
 
function out( $msg )
{
	echo $msg .&#092;"n<br>\";
}
 
$controll = new controll( 'home page' );
$controll->run( );
$controll->display( );
$controll->_die( );
 
?>
[PHP] pobierz, plaintext 

[dag]

Powinno być raczej $Controller a nie $controll ;-). Ja to troszkę inaczej skonstruowałem.

Ten post edytował dag 22.10.2004, 15:22:45

[bela]

@dag: nie chwal sie, tylko powiedz jak

[splatch]

Dlatego mnie wkurza to forum - są odpowiedzi, czasami nawet dużo, ale 0 konkretów. Pytam o opinie i rady. Dostaje w odpowiedzi switchom wielkim nie i skierowanie do MVC, przedstawiam kod oparty o wcześniej wspomiany wzorzec i co dostaje w odpowiedzi.. ? Jak narazie "Ja to troszkę inaczej skonstruowałem." (IMG:http://forum.php.pl/style_emoticons/default/dry.gif)

[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.

[dag]

Nie mogłem znaleźć aktualnego projektu w UML, więc dam "trochę" CHAOTYCZNĄ wersję.

Wprowadziłem template. Jest to pierwsza faza jego projektowania więc jest mocno uogólniony.

image ~60 KB - 1306x876px

to samo co wyżej, tylko ~37 KB - 1000x671px

PS. Przepraszam za ten projekt, jest MOCNO niezgodny ze standardami UML, ale innego w tej chwili nie mam.

Eh, aż sam się dziwię, że coś rozumiem z tego wykresu ;-) podąrzajcie za strzałkami, jeśli jest <<extends>> to nie musi się zawsze tego wykonywać.

Co sądzicie?

Ten post edytował dag 22.10.2004, 21:21:44

[bela]

@dag: a gorzej już się nie dało narysować (IMG:http://forum.php.pl/style_emoticons/default/winksmiley.jpg) ?

[dag]

Zaraz przerysuje ;-) Pisałem, że to jest stare i chaotyczne...

EDIT:

MVC

Ten post edytował dag 22.10.2004, 23:01:03

[NuLL]

Ja odnosze wrazenie - ze mamy tu dyskusje o MVC - z drugiej strony odnosze wrazenie ze pisanie systemu aukcyjnego jest podobne do systemu zarzadzania trescia tyle ze iwkeszej ilosci powiazan - idee co do podstaw jadra sa chyba takie same. Piszemy inne moduly, troche wicej powiazan i tyle.

[splatch]

null:
http://forum.php.pl/index.php?showtopic=22...ndpost&p=132351

[dag]

Heh nie zrozumieliśmy się.

Chodzi o to, że radzę Ci zrobić system aukcyjny w oparciu o framework zgodny ze wzorem MVC.

Wtedy tylko robisz widoki, modele i akcje odpowiedzialne za aukcje. Potem dorabiasz tak samo CMS i masz gotowy i dość łatwy w utrzymaniu kod.

[splatch]

Poprawiłem nieco całość. Za jakiś czas wrzucę schemat ralacji.