Как оставить доступным GET параметр после обрезания?

Question

[QNA-1976]

обрезание :)

private static function removeQueryString($url)
    {
        if ($url) {
            $params = explode('&', $url, 2); // отрезали что-то вроде ?page=1
            if (strpos($params[0], '=') === false) {
                $params = explode('/', $params[0]); // отрезали до первого слеша
                return rtrim($params[0], '/'); // вернули чистое имя контроллера
            }
        }
        return '';
    }

правила

Router::add('^operation$', ['controller' => 'Operations', 'action' => 'operation']); // 'id' => 1 как-то сюда надо
Router::add('^$', ['controller' => 'Main']);
Router::add('^(?P<controller>[a-z-]+)/?(?P<id>\d+)?$');
Router::add('^(?P<controller>[a-z-]+)/?(?P<action>[a-z-]+)?/?(?P<id>\d+)?$');

Теперь если я перехожу по /operation/1, то всё нормально - отработало правило 1, но нет GET параметра id со значением 1

Array
(
    [controller] => Operations
    [action] => operation
    тут нужно получить [id] => 1 
)

то есть правило 3 естественно не отработало. Как оставить id? Добавить в правило? А как передать значение в правило?

Comments

[Дмитрий]

Думаю надо перестать заниматься ерундой и использовать готовые функции языка
https://www.php.net/manual/ru/function.parse-str.php

[QNA-1976]

Дмитрий, не очень пойму, как это работает при изменении реального контроллера на свою ссылку

[Дмитрий]

QNA-1976, вам надо разобрать query string , вы его разбираете через три функции, делаете проверку поиском и тримите.

[QNA-1976]

Дмитрий, мне после разбора в self::$routes собирать их потом?

private static function matchRoute($url)
    {
        foreach (self::$routes as $pattern => $route) {
            if (preg_match("@{$pattern}@i", $url, $matches)) {
                debug($matches);
                foreach ($matches as $k => $v) {
                    if (is_string($k)) {
                        $route[$k] = $v;
                    }
                }
                if (!isset($route['action'])) {
                    $route['action'] = 'index';
                }
                debug($route);
                debug($_GET);
                self::$route['controller'] = self::upperCamelCase($route['controller']);
                self::$route['action'] = self::lowerCamelCase($route['action']);
                return true;
            }
        }
        return false;
    }

[Дмитрий]

QNA-1976, есть вагон готовых роутеров, у вас какой то собственный велосипед, который ещё и не поехал толком, ещё и на статике

[Дмитрий]

QNA-1976, если по простому, первых два элемента будут экшон и контроллер, прочее это аргументы

[QNA-1976]

Дмитрий, поехал уже давно, только теперь появилась цель ЧПУ обрабатывать, так-то ?page=1&id=2 без проблем

[QNA-1976]

Дмитрий, первого и второго элемента может не быть: стартовая страница с пагинацией, например ))), контроллер с экшеном index, своё правило

[Дмитрий]

private function getRouteArgs(): array
{
parse_str(
    $_SERVER['QUERY_STRING'],
    $args
);

$controler = array_shift($args);
$action = array_shift($args);

return [
    'controller' => $controller,
    'action' => $action,
    'args' => $args
];
}

$c = new $controller;
$result = $c->{$action}(... $args);

[QNA-1976]

Дмитрий, что попадает в action, если в адресной строке /page/1
?

[Дмитрий]

QNA-1976, я вам дал пример обычного роутера, в вашем случае вы хотите сделать роутер, чтобы по регулярке хватал аргументы, что то подобное в codeIgniter

[Дмитрий]

Ну у вас должны быть регулярки и ссылки на контроллеры
/page-(\d+) => index/page/$1

[QNA-1976]

Дмитрий, да, я уже не помню точно с каких примеров или с игнитера или с кейка такой подход, мне не очень сильно и надо, но хотел попробовать ЧПУ под него, я так понял всё упирается в проверку по регулярке входящего url, я если по правилу зайду, то мне нужно обрезать гет параметры а без них не будет найдено совпадение в массиве маршрутов (правил), обычные нормально обрезаются, но со слешем проблемы

[QNA-1976]

/page-(\d+) => index/page/$1

Router::add('^(?P<controller>[a-z-]+)/?(?P<id>\d+)?$');

:)

[QNA-1976]

Дмитрий, ааааааа, понял - контроллер с цифрой после дефиса

[QNA-1976]

Дмитрий, мне потом цифру из имени контроллера вырезать и присваивать её GET параметру? )

[Дмитрий]

QNA-1976, ни чего не надо вырезать, вам надо написать регулярку маршрута, в регулярке выделить скобками динамические элементы, чтоб передать их аргументами в экшон контроллера
Сделайте проще, есть регулярка, и определяем нужное от этой регулярки

/page-(\d+) => [
'controler' => 'index',
'action' => 'page',
'args' => [$1, $2, $3]
]
Пример абстрактный

[QNA-1976]

Дмитрий, я понял суть, спс

[QNA-1976]

Дмитрий, но это дофига просто правил будет, там где применяем page-(\d+) нужно будет правило создавать )

[Дмитрий]

QNA-1976, ну либо без чпу по простому, либо правила, обычно в фреймворках делают группы правил

[QNA-1976]

Дмитрий, да-да, благодарю, что потратили время и уделили внимание, а то я бы ещё, наверное код долбил, правила же есть, просто я затупил, что регулярку можно в своё правило вставить, а не просто жёстко строку (operation) )))

Answer 1

[Евгений Матвеев]

может надо сразу разбить всю строку, чтобы не путаться?
function.parse-str.php

Comments

[QNA-1976]

спасибо, что зашли, но уже решил

Router::add('^operation-(?P<oid>\d+)/missions$', ['controller' => 'Operations', 'action' => 'missions']);

/operation-2/missions

Array
(
    [controller] => Operations
    [action] => missions
    [oid] => 2
)

из роутера кидаем в контроллер и используем