Forum PHP.pl

napisałem takiego oto potworka, z założenia jest to projekt szkoleniowy, który rozwija się w miarę przyswajania nowej wiedzy, składa się z kilku elementów

proszę o krytykę, mam wrażenie, że trochę błądzę w założeniach MVC i chcę mieć pewność w których miejscach

proszę tylko nie do końca zwracać uwagę na "niefortunnie" dobrane nazwy klas i funkcji, wiem, że wymaga to staranniejszej organizacji, na razie jest tak, by __autoload łapał wszystkie potrzebne mi klasy

o dziwo potworek ten działa tak jak zakładałem .htacces przekierowuje wszystko na index.php a linki buduję np. tak www.mojastrona.pl/product/viewproduct/25



index.php - załącza ini.php, tworzy router, tworzy dispatcher, przekazuje router do dispatchera

[PHP] pobierz, plaintext 
<?php
include ('core/ini.php');
$router=router::getInstance();
$dispatcher=dispatcher::getInstance();
$dispatcher->start($router);
?>
[PHP] pobierz, plaintext 

ini.php - ustala ścieżki i załącza funkcję __autoload

[PHP] pobierz, plaintext 
<?php
set_include_path(get_include_path().PATH_SEPARATOR."core");
set_include_path(get_include_path().PATH_SEPARATOR."core/main");
set_include_path(get_include_path().PATH_SEPARATOR."core/cfg");
set_include_path(get_include_path().PATH_SEPARATOR."app/controllers");
set_include_path(get_include_path().PATH_SEPARATOR."app/views");
set_include_path(get_include_path().PATH_SEPARATOR."app/models");
 
function __autoload($object){	
	require_once("c{$object}.php");
}
?>
[PHP] pobierz, plaintext 

cConfigs.php - udostępnia dane

[PHP] pobierz, plaintext 
<?php
 
class configs{	
 
	const ALLOWED_URL_CHARS="/[^A-z0-9\/\^]/";	
	const DEFAULT_CONTROLLER="products";	
	const DEAFAULT_ACTION="index";	
	const DB_USER="test";
	const DB_PASSWORD="*******";
	const MYSQL="mysql:host=localhost; dbname=test";	
	const PATH_CORE="core";
	const PATH_CONTROLLERS="app/controllers";
	const PATH_VIEWS="app/views";	
	const ERROR_NO_PAGE="errornopage";
 
	private static $ALLOWED_CONTROLLERS=array("start","products");
 
	public static function getAllowed_Controllers()
	{
		return self::$ALLOWED_CONTROLLERS;
	}
}
?>
[PHP] pobierz, plaintext 

cRouter.php - obsługa adresu wywołania (dzieli url na parametry)

[PHP] pobierz, plaintext 
<?php
 
class router
 {	
	private static $instance=false;
	private $url;
	private $routeParts;
	private $controller;
	private $action;
	private $parametry;	
 
	private function __construct()
	{	
		$this->url=array_keys($_GET);		
		$this->executeRouting();		
	}
 
	public static function getInstance()
	{
		if(self::$instance==false){		
			self::$instance= new router();						
		}	
		return self::$instance;	
	}
 
	private function executeRouting()
	{			
		$patern=configs::ALLOWED_URL_CHARS;
		$url=$this->url;		
 
 
	if(isset($url['0']))
	{
		$route=$url['0'];
		$route=preg_replace($patern,"",$route);
		$route=str_replace("^","",$route);
		$routeParts=array();		
		$routeParts=explode("/", $route);
 
		if(isset($routeParts['0']))
		{
			$this->controller=$routeParts['0'];
 
			if(isset($routeParts['1']))
			{
				$this->action=$routeParts['1'];
 
				if(count($routeParts>2)){
 
					$routePartsParam=$routeParts;
					array_shift($routePartsParam);
					array_shift($routePartsParam);
					$this->parametry=$routePartsParam;
 
				}else{
					$this->parametry=false;
				}
 
			}else{
				$this->action=false;
				$this->parametry=false;
			}
		}else{
			$this->controller=false;
			$this->action=false;
			$this->parametry=false;
		}	
 
	}
	}
 
	public function getController()
	{
		return $this->controller;
	}
 
 	public function getAction()
	{
		return $this->action;
	}
 
 	public function getParametry()
	{
		return $this->parametry;
	}
 }
?>
[PHP] pobierz, plaintext 

cDispatcher.php - pobiera dane z routera i przetwarza je (sprawdza czy są poprawne i dozwolone, zwraca wartości domyślne - inicjuje właściwe kontrolery i akcje)

[PHP] pobierz, plaintext 
 
<?php
 
class dispatcher
{
	private static $instance=false;
	private $controller;
	private $action;
	private $parametry;
 
	public static function getInstance()
	{
		if(self::$instance==false){		
			self::$instance= new dispatcher();					
		}
		return self::$instance;	
	}
 
 
	public function start($router)
	{
		$controller=$router->getController();
		$action=$router->getAction();
		$parametry=$router->getParametry();		
		self::setParams($controller,$action,$parametry);
		self::dispatch();		
	}
 
	private function setParams($controller,$action,$parametry)
	{			
		//POPRAWNY KONTROLER i AKCJA
		if($controller!=false && in_array($controller,configs::getAllowed_Controllers())&& $action!=false && method_exists($controller, $action))
		{
			$this->controller=$controller;
			$this->action=$action;
			if(count($parametry)>0)
			{
				$this->parametry=$parametry;
			}
			else{
				$this->parametry=false;					
			}
		}
		//NIEPOPRAWNY KONTROLER LUB POPRAWNY KONTROLER I NIEPOPRAWNA AKCJA
		elseif(($controller!=false && !in_array($controller,configs::getAllowed_Controllers()))||($action!=false && !method_exists($controller, $action)))
		{
			$this->controller=configs::ERROR_NO_PAGE;
			$this->action=configs::DEAFAULT_ACTION;
			$this->parametry=false;
		}
		//BRAK KONTROLERA LUB POPRAWNY KONTROLER I BRAK AKCJI
		elseif(($controller==false)||($controller!=false && in_array($controller,configs::getAllowed_Controllers())&& $action==false))
		{
			$this->controller=configs::DEFAULT_CONTROLLER;
			$this->action=configs::DEAFAULT_ACTION;
			$this->parametry=false;
		}			
	}
 
	private function dispatch()
	{
		$controller=$this->controller;
		$action=$this->action;
		$parametry=$this->parametry;
 
		$app = new $controller();
 
		if($parametry!=false){			
			$app->setParametry($parametry);
		}
 
		$app->$action();
	}
}
?>
[PHP] pobierz, plaintext 

cController.php - kontroler rodzic, ustawia parametry dla akcji, sprawdza ich ilość, ustawia model i widok

[PHP] pobierz, plaintext 
abstract class controller implements interfaceBasic
{
	protected $parametry;
	protected $view;
	protected $model;
	protected $viewname;
 
	public function setParametry($parametry)
	{	
	$this->parametry=$parametry;	
	}
 
	public function countParametry($parametry, $parNum)
	{
		if(count($parametry)!= $parNum || $parametry['0']=="")
		{
			$controller=configs::ERROR_NO_PAGE;
			$action=configs::DEAFAULT_ACTION;
			$app= new $controller;
			$app->$action();
			return $error=true;
		}		
	}
 
	protected function setModel()
	{
		$model= new model();
		$this->model=$model;		
	}
 
	protected function setView($viewname)
	{			
		$widokname="view".$viewname;
		$widok=new $widokname();
		$this->widok=$widok;
	}
}
?>
[PHP] pobierz, plaintext 

cInterfaceBasic.php - żąda funkcji index() która jest domyślną dla wszystkich kontrolerów

[PHP] pobierz, plaintext 
<?php
interface interfaceBasic
{
	function index();
	function countParametry($parametry, $parNum);	
}
[PHP] pobierz, plaintext 

cProducts.php- kontroler produktów - domyślny kontroler - wywołuje akcję index() która listuje wszystkie produkty lub akcję viewproduct pokazującą pojedyńczy produkt po parametrze $id produktu

[PHP] pobierz, plaintext 
<?php
 
class products extends controller
{
	public $use_layout=false;
 
	public function __construct()
	{				
		parent::setModel();		
	}
 
	public function index()
	{
		parent::setView("products");
		$model=$this->model;
		$widok=$this->widok;
		$widok->setModelInView($model);
		$widok->render();		
	}
 
	public function viewproduct()
	{
		parent::setView(__FUNCTION__);
		$parametry=$this->parametry;
		$parNum=1;
		$error=self::countParametry($parametry,$parNum);
 
		if($error!=true)
		{
 
		$model=$this->model;
		$widok=$this->widok;
		$widok->setModelInView($model);
		$widok->setParamsInView($parametry);
		$widok->render();
		}
	}
}
?>
[PHP] pobierz, plaintext 

cErrornopage.php - klasa błędu - tylko na potrzeby zaznaczenia momentu błedu

[PHP] pobierz, plaintext 
<?php
class errornopage extends controller implements interfaceBasic
{
	function index()
	{
		echo "wrong site : 404";
	}
 
	function __construct()
	{
 
	}
}
?>
[PHP] pobierz, plaintext 

cDB.php - baza danych

[PHP] pobierz, plaintext 
<?php
abstract class DB
{	
	private static $user;
	private static $password;
	private static $mysql;
	protected static $pdo;
 
	public static function setParams()
	{		
		self::$user=configs::DB_USER;
		self::$password=configs::DB_PASSWORD;		
		self::$mysql=configs::MYSQL;
	}
 
	public static function setConnection()
	{
		self::setParams();		
		$user=self::$user;
		$password=self::$password;
		$mysql=self::$mysql;		
		self::$pdo=new PDO($mysql,$user,$password);                       
     } 
}
?>
[PHP] pobierz, plaintext 

cModel.php - model

[PHP] pobierz, plaintext 
<?php
 
class model extends DB
{
	private $result;
 
	public function __construct()
	{		
		parent::setConnection();			
	}
 
	public function getAllProducts()
	{				
		$pdo=self::$pdo;		
		$query=$pdo->query('SELECT * FROM products ');
		$this->result=$query;		
	}
 
	public function getProduct($id)
	{
 
		$pdo=self::$pdo;		
		$query=$pdo->query("SELECT * FROM products WHERE PRODUCTID=$id")or die ('muka');
		$this->result=$query;
 
	}
 
	public function getResult()
	{
		return $this->result;
	}
}
?>
[PHP] pobierz, plaintext 

cView.php - klasa widok - rodzic

[PHP] pobierz, plaintext 
<?php
abstract class view
{
	protected $model;
	protected $parametry;	
 
	public function setModelInView($model)
	{
		$this->model=$model;
	}
 
	public function setParamsInView($parametry){
		$this->parametry=$parametry;
	}	
}
?>
[PHP] pobierz, plaintext 

CViewProducts.php - widok dla listy produktów

[PHP] pobierz, plaintext 
<?php
class viewproducts extends view
{
	public function render()
	{		
		$model=$this->model;
		$model->getAllProducts();
		$result=$model->getResult();		
 
		while($row=$result->fetch())
		{			
		echo '<hr/>
          <p><b>iD:</b> '.$row['PRODUCTID'].'</p>
          <p><b>Nazwa:</b> <a href="products/viewproduct/'.$row['PRODUCTID'].'">'.$row['PRODUCTNAME'].'</a></p>
          <p><b>Cena:</b> '.$row['UNITPRICE'].'</p>
          <p><b>Na Magazynie:</b> '.$row['UNITSINSTOCK'].'</p>';		 
		}	
	}
} 
?>
[PHP] pobierz, plaintext 

cViewViewproducts.php - widok dla pojedynczego produktu

[PHP] pobierz, plaintext 
<?php
 
class viewviewproduct extends view
{
 
	public function render()
	{		
		$parametry=$this->parametry;
		$model=$this->model;
		$model->getProduct($parametry['0']);
		$result=$model->getResult();
		$row=$result->fetch();
 
				echo '<hr/>
          <p><b>iD:</b> '.$row['PRODUCTID'].'</p>
          <p><b>Nazwa:</b> '.$row['PRODUCTNAME'].'</p>
          <p><b>Cena:</b> '.$row['UNITPRICE'].'</p>
          <p><b>Na Magazynie:</b> '.$row['UNITSINSTOCK'].'</p>'; 		
	}
} 
?>
[PHP] pobierz, plaintext 

baza danych zapożyczona z innego przykładu

[SQL] pobierz, plaintext 
 
CREATE TABLE products (
  PRODUCTID int(11) NOT NULL AUTO_INCREMENT,
  PRODUCTNAME varchar(255) NOT NULL DEFAULT '',
  UNITPRICE varchar(255) NOT NULL DEFAULT '',
  UNITSINSTOCK varchar(255) NOT NULL DEFAULT '',
  DISCONTINUED varchar(255) NOT NULL DEFAULT '',
  PRIMARY KEY  (PRODUCTID)
) TYPE=MyISAM;
 
#
# Dumping data for table `products`
#
 
INSERT INTO products VALUES (1, 'Chai', '18', '39', '0');
INSERT INTO products VALUES (2, 'Chang', '19', '17', '0');
INSERT INTO products VALUES (3, 'Aniseed Syrup', '10', '13', '0');
INSERT INTO products VALUES (4, 'Chef Antons Cajun Seasoning', '22', '53', '0');
INSERT INTO products VALUES (5, 'Chef Anton Gumbo Mix', '21.35', '0', '1');
INSERT INTO products VALUES (6, 'Grandma Boysenberry Spread', '25', '120', '0');
INSERT INTO products VALUES (7, 'Uncle Bob Organic Dried Pears', '30', '15', '0');
INSERT INTO products VALUES (8, 'Northwoods Cranberry Sauce', '40', '6', '0');
INSERT INTO products VALUES (9, 'Mishi Kobe Niku', '97', '29', '1');
INSERT INTO products VALUES (10, 'Ikura', '31', '31', '0');
INSERT INTO products VALUES (11, 'Queso Cabrales', '21', '22', '0');
INSERT INTO products VALUES (12, 'Queso Manchego La Pastora', '38', '86', '0');
INSERT INTO products VALUES (13, 'Konbu', '6', '24', '0');
INSERT INTO products VALUES (14, 'Tofu', '23.25', '35', '0');
INSERT INTO products VALUES (15, 'Genen Shouyu', '15.5', '39', '0');
 
 
[SQL] pobierz, plaintext 

Ten post edytował olechafm 18.05.2011, 15:50:30

to jest akurat na żywca z książki (Programowanie Obiektowe w PHP5 autorstwa Hasina Haydera oczywiście, jakby ktoś chciał się pośmiać ) , cala funkcja __autoloda będzie inaczej napisana (się pisze właśnie) jak tylko ustalę sam ze sobą strukturę katalogów i nazewnictwo plików, i będzie miała jasno określone ścieżki bez tych elementów

set_include_path(get_include_path().PATH_SEPARATOR."core");

up

Dzięki za odpowiedź, to była jedna z rzeczy których nie byłem pewien. A co do PSR-0 (może ktoś ma link do jakiegoś dobrego przykładu albo po ludzku napisanej dokumentacji?) to tak, oczywiście wiem, że istnieje i chętnie zacznę go używać jak tylko go dostatecznie zrozumiem, tak jak napisałem już wcześniej nie da się wszystkiego od razu robić dobrze, na początku zbyt wiele jest do nauki...

Oj jest, może wydaje się, że to oczywistości są, ale gdy wcześniej nauczyło się złych rzeczy to przestawić się później trudno, szczególnie, że jak i we wszystkich innych przypadkach, ciężko o napisaną po ludzku wersję angielskiej dokumentacji czy przykład pokazujący chociażby kilka klas itd...

niema to jak autopromocja tak wiem, że one tam są i przedzieram się przez nie tak samo jak i przez całego bloga bądź co bądź są to jednakowoż przykłady bardzo złożone






odnośnie jeszcze tego, rozumiem, że model ma sobie wyciągnąć z klasy DB uchwyt połączenia z bazą po prostu, ale same zapytania do bazy

[PHP] pobierz, plaintext 
$query=$pdo->query('SELECT * FROM products ');
$this->result=$query; 
[PHP] pobierz, plaintext 

pozostają w model czy też lądują w klasie DB i przy ich użyciu model te dane pobiera?

Ten post edytował olechafm 27.05.2011, 11:38:14

aaaaaaaaa co nie zadam pytania to dorzucacie mi jakieś nowe rzeczy do nauczenia się...

[PHP] pobierz, plaintext 
public function __construct(DatabaseConnection $db) // DatabaseConnection to jest ten kontener?
[PHP] pobierz, plaintext 

dzięki, czytam też to: www.rekurencja.pl/php/symfony2/czym-jest-dependency-injection.html

Ten post edytował olechafm 27.05.2011, 13:30:10