В правильном ли направлении строится система роутингов?

Question

[dj_maestro]

Господа, доброго времени суток. Будьте добры, помогите разобраться в направлении PHP ООП. Есть класс Roure и в данной реализации он работает! ( обрабатывает URI ). Он вызывает контроллеры и методы которые описаны в $_GET запросе. НЕ уверен что всё правильно сделано.... Опыт не большой...

<?php
namespace core\Controllers;

class Route {

    public $uri;
    public $controller;
    public $action;

    public function __construct(){

        $url = htmlspecialchars( trim( parse_url($_SERVER['REQUEST_URI'], PHP_URL_PATH)) );

            if (substr($url, -1)=='/') {
                $this->uri = substr($url, 0, -1);
            } else {
                $this->uri = $url;
            }
    }

    public function get( $pathUrl, $nameController, $action ) {

        if ($pathUrl == $this->uri) {

            $str = __DIR__.'/../../App/Controllers/'.$nameController.'.php';
            $pathClass = str_replace('/', '\\', $str);

            if ( file_exists( $pathClass ) ){

                $str2 = 'App\\Controllers\\'.$nameController;

                $this->controller = new $str2();

                if ( method_exists($this->controller, $action) ){

                    return $this->controller->$action();

                } else {

                    echo 'Не найден метод класса-контроллера';
                }

            } else {

                echo 'Не найден контроллер';
            }
        } else {
            echo 'Не равен';
        }

    }

}

Answer 1

[Stalker_RED]

Направление более-менее правильное, но путь еще предстоит большой.

Что в роутинге делают htmlspecialchars и trim? Если в запросе пришел мусор, лучше не устраивать лотерею, а честно показывать 404. По крайней мере поначалу.

str_replace('/', '\\', $str)

Тестовый сервер на windows? В последствии он только на windows и будет крутится?
(Хинт: есть такая штука как DIRECTORY_SEPARATOR)

Переменные с названиями $str и $str2 пораюдуют вас, когда вы попытаетесь вспомнить что делает этот код всего-лишь через полгодика.

if (file_exists ...

Не лучше ли поручить подключение классов автолоадеру? Он специально для таких случаев и придуман.

Ну и echo с выводом ошибок в браузер - это жесть. Надеюсь, что это временная заглушка.

Comments

[Falseclock]

Ай маладец! Нравятся мне такие ответы. эхи можно заменить на throw new Exception("ашипка")

[dj_maestro]

Stalker_RED, спасибо за DIRECTORY_SEPARATOR. Автолоадер задействован. Я так понимаю что проверка на существование файла лишняя, наверное лучше проверять ( используя namespace ) существует ли класс-контроллер который будет вызывать описанный в нём метод при использовании определённого маршрута к примеру такого как $route->get('/article', 'IndexController', 'show'); Переменные с названиями $str и $str2- никогда в реальном проекте использовать не хорошо, также как и echo с выводом ошибок в браузер ) Просто хочется пощупать как это работает. Спасибо за ответ!

[Falseclock]

dj_maestro: кстати, где-нибудь глобально зафиксируйте DEFINE('EXT_PHP', '.php'); и используйте его также как сепаратор директорий. А вдруг вы захотите завтра изменить расширение всех файлов? Вряд ли конечно, но опять же, правилом хорошего тона считается в глобальных классах вообще не использовать строчные значения. Все должно быть на уровне переменных. Некоторые фрики даже Exception переносят в отдельный класс, расширяя его... А знаете почему? Потому что потом проще подключать языки к вашему сайту и выводить ошибку не на вашем родном, а на том, на котором говорит посетитель.

Answer 2

[Falseclock]

Как уже сказал Stalker_RED направление правильное, но надо еще больше отделить мух от котлет.

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

А вы вместо этого вы вызываете $action, делая медвежью услугу.

А вдруг вам завтра надо будет не сразу выводить данные в браузер, а получить контент по данному запросу? Вам придется шаманить сам рутинг и вот тут вот начнутся костыли.

Comments

[Falseclock]

при этом ваш nameController должен имплементить интерфейс или расширять другой класс, в котором вы опишите как минимум две функции: render и getContent.

render - вывод в браузер
getContent - отдача контента другому классу

interface View
{
	public function render($model = null);
	public function content($model = null);
}

В подключаемом классе описываете что нужно вообще делать

class Handler implements View {
	public function content($module = null)
	{
		ob_start();
		ob_end_clean();
               // возврат значения
	}

	public function render($module = null)
	{
               // эхо вашего HTML
	}
}

а потом уже в вашем самом классе

class MainHandler extends Handler {
 // а тут реализуете что вам нужно.
}

[dj_maestro]

Falseclock, спасибо! php изучаю буквально три месяца, хочется различные реализации пощупать ручками и понять как это всё работает, также успел познакомиться с Laravel, работая с ним всё вроде бы понятно и легко реализуется но почему-то хочется написать свой велосипед в целях обучения.

[Falseclock]

и еще из замечаний.

__construct() - это только конструктор. Правилом хорошего тона будет только принимает переданные значения и только, но не обрабатывать и не модифицировать.

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

Потому что завтра может быть так, что вам нужно будет получить два экземпляра класса, а вы передали значение, которое становится статическим. Вызывая второй класс, вы переопределили статику и в первом экземпляре начинаете ловить неожиданные баги... это так.. для примера.

[Stalker_RED]

dj_maestro: Можете еще посмотреть как устроен роутинг в Yii, в Symfony, в Silex (который типа обрезаный Symfony, но подход к роутингу у него значительно отличается.)

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

[dj_maestro]

Вот прям в точку! Спасибо огромное!

[Falseclock]

dj_maestro: Вся правильно! Пишите свой велосипед. Я за 15 лет так и не пощупал что такое "Yii, в Symfony, в Silex " и без понятия. Как-то открыл исходники Worpdress и меня хватил удар.

Все что вам нужно будет в разработке - это composer, через который вы будете подключать огромные классы, например работа с XML, XLS, БД, кэшем, демонами, сокетами и т.д и т.д., которыми пользуются десятки тысяч и 99% багов исправлены, а функционал достаточен для ваших нужд. В крайнем случае можно всегда сделать extend, но мне не приходилось. Тем более вы изобретаете не велосипед, а реализуете свой требуемый функционал. Изобрести велосипед - это если не пользоваться готовыми встроенными в PHP функциями и классами. Например сделать свою обертку с работой Memcached.

[Falseclock]

dj_maestro: и кстати, если у вас рутинг для всех и для любого, то уберите public для переменных. Они же для каждого вызова уникальные раз и никто не должен их изменять - это два. Если вы не заимеете привычку правильно описывать переменные и классы, область видимости и доступ, то опять же можете нарваться на баги, которые с трудом отыскиваются.

[dj_maestro]

Composer использую. Автолоадер задействован. Я так понимаю что проверка на существование файла лишняя, наверное лучше проверять ( используя namespace ) существует ли класс-контроллер который будет вызывать описанный в нём метод при использовании определённого маршрута к примеру такого как $route->get('/article', 'IndexController', 'show'); Переменные с названиями $str и $str2- никогда в реальном проекте использовать не хорошо, также как и echo с выводом ошибок в браузер ) Просто хочется пощупать как это работает. Спасибо за ответ!

[Falseclock]

dj_maestro: проверяйте через instanceof

[dj_maestro]

Огромное спасибо Вам за ответ!