Witam. Od jakiegoś czasu myślę nad zrobieniem routingu. Zrobiłem coś bardzo prostego i moje pytanie brzmi, czy takie coś jest poprawne?

[PHP] pobierz, plaintext 
<?php
$request = 'news-2';
$route = array(
	'news-([0-9])' => 'index/news/id/$1',
	'pokaz' => 'index/news',
	);
 
foreach($route as $r => $key) {
	if(preg_match('/^'.$r.'$/i',$request)) {
		$router = preg_replace('/^'.$r.'$/i',$key,$request);
	}
}
echo $router;
[PHP] pobierz, plaintext 

Nie ma czegoś takiego jak "poprawny" router z naszego punktu widzienia.
To od ciebie zależy jakie konkretnie ma on mieć zastosowanie; Zadaj sobie pytanie jak bardzo ma być zaawansowany itd.
Jeżeli uznasz że to co masz wystarczy, zamij się kwestią bezpieczeństwa i wtedy dopiero można go oceniać
A napisać to można na kilkadziesiąt róznych sposobów.

Ale jak chcesz to mogę Ci na pw podesłać prostu routing z którego korzystam.

Też jestem na etapie tworzenia i mam pytanie. Myślałeś jak uruchomić metodę z przekazaniem kilku argumentów do niej?

Tzn. tak jak u Ciebie
index -> uruchamia news

ale np niech będzie uri
/gallery/show/1/4/big
gdzie: 1,4,big to argumenty, które trzeba przekazać, początkowo są one array.

na razie doszedłem do tego:
$class->method($args);

no i w klasie:

[PHP] pobierz, plaintext 
public function method($args)
{
    var_dump($args);
}
[PHP] pobierz, plaintext 

ale czy jest jakaś prosta możliwość przekazania np
$class->method($id, $num, $size)...

@Szymciosek ja mam to rozwiązane tak:

[PHP] pobierz, plaintext 
<?php
namespace Lib;
use Lib\Config;
class Request {
	//vars define
	private $_controller;
	private $_action;
	private $_args = array();
	public function __construct() {
		//route from config 
		$route = array(
					'\\/home' => 'index/index', //controller index, action: index
					);
 
		foreach($route as $r => $key) {
			echo $r;
			if(preg_match('/^'.$r.'$/',$_SERVER['REQUEST_URI'])) {
				$_SERVER['REQUEST_URI'] = preg_replace('/^'.$r.'$/i',$key,$_SERVER['REQUEST_URI']);
				break;
			}
		}
		//end config route
		$parts = explode('/',$_SERVER['REQUEST_URI']);
		$parts = array_filter($parts);
 
		$this->_controller = ($c = array_shift($parts))? $c : 'index';
		$this->_action = ($c = array_shift($parts))? $c : 'index';
		//param
		for($i=0;$i<count($parts);$i=$i+2) {
			$this->_args[$parts[$i]] = (isset($parts[$i+1]))?$parts[$i+1]:'';
		}
 
		//args
	}
	public function getController() {
		return ucfirst(strtolower($this->_controller));
	}
	public function getAction() {
		return ucfirst(strtolower($this->_action));
	}
 
	public function getParam($name=null) {
		if(array_key_exists($name,$this->_args)) {
			return $this->_args[$name];
		}
	}
	public function goUrl($url) {
		return header('location: '.strtolower($url));
	}
}
[PHP] pobierz, plaintext 

I mnie pobiera argumenty na zasadzie "co drugi", czyli mam: index/index/id/2/idk/5 i funkcja getParam('id') => 2, getParam('idk')=>5.

Mam pytanie jak zrobić zeby przed "/" dodawało "\\" czyli w wyniku ma być: \\/

No tak, ale tutaj raczej nadal operujesz na array, a mi chodziło o automatyczne rozbicie tych zmiennych do postaci takich jakie powinny mieć po przekazaniu do klas.

Tak ma np zrobione Symfony2.

/*
* @Route("/test/{var}/{id}")
*/
public function name($var, $id)...

tylko, że tutaj są akurat adnotacje, ale z linku od razu wiem, gdzie jest var i gdzie jest id.

Myślę, że dojdę do tego z czasem jeszcze.

U siebie daje w routerze:

[PHP] pobierz, plaintext 
'/kategoria/([0-9])/strona/([0-9])' => 'kategoria/index/idk/$1/page/$2'
[PHP] pobierz, plaintext 

I otwierając sobie w adresie: kategoria/2/strona/9 to mogę sobie pobrac idk (2) i page(9)

To mi w zupełności wystarcza.