Роутер. Разумно ли использовать статические методы?

Question

[dev400]

А как делаете вы?

class Router
{
	private static $uri;

	public static function notFound() {
		header("Status: 404 Not Found");
		header('Location:' . $_SERVER['HTTP_HOST'] . 'Error');
	}

	private static function getController() {

		$array['controller'] = 'index';
		$array['method'] = 'mainMethod';

		self::$uri = explode('/', $_SERVER['REQUEST_URI']);

		if (!empty(self::$uri[1])) {

			$array['controller'] = self::$uri[1];

		}


		if ( !empty(self::$uri[2]) ) {

			$array['method'] = self::$uri[2];

		}

		return $array;

	}

	public static function getRoute() {

		$route = self::getController();
		$controller = "App_Controllers_".$route['controller'];
		$obj = new $controller;
		if(method_exists($obj, $route['method'])) {
			$obj->$route['method']();
		}
		else {
			self::notFound();
		}
	
	}

}

Comments

[Stalker_RED]

Считается, что делать редирект при 404 - нехорошо. Страничку 404 показать конечно нужно, вызвав соответствующий action, но url в адресной строке лучше сохранить, чтобы человек мог исправить опечатку, например.

Answer 1

[Александр]

По поводу статики - нет, не стоит, ибо если вы потом захотите покрыть ваш роутер тестами - у вас возникнут с этим проблемы. А вообще прежде чем писать свой велосипед, стоит ознакомится с популярными решениями на эту тему, разобраться как они работают и уже потом что-то лепить на коленке.

Comments

[dev400]

какие лучше посмотреть популярные решения?

[Александр]

Андрей Безруков: https://github.com/nikic/FastRoute - это как пример отдельной библиотеки и конечно можно глянуть реализации в популярных фреймворках: Yii, Symfony, Laravel и тд

[dev400]

Александр: Спасибо!